Skip to content
  1. Aug 04, 2012
  2. Aug 03, 2012
    • Jesse Glick's avatar
      [JENKINS-14495] Refined fix - needed to pull uniqueness check out of apply()... · f39fbdb3
      Jesse Glick authored
      [JENKINS-14495] Refined fix - needed to pull uniqueness check out of apply() fn in case we were being passed an Array of nodes.
      Fixes test failures introduced by  dbb100da, and adds a test for it now that I know it can be done.
      f39fbdb3
    • Jesse Glick's avatar
      [FIXED JENKINS-14495] Hetero lists not working correctly after adding elements. · dbb100da
      Jesse Glick authored
      Unlike [JENKINS-14514] this is a true fix rather than a workaround (now removed), and is more general.
      cjo9900 discovered that behaviors were being redundantly registered (as of 1.474 the monolithic JS is broken up);
      this caused some behaviors to be run repeatedly on the same elements, breaking reasonable expectations of some behaviors.
      The ideal fix would be to change Behavior.register to be idempotent: for example, key it by selector, then maintain a set of distinct behavior functions for each.
      Unfortunately some adjuncts directly call Behavior.list.unshift, bypassing register(...), which would be tricky to intercept (would need to make a mock of Array).
      The known one cases are in core, but it is possible plugin adjuncts do this too, in which case it would be incompatible to (say) change the Array<Map<String,Behavior>> to a Map<String,Array<Behavior>>.
      Instead, permitting redundant registrations as before, and just silently skipping all but the first at runtime when applying behaviors.
      Beware that since adjuncts are loaded from multiple places, different JS function objects are registered each time, so a naive set of behavior functions does not work;
      have to identify functions by their toString in order to ensure that each is run only once.
      (Currently once _per selector_, conceivably >1x per element; could if necessary be refined to make sure a given behavior is only run once on a given element during one call to applySubtree even if the element matches multiple selectors.)
      dbb100da
    • Nicolas De Loof's avatar
      getExtensionList never returns null · 8349d0d8
      Nicolas De Loof authored
      8349d0d8
    • Nicolas De Loof's avatar
      fix typo · a0b29cd3
      Nicolas De Loof authored
      a0b29cd3
  3. Aug 02, 2012
  4. Jul 31, 2012
  5. Jul 30, 2012
  6. Jul 28, 2012
  7. Jul 26, 2012
Loading