Index: lams_admin/conf/language/lams/ApplicationResources.properties =================================================================== diff -u -rd0de65cc054fe3d424e7d58d4ce75826d14ad5a6 -r0484f09a01d9b6b6d0c8ef375206c651c6694dc1 --- lams_admin/conf/language/lams/ApplicationResources.properties (.../ApplicationResources.properties) (revision d0de65cc054fe3d424e7d58d4ce75826d14ad5a6) +++ lams_admin/conf/language/lams/ApplicationResources.properties (.../ApplicationResources.properties) (revision 0484f09a01d9b6b6d0c8ef375206c651c6694dc1) @@ -155,8 +155,7 @@ sysadmin.server.add =Add new server sysadmin.organisation.create =Create one sysadmin.organisation.select =Please select... -sysadmin.login.logo =Upload a new logo. Use a height of 90 pixels and width between 180 and 340 pixels for best effect: -sysadmin.login.text =Update the text under the logo on the login page: +sysadmin.login.text =Update the page text: sysadmin.headline =System Administration sysadmin.config.settings.edit =Edit configuration settings sysadmin.batch.temp.file.delete =Delete old temporary files Index: lams_admin/conf/language/lams/ApplicationResources_en_AU.properties =================================================================== diff -u -r60e44b19b8de02a00faa437fba8117928baa3d73 -r0484f09a01d9b6b6d0c8ef375206c651c6694dc1 --- lams_admin/conf/language/lams/ApplicationResources_en_AU.properties (.../ApplicationResources_en_AU.properties) (revision 60e44b19b8de02a00faa437fba8117928baa3d73) +++ lams_admin/conf/language/lams/ApplicationResources_en_AU.properties (.../ApplicationResources_en_AU.properties) (revision 0484f09a01d9b6b6d0c8ef375206c651c6694dc1) @@ -154,8 +154,7 @@ sysadmin.server.add =Add new server sysadmin.organisation.create =Create one sysadmin.organisation.select =Please select... -sysadmin.login.logo =Upload a new logo. Use a height of 90 pixels and width between 180 and 340 pixels for best effect: -sysadmin.login.text =Update the text under the logo on the login page: +sysadmin.login.text =Update the page text: sysadmin.headline =System Administration sysadmin.config.settings.edit =Edit configuration settings sysadmin.batch.temp.file.delete =Delete old temporary files Index: lams_admin/src/java/org/lamsfoundation/lams/admin/web/LoginSaveAction.java =================================================================== diff -u -rc92ea51a973dc03ece970c057dd9369eda6a8c24 -r0484f09a01d9b6b6d0c8ef375206c651c6694dc1 --- lams_admin/src/java/org/lamsfoundation/lams/admin/web/LoginSaveAction.java (.../LoginSaveAction.java) (revision c92ea51a973dc03ece970c057dd9369eda6a8c24) +++ lams_admin/src/java/org/lamsfoundation/lams/admin/web/LoginSaveAction.java (.../LoginSaveAction.java) (revision 0484f09a01d9b6b6d0c8ef375206c651c6694dc1) @@ -26,7 +26,6 @@ import java.io.BufferedWriter; import java.io.File; import java.io.FileOutputStream; -import java.io.IOException; import java.io.OutputStreamWriter; import java.nio.charset.Charset; @@ -36,44 +35,21 @@ import org.apache.struts.action.ActionForm; import org.apache.struts.action.ActionForward; import org.apache.struts.action.ActionMapping; -import org.apache.struts.action.ActionMessage; -import org.apache.struts.action.ActionMessages; import org.apache.struts.action.DynaActionForm; -import org.apache.struts.upload.FormFile; import org.lamsfoundation.lams.util.Configuration; import org.lamsfoundation.lams.util.ConfigurationKeys; import org.lamsfoundation.lams.web.action.LamsDispatchAction; /** - *
- * View Source - *
- * * Use DispatchAction for future extension convenience, e.g. add preview feature * * @author Fei Yang */ - -/** - * struts doclet - * - * - * "method" scope = "request" input = ".loginmaintian" validate = - * "false" - * - * - * - */ public class LoginSaveAction extends LamsDispatchAction { - private static final String IMAGE_FOLDER_SUFFIX = File.separatorChar + "lams-www.war" + File.separatorChar - + "images"; - private static final String NEWS_PAGE_PATH_SUFFIX = File.separatorChar + "lams-www.war" + File.separatorChar + "news.html"; - private static final String LOGO_FILENAME = "lams_login.gif"; - public ActionForward save(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) throws Exception { @@ -82,68 +58,21 @@ } DynaActionForm loginMaintainForm = (DynaActionForm) form; - ActionMessages errors = new ActionMessages(); - FormFile file = (FormFile) loginMaintainForm.get("logo"); - if ((file != null) && (file.getFileSize() != 0)) { - checkFile(errors, file); - } - if (errors.isEmpty()) { - if ((file != null) && (file.getFileSize() != 0)) { - updateImageFile(file, LOGO_FILENAME); - - } - updateNewsPage(loginMaintainForm.getString("news")); - } else { - saveErrors(request, errors); - return mapping.findForward("loginmaintain"); - } - return mapping.findForward("sysadmin"); - } - - private void updateNewsPage(String news) throws IOException { BufferedWriter bWriter = null; try { - OutputStreamWriter ow = new OutputStreamWriter( - new FileOutputStream(Configuration.get(ConfigurationKeys.LAMS_EAR_DIR) + NEWS_PAGE_PATH_SUFFIX), + OutputStreamWriter ow = new OutputStreamWriter(new FileOutputStream( + Configuration.get(ConfigurationKeys.LAMS_EAR_DIR) + NEWS_PAGE_PATH_SUFFIX), Charset.forName("UTF-8")); bWriter = new BufferedWriter(ow); - bWriter.write(news); + bWriter.write(loginMaintainForm.getString("news")); bWriter.flush(); } finally { if (bWriter != null) { bWriter.close(); } } - } - private void updateImageFile(FormFile file, String fileName) throws IOException { - File imagesFolder = new File(Configuration.get(ConfigurationKeys.LAMS_EAR_DIR) + IMAGE_FOLDER_SUFFIX); - if (!imagesFolder.exists()) { - imagesFolder.mkdir(); - } - String imageFilePath = Configuration.get(ConfigurationKeys.LAMS_EAR_DIR) + IMAGE_FOLDER_SUFFIX - + File.separatorChar + fileName; - FileOutputStream out = null; - try { - out = new FileOutputStream(imageFilePath); - out.write(file.getFileData()); - out.flush(); - } finally { - if (out != null) { - out.close(); - } - } - + return mapping.findForward("sysadmin"); } - private void checkFile(ActionMessages errors, FormFile file) { - boolean imgFormat = file.getContentType().contains("image"); - if (!imgFormat) { - errors.add("format", new ActionMessage("error.img.format")); - } - if (file.getFileSize() > 4096 * 1024) { - errors.add("size", new ActionMessage("error.img.size")); - } - } - } Index: lams_admin/web/WEB-INF/struts-config.xml =================================================================== diff -u -r60e44b19b8de02a00faa437fba8117928baa3d73 -r0484f09a01d9b6b6d0c8ef375206c651c6694dc1 --- lams_admin/web/WEB-INF/struts-config.xml (.../struts-config.xml) (revision 60e44b19b8de02a00faa437fba8117928baa3d73) +++ lams_admin/web/WEB-INF/struts-config.xml (.../struts-config.xml) (revision 0484f09a01d9b6b6d0c8ef375206c651c6694dc1) @@ -94,7 +94,6 @@