refactored global object intiialization

Author: mdorn

src/proose/libraries/goose.js

@@ -6,6 +6,7 @@ try {
 } catch(error) {
     application.logger.warn("Google Translate Java API not found: Translation feature unavailable.")
 }
+document.execute('register/')
 
 var Goose = Goose || function() {
 	var Public = {
@@ -21,7 +22,7 @@ var Goose = Goose || function() {
                         "text": String(StringEscapeUtils.unescapeHtml(article.getCleanedArticleText()))
                     }
                 } catch(error) {
-                    application.logger.info(error + ": " + uri)
+                    application.logger.debug(error + ": " + uri)
                     return null
                 }
                 if (srclang && tlang) {
@@ -37,33 +38,14 @@ var Goose = Goose || function() {
     		}
 	    }
 	}
-	
-    // Initialize config
-    Public.config = application.globals.get('proose.config')
-    if (!Public.config) {
-        var config = new Configuration()
-        config.setEnableImageFetching(false)
-        Public.config = application.getGlobal('proose.config', config)
+    // Initialize
+    Public.config = register(Configuration, null, {'setEnableImageFetching': false})
+    Public.extractor = register(ContentExtractor, Public.config)
+    try {
+        Public.translate = register(Translate, null, {'setHttpReferrer': application.globals.get('proose.settings.httpReferrer')})
+    } catch(error) {
+        // Google Translate library is optional
     }
-
-    // Initialize extractor
-    Public.extractor = application.globals.get('proose.extractor')
-    if (!Public.extractor && Public.config) {
-        Public.extractor = application.getGlobal('proose.extractor', new ContentExtractor(Public.config))
-    }
-
-    // Initialize translate (optional)
-    Public.translate = application.globals.get('proose.translate')
-    if (!Public.translate) {
-        try {
-            var translate = new Translate()
-            translate.setHttpReferrer(application.globals.get('proose.settings.httpReferrer'))
-            Public.translate = application.getGlobal('proose.translate', translate)
-        }
-        catch (x) {
-            // translate is optional
-        }
-    }    
 	return Public
 }()
 

src/proose/libraries/register.js

@@ -0,0 +1,28 @@
+function register(cls, params, attributes) {
+    namespace = application.globals.get('proose.settings.namespace')
+    // don't have access to cls.class.getName() in JS, hence regex
+    clsname = String(cls).match(/ (.*)]/)[1]
+    globname = namespace + '.' + clsname
+    globinst = application.globals.get(globname)
+    if (!globinst) {
+        if (params) {
+            var instance = new cls(params)
+        } else {
+            var instance = new cls()
+        }
+        var cls = instance.getClass()
+        // can't use java.lang.Class in Rhino, hence no getMethod()
+        var methods = cls.getMethods()
+        var method_names = []
+        for (m in methods) {
+            method_names[m] = String(methods[m].getName())
+        }
+        for (a in attributes) {
+            method_index = method_names.indexOf(a)
+            method = methods[method_index]
+            method.invoke(instance, attributes[a])
+        }
+        globinst = application.getGlobal(globname, instance)
+    }
+    return globinst
+}

src/proose/settings.js

@@ -8,3 +8,4 @@ applicationHomeURL = 'http://github.com/mdorn/proose'
 applicationContactEmail = '[email protected]'
 
 predefinedGlobals['proose.settings.httpReferrer'] = 'http://example.com'
+predefinedGlobals['proose.settings.namespace'] = 'proose'