Update Documentation page, improve function toJSONAttribute and Base Model URL function

Author: enzolutions

README.md

@@ -23,6 +23,21 @@ I strong recommend to use the Drupal module <a href="https://drupal.org/project/
 
 **Models**: Created Backbone models for Nodes, Users, Comments Entities
 
+You can add extra fields to modules to use as extra information in you application, these extra fields could be mark as *noSaveAttributes*. Check the following example.
+
+````
+var Property = Backbone.Drupal.Models.Node.extend({
+      initialize : function(options) {
+        // Setting the Id Attribute for Drupal model
+        this.attributes.nid = options.property_id;
+        this.noSaveAttributes = ['property_id'];
+
+        // Extended Backbone.Drupal.Models.Node to my own service for Drupal Nodes.
+        // This Rest service return absolute URL for field pictures
+        this.urlSource = "node_waterbed";
+       },
+````
+
 **Collections**: Created Backbone collection for Users, Nodes and Views
 
 **REST**: Integration with Services Server type REST
@@ -61,31 +76,32 @@ I strong recommend to use the Drupal module <a href="https://drupal.org/project/
         var Auth = new Backbone.Drupal.Auth({crossDomain: true});
         // Request executed in sync mode
         // If status is token further ajax will use the proper token
-        var status = Auth.login('admin', 'admin');
-
-        console.log(status);
-
-        /*
-          Check user retrieve
-        */
-
-        var User = new Backbone.Drupal.Models.User({uid: 1});
-        User.fetch({
-          success: function (user) {
-            // Check information retrived, could be used directly in a template
-            console.log(user.attributes.mail);
-          }
-        });
-        /*
-          Check users retrive
-        */
-        var Users = new Backbone.Drupal.Collections.UserIndex();
-        Users.fetch({
-          success: function (users) {
-            // Check information retrived, could be used directly in a template
-            console.log(users.models[0].attributes.uri);
-          }
-        });
+        var auth_status =  = Auth.login('admin', 'admin');
+
+        if(auth_status) {
+
+          // Check user retrieve
+
+          var User = new Backbone.Drupal.Models.User({uid: 1});
+          User.fetch({
+            success: function (user) {
+              // Check information retrived, could be used directly in a template
+              console.log(user.attributes.mail);
+            }
+          });
+
+          //  Check users retrive
+
+          var Users = new Backbone.Drupal.Collections.UserIndex();
+          Users.fetch({
+            success: function (users) {
+              // Check information retrived, could be used directly in a template
+              console.log(users.models[0].attributes.uri);
+            }
+          });
+        } else {
+          alert('Auth Error');
+        }
       });
 
     </script>

backbone.drupal.js

@@ -61,9 +61,6 @@ Backbone.Drupal.Auth = (function(Backbone, $, _){
           //dataType : 'json',
           error: function(XMLHttpRequest, textStatus, errorThrown) {
               return false;
-              /*console.log(JSON.stringify(XMLHttpRequest));
-              console.log(JSON.stringify(textStatus));
-              console.log(JSON.stringify(errorThrown));*/
             },
           success : function(data) {
 
@@ -181,8 +178,10 @@ Backbone.Drupal.Models.Base = Backbone.Model.extend({
 
   // #### toJSONAttribute: process attribute into JSON if process funciton given.
   toJSONAttribute: function(attributeValue, attributeName) {
-    if (this.toJSONProcessors[attributeName]) {
-      attributeValue = this.toJSONProcessors[attributeName].call(this, attributeValue);
+    if (typeof this.toJSONProcessors !== 'undefined') {
+      if (this.toJSONProcessors[attributeName]) {
+        attributeValue = this.toJSONProcessors[attributeName].call(this, attributeValue);
+      }
     }
     return attributeValue;
   },
@@ -196,9 +195,10 @@ Backbone.Drupal.Models.Base = Backbone.Model.extend({
     // Modified from Backbone.js to ignore collection and add ".format" extension.
     var restEndpoint = Backbone.Drupal.restEndpoint.root + (Backbone.Drupal.restEndpoint.root.charAt(Backbone.Drupal.restEndpoint.root.length - 1) === '/' ? '' : '/');
     var base = restEndpoint + this.urlSource;
+
     if (this.isNew()) { return base; }
     // Add .json for format here.
-    return base + (base.charAt(base.length - 1) === '/' ? '' : '/') + encodeURIComponent(this.id) + Backbone.Drupal.restEndpoint.dataType;
+    return base + (base.charAt(base.length - 1) === '/' ? '' : '/') + encodeURIComponent(this.get(this.idAttribute)) + Backbone.Drupal.restEndpoint.dataType;
   },
 
   // TODO: evaluate moving all of this to Views.Base

backbone.drupal.services.js

@@ -118,7 +118,13 @@ Backbone.Drupal.Collections.NodeView = Backbone.Drupal.Collections.Base.extend({
   },
   url: function() {
       var restEndpoint = Backbone.Drupal.restEndpoint.root + (Backbone.Drupal.restEndpoint.root.charAt(Backbone.Drupal.restEndpoint.root.length - 1) === '/' ? '' : '/');
-      return restEndpoint + "views/" + this.viewName  + Backbone.Drupal.restEndpoint.dataType;
+
+      return restEndpoint + "views/" + this.viewName  + Backbone.Drupal.restEndpoint.dataType + (typeof(this.filters) !== "undefined"? this.filters: '');
+  },
+  setFilters: function (filters) {
+    if(filters !== '') {
+      this.filters = filters;
+    }
   }
 });