"for var in" is good practise in such a case, as we don't need additional functionality of Base.each().

This commit is contained in:
Jürg Lehni 2011-06-01 09:57:18 +01:00
parent ad335374cd
commit 2a5f885f50

View file

@ -164,8 +164,6 @@ var Project = this.Project = Base.extend({
* Deselects all selected items in the project.
*/
deselectAll: function() {
// TODO: is using for var i in good practice?
// or should we use Base.each? (JP)
for (var i in this._selectedItems)
this._selectedItems[i].setSelected(false);
},