quickstart-code-sample icon indicating copy to clipboard operation
quickstart-code-sample copied to clipboard

everyProperty is deprecated...

Open jodarove opened this issue 11 years ago • 0 comments

On todos_controller.js, specifically in the allAreDone computed property there is:

return !!this.get('length') && this.everyProperty('isCompleted', true);

i think it may be:

return this.get('length') && this.everyBy('isCompleted', true);

Note that i also removed the double "!!"

i think also should be changed in the starting guide from emberjs.com (http://emberjs.com/guides/getting-started/show-when-all-todos-are-complete/ and http://emberjs.com/guides/getting-started/toggle-all-todos/) .

--- js/controllers/todos_controller.js  (revision 81801d87da42d0c83685ff946c46de68589ce38f)
+++ js/controllers/todos_controller.js  (revision )
@@ -43,7 +43,7 @@

   allAreDone: function (key, value) {
     if (value === undefined) {
-      return !!this.get('length') && this.everyProperty('isCompleted', true);
+      return this.get('length') && this.everyBy('isCompleted', true);
     } else {
       this.setEach('isCompleted', value);
       this.invoke('save')

jodarove avatar Oct 08 '13 15:10 jodarove