2016-09-26 77 views
0

我在过去的几天里一直在努力理解我做错了什么。 我正在实施我的应用程序列表的基础结构,其中可能包括paging/infinite scroll/filtering/grouping/等。该实现基于扩展控制器(不是阵列控制器,我想要成为Ember 2.0安全),其中包含数据属性为content的数组。Ember.computed.sort属性没有更新

我正在使用Ember.computed.sort进行排序,它正在工作,但当我尝试更改分拣机时,我遇到了一个奇怪的行为。 在displayContent内没有更新,即使更新了定义sortingDefinitions

这会导致一个奇怪的行为,它只会排序如果我sort它两次,就好像排序是异步的。

我使用Ember 1.5(但它也发生在1.8) (附加的代码解释我的问题的一个片段)

sortingDefinitions: function(){ 
    var sortBy = this.get('sortBy'); 
    var sortOrder = this.get('sortOrder') || 'asc'; 

    if (_.isArray(sortBy)) { 
     return sortBy; 
    } 
    else { 
     return (sortBy ? [sortBy + ':' + sortOrder] : []); 
    } 
}.property('sortBy', 'sortOrder'), 

sortedContent: Ember.computed.sort('content', 'sortingDefinitions'), 




displayContent: function() { 
    var that = this; 
    var sortBy = this.get('sortBy'); 
    var sortOrder = this.get('sortOrder'); 
    var list = (sortBy ? this.get('sortedContent') : this.get('content')); 

    var itemsPerPage = this.get('itemsPerPage'); 
    var currentPage = this.get('currentPage'); 
    var listItemModel = this.get('listItemModel'); 

    return list.filter(function(item, index, enumerable){ 
     return ((index >= (currentPage * itemsPerPage)) && (index < ((currentPage + 1) * itemsPerPage))); 
    }).map(function(item) { 
     var listItemModel = that.get('listItemModel'); 
     if (listItemModel) { 
      return listItemModel.create(item); 
     } 
     else { 
      return item; 
     } 
    }); 

}.property('content.length', 'sortBy', 'sortOrder', 'currentPage', 'itemsPerPage') 

编辑: 通过添加另一个依赖于displayContent(sortedContent固定。 []):

displayContent: function() { 
    .... 

}.property('content.length', 'sortBy', 'sortOrder', 'currentPage', 'itemsPerPage' , 'sortedContent.[]') 
+0

注意,它适用于我的单元测试的罚款。渲染循环的东西在这里搞砸了。 – Manu

回答

1

你的排序函数正在观看整个阵列sortingDefinitions代替的阵列中的每个元件。如果数组更改为字符串或其他变量,它会更新,但如果数组中的元素发生更改,则不会更新。

要正确确保您的计算机属性更新,添加.[]到数组的末尾,以便它看起来像这样:Ember.computed.sort('content', 'sortingDefinitions.[]')

+0

但整个阵列正在被替换。真正奇怪的部分是,当它排序两次它的作品。 (试过你的解决方案,无论如何,没有工作:) :( – Manu

+0

好吧,我发现这个问题,并没有那么不同 添加'sortedContent。[]'到displayContent属性列表做了伎俩。 – Manu

相关问题