svn commit: r561427 - in /roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business: GuiceWebloggerProvider.java WebloggerFactory.java WebloggerProvider.java

[snoopdave]

Author: snoopdave
Date: Tue Jul 31 11:34:18 2007
New Revision: 561427

URL: http://svn.apache.org/viewvc?view=rev&rev=561427
Log:
Allen's right (again, grrr...) , we've already got enough pluggabilty for 
custom managers and it's better to encourage folks to extend Weblogger instead 
of injecting arbitrary objects.

Modified:
    
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/GuiceWebloggerProvider.java
    
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/WebloggerFactory.java
    
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/WebloggerProvider.java

Modified: 
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/GuiceWebloggerProvider.java
URL: 
http://svn.apache.org/viewvc/roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/GuiceWebloggerProvider.java?view=diff&rev=561427&r1=561426&r2=561427
==============================================================================
--- 
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/GuiceWebloggerProvider.java
 (original)
+++ 
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/GuiceWebloggerProvider.java
 Tue Jul 31 11:34:18 2007
@@ -95,10 +95,4 @@
         return webloggerInstance;
     }
     
-    /**
-     * Get injected instance, for custom managers.
-     */
-    public <T> T getInstance(Class<T> type) {
-        return injector.getInstance(type);
-    }   
 }

Modified: 
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/WebloggerFactory.java
URL: 
http://svn.apache.org/viewvc/roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/WebloggerFactory.java?view=diff&rev=561427&r1=561426&r2=561427
==============================================================================
--- 
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/WebloggerFactory.java
 (original)
+++ 
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/WebloggerFactory.java
 Tue Jul 31 11:34:18 2007
@@ -65,18 +65,6 @@
     
     
     /**
-     * Get injected instance, for custom managers.
-     */
-    public <T> T getInstance(Class<T> type) {
-        if (webloggerProvider == null) {
-            throw new IllegalStateException("Roller Weblogger has not been 
bootstrapped yet");
-        }
-        
-        return webloggerProvider.getInstance(type);
-    }
-    
-    
-    /**
      * Bootstrap the Roller Weblogger business tier, uses default 
WebloggerProvider.
      *
      * Bootstrapping the application effectively instantiates all the necessary

Modified: 
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/WebloggerProvider.java
URL: 
http://svn.apache.org/viewvc/roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/WebloggerProvider.java?view=diff&rev=561427&r1=561426&r2=561427
==============================================================================
--- 
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/WebloggerProvider.java
 (original)
+++ 
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/WebloggerProvider.java
 Tue Jul 31 11:34:18 2007
@@ -35,9 +35,4 @@
      */
     public Weblogger getWeblogger();
     
-    
-    /**
-     * Get injected instance, for custom managers.
-     */
-    public <T> T getInstance(Class<T> type);
 }