cleanup + misc. fixes
[mir.git] / source / mircoders / servlet / ServletModuleUploadedMedia.java
index d40eb37..87c321c 100755 (executable)
@@ -34,14 +34,19 @@ import mir.config.MirPropertiesConfiguration;
 import mir.entity.Entity;
 import mir.entity.adapter.EntityAdapter;
 import mir.entity.adapter.EntityAdapterEngine;
-import mir.log.LoggerWrapper;
 import mir.media.MediaHandler;
-import mir.servlet.ServletModule;
+import mir.servlet.AdminServletModule;
 import mir.servlet.ServletModuleExc;
 import mir.servlet.ServletModuleFailure;
 import mir.servlet.ServletModuleUserExc;
 import mir.session.UploadedFile;
-import mir.util.*;
+import mir.session.CommonsUploadedFileAdapter;
+import mir.util.ExceptionRoutines;
+import mir.util.HTTPParsedRequest;
+import mir.util.HTTPRequestParser;
+import mir.util.JDBCStringRoutines;
+import mir.util.SQLQueryBuilder;
+import mir.util.URLBuilder;
 import mircoders.entity.EntityComment;
 import mircoders.entity.EntityContent;
 import mircoders.entity.EntityUploadedMedia;
@@ -57,30 +62,19 @@ import javax.servlet.ServletOutputStream;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 import java.io.InputStream;
-import java.util.*;
-
-/**
- *
- * <p>Title: </p>
- * <p>Description: </p>
- * <p>Copyright: Copyright (c) 2003</p>
- * <p>Company: </p>
- * @author the mir coders
- * @version 1.0
- */
-
-public abstract class ServletModuleUploadedMedia extends ServletModule {
-  protected String moduleName = "UploadedMedia";
-
-  public static ServletModule getInstance() {
-    return null;
-  }
-
-  public ServletModuleUploadedMedia() {
-    super();
-
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Locale;
+import java.util.Map;
+
+public abstract class ServletModuleUploadedMedia extends AdminServletModule {
+  private String moduleName;
+
+  protected ServletModuleUploadedMedia(String aModuleName) {
+    moduleName = aModuleName;
     definition = "uploadedMedia";
-    logger = new LoggerWrapper("ServletModule.UploadedMedia");
   }
 
   public void insert(HttpServletRequest aRequest, HttpServletResponse aResponse)
@@ -88,9 +82,9 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
 
     try {
       HTTPParsedRequest parsedRequest = new HTTPParsedRequest(aRequest,
-          configuration.getString("Mir.DefaultEncoding"),
-          configuration.getInt("MaxMediaUploadSize")*1024,
-          configuration.getString("TempDir"));
+          getConfiguration().getString("Mir.DefaultEncoding"),
+          getConfiguration().getInt("MaxMediaUploadSize")*1024,
+          getConfiguration().getString("TempDir"));
 
       Map mediaValues = new HashMap();
 
@@ -108,7 +102,7 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
 
       i = parsedRequest.getFiles().iterator();
       while (i.hasNext()) {
-        UploadedFile file = new mir.session.CommonsUploadedFileAdapter((FileItem) i.next());
+        UploadedFile file = new CommonsUploadedFileAdapter((FileItem) i.next());
 
         String suffix = file.getFieldName().substring(5);
         mediaValues.put("title", parsedRequest.getParameter("media_title" + suffix));
@@ -120,19 +114,19 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
       String commentid = parsedRequest.getParameter("commentid");
 
       if (articleid!=null) {
-        EntityContent entContent = (EntityContent) DatabaseContent.getInstance().selectById(articleid);
+        EntityContent entContent = (EntityContent)
+            DatabaseContent.getInstance().selectById(articleid);
 
         i = mediaList.iterator();
 
         while (i.hasNext()) {
-          String id = ((EntityUploadedMedia) i.next()).getId();
+          String id = ((Entity) i.next()).getId();
 
           entContent.attach(id);
           logAdminUsage(aRequest, id, "object attached to article " + articleid);
         }
 
-        (ServletModuleContent.getInstance()).editObject(aRequest, aResponse, articleid);
-
+        ServletHelper.redirect(aResponse, "module=content&do=edit&id=" + articleid);
 
         return;
       }
@@ -150,7 +144,7 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
           logAdminUsage(aRequest, id, "object attached to comment " + commentid);
         }
 
-        (ServletModuleComment.getInstance()).editObject(aRequest, aResponse, commentid);
+        ServletHelper.redirect(aResponse, "module=comment&do=edit&id=" + commentid);
 
         return;
       }
@@ -173,26 +167,29 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
 
     try {
       HTTPParsedRequest parsedRequest = new HTTPParsedRequest(aRequest,
-          configuration.getString("Mir.DefaultEncoding"),
-          configuration.getInt("MaxMediaUploadSize")*1024,
-          configuration.getString("TempDir"));
+          getConfiguration().getString("Mir.DefaultEncoding"),
+          getConfiguration().getInt("MaxMediaUploadSize")*1024,
+          getConfiguration().getString("TempDir"));
+
       Map mediaValues = new HashMap();
 
       Iterator i = mainModule.getStorageObject().getFieldNames().iterator();
       while (i.hasNext()) {
         String field = (String) i.next();
         String value = parsedRequest.getParameter(field);
-        if (value!=null)
+        if (value!=null) {
           mediaValues.put(field, value);
+        }
       }
 
       mediaValues.put("to_publisher", ServletHelper.getUser(aRequest).getId());
       mediaValues.put("is_produced", "0");
-      if (!mediaValues.containsKey("is_published"))
+      if (!mediaValues.containsKey("is_published")) {
         mediaValues.put("is_published", "0");
+      }
 
       String id = mainModule.set(mediaValues);
-      logger.debug("update: media ID = " + id);
+      getLogger().debug("update: media ID = " + id);
       logAdminUsage(aRequest, id, "object modified");
 
       editUploadedMediaObject(id, aRequest, aResponse);
@@ -248,7 +245,7 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
 
     try {
       List list =
-         EntityAdapterEngine.retrieveAdapterList(model, definition, aWhereClause, anOrderByClause, nrEntitiesPerListPage, anOffset);
+         EntityAdapterEngine.retrieveAdapterList(getModel(), definition, aWhereClause, anOrderByClause, nrEntitiesPerListPage, anOffset);
 
       urlBuilder.setValue("module", moduleName);
       urlBuilder.setValue("do", "list");
@@ -337,7 +334,7 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
       Map responseData = ServletHelper.makeGenerationData(aRequest, aResponse, new Locale[] {getLocale(aRequest), getFallbackLocale(aRequest)});
 
       int nrMedia = requestParser.getIntegerWithDefault("nrmedia", 1);
-      int maxNrMedia=configuration.getInt("ServletModule.OpenIndy.MaxMediaUploadItems", 20);
+      int maxNrMedia = getConfiguration().getInt("ServletModule.OpenIndy.MaxMediaUploadItems", 20);
 
       List fields = mainModule.getStorageObject().getFieldNames();
       Map media = new HashMap();
@@ -382,11 +379,11 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
   }
 
   private void editUploadedMediaObject(String idParam, HttpServletRequest aRequest, HttpServletResponse aResponse) throws ServletModuleExc {
-    if (idParam != null && !idParam.equals("")) {
+    if (idParam != null && !"".equals(idParam)) {
       try {
         Map responseData = ServletHelper.makeGenerationData(aRequest, aResponse, new Locale[] {getLocale(aRequest), getFallbackLocale(aRequest)});
         EntityAdapter object =
-            model.makeEntityAdapter(definition, mainModule.getById(idParam));
+            getModel().makeEntityAdapter(definition, mainModule.getById(idParam));
         HTTPRequestParser requestParser = new HTTPRequestParser(aRequest);
         responseData.put("uploadedmedia", object);
         responseData.put("new", Boolean.FALSE);
@@ -442,7 +439,9 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
         throw new ServletModuleFailure(e);
       }
     }
-    else logger.error("id not specified.");
+    else {
+      getLogger().error("id not specified.");
+    }
     // no exception allowed
   }
 
@@ -489,15 +488,18 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
         }
       }
       catch (Throwable e) {
-        logger.error("getIcon: " + e.toString());
+        getLogger().error("getIcon: " + e.toString());
       }
     }
