Ticket #321 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

empty controls show up after setwirevalue in collection-performance (forms sample)

Reported by: freya Owned by: mpo
Priority: major Milestone: 0.4
Component: modules/kauri-forms Version:
Keywords: Cc: kauri-discuss@…

Description (last modified by mpo) (diff)

When pushing the 'setWireValue' repeatingly on collection.performance in the forms sample, once in a while (quite often, in fact) a empty line shows up: a table row where the values are not set.

Screenshot attached

Attachments

collection-performance.png (8.9 KB) - added by freya 3 years ago.

Change History

Changed 3 years ago by freya

comment:1 Changed 3 years ago by freya

  • Status changed from new to closed
  • Resolution set to fixed

comment:2 Changed 3 years ago by sdm

  • Status changed from closed to reopened
  • Resolution fixed deleted

I'm still seeing this problem, both in FF and IE. You do have to push the setWireValue button quite often though to get an empty line.

comment:3 Changed 3 years ago by jgou

  • Type changed from task to defect
  • Component changed from -- unknown -- to modules/kauri-forms
  • Milestone set to 0.4

comment:4 Changed 3 years ago by karel

This script helps to reproduce the problem:

      $(function() {
        var tint;
        var tmax = 100;
        var tcount = tmax;
        var t = function() {
          $("#loadNativeSampleData").click();
          $("#form1 input").each(function() {
            if (this.value=="") {
              debugger;
            }
          });
          if (--tcount == 0) {
            alert('Button click simulated ' + tmax + ' times. Stopping simulation.');
            clearInterval(tint);
          }
        }
        tint = setInterval(t, 0);
      });

Using this script I can reproduce the problem on

  • chromium 6.0.472.63 (Developer Build 59945) Ubuntu 10.10
  • Mozilla/5.0 (X11; U; Linux i686; nl; rv:1.9.2.11) Gecko/20101013 Ubuntu/10.10 (maverick) Firefox/3.6.11

(I didn't try IE)

comment:5 Changed 3 years ago by karel

  • Status changed from reopened to closed
  • Resolution set to fixed

(In [1757]) Properly clean up element index when removing

fixes #321

comment:6 Changed 3 years ago by karel

  • Status changed from closed to reopened
  • Resolution fixed deleted

[1757] removed the 'removeChildById' method, but it is still used in the collection control

comment:7 Changed 3 years ago by mpo

  • Description modified (diff)

I scanned the code but can't find this extra lingering reference to removeChildById
Am I missing something?

comment:8 Changed 2 years ago by karel

  • Status changed from reopened to closed
  • Resolution set to fixed

Fixed in r1766, see #386

(Hint: found this using git log -SremoveChildById?)

Note: See TracTickets for help on using tickets.