{ "id": "164124", "key": "TIMOB-24059", "fields": { "issuetype": { "id": "1", "description": "A problem which impairs or prevents the functions of the product.", "name": "Bug", "subtask": false }, "project": { "id": "10153", "key": "TIMOB", "name": "Titanium SDK/CLI", "projectCategory": { "id": "10100", "description": "Titanium and related SDKs used in application development", "name": "Client" } }, "fixVersions": [ { "id": "16980", "description": "New V8", "name": "Release 6.0.0", "archived": false, "released": true, "releaseDate": "2016-11-15" } ], "resolution": { "id": "1", "description": "A fix for this issue is checked into the tree and tested.", "name": "Fixed" }, "resolutiondate": "2016-10-28T05:46:01.000+0000", "created": "2016-10-21T00:21:15.000+0000", "priority": { "name": "Medium", "id": "3" }, "labels": [ "qe-6.0.0" ], "versions": [ { "id": "16980", "description": "New V8", "name": "Release 6.0.0", "archived": false, "released": true, "releaseDate": "2016-11-15" } ], "issuelinks": [], "assignee": { "name": "cwilliams", "key": "cwilliams", "displayName": "Christopher Williams", "active": true, "timeZone": "America/New_York" }, "updated": "2016-11-01T17:28:58.000+0000", "status": { "description": "The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.", "name": "Closed", "id": "6", "statusCategory": { "id": 3, "key": "done", "colorName": "green", "name": "Done" } }, "components": [ { "id": "10202", "name": "Android", "description": "Android Platform" } ], "description": "h5.Steps to reproduce:\r\n1. In an android app make sure you have ti.cloud module required.\r\n2. Build for android device/emulator.\r\n\r\nh5.Actual results:\r\n1. The app builds fine but we see lot of these debug logs:\r\n{code}\r\n[DEBUG] : Module: require called with un-prefixed module id, should be a core or CommonJS module. Falling back to old Ti behavior and assuming it's an absolute file\r\n[DEBUG] : Module: require called with un-prefixed module id, should be a core or CommonJS module. Falling back to old Ti behavior and assuming it's an absolute file\r\n[DEBUG] : Module: require called with un-prefixed module id, should be a core or CommonJS module. Falling back to old Ti behavior and assuming it's an absolute file\r\n[DEBUG] : Module: require called with un-prefixed module id, should be a core or CommonJS module. Falling back to old Ti behavior and assuming it's an absolute file\r\n[DEBUG] : Module: require called with un-prefixed module id, should be a core or CommonJS module. Falling back to old Ti behavior and assuming it's an absolute file\r\n{code}\r\n\r\nh5.Expected results:\r\n1. These should not be debug logs like these.\r\n2. Need to update ti.cloud require to new require standards. ", "attachment": [], "flagged": false, "summary": "Android: Update ti.cloud require to new require standards ", "creator": { "name": "lchoudhary", "key": "lchoudhary", "displayName": "Lokesh Choudhary", "active": true, "timeZone": "America/Los_Angeles" }, "subtasks": [], "reporter": { "name": "lchoudhary", "key": "lchoudhary", "displayName": "Lokesh Choudhary", "active": true, "timeZone": "America/Los_Angeles" }, "environment": "Appc Studio : 4.8.0.201610171310\r\nSDK Version : 6.0.0.v20161019091022\r\nMac OS Version : 10.11.6\r\nXcode Version : Xcode 8.0 Build version 8A218a\r\nAppc CLI AND Appc NPM : {\"NPM\":\"4.2.8-9\",\"CLI\":\"6.0.0-63\"}\r\nTi CLI : 5.0.10\r\nAlloy : 1.9.3\r\nNode : v4.6.0", "closedSprints": [ { "id": 741, "state": "closed", "name": "2016 Sprint 22 SDK", "startDate": "2016-10-22T00:02:29.945Z", "endDate": "2016-11-05T00:02:00.000Z", "completeDate": "2016-11-07T04:38:58.335Z", "originBoardId": 114 } ], "comment": { "comments": [ { "id": "399548", "author": { "name": "cng", "key": "cng", "displayName": "Chee Kiat Ng", "active": false, "timeZone": "America/Los_Angeles" }, "body": "This ticket may have to be re-described, since we decided to just disable the debug logs above.", "updateAuthor": { "name": "cng", "key": "cng", "displayName": "Chee Kiat Ng", "active": false, "timeZone": "America/Los_Angeles" }, "created": "2016-10-21T02:36:22.000+0000", "updated": "2016-10-21T02:36:22.000+0000" }, { "id": "399573", "author": { "name": "hknoechel", "key": "hansknoechel", "displayName": "Hans Knöchel", "active": true, "timeZone": "Europe/Berlin" }, "body": "I am wondering something else here: The log does not include the actual effected path. That was changed in [this commit|https://github.com/appcelerator/titanium_mobile/commit/73b905782cffaf4fe70d8a71cdf778403c0882da] and [this line|https://github.com/appcelerator/titanium_mobile/commit/73b905782cffaf4fe70d8a71cdf778403c0882da#diff-f43b9c568b23f950dc102d13a961daaeR1191], so I'm wondering if you may have an outdated 6.0.0 you are testing with.\r\n\r\nBut the issue is still valid, it would just help to see the actual paths that are effected. Thx!\r\n\r\n*EDIT*: And please provide a sample code that reproduces the issue, thx!\r\n*EDIT2*: Dump me, you talked about Android. We may still want to update Android as well (filed TIMOB-24061) :-)", "updateAuthor": { "name": "hknoechel", "key": "hansknoechel", "displayName": "Hans Knöchel", "active": true, "timeZone": "Europe/Berlin" }, "created": "2016-10-21T13:52:51.000+0000", "updated": "2016-10-21T14:05:35.000+0000" }, { "id": "399865", "author": { "name": "cwilliams", "key": "cwilliams", "displayName": "Christopher Williams", "active": true, "timeZone": "America/New_York" }, "body": "[~hansknoechel] Kita and I discussed this in the flow, but basically we can't really update the require paths used in this module because of the way our module system works. Since the new valid require paths aren't backwards compatible, if we update ti.cloud and a user has an app on 5.5.0.GA or lower, it'd automatically use the newest ti.cloud by default which would include the new non-backwards compatible requires. So until 5.x is end of life, we can't really update requires used by global pre-packaged modules (yuck!).\r\n\r\nSo, I think your suggestion of just commenting out this warning message about legacy require syntax wins the day, and we'll just have to add some docs/notes about the new require syntax and side effects of using old/new on various SDK versions.", "updateAuthor": { "name": "cwilliams", "key": "cwilliams", "displayName": "Christopher Williams", "active": true, "timeZone": "America/New_York" }, "created": "2016-10-26T14:30:51.000+0000", "updated": "2016-10-26T14:30:51.000+0000" }, { "id": "400009", "author": { "name": "cng", "key": "cng", "displayName": "Chee Kiat Ng", "active": false, "timeZone": "America/Los_Angeles" }, "body": "PRs merged", "updateAuthor": { "name": "cng", "key": "cng", "displayName": "Chee Kiat Ng", "active": false, "timeZone": "America/Los_Angeles" }, "created": "2016-10-28T05:46:01.000+0000", "updated": "2016-10-28T05:46:01.000+0000" }, { "id": "400189", "author": { "name": "lchoudhary", "key": "lchoudhary", "displayName": "Lokesh Choudhary", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Verified the fix.\r\n\r\nClosing.\r\n\r\nAppc Studio : 4.8.0.201610171310\r\nSDK Version : 6.0.0.v20161031201605\r\nMac OS Version : 10.11.6\r\nXcode Version : Xcode 8.1 Build version 8B62\r\nAppc CLI AND Appc NPM : {\"NPM\":\"4.2.8-9\",\"CLI\":\"6.0.0-67\"}\r\nTi CLI : 5.0.10\r\nAlloy : 1.9.4\r\nNode : v4.6.0\r\n\r\n", "updateAuthor": { "name": "lchoudhary", "key": "lchoudhary", "displayName": "Lokesh Choudhary", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2016-11-01T17:28:51.000+0000", "updated": "2016-11-01T17:28:51.000+0000" } ], "maxResults": 5, "total": 5, "startAt": 0 } } }