Titanium JIRA Archive
Titanium SDK/CLI (TIMOB)

[TIMOB-11525] Android: Relative path for Window URL does not work on 3_0_X

GitHub Issuen/a
TypeSub-task
PriorityHigh
StatusClosed
ResolutionFixed
Resolution Date2012-10-30T18:12:02.000+0000
Affected Version/sRelease 3.0.0, Release 3.1.0
Fix Version/sRelease 3.0.0, Release 3.1.0, 2012 Sprint 22 API, 2012 Sprint 22
ComponentsAndroid
Labels3.0.0beta2, api
ReporterPing Wang
AssigneePing Wang
Created2012-10-17T22:30:38.000+0000
Updated2017-03-08T23:47:37.000+0000

Description

Android has supported relative path for window URL since 1.8. Due to [PR#3057](https://github.com/appcelerator/titanium_mobile/pull/3057/), it is not supported any more on 3_0_X and master. PR#3057 was trying to fix a regression (TIMOB-10783) which was caused by [PR#2027](https://github.com/appcelerator/titanium_mobile/pull/2720). Steps to reproduce: Run the attached project. Actual result: An error shows in the console:
E/TiAssetHelper(14013): Error while reading asset "Resources/dir/relative2.js":
E/TiAssetHelper(14013): java.io.FileNotFoundException: Resources/dir/relative2.js
Expected result: A blue window opens first, then a red window opens, and finally both windows close. The log shows in the console:
I/TiAPI   ( 8074):  ************************* relative2.js: win closed
I/TiAPI   ( 8074):  ************************* relative.js: win closed
I/TiAPI   ( 8074):  ************************* app.js: win closed
PS: This regression causes the failure of several Anvil tests: ui/ui->absoluteAndRelativeWinURLs, includes/includes->includeFromUrlWindow and includes/includes->secondContextRequire.

Attachments

FileDateSize
timob11525.zip2012-10-24T18:21:31.000+00003694669

Comments

  1. Ping Wang 2012-10-25

    Steps for FR: 1. Run the attached project "timob11525" and follow the instruction in the ticket. 2. Run the test cases in TIMOB-10783, TIMOB-11408, TIMOB-11307, TIMOB-5434, TIMOB-10091 and TIMOB-5748. 3. Run Anvil on both v8 and rhino. ui/ui->absoluteAndRelativeWinURLs, includes/includes->includeFromUrlWindow and includes/includes->secondContextRequire should pass without any new regression.
  2. Ping Wang 2012-10-25

    PR https://github.com/appcelerator/titanium_mobile/pull/3321
  3. Ping Wang 2012-10-30

    3_0_X PR: https://github.com/appcelerator/titanium_mobile/pull/3356
  4. Lee Morris 2017-03-08

    Closing ticket as fixed.

JSON Source