-    else logger.error("getIcon: id not specified.");
+    else {
+      getLogger().error("getIcon: id not specified.");
+    }
   }
 
   protected void addExtraData(Map aTarget) throws ServletModuleExc, ServletModuleFailure {
     try {
-      aTarget.put("mediafolders", EntityAdapterEngine.retrieveAdapterList(model, "mediaFolder", "", "", -1, 0));
+      aTarget.put("mediafolders",
+          EntityAdapterEngine.retrieveAdapterList(getModel(), "mediaFolder", "", "", -1, 0));
     }
     catch (Throwable t) {
       throw new ServletModuleFailure(t);
@@ -509,7 +511,7 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
     if (idParam!=null && !idParam.equals("")) {
       try {
         // TODO rewrite as relational sql
-        ServletModuleContent.getInstance().returnList(
+        ServletHelper.getServletModuleContent().returnList(
             aRequest,
             aResponse,
             "exists (select * from content_x_media where content_id=content.id and media_id='" +
@@ -519,7 +521,9 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
         throw new ServletModuleFailure(t);
       }
     }
-    else logger.error("showarticles: id not specified.");
+    else {
+      getLogger().error("showarticles: id not specified.");
+    }
   }
 
   public void showcomments(HttpServletRequest aRequest, HttpServletResponse aResponse) throws ServletModuleExc
@@ -528,7 +532,7 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
     if (idParam!=null && !idParam.equals("")) {
       try {
         // TODO rewrite as relational sql
-        ServletModuleComment.getInstance().returnList(
+        ServletHelper.getServletModuleComment().returnList(
             aRequest,
             aResponse,
             "exists (select * from comment_x_media where comment_id=comment.id and media_id='" + JDBCStringRoutines.escapeStringLiteral( idParam ) + "')", "", 0);
@@ -537,6 +541,8 @@ public abstract class ServletModuleUploadedMedia extends ServletModule {
         throw new ServletModuleFailure(t);
       }
     }
-    else logger.error("showcomments: id not specified.");
+    else {
+      getLogger().error("showcomments: id not specified.");
+    }
   }
 }
\ No newline at end of file