sproutcore 1.6.0.beta.3 → 1.6.0.rc.1

data/lib/frameworks/sproutcore/frameworks/ajax/tests/system/request.js

@@ -274,3 +274,32 @@ test("Timeouts - Status listener callback", function() {
   }, 500);
 });
 
+test("Test Multiple listeners per single status response", function() {
+
+
+  var numResponses = 0;
+  var response;
+
+  expect(4);
+
+  request.notify(200, this, function(response) {
+    numResponses++;
+    ok(true, "Receieved a response");
+
+    if (numResponses === 2) window.start();
+  });
+
+  request.notify(200, this, function(response) {
+    numResponses++;
+    ok(true, "Receieved a response");
+
+    if (numResponses === 2) window.start();
+  });
+
+  response = request.send();
+  ok(response !== null, 'request.send() should return a response object');
+  ok(response.get('status')<0, 'response should still not have a return code since this should be async');
+
+  stop() ; // stops the test runner - wait for response
+});
+

data/lib/frameworks/sproutcore/frameworks/core_foundation/controllers/array.js

@@ -448,6 +448,8 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
       }
 
       lastContent.removeObserver('status', this, sfunc);
+
+      this.teardownEnumerablePropertyChains(lastContent);
     }
 
     // save new cached values
@@ -480,6 +482,8 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
       // Observer for changes to the status property, in case this is an
       // SC.Record or SC.RecordArray.
       content.addObserver('status', this, sfunc);
+
+      this.setupEnumerablePropertyChains(content);
     } else {
       newlen = SC.none(content) ? 0 : 1;
     }
@@ -487,6 +491,7 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
     // finally, notify enumerable content has changed.
     this._scac_length = newlen;
     this._scac_contentStatusDidChange();
+
     this.arrayContentDidChange(0, 0, newlen);
     this.updateSelectionAfterContentChange();
   }.observes('content'),

data/lib/frameworks/sproutcore/frameworks/core_foundation/controls/button.js

@@ -20,13 +20,18 @@ SC.Button = SC.TemplateView.extend({
     }
   },
 
+  rootResponder: function() {
+    var pane = this.get('pane');
+    return pane.get('rootResponder');
+  }.property('pane').cacheable(),
+
   // Setting isActive to false will remove 'is-active' from our
   // layer's class names.
   mouseUp: function(event) {
     if (this.get('isActive')) {
       var action = this.get('action'),
           target = this.get('target') || null,
-          rootResponder = this.getPath('pane.rootResponder');
+          rootResponder = this.get('rootResponder');
 
       if (action && rootResponder) {
         rootResponder.sendAction(action, target, this, this.get('pane'), null, this);
@@ -46,4 +51,3 @@ SC.Button = SC.TemplateView.extend({
     this.mouseUp(touch);
   }
 });
-

data/lib/frameworks/sproutcore/frameworks/core_foundation/ext/handlebars/bind.js

@@ -54,20 +54,28 @@ sc_require('ext/handlebars');
         previousContext: this
       });
 
+      var observer, invoker;
+
       view.get('childViews').push(bindView);
 
-      // Observe the given property on the context and
-      // tells the SC._BindableSpan to re-render.
-      this.addObserver(property, this, function observer() {
+      observer = function() {
         if (bindView.get('layer')) {
           bindView.rerender();
         } else {
           // If no layer can be found, we can assume somewhere
           // above it has been re-rendered, so remove the
           // observer.
-          this.removeObserver(property, this, observer);
+          this.removeObserver(property, invoker);
         }
-      });
+      };
+
+      invoker = function() {
+        this.invokeOnce(observer);
+      };
+
+      // Observe the given property on the context and
+      // tells the SC._BindableSpan to re-render.
+      this.addObserver(property, invoker);
 
       var context = bindView.renderContext(bindView.get('tagName'));
       bindView.renderToContext(context);
@@ -142,10 +150,9 @@ Handlebars.registerHelper('bindAttr', function(options) {
     var property = attrs[attr];
     var value = this.getPath(property);
 
-    // Add an observer to the view for when the property changes.
-    // When the observer fires, find the element using the
-    // unique data id and update the attribute to the new value.
-    this.addObserver(property, function observer() {
+    var observer, invoker;
+
+    observer = function observer() {
       var result = this.getPath(property);
       var elem = view.$("[data-handlebars-id='" + dataId + "']");
 
@@ -154,7 +161,7 @@ Handlebars.registerHelper('bindAttr', function(options) {
       // In that case, we can assume the template has been re-rendered
       // and we need to clean up the observer.
       if (elem.length === 0) {
-        this.removeObserver(property, observer);
+        this.removeObserver(property, invoker);
         return;
       }
 
@@ -170,7 +177,16 @@ Handlebars.registerHelper('bindAttr', function(options) {
       } else {
         elem.attr(attr, result);
       }
-    });
+    };
+
+    invoker = function() {
+      this.invokeOnce(observer);
+    };
+
+    // Add an observer to the view for when the property changes.
+    // When the observer fires, find the element using the
+    // unique data id and update the attribute to the new value.
+    this.addObserver(property, invoker);
 
     // Use the attribute's name as the value when it is YES
     if (value === YES) {
@@ -246,9 +262,11 @@ SC.Handlebars.bindClasses = function(context, classBindings, view, id) {
     // the property changes.
     var oldClass;
 
+    var observer, invoker;
+
     // Set up an observer on the context. If the property changes, toggle the
     // class name.
-    var observer = function() {
+    observer = function() {
       // Get the current value of the property
       newClass = classStringForProperty(property);
       elem = id ? view.$("[data-handlebars-id='" + id + "']") : view.$();
@@ -256,7 +274,7 @@ SC.Handlebars.bindClasses = function(context, classBindings, view, id) {
       // If we can't find the element anymore, a parent template has been
       // re-rendered and we've been nuked. Remove the observer.
       if (elem.length === 0) {
-        context.removeObserver(property, observer);
+        context.removeObserver(property, invoker);
       } else {
         // If we had previously added a class to the element, remove it.
         if (oldClass) {
@@ -274,7 +292,11 @@ SC.Handlebars.bindClasses = function(context, classBindings, view, id) {
       }
     };
 
-    context.addObserver(property, observer);
+    invoker = function() {
+      this.invokeOnce(observer);
+    };
+
+    context.addObserver(property, invoker);
 
     // We've already setup the observer; now we just need to figure out the correct
     // behavior right now on the first pass through.

data/lib/frameworks/sproutcore/frameworks/core_foundation/ext/handlebars/collection.js

@@ -10,11 +10,11 @@ Handlebars.registerHelper('collection', function(path, options) {
   var collectionClass, collectionObject;
 
   collectionClass = path ? SC.objectForPropertyPath(path) : SC.TemplateCollectionView;
-  //@ if (debug)
+  // @if (debug)
   if (!collectionClass) {
     throw "%@ #collection: Could not find %@".fmt(data.view, path);
   }
-  //@ endif
+  // @endif
 
   var extensions = {};
 
@@ -51,7 +51,7 @@ Handlebars.registerHelper('collection', function(path, options) {
 });
 
 Handlebars.registerHelper('each', function(path, options) {
-  options.hash.contentBinding = SC.Binding.from('*'+path, this);
+  options.hash.contentBinding = SC.Binding.from('*'+path, this).oneWay();
   options.hash.itemContextProperty = 'content';
   return Handlebars.helpers.collection.call(this, null, options);
 });

data/lib/frameworks/sproutcore/frameworks/core_foundation/panes/pane.js

@@ -249,44 +249,43 @@ SC.Pane = SC.View.extend(SC.ResponderContext,
     @param {Event} evt that cause this to become first responder
     @returns {SC.Pane} receiver
   */
-  makeFirstResponder: function(view, evt) {
+  makeFirstResponder: function(original, view, evt) {
+    // firstResponder should never be null
+    if(!view) view = this;
+
     var current=this.get('firstResponder'), isKeyPane=this.get('isKeyPane');
     if (current === view) return this ; // nothing to do
     if (SC.platform.touch && view && view.kindOf(SC.TextFieldView) && !view.get('focused')) return this;
 
-    // notify current of firstResponder change
-    if (current) current.willLoseFirstResponder(current, evt);
-
     // if we are currently key pane, then notify key views of change also
     if (isKeyPane) {
       if (current) { current.tryToPerform('willLoseKeyResponderTo', view); }
       if (view) { view.tryToPerform('willBecomeKeyResponderFrom', current); }
     }
 
-    // change setting
     if (current) {
-      current.beginPropertyChanges()
-        .set('isFirstResponder', NO).set('isKeyResponder', NO)
-      .endPropertyChanges();
+      current.beginPropertyChanges();
+      current.set('isKeyResponder', NO);
     }
 
-    this.set('firstResponder', view) ;
-
     if (view) {
-      view.beginPropertyChanges()
-        .set('isFirstResponder', YES).set('isKeyResponder', isKeyPane)
-      .endPropertyChanges();
+      view.beginPropertyChanges();
+      view.set('isKeyResponder', isKeyPane);
     }
 
+    original(view, evt);
+
+    if(current) current.endPropertyChanges();
+    if(view) view.endPropertyChanges();
+
     // and notify again if needed.
     if (isKeyPane) {
       if (view) { view.tryToPerform('didBecomeKeyResponderFrom', current); }
       if (current) { current.tryToPerform('didLoseKeyResponderTo', view); }
     }
 
-    if (view) view.didBecomeFirstResponder(view);
     return this ;
-  },
+  }.enhance(),
 
   /**
     Called just before the pane loses it's keyPane status.  This will notify

data/lib/frameworks/sproutcore/frameworks/core_foundation/panes/template.js

@@ -16,26 +16,26 @@
     })
   }
 */
-SC.TemplatePane = SC.Object.extend({
-  touchStart: function(touch) {
-    touch.allowDefault();
-  },
-
-  touchesDragged: function(evt, touches) {
-    evt.allowDefault();
-  },
-
-  touchEnd: function(touch) {
-    touch.allowDefault();
-  }
-});
+SC.TemplatePane = SC.Object.extend({});
 
 SC.mixin(SC.TemplatePane, {
   append: function(attrs) {
     var pane = SC.MainPane.extend({
       childViews: ['contentView'],
 
-      contentView: SC.TemplateView.design(attrs)
+      contentView: SC.TemplateView.design(attrs),
+
+      touchStart: function(touch) {
+        touch.allowDefault();
+      },
+
+      touchesDragged: function(evt, touches) {
+        evt.allowDefault();
+      },
+
+      touchEnd: function(touch) {
+        touch.allowDefault();
+      }
     });
 
     return pane.create().append();

data/lib/frameworks/sproutcore/frameworks/core_foundation/system/application.js

@@ -7,8 +7,8 @@
 
 /** @class
 
-  The root object for a SproutCore application.  Usually you will create a
-  single SC.Application instance as your root namespace.  SC.Application is
+  The root object for a SproutCore application. Usually you will create a
+  single SC.Application instance as your root namespace. SC.Application is
   required if you intend to use SC.Responder to route events.
 
   ## Example
@@ -19,11 +19,6 @@
         // add other useful properties here
       });
 
-  ## Sending Events
-
-  You can send actions and events down an application-level responder chain
-  by
-
   @extends SC.ResponderContext
   @since SproutCore 1.0
 */

data/lib/frameworks/sproutcore/frameworks/core_foundation/system/event.js

@@ -76,14 +76,14 @@ SC.Event = function(originalEvent) {
   if (!this.which && this.button) {
     this.which = ((this.button & 1) ? 1 : ((this.button & 2) ? 3 : ( (this.button & 4) ? 2 : 0 ) ));
   }
-  
+
   // Normalize wheel delta values for mousewheel events
   if (this.type === 'mousewheel' || this.type === 'DOMMouseScroll' || this.type === 'MozMousePixelScroll') {
     var deltaMultiplier = SC.Event.MOUSE_WHEEL_MULTIPLIER,
         version = parseFloat(SC.browser.version);
 
     // normalize wheelDelta, wheelDeltaX, & wheelDeltaY for Safari
-    if (SC.browser.webkit && originalEvent.wheelDelta!==undefined) {
+    if (SC.browser.webkit && originalEvent.wheelDelta !== undefined) {
       this.wheelDelta = 0-(originalEvent.wheelDeltaY || originalEvent.wheelDeltaX);
       this.wheelDeltaY = 0-(originalEvent.wheelDeltaY||0);
       this.wheelDeltaX = 0-(originalEvent.wheelDeltaX||0);
@@ -102,7 +102,7 @@ SC.Event = function(originalEvent) {
 
     // handle all other legacy browser
     } else {
-      this.wheelDelta = this.wheelDeltaY = SC.browser.msie ? 0-originalEvent.wheelDelta : originalEvent.wheelDelta ;
+      this.wheelDelta = this.wheelDeltaY = SC.browser.msie || SC.browser.opera ? 0-originalEvent.wheelDelta : originalEvent.wheelDelta ;
       this.wheelDeltaX = 0 ;
     }
 

data/lib/frameworks/sproutcore/frameworks/core_foundation/system/platform.js

@@ -109,9 +109,9 @@ SC.platform = SC.Object.create({
   simulateTouchEvents: function() {
     // Touch events are supported natively, no need for this.
     if (this.touch) {
-      //@ if (debug)
+      // @if (debug)
       SC.Logger.info("Can't simulate touch events in an environment that supports them.");
-      //@ endif
+      // @endif
       return;
     }
     

data/lib/frameworks/sproutcore/frameworks/core_foundation/system/responder.js

@@ -15,7 +15,7 @@
   @extends SC.Object
   @since SproutCore 1.0
 */
-SC.Responder = SC.Object.extend( /** SC.Responder.prototype */ {
+SC.Responder = SC.Object.extend( /** @scope SC.Responder.prototype */ {
 
   isResponder: YES,
   

data/lib/frameworks/sproutcore/frameworks/core_foundation/system/root_responder.js

@@ -18,7 +18,8 @@ SC.CAPTURE_BACKSPACE_KEY = NO ;
   RootResponder directly.  Instead you will work with Pane objects, which
   register themselves with the RootResponder as needed to receive events.
 
-  ## RootResponder and Platforms
+  RootResponder and Platforms
+  ---
 
   RootResponder contains core functionality common among the different web
   platforms. You will likely be working with a subclass of RootResponder that
@@ -27,7 +28,8 @@ SC.CAPTURE_BACKSPACE_KEY = NO ;
   The correct instance of RootResponder is detected at runtime and loaded
   transparently.
 
-  ## Event Types
+  Event Types
+  ---
 
   RootResponders can route four types of events:
 
@@ -46,7 +48,8 @@ SC.CAPTURE_BACKSPACE_KEY = NO ;
      explicit target, or allow the action to traverse the hierarchy until a
      view is found that handles it.
 */
-SC.RootResponder = SC.Object.extend({
+SC.RootResponder = SC.Object.extend(
+  /** @scope SC.RootResponder.prototype */{
 
   /**
     Contains a list of all panes currently visible on screen.  Everytime a
@@ -68,7 +71,7 @@ SC.RootResponder = SC.Object.extend({
   // MAIN PANE
   //
 
-  /** @property
+  /**
     The main pane.  This pane receives shortcuts and actions if the
     focusedPane does not respond to them.  There can be only one main pane.
     You can swap main panes by calling makeMainPane() here.
@@ -76,6 +79,8 @@ SC.RootResponder = SC.Object.extend({
     Usually you will not need to edit the main pane directly.  Instead, you
     should use a MainPane subclass, which will automatically make itself main
     when you append it to the document.
+    
+    @type SC.MainPane
   */
   mainPane: null,
 
@@ -89,7 +94,7 @@ SC.RootResponder = SC.Object.extend({
     document body.  That will be handled by the Pane itself.
 
     @param {SC.Pane} pane
-    @returns {SC.RootResponder} receiver
+    @returns {SC.RootResponder}
   */
   makeMainPane: function(pane) {
     var currentMain = this.get('mainPane') ;
@@ -682,18 +687,19 @@ SC.RootResponder = SC.Object.extend({
     }
     SC.Event.add(document, mousewheel, this, this.mousewheel);
 
-    // If the browser is identifying itself as a touch-enabled browser, but
-    // touch events are not present, assume this is a desktop browser doing
-    // user agent spoofing and simulate touch events automatically.
-    if (SC.browser && SC.platform && SC.browser.mobileSafari && !SC.platform.touch) {
-      SC.platform.simulateTouchEvents();
-    }
-
     // do some initial set
     this.set('currentWindowSize', this.computeWindowSize()) ;
     this.focus(); // assume the window is focused when you load.
 
     if (SC.browser.mobileSafari) {
+
+      // If the browser is identifying itself as a touch-enabled browser, but
+      // touch events are not present, assume this is a desktop browser doing
+      // user agent spoofing and simulate touch events automatically.
+      if (SC.platform && !SC.platform.touch) {
+        SC.platform.simulateTouchEvents();
+      }
+
       // Monkey patch RunLoop if we're in MobileSafari
       var f = SC.RunLoop.prototype.endRunLoop, patch;
 
@@ -1707,6 +1713,7 @@ SC.RootResponder = SC.Object.extend({
   mousedown: function(evt) {
     if (SC.platform.touch) {
       evt.allowDefault();
+      this._lastMouseDownCustomHandling = YES;
       return YES;
     }
 
@@ -1764,6 +1771,7 @@ SC.RootResponder = SC.Object.extend({
   mouseup: function(evt) {
     if (SC.platform.touch) {
       evt.allowDefault();
+      this._lastMouseUpCustomHandling = YES;
       return YES;
     }