restructuring producer startpage
[mir.git] / source / mircoders / servlet / ServletModuleOpenIndy.java
index 9c6d501..573cd3b 100755 (executable)
@@ -158,14 +158,15 @@ public class ServletModuleOpenIndy extends ServletModule
     }
     mergeData.put("medianum",numOfMedia);
     mergeData.put("mediafields",mediaFields);
-    mergeData.put("themenPopupData", themenModule.getTopicsAsSimpleList());
     
     
     /** @todo popups missing */
     try{
       mergeData.put("languagePopUpData",DatabaseLanguage.getInstance().getPopupData());
+      mergeData.put("themenPopupData", themenModule.getTopicsAsSimpleList());
     } catch (Exception e) {
-      theLog.printError("languagePopUpData failed");
+      theLog.printError("languagePopUpData or getTopicslist failed");
+      throw new ServletModuleException("smod -- openindy -- getting language or topics failed: "+e.toString());
     }
     deliver(req, res, mergeData, postingFormTemplate);
   }
@@ -180,6 +181,7 @@ public class ServletModuleOpenIndy extends ServletModule
   {
     SimpleHash mergeData = new SimpleHash();
     boolean setMedia=false;
+               boolean setTopic = false;
 
     try {
       WebdbMultipartRequest mp = new WebdbMultipartRequest(req);
@@ -226,7 +228,7 @@ public class ServletModuleOpenIndy extends ServletModule
       theLog.printDebugInfo("id: "+cid);
       //insert was not successfull
       if(cid==null){
-        //How do we know that it was not succesful cause of a 
+        //How do we know that it was not succesful cause of a
         //dupe, what if it failed cause of "No space left on device"?
         //Or is there something I am missing? Wouldn't it be better
         //to have an explicit dupe check and then insert? I have no
@@ -236,12 +238,16 @@ public class ServletModuleOpenIndy extends ServletModule
       }
 
       String[] to_topicsArr = mp.getParameterValues("to_topic");
-      if (to_topicsArr != null && to_topicsArr.length > 0) {
+      
+                       if (to_topicsArr != null && to_topicsArr.length > 0) {
         try{
           DatabaseContentToTopics.getInstance().setTopics(cid,to_topicsArr);
-          theLog.printError("setting content_x_topic success");
+               setTopic = true;
+                                       theLog.printError("setting content_x_topic success");
         } catch (Exception e) {
           theLog.printError("setting content_x_topic failed");
+          contentModule.deleteById(cid);
+          throw new ServletModuleException("smod - openindy :: insposting: setting content_x_topic failed: "+e.toString());
         } //end try
       } //end if
         
@@ -261,20 +267,20 @@ public class ServletModuleOpenIndy extends ServletModule
         //if the client browser sent us unknown (text/plain is default)
         //or if we got application/octet-stream, it's possible that
         //the browser is in error, better check against the file extension
-        if (contentType.equals("text/plain") || 
+        if (contentType.equals("text/plain") ||
             contentType.equals("application/octet-stream")) {
-            /** 
+            /**
              * This is just a temporary way to get the content-type via
              * the .extension , we could maybe use a magic method, by looking
              * at the header (first few bytes) of the file. (like the file(1)
              * command).
-             * The Oreilly method  relies on the content-type that the client 
+             * The Oreilly method  relies on the content-type that the client
              * browser sends and that sometimes is application-octet stream with
              * broken/mis-configured browsers.
-             * 
-             * The map file should be Mir/content-types.properties, it's the 
-             * default Sun Java file with some additional entries that it did 
-             * not have. So if you support a new media type you have to make 
+             *
+             * The map file should be Mir/content-types.properties, it's the
+             * default Sun Java file with some additional entries that it did
+             * not have. So if you support a new media type you have to make
              * sure that it is in this file -mh
              */
             contentType = FileUtil.guessContentTypeFromName(fileName);
@@ -286,7 +292,7 @@ public class ServletModuleOpenIndy extends ServletModule
 
         theLog.printError("CONTENT TYPE IS: "+contentType);
         
-        if (contentType.equals("text/plain") || 
+        if (contentType.equals("text/plain") ||
             contentType.equals("application/octet-stream")) {
           throw new ServletModuleUserException("One or more files of unrecognized types");
         }
@@ -306,7 +312,7 @@ public class ServletModuleOpenIndy extends ServletModule
 
         //the where clause to find the media_type entry
         //from the content-type.
-        //we use the media type entry to lookup the 
+        //we use the media type entry to lookup the
         //media Handler/Storage classes
         String wc = " mime_type='"+contentType+"'";
 
@@ -329,42 +335,38 @@ public class ServletModuleOpenIndy extends ServletModule
           String MediaId;
           Entity mediaEnt = null;
           try {
-                Class mediaStorageClass = Class.forName("mircoders.storage.Database"+mediaStorageName);
-                Method m = mediaStorageClass.getMethod("getInstance", null);
-                Database mediaStorage = (Database)m.invoke(null, null);
-                mediaEnt = (Entity)mediaStorage.getEntityClass().newInstance();
-                mediaEnt.setStorage(mediaStorage);
-                mediaEnt.setValues(mediaValues);
-                mediaId = mediaEnt.insert();
-
-                Class mediaHandlerClass = Class.forName("mir.media.MediaHandler"+mediaHandlerName);
-                MirMedia mediaHandler = (MirMedia)mediaHandlerClass.newInstance();
-                //save and store the media data/metadata
-                mediaHandler.set(mpReq.getMedia(), mediaEnt,mediaTypesList.elementAt(0));
-
-                //were done with mpReq at this point, dereference it.
-                //as it contains mucho mem. -mh 01.10.2001
-                mpReq=null;
-              
-          } catch (Exception e) {
-                theLog.printError("setting uploaded_media failed: "+e.toString());
-          } //end try-catch
+            Class mediaStorageClass = Class.forName("mircoders.storage.Database"+mediaStorageName);
+            Method m = mediaStorageClass.getMethod("getInstance", null);
+            Database mediaStorage = (Database)m.invoke(null, null);
+            mediaEnt = (Entity)mediaStorage.getEntityClass().newInstance();
+            mediaEnt.setStorage(mediaStorage);
+            mediaEnt.setValues(mediaValues);
+            mediaId = mediaEnt.insert();
+
+            Class mediaHandlerClass = Class.forName("mir.media.MediaHandler"+mediaHandlerName);
+            MirMedia mediaHandler = (MirMedia)mediaHandlerClass.newInstance();
+            //save and store the media data/metadata
+            mediaHandler.set(mpReq.getMedia(), mediaEnt,mediaTypesList.elementAt(0));
+
+            //were done with mpReq at this point, dereference it.
+            //as it contains mucho mem. -mh 01.10.2001
+            mpReq=null;
               
-          //we got this far, associate the media to the article
-          try{
-              theLog.printError("ID"+mediaId);
-              DatabaseContentToMedia.getInstance().addMedia(cid,mediaId);
-              mediaEnt.setValueForProperty("is_published","1");
-              mediaEnt.update();
-              new ProducerMedia().handle(null,null,false,false,mediaId);
-              theLog.printError("setting content_x_media success");
+            //we got this far, associate the media to the article
+            theLog.printError("ID"+mediaId);
+            mediaEnt.setValueForProperty("is_published","1");
+            mediaEnt.update();
+            new ProducerMedia().handle(null,null,false,false,mediaId);
+            DatabaseContentToMedia.getInstance().addMedia(cid,mediaId);
           } catch (Exception e) {
-              theLog.printError("setting content_x_media failed");
+            theLog.printError("setting media failed: "+e.toString());
+            contentModule.deleteById(cid);
+            throw new ServletModuleException("setting media failed: "+e.toString());
           }
 
         } else {
           contentModule.deleteById(cid);
-          theLog.printDebugInfo("Wrong file uploaded!: " + fileName);
+          theLog.printDebugInfo("Wrong file type uploaded!: " + fileName);
           throw new ServletModuleUserException("One or more files of unrecognized types");
         } // end if-else mediaTypesList.size() > 0
           
@@ -385,6 +387,10 @@ public class ServletModuleOpenIndy extends ServletModule
       //if direct op producing startpage
       if (directOp.equals("yes")) new ProducerStartPage().handle(null,null);
       
+                       //produce the topicPages if set
+                       //should be more intelligent
+                       //if(setTopic==true) new ProducerTopics().handle(null,null);
+                       
       // sync the server
       //should be configureable
       int exitValue = Helper.rsync();