X-Git-Url: http://erislabs.net/gitweb/?a=blobdiff_plain;f=source%2Fmircoders%2Fservlet%2FServletModuleOpenIndy.java;h=31b99bff38e6ce748bb62a7c68a477be230698f1;hb=9065f22c307993d6dd19be86aafcef915f52f36d;hp=db4e24721ee51c0a10d756d35cdb52fd39388ad6;hpb=52183ff24543440c1a0e5d7377c81879d268124a;p=mir.git

diff --git a/source/mircoders/servlet/ServletModuleOpenIndy.java b/source/mircoders/servlet/ServletModuleOpenIndy.java
index db4e2472..31b99bff 100755
--- a/source/mircoders/servlet/ServletModuleOpenIndy.java
+++ b/source/mircoders/servlet/ServletModuleOpenIndy.java
@@ -294,6 +294,7 @@ public class ServletModuleOpenIndy extends ServletModule
         
         if (contentType.equals("text/plain") ||
             contentType.equals("application/octet-stream")) {
+          System.err.println("IN HERE");
           throw new ServletModuleUserException("One or more files of unrecognized types");
         }
 
@@ -316,37 +317,44 @@ public class ServletModuleOpenIndy extends ServletModule
         //media Handler/Storage classes
         String wc = " mime_type='"+contentType+"'";
 
-        EntityList mediaTypesList = DatabaseMediaType.getInstance().selectByWhereClause(wc);
+        DatabaseMediaType mediaTypeStor = DatabaseMediaType.getInstance();
+        EntityList mediaTypesList = mediaTypeStor.selectByWhereClause(wc);
 
         String mediaTypeId = null;
-        String mediaStorageName = null;
-        String mediaHandlerName = null;
+        MirMedia mediaHandler;
+        Database mediaStorage;
  
         //if we found an entry matching the
         //content-type int the table.
         if (mediaTypesList.size() > 0) {
           //get the class names from the media_type table.
           mediaTypeId = mediaTypesList.elementAt(0).getId();
-          mediaStorageName = mediaTypesList.elementAt(0).getValue("tablename");
-          mediaHandlerName = mediaTypesList.elementAt(0).getValue("classname");
+          try {
+            // ############### TODO: merge these and the getURL call into one
+            // getURL helper call that just takes the Entity as a parameter
+            // along with media_type
+            mediaHandler = MediaHelper.getHandler(mediaTypesList.elementAt(0));
+            mediaStorage = MediaHelper.getStorage(mediaTypesList.elementAt(0),
+                                                "mircoders.storage.Database");
+          } catch (MirMediaException e) {
+            throw new ServletModuleException("getting media handler failed: "
+                                            +e.toString());
+          }
+
           mediaValues.put("to_media_type",mediaTypeId);
          
           //load the classes via reflection
           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));
+            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
@@ -389,7 +397,7 @@ public class ServletModuleOpenIndy extends ServletModule
       
 			//produce the topicPages if set
 			//should be more intelligent
-			if(setTopic==true) new ProducerTopics().handle(null,null);
+			//if(setTopic==true) new ProducerTopics().handle(null,null);
 			
       // sync the server
       //should be configureable