Merge pull request #26 from saponifi3d/fix_double_initialization

Updated the view helper to only create an instance of the view

Author: saponifi3d

package.json

@@ -1,6 +1,6 @@
 {
   "name": "rendr-handlebars",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Glue handlebars templates into a Rendr app.",
   "main": "index.js",
   "scripts": {
@@ -15,7 +15,7 @@
     "handlebars": "1.3.0"
   },
   "peerDependencies": {
-    "rendr": ">=0.5.0"
+    "rendr": ">=0.5.1"
   },
   "keywords": [
     "rendr",

shared/helpers.js

@@ -17,28 +17,49 @@ module.exports = function(Handlebars, getTemplate) {
       }
       viewOptions = options.hash || {};
 
-      // Pass through a reference to the app.
       var app = getProperty('_app', this, options);
+      // Pass through a reference to the app.
       if (app) {
         viewOptions.app = app;
         viewName = app.modelUtils.underscorize(viewName);
       } else{
         throw new Error("An App instance is required when rendering a view, it could not be extracted from the options.")
       }
 
-      // Pass through a reference to the parent view.
-      var parentView = getProperty('_view', this, options);
-      if (parentView) {
-        viewOptions.parentView = parentView;
+      if (isServer) {
+        // Pass through a reference to the parent view.
+        var parentView = getProperty('_view', this, options);
+        if (parentView) {
+          viewOptions.parentView = parentView;
+        }
+
+        // get the Backbone.View based on viewName
+        ViewClass = BaseView.getView(viewName, app.options.entryPath);
+        view = new ViewClass(viewOptions);
+
+        // create the outerHTML using className, tagName
+        html = view.getHtml();
+        return new Handlebars.SafeString(html);
       }
 
-      // get the Backbone.View based on viewName
-      ViewClass = BaseView.getView(viewName, app.options.entryPath);
-      view = new ViewClass(viewOptions);
+      // only create an element to pass attributes to a single point for view instantiation for client-side
 
-      // create the outerHTML using className, tagName
-      html = view.getHtml();
-      return new Handlebars.SafeString(html);
+      // Builds a fetch_summary attribute
+      viewOptions = BaseView.parseModelAndCollection(app.modelUtils, viewOptions);
+      var fetchSummary = BaseView.extractFetchSummary(app.modelUtils, viewOptions);
+      fetchSummary = JSON.stringify(fetchSummary)
+
+      viewOptions['fetch_summary'] = fetchSummary
+      viewOptions = _.omit(viewOptions, ['model', 'collection', 'app'])
+
+      // create a list of data attributes
+      var attrString = _.inject(viewOptions, function(memo, value, key) {
+        return memo += " data-" + key + "=\"" + _.escape(value) + "\"";
+      }, '');
+
+      return new Handlebars.SafeString(
+        '<div data-render="true" ' + attrString +' data-view="'+ viewName +'"></div>'
+      )
     },
 
     partial: function(templateName, options) {