Re: [qooxdoo-devel] controler list bug on new version

2011-01-04 Thread Martin Wittemann
Hello David, Thats a good one! Thanks for the report. Your patch looks like a good fix to me too. This was added unintentionally as you already guessed. I simply haven't thought of the converter which of course needs the model in some cases on the initial bind. Would you be so kind and open up a

Re: [qooxdoo-devel] controler list bug on new version

2011-01-04 Thread d2
>I believe you're trying to concat two models. I'm not, I just trying to fix a bug that was not on previews release ;) My playground example is clear, his only purpose is to show you the bug but the code has nothing to do with our real code. Of course I can add a if(model === null) {return;} but

Re: [qooxdoo-devel] controler list bug on new version

2011-01-04 Thread Tristan Koch
Hi David, I believe you're trying to concat two models. qx.data.marshal.Json.createModel plus some JSON data is everything you need. See http://tinyurl.com/3a7gkl2. Tristan Am 04.01.2011 um 10:03 schrieb David Charbonnier: > Hi 2, > > New version introduce new functionalities and new bug ;) >

[qooxdoo-devel] controler list bug on new version

2011-01-04 Thread David Charbonnier
Hi 2, New version introduce new functionalities and new bug ;) To reproduce : http://tinyurl.com/2aqm7ue Error : TypeError: model is null Patch : --- qooxdoo-1.3-sdk.original/framework/source/class/qx/data/controller/List.js 2011-01-04 09:55:11.0 +0100 +++ qooxdoo-1.3-sdk.oxys/frame