Changes between Version 2 and Version 3 of Ticket #16703


Ignore:
Timestamp:
Feb 13, 2013, 10:44:11 PM (7 years ago)
Author:
bill
Comment:

Refs #14866. OK, I'll check in that patch, thanks.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #16703

    • Property Status changed from new to assigned
    • Property Version changed from 1.8.3 to 1.8.0
    • Property Milestone changed from tbd to 1.8.4
    • Property Owner changed from Douglas Hays to bill
  • Ticket #16703 – Description

    v2 v3  
    11It appears to be a bug in dijit/form/_FormSelectWidget.js
    22
    3         When Bill converted to use/allow the new dojo.store API, he broke a check that actually handles creating the sorter function.  It doesn't test right/properly to identify the old store API
     3When Bill converted to use/allow the new dojo.store API, he broke a check that actually handles creating the sorter function.  It doesn't test right/properly to identify the old store API
    44
    55This:
    6                                 if(this.sortByLabel && !fetchArgs.sort && items.length){
    7                                         if(items[0].getValue){
    8                                                 // Old dojo.data API to access items, remove for 2.0
    9                                                 items.sort(sorter.createSortFunction([{
    10                                                         attribute: store.getLabelAttributes(items[0])[0]
    11                                                 }], store));
    12                                         }else{
    13                                                 var labelAttr = this.labelAttr;
    14                                                 items.sort(function(a, b){
    15                                                         return a[labelAttr] > b[labelAttr] ? 1 :  b[labelAttr] > a[labelAttr] ? -1 : 0;
    16                                                 });
    17                                         }
    18                                 }
     6
     7{{{
     8if(this.sortByLabel && !fetchArgs.sort && items.length){
     9        if(items[0].getValue){
     10...
     11}}}
    1912
    2013Should be:
    2114
    22                                 if (this.sortByLabel && !fetchArgs.sort && items.length) {
    23                                         if (store.getValue) {
    24                                                 // Old dojo.data API to access items, remove for 2.0
    25                                                 items.sort(sorter.createSortFunction([{
    26                                                         attribute: store.getLabelAttributes(items[0])[0]
    27                                                 }], store));
    28                                         } else {
    29                                                 var labelAttr = this.labelAttr;
    30                                                 items.sort(function (a, b) {
    31                                                         return a[labelAttr] > b[labelAttr] ? 1 : b[labelAttr] > a[labelAttr] ? -1 : 0;
    32                                                 });
    33                                         }
    34                                 }
     15{{{
     16if (this.sortByLabel && !fetchArgs.sort && items.length) {
     17        if (store.getValue) {
     18...
     19}}}