Results 1 to 5 of 5

Thread: Ext.util.Collection key configuration application and cloning is broken

    You found a bug! We've classified it as EXTJS-21054 . We encourage you to continue the discussion and to find an acceptable workaround while we work on a permanent fix.
  1. #1
    Sencha Premium Member MaximGB's Avatar
    Join Date
    Jun 2007
    Location
    Moscow, Russia
    Posts
    542
    Vote Rating
    7
      0  

    Default Ext.util.Collection key configuration application and cloning is broken

    Ext version tested:
    • Ext 6.0.2 rev 437



    Browser versions tested against:
    Any browser is affected the issue is not GUI/DOM related


    DOCTYPE tested against:
    Any doctype is affected the issue is not GUI/DOM related


    Description:
    Ext.util.CollectionKey missuses Ext's configuration facility, if one provides a `keyFn` configuration option it's value is transformed and assigned to key's `getKey` property inside key's `applyKeyFn` method. The method returns `undefined` thus the key instance stores and `getKeyFn` method returns undifined value. This in turn leads to an exception when such a key is being cloned via key's `clone` method.


    Steps to reproduce the problem:
    https://fiddle.sencha.com/#fiddle/19bl


    The result that was expected:
    Instead of wrongly using `applyKeyFn` method the Ext.util.CollectionKey class should use `updateKeyFn`. And the fiddle provided should show no alert messages.


    The result that occurs instead:
    The oposite to expected.
    Use the force - read the source.

  2. #2
    Sencha - Sustaining Engineer tristan.lee's Avatar
    Join Date
    Mar 2015
    Location
    Central Ohio
    Posts
    1,579
    Vote Rating
    77
      0  

    Default

    Thanks for the report! I have opened a bug in our bug tracker.

  3. #3

  4. #4
    Sencha Premium User mankz's Avatar
    Join Date
    Nov 2007
    Location
    Stockholm, Sweden
    Posts
    3,153
    Vote Rating
    203
      0  

    Default

    And still broken in 6.5.1, guessing this is something you could fix easily?

  5. #5
    Sencha Premium User
    Join Date
    Oct 2013
    Location
    Kharkiv, Ukraine
    Posts
    28
    Vote Rating
    6
      0  

    Default

    BUMP: still broken in 6.6.0

Similar Threads

  1. [CLOSED] Ext.util.Collection - minor typo
    By LesJ in forum Ext JS 6.x Bugs
    Replies: 1
    Last Post: 17 Jun 2015, 9:49 AM
  2. Ext.util.Collection sorting/unsorting
    By mozenda in forum Ext 5: Q&A
    Replies: 2
    Last Post: 30 Sep 2014, 7:21 AM
  3. [FIXED] Performancetuning of Store by optimizing Ext.util.Collection
    By crp_spaeth in forum Sencha Touch 2.x: Bugs
    Replies: 2
    Last Post: 12 Mar 2013, 1:31 PM
  4. [FIXED] Ext.util.Collection remove bug if filtered
    By jensen83 in forum Sencha Touch 2.x: Bugs
    Replies: 2
    Last Post: 11 Sep 2012, 7:48 AM
  5. [FIXED] PR4: broken replace method in Ext.util.Collection
    By jensen83 in forum Sencha Touch 2.x: Bugs
    Replies: 2
    Last Post: 31 Jan 2012, 12:07 PM

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •