On 08/16/2013 12:34 AM, Stef wrote:
> Hi, thanks for the info.
> Topic is not that important. There's no todo left.
>
> For completeness or reference, just a remark on migration from 2.1.1 to 3.0:
>
> thron7-2 wrote
>> [...] But you should have received numerous messages about this
>> during migration and running the source version of your app under 2.1,
>> no?! The migration would also automatically have re-written your code to
>> use the instance methods, so you should be fine with existing code.
> The demo code of Application.js was:
> ...
> var s1 = " demo-string ";
> var s2 = qx.lang.String.trim(s1);
> alert("/" + s2 + "/");
> ...
>
> In the 2.1.1 version, no compile error, no lint warning, no runtime error.
>
> When creating a new 3.0 skeleton, copying the previous Application.js into
> the project and then running "generate.py migration", there's neiter warning
> about trim(), nor automatic code update.
Look, if you migrate from 2.1.1 to 3.0 the transformation of .trim() is
not applied as it already occurred in 2.1. So if you enter "2.1.1" as
your current version during migration but effectively have pre-2.1 code
you're cheating to the Generator, and necessary checks and
transformations are not applied.
You took advantage of the fact that APIs that are deprecated and pot.
rewritten are still available for this version, and will only be removed
with the next. And everytime you ran the above code under 2.1.1 there
was a console warning saying that qx.lang.String.trim is deprecated,
definitely (unless you deliberately removed debug logging for the source
version in your app).
> Of course, a warning is dropped for
> the replaced "asset" syntax.
> Also "generate.py source" and "generate.py lint" doesn't recognize any error
> or warning. But at runtime the browser throws the error "TypeError:
> qx.lang.String.trim is not a function"
The Generator doesn't do an API check down to the method level. It
checks for the class, like qx.lang.String, and if that is available it
thinks you know what you are doing. But if you feel there should be a
full API check, including special-casing deprecations, you can open an
enhancement bug for this.
T.
------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite!
It's a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel