Added ng:options directive

Closes angular#301

Author: mhevery

CHANGELOG.md

@@ -1,13 +1,15 @@
 <a name="0.9.16"><a/>
 # <angular/> 0.9.16 weather-control (in-progress) #
 
+### New Features
+- Added ng:options directive (http://docs.angularjs.org/#!angular.directive.ng:options)
+
 ### Breaking changes
+- Dynamic Iteration (ng:repeater) on <option> elements is no longer supported. Use ng:options
 - $service now has $service.invoke for method injection ($service(self, fn) no longer works)
 - injection name inference no longer supports method curry and linking functions. Both must be
   explicitly specified using $inject property.
 
-
-
 <a name="0.9.15"><a/>
 # <angular/> 0.9.15 lethal-stutter (2011-04-11) #
 

src/Angular.js

@@ -54,7 +54,6 @@ function fromCharCode(code) { return String.fromCharCode(code); }
 
 var _undefined        = undefined,
     _null             = null,
-    $$update          = '$update',
     $$scope           = '$scope',
     $$validate        = '$validate',
     $angular          = 'angular',

src/Scope.js

@@ -716,10 +716,7 @@ Scope.prototype = {
    * @returns {*} The result of evaluating the expression.
    */
   $eval: function(expr) {
-    var fn = isString(expr)
-      ? parser(expr).statements()
-      : expr || noop;
-    return fn(this);
+    return (isString(expr) ? compileExpr(expr) : expr || noop)(this);
   },
 
   /**

src/directives.js

@@ -425,10 +425,8 @@ var REMOVE_ATTRIBUTES = {
 angularDirective("ng:bind-attr", function(expression){
   return function(element){
     var lastValue = {};
-    var updateFn = element.data($$update) || noop;
     this.$observe(function(scope){
-      var values = scope.$eval(expression),
-          dirty = noop;
+      var values = scope.$eval(expression);
       for(var key in values) {
         var value = compileBindTemplate(values[key])(scope, element),
             specialName = REMOVE_ATTRIBUTES[lowercase(key)];
@@ -446,10 +444,8 @@ angularDirective("ng:bind-attr", function(expression){
           } else {
             element.attr(key, value);
           }
-          dirty = updateFn;
         }
       }
-      dirty();
     });
   };
 });

src/formatters.js

@@ -181,6 +181,7 @@ angularFormatter.trim = formatter(
  * @workInProgress
  * @ngdoc formatter
  * @name angular.formatter.index
+ * @deprecated
  * @description
  * Index formatter is meant to be used with `select` input widget. It is useful when one needs
  * to select from a set of objects. To create pull-down one can iterate over the array of object
@@ -227,6 +228,7 @@ angularFormatter.trim = formatter(
      </doc:scenario>
    </doc:example>
  */
+//TODO: delete me since this is replaced by ng:options
 angularFormatter.index = formatter(
   function(object, array){
     return '' + indexOf(array || [], object);

src/jqLite.js

@@ -356,6 +356,20 @@ forEach({
     });
   },
 
+  prepend: function(element, node) {
+    if (element.nodeType === 1) {
+      var index = element.firstChild;
+      forEach(new JQLite(node), function(child){
+        if (index) {
+          element.insertBefore(child, index);
+        } else {
+          element.appendChild(child);
+          index = child;
+        }
+      });
+    }
+  },
+
   remove: function(element) {
     JQLiteDealoc(element);
     var parent = element.parentNode;

src/parser.js

@@ -239,6 +239,8 @@ function parser(text, json){
       pipeFunction =
         function (){ throwError("is not valid json", {text:text, index:0}); };
   }
+  //TODO: Should'nt all of the public methods have assertAllConsumed?
+  //TODO: I think these should be public instead of scope.$eval() being the enterenece.
   return {
       assignable: assertConsumed(assignable),
       primary: assertConsumed(primary),
@@ -748,12 +750,17 @@ function getterFn(path){
 
 ///////////////////////////////////
 
-//TODO: Deprecate? Remove?
+//TODO: Should this function be public?
+function compileExpr(expr) {
+  return parser(expr).statements();
+}
+
+//TODO: Deprecate? Remove!
 function expressionCompile(exp){
   if (typeof exp === $function) return exp;
   var fn = compileCache[exp];
   if (!fn) {
-    fn = compileCache[exp] =  parser(exp).statements();
+    fn = compileCache[exp] =  compileExpr(exp);
   }
   return fn;
 }