Do you by chance have a testcase that demonstrates this? If you could add one, I;ll go ahead and commit your change..
On my quest to get more testcases! Eric -----Original Message----- From: Peter Courcoux [mailto:[EMAIL PROTECTED] Sent: Friday, June 13, 2003 5:43 PM To: Turbine Developers List Subject: [PATCH] UIManager Hi, I attach a patch for your consideration. Skins for the UIManager were being corrupted by other users skins, when the UIManager is being used as a session scoped tool. Turns out that the Skin Properties are held in a static property of the class! The patch simply removes the 'static' keyword. Regards, Peter Index: UIManager.java =================================================================== RCS file: /home/cvspublic/jakarta-turbine-2/src/java/org/apache/turbine/services/pull/ util/UIManager.java,v retrieving revision 1.9 diff -u -r1.9 UIManager.java --- UIManager.java 3 Jun 2003 13:41:27 -0000 1.9 +++ UIManager.java 13 Jun 2003 21:32:48 -0000 @@ -182,7 +182,7 @@ * Properties to hold the name/value pairs * for the skin. */ - private static Properties skinProperties; + private Properties skinProperties; /** * Initialize the UIManager object. -- Peter Courcoux <[EMAIL PROTECTED]> --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
