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.10 -r1.11 --- lams_common/src/java/org/lamsfoundation/lams/themes/service/ThemeService.java 17 Sep 2006 06:14:25 -0000 1.10 +++ lams_common/src/java/org/lamsfoundation/lams/themes/service/ThemeService.java 17 Jan 2007 01:16:51 -0000 1.11 @@ -58,10 +58,6 @@ protected MessageService messageService; protected IUserManagementService userManagementService; - /** for sending acknowledgment/error messages back to flash */ - private FlashMessage flashMessage; - - public ThemeService() { } @@ -123,6 +119,7 @@ */ public String storeTheme(String wddxPacket) throws Exception { + FlashMessage flashMessage= null; Hashtable table = (Hashtable)WDDXProcessor.deserialize(wddxPacket); CSSThemeDTO themeDTO = new CSSThemeDTO(table); @@ -157,6 +154,7 @@ * @throws Exception */ public String getTheme(Long themeId)throws IOException { + FlashMessage flashMessage= null; CSSThemeVisualElement theme = themeDAO.getThemeById(themeId); if(theme==null) flashMessage = FlashMessage.getNoSuchTheme("wddxPacket",themeId); @@ -177,6 +175,7 @@ * @throws IOException */ public String getThemes() throws IOException { + FlashMessage flashMessage= null; List themes = themeDAO.getAllThemes(); ArrayList themeList = new ArrayList(); Iterator iterator = themes.iterator(); @@ -195,6 +194,7 @@ * @throws IOException */ private FlashMessage setTheme(Integer userId, Long themeId, String type) throws IOException, ThemeException, UserException { + FlashMessage flashMessage= null; User user = (User)userManagementService.findById(User.class,userId); CSSThemeVisualElement theme = themeDAO.getThemeById(themeId); Index: lams_common/src/java/org/lamsfoundation/lams/util/wddx/FlashMessage.java =================================================================== RCS file: /usr/local/cvsroot/lams_common/src/java/org/lamsfoundation/lams/util/wddx/FlashMessage.java,v diff -u -r1.14 -r1.15 --- lams_common/src/java/org/lamsfoundation/lams/util/wddx/FlashMessage.java 17 Sep 2006 06:14:17 -0000 1.14 +++ lams_common/src/java/org/lamsfoundation/lams/util/wddx/FlashMessage.java 17 Jan 2007 01:16:51 -0000 1.15 @@ -29,9 +29,11 @@ /** * @author Manpreet Minhas * This class represents the message sent by the server to the - * Flash client. - * - * TODO Flash messages require internationalisation. + * Flash client. The error messages in this class should only + * be generic error messages that may be used in various modules. + * The errors are not I18N'd as we assume Flash will not show the + * error to the user - but the error is included in the Flash dump + * file so it should still be meaningful. */ public class FlashMessage implements Serializable { @@ -116,7 +118,7 @@ } return wddxPacket; } - public static FlashMessage getNoSuchUserExists(String methodName, Integer userID){ + public static FlashMessage getNoSuchUserExists(String methodName, Integer userID){ return new FlashMessage(methodName, "No such User with a user_id of :" + userID + " exists", ERROR);