Index: lams_admin/src/java/org/lamsfoundation/lams/admin/web/UserAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_admin/src/java/org/lamsfoundation/lams/admin/web/UserAction.java,v diff -u -r1.49 -r1.50 --- lams_admin/src/java/org/lamsfoundation/lams/admin/web/UserAction.java 20 Aug 2009 06:40:01 -0000 1.49 +++ lams_admin/src/java/org/lamsfoundation/lams/admin/web/UserAction.java 20 Aug 2009 23:57:47 -0000 1.50 @@ -124,7 +124,7 @@ } } Theme defaultFlashTheme = themeService.getDefaultFlashTheme(); - for (Theme theme : cssThemes) { + for (Theme theme : flashThemes) { if (theme.getThemeId().equals(defaultFlashTheme.getThemeId())) { userForm.set("userFlashTheme", theme.getThemeId()); break; Index: lams_common/src/java/org/lamsfoundation/lams/themes/service/ThemeService.java =================================================================== RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/themes/service/ThemeService.java,v diff -u -r1.15 -r1.16 --- lams_common/src/java/org/lamsfoundation/lams/themes/service/ThemeService.java 20 Aug 2009 04:37:19 -0000 1.15 +++ lams_common/src/java/org/lamsfoundation/lams/themes/service/ThemeService.java 20 Aug 2009 23:57:48 -0000 1.16 @@ -300,8 +300,9 @@ */ public Theme getDefaultCSSTheme() { List themes = getAllThemes(); + String defaultTheme = Configuration.get(ConfigurationKeys.DEFAULT_HTML_THEME); for (Theme theme : themes) { - if (theme.getName().equals(Configuration.get(ConfigurationKeys.DEFAULT_HTML_THEME))); { + if (theme.getName().equals(defaultTheme)) { return theme; } } @@ -314,8 +315,9 @@ */ public Theme getDefaultFlashTheme() { List themes = getAllThemes(); + String defaultTheme = Configuration.get(ConfigurationKeys.DEFAULT_FLASH_THEME); for (Theme theme : themes) { - if (theme.getName().equals(Configuration.get(ConfigurationKeys.DEFAULT_FLASH_THEME))); { + if (theme.getName().equals(defaultTheme)) { return theme; } }