sproutcore 1.5.0.rc.1 → 1.5.0.rc.2

data/CHANGELOG

@@ -1,3 +1,7 @@
+*SproutCore 1.5.0.rc.2 (April 11, 2011)*
+
+* Fixed a compatibility issue with Windows environments
+
 *SproutCore 1.5.0.rc.1 (April 1, 2011)*
 
 * Made it possible to share index.html across multiple languages vs. duplicating it for each.

data/VERSION.yml

@@ -1,4 +1,4 @@
 --- 
 :major: 1
 :minor: 5
-:patch: 0.rc.1
+:patch: 0.rc.2

data/lib/frameworks/sproutcore/Buildfile

@@ -27,12 +27,14 @@ end
 # CORE FRAMEWORKS
 config :bootstrap,  :required => [], :use_modules => false
 
-config :jquery,          :required => []
+config :jquery,          :required => [], :test_required => [], :debug_required => []
 config :handlebars,      :required => []
 config :runtime,         :required => [:jquery]
-config :datetime,        :required => [:runtime]
+config :'datetime/core', :required => [:runtime]
+config :datetime,        :required => [:'datetime/core']
 config :core_foundation, :required => [:runtime, :handlebars]
-config :foundation,      :required => [:core_foundation, :datetime]
+config :'datetime/localized', :required => [:core_foundation]
+config :foundation,      :required => [:core_foundation, :datetime, :'datetime/localized']
 config :datastore,       :required => [:runtime, :datetime]
 config :desktop,         :required => [:foundation]
 config :media,           :required => [:desktop]
@@ -54,9 +56,12 @@ config :mini, :required => [:runtime, :datastore]
 config :animation, :required => :foundation
 config :forms, :required => :desktop
 
+config :qunit, :required => []
+config :testing, :required => [:jquery], :test_required => [], :debug_required => []
+
 # SPECIAL FRAMEWORKS AND THEMES
 # These do not require any of the built-in SproutCore frameworks
-%w(testing debug legacy_theme empty_theme).each do |target_name|
+%w(debug legacy_theme empty_theme).each do |target_name|
   config target_name, 
     :required => [], :test_required => [], :debug_required => []
 end

data/lib/frameworks/sproutcore/README.md

@@ -33,6 +33,7 @@ code and ideas:
 - Erich Ocean
 - Evin Grano
 - Geoffrey Donaldson
+- James A. Rosen
 - James Austin
 - Jason Ketterman
 - Jonathan Lewis

data/lib/frameworks/sproutcore/frameworks/animation/core.js

@@ -8,31 +8,29 @@
 /*globals */
 
 /** @namespace
-A simple mixin called Animatable is provided. What does it do?
-It makes CSS transitions for you, and if they aren't available,
-implements them in JavaScript.
-
-Animatable things:
-- layout. You can animate any layout property, even centerX and centerY
-- opacity.
-- display, in a way. All animating display does is delay setting display:none
-  until <em>after</em> the transition duration has passed. This allows you
-  to set display:none after fading out. If mixing with CSS transitions, you will
-  need to set the delay a tad longer to accomodate any delays in beginning the
-  transition.
-
-@class SC.Animatable
-@example Example Usage:
-{{{
-aView: SC.LabelView.design(SC.Animatable, {
-  transitions: {
-    left: {duration: .25},
-    top: .25, // only possible during design; otherwise you must use long form.
-    width: {duration: .25, timing: SC.Animatable.TRANSITION_EASE_IN_OUT }
-  }
-})
-}}}
-@extends SC.Object
+  A simple mixin called Animatable is provided. What does it do?
+  It makes CSS transitions for you, and if they aren't available,
+  implements them in JavaScript.
+
+  ## Animatable things:
+
+    - layout. You can animate any layout property, even centerX and centerY
+    - opacity.
+    - display, in a way. All animating display does is delay setting display:none
+      until <em>after</em> the transition duration has passed. This allows you
+      to set display:none after fading out. If mixing with CSS transitions, you will
+      need to set the delay a tad longer to accomodate any delays in beginning the
+      transition.
+
+  ## Example Usage:
+
+      aView: SC.LabelView.design(SC.Animatable, {
+        transitions: {
+          left: {duration: .25},
+          top: .25, // only possible during design; otherwise you must use long form.
+          width: {duration: .25, timing: SC.Animatable.TRANSITION_EASE_IN_OUT }
+        }
+      })
 */
 SC.Animatable = {
   /** @scope SC.Animatable.prototype */

data/lib/frameworks/sproutcore/frameworks/bootstrap/system/browser.js

@@ -17,49 +17,209 @@ SC._detectBrowser = function(userAgent, language) {
   version = browser.version = (userAgent.match( /.*(?:rv|chrome|webkit|opera|ie)[\/: ](.+?)([ \);]|$)/ ) || [])[1];
   webkitVersion = (userAgent.match( /webkit\/(.+?) / ) || [])[1];
 
+  /**
+    @name SC.browser.isWindows
+    @type Boolean
+  */
   browser.windows = browser.isWindows = !!/windows/.test(userAgent);
+  
+  /**
+    @name SC.browser.isMac
+    @type Boolean
+  */
   browser.mac = browser.isMac = !!/macintosh/.test(userAgent) || (/mac os x/.test(userAgent) && !/like mac os x/.test(userAgent));
+  
+  /**
+    @name SC.browser.isiPhone
+    @type Boolean
+  */
   browser.iPhone = browser.isiPhone = !!/iphone/.test(userAgent);
+  
+  /**
+    @name SC.browser.isiPod
+    @type Boolean
+  */
   browser.iPod = browser.isiPod = !!/ipod/.test(userAgent);
+  
+  /**
+    @name SC.browser.isiPad
+    @type Boolean
+  */
   browser.iPad = browser.isiPad = !!/ipad/.test(userAgent);
+  
+  /**
+    @name SC.browser.isiOS
+    @type Boolean
+  */
   browser.iOS = browser.isiOS = browser.iPhone || browser.iPod || browser.iPad;
+  
+  /**
+    @name SC.browser.isAndroid
+    @type Boolean
+  */
   browser.android = browser.isAndroid = !!/android/.test(userAgent);
 
+  /**
+    @name SC.browser.opera
+    @type String
+  */
   browser.opera = /opera/.test(userAgent) ? version : 0;
+  
+  /**
+    @name SC.browser.isOpera
+    @type Boolean
+  */
   browser.isOpera = !!browser.opera;
 
+  /**
+    @name SC.browser.msie
+    @type String
+  */
   browser.msie = /msie/.test(userAgent) && !browser.opera ? version : 0;
+  
+  /**
+    @name SC.browser.isIE
+    @type Boolean
+  */
   browser.isIE = !!browser.msie;
+  
+  /**
+    @name SC.browser.isIE8OrLower
+    @type Boolean
+  */
   browser.isIE8OrLower = !!(browser.msie && parseInt(browser.msie, 10) <= 8);
 
+  /**
+    @name SC.browser.mozilla
+    @type String
+  */
   browser.mozilla = /mozilla/.test(userAgent) && !/(compatible|webkit|msie)/.test(userAgent) ? version : 0;
+  
+  /**
+    @name SC.browser.isMozilla
+    @type Boolean
+  */
   browser.isMozilla = !!browser.mozilla;
 
+  /**
+    @name SC.browser.webkit
+    @type String
+  */
   browser.webkit = /webkit/.test(userAgent) ? webkitVersion : 0;
+  
+  /**
+    @name SC.browser.isWebkit
+    @type Boolean
+  */
   browser.isWebkit = !!browser.webkit;
 
+  /**
+    @name SC.browser.chrome
+    @type String
+  */
   browser.chrome = /chrome/.test(userAgent) ? version: 0;
+  
+  /**
+    @name SC.browser.isChrome
+    @type Boolean
+  */
   browser.isChrome = !!browser.chrome;
 
+  /**
+    @name SC.browser.mobileSafari
+    @type String
+  */
   browser.mobileSafari = /apple.*mobile.*safari/.test(userAgent) && browser.iOS ? webkitVersion : 0;
+  
+  /**
+    @name SC.browser.isMobileSafari
+    @type Boolean
+  */
   browser.isMobileSafari = !!browser.mobileSafari;
 
+  /**
+    @name SC.browser.iPadSafari
+    @type String
+  */
   browser.iPadSafari = browser.iPad && browser.isMobileSafari ? webkitVersion : 0;
+  
+  /**
+    @name SC.browser.isiPadSafari
+    @type Boolean
+  */
   browser.isiPadSafari = !!browser.iPadSafari;
 
+  /**
+    @name SC.browser.iPhoneSafari
+    @type String
+  */
   browser.iPhoneSafari = browser.iPhone && browser.isMobileSafari ? webkitVersion : 0;
+  
+  /**
+    @name SC.browser.isiPhoneSafari
+    @type Boolean
+  */
   browser.isiPhoneSafari = !!browser.iphoneSafari;
 
+  /**
+    @name SC.browser.iPodSafari
+    @type String
+  */
   browser.iPodSafari = browser.iPod && browser.isMobileSafari ? webkitVersion : 0;
+  
+  /**
+    @name SC.browser.isiPodSafari
+    @type Boolean
+  */
   browser.isiPodSafari = !!browser.iPodSafari;
 
+  /**
+    @name SC.browser.safari
+    @type String
+  */
   browser.safari = browser.webkit && !browser.chrome && !browser.iOS && !browser.android ? webkitVersion : 0;
+  
+  /**
+    @name SC.browser.isSafari
+    @type Boolean
+  */
   browser.isSafari = !!browser.safari;
 
+  /**
+    @name SC.browser.language
+    @type String
+  */
   browser.language = language.split('-', 1)[0];
   
+  /**
+    Possible values:
+    
+      - 'msie'
+      - 'mozilla'
+      - 'chrome'
+      - 'safari'
+      - 'opera'
+      - 'mobile-safari'
+      - 'unknown'
+
+    @name SC.browser.current
+    @type String
+    @default 'unknown'
+  */
   browser.current = browser.msie ? 'msie' : browser.mozilla ? 'mozilla' : browser.chrome ? 'chrome' : browser.safari ? 'safari' : browser.opera ? 'opera' : browser.mobileSafari ? 'mobile-safari' : 'unknown';
   return browser;
 };
 
+
+/** @class
+  
+  Contains information about the browser environment that SproutCore
+  is running in. String properties, such as `SC.browser.webkit` or
+  `SC.browser.msie`, will have a value that represents the browser build
+  number if that browser is being used. Otherwise, they will have a
+  falsey value. For convenience, Boolean counterparts for all of the
+  versioned properties are provided.
+  
+  @since SproutCore 1.0
+*/
 SC.browser = SC._detectBrowser();

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

@@ -293,7 +293,8 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
       }
     }
 
-    if (content) content.replace(start, amt, objects);
+    if (content) { content.replace(start, amt, objects); }
+
     for(i=0, objsLen = objsToDestroy.length; i<objsLen; i++){
 
       objsToDestroy[i].destroy();
@@ -392,6 +393,27 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
     return (this._scac_cached = content.toArray().sort(func)) ;
   },
 
+  propertyWillChange: function(key) {
+    if (key === 'content') {
+      this.arrayContentWillChange(0, this.get('length'), 0);
+    } else {
+      return sc_super();
+    }
+  },
+
+  _scac_arrayContentWillChange: function(start, removed, added) {
+    this.arrayContentWillChange(start, removed, added);
+    var removedObjects = this.slice(start, start+removed);
+    this.teardownEnumerablePropertyChains(removedObjects);
+  },
+
+  _scac_arrayContentDidChange: function(start, removed, added) {
+    this.arrayContentDidChange(start, removed, added);
+    var addedObjects = this.slice(start, start+added);
+    this.setupEnumerablePropertyChains(addedObjects);
+    this.updateSelectionAfterContentChange();
+  },
+
   /** @private
     Whenever content changes, setup and teardown observers on the content
     as needed.
@@ -400,81 +422,74 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
 
     this._scac_cached = NO; // invalidate observable content
 
-    var cur    = this.get('content'),
-        orders = !!this.get('orderBy'),
-        last   = this._scac_content,
-        oldlen = this._scac_length || 0,
-        func   = this._scac_rangeDidChange,
-        efunc  = this._scac_enumerableDidChange,
-        cfunc  = this._scac_enumerableContentDidChange,
-        sfunc  = this._scac_contentStatusDidChange,
-        ro     = this._scac_rangeObserver,
+    var content     = this.get('content'),
+        orders      = !!this.get('orderBy'),
+        lastContent = this._scac_content,
+        oldlen      = this._scac_length || 0,
+        didChange   = this._scac_arrayContentDidChange,
+        willChange  = this._scac_arrayContentWillChange,
+        sfunc       = this._scac_contentStatusDidChange,
+        efunc       = this._scac_enumerableDidChange,
         newlen;
 
-    if (last === cur) { return this; } // nothing to do
+    if (content === lastContent) { return this; } // nothing to do
 
     // teardown old observer
-    if (last) {
-      if (last.isSCArray) {
-        if (ro) { last.removeRangeObserver(ro); }
-        last.removeEnumerableObserver(this, cfunc);
+    if (lastContent) {
+      if (lastContent.isSCArray) {
+        lastContent.removeArrayObservers({
+          target: this,
+          didChange: didChange,
+          willChange: willChange
+        });
+      } else if (lastContent.isEnumerable) {
+        lastContent.removeObserver('[]', this, efunc);
       }
-      else if (last.isEnumerable) { last.removeObserver('[]', this, efunc); }
-      last.removeObserver('status', this, sfunc);
-    }
 
-    ro = null;
+      lastContent.removeObserver('status', this, sfunc);
+    }
 
     // save new cached values
     this._scac_cached = NO;
-    this._scac_content = cur ;
+    this._scac_content = content ;
 
-    // setup new observers
+    // setup new observer
     // also, calculate new length.  do it manually instead of using
     // get(length) because we want to avoid computed an ordered array.
-    if (cur) {
-      if (!orders && cur.isSCArray) { ro = cur.addRangeObserver(null, this, func); }
-      if (cur.isSCArray) { cur.addEnumerableObserver(this, cfunc); }
-      else if (cur.isEnumerable) { cur.addObserver('[]', this, efunc); }
-      newlen = cur.isEnumerable ? cur.get('length') : 1;
-      cur.addObserver('status', this, sfunc);
+    if (content) {
+      // Content is an enumerable, so listen for changes to its
+      // content, and get its length.
+      if (content.isSCArray) {
+        content.addArrayObservers({
+          target: this,
+          didChange: didChange,
+          willChange: willChange
+        });
+
+        newlen = content.get('length');
+      } else if (content.isEnumerable) {
+        content.addObserver('[]', this, efunc);
+        newlen = content.get('length');
+      } else {
+        // Assume that someone has set a non-enumerable as the content, and
+        // treat it as the sole member of an array.
+        newlen = 1;
+      }
 
+      // Observer for changes to the status property, in case this is an
+      // SC.Record or SC.RecordArray.
+      content.addObserver('status', this, sfunc);
     } else {
-      newlen = SC.none(cur) ? 0 : 1;
+      newlen = SC.none(content) ? 0 : 1;
     }
 
-    this._scac_rangeObserver = ro;
-
     // finally, notify enumerable content has changed.
     this._scac_length = newlen;
     this._scac_contentStatusDidChange();
-    this.enumerableContentDidChange(0, newlen, newlen - oldlen, this, last||[]);
+    this.arrayContentDidChange(0, 0, newlen);
     this.updateSelectionAfterContentChange();
   }.observes('content'),
 
-  /** @private
-    Whenever array content changes, need to simply forward notification.
-
-    Assumes that content is not null and is SC.Array.
-  */
-  _scac_rangeDidChange: function(array, objects, key, indexes) {
-    if (key !== '[]') { return ; } // nothing to do
-
-    var content = this.get('content');
-    this._scac_length = content.get('length');
-    this._scac_cached = NO; // invalidate
-
-    // if array length has changed, just notify every index from min up
-    if (indexes) {
-      this.beginPropertyChanges();
-      indexes.forEachRange(function(start, length) {
-        this.enumerableContentDidChange(start, length, 0);
-      }, this);
-      this.endPropertyChanges();
-      this.updateSelectionAfterContentChange();
-    }
-  },
-
   /**
     @private
 
@@ -490,33 +505,6 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
     @param {Number} start the index at which the positions occurred
   */
   _scac_enumerableContentDidChange: function(addedObjects, removedObjects, start) {
-    var enumerableChanges = this._scac_enumerableChanges || [];
-
-    enumerableChanges.push([addedObjects, removedObjects, start]);
-
-    this._scac_enumerableChanges = enumerableChanges;
-    this.invokeOnce(this._scac_propagateEnumerableObservers);
-
-    this.setupPropertyChainsForEnumerableContent(addedObjects, removedObjects);
-  },
-
-  /**
-    @private
-
-    At the end of the run loop, notifies enumerable observers on this array
-    controller of changes we received from the content object.
-  */
-  _scac_propagateEnumerableObservers: function() {
-    var enumerableChanges = this._scac_enumerableChanges;
-    var idx, len, change;
-
-    len = enumerableChanges.get('length');
-    for (idx = 0; idx < len; idx++) {
-      change = enumerableChanges[idx];
-      this._notifyEnumerableObservers(change[0], change[1], change[2]);
-    }
-
-    this._scac_enumerableChanges = null;
   },
 
   /** @private
@@ -534,7 +522,10 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
     this._scac_length = newlen;
     this.beginPropertyChanges();
     this._scac_cached = NO; // invalidate
-    this.enumerableContentDidChange(0, newlen, newlen-oldlen);
+    // If this is an unordered enumerable, we have no way
+    // of knowing which indices changed. Instead, we just
+    // invalidate the whole array.
+    this.arrayContentDidChange(0, oldlen, newlen);
     this.endPropertyChanges();
     this.updateSelectionAfterContentChange();
   }.observes('orderBy'),

data/lib/frameworks/sproutcore/frameworks/core_foundation/core.js

@@ -175,9 +175,7 @@ SC.mixin(Function.prototype, /** @scope Function.prototype */ {
     to the function as well.  Otherwise the function should have the 
     signature:
     
-    {{{
-      function functionName(timer)
-    }}}
+        function functionName(timer)
 
     @param target {Object} optional target object to use as this
     @param interval {Number} the time to wait, in msec