You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@royale.apache.org by GitBox <gi...@apache.org> on 2020/08/26 15:01:35 UTC

[GitHub] [royale-asjs] jleistner commented on issue #900: Sorting arrays with Array.NUMERIC not working correctly

jleistner commented on issue #900:
URL: https://github.com/apache/royale-asjs/issues/900#issuecomment-680935722


   Hi Harbs,
   
   I did some testing and
   ```
   a = +a;
   b = +b;
   ```
   is not working for me (`Implicit coercion of a value with static type Object to a possibly unrelated type Number.`) but
   ```
   a = Number(a);
   b = Number(b);
   ```
   is. If casting the values like this is acceptable I would open a PR.
   
   On a side note:
   To be able to build Apache Royale on Linux (not sure if this is OS dependent) I had to rename [royale-asjs/frameworks/js/projects/JQueryJS ](https://github.com/apache/royale-asjs/tree/develop/frameworks/js/projects/JQueryJS) to jQueryJS (lower case 'j') because the project name in [royale-asjs/frameworks/projects/JQuery/build.xml:22](https://github.com/apache/royale-asjs/blob/develop/frameworks/js/projects/JQueryJS/build.xml#L22) uses a lower case 'j' as well. Should I address this in the PR as well and if so, would it be better to rename the folder or to change the project name?
   
   Thanks
   Jonathan


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org