[TIMOB-24170] LiveView breaks relative path usage in require

GitHub Issue	n/a
Type	Bug
Priority	Critical
Status	Closed
Resolution	Fixed
Resolution Date	2017-09-06T06:47:06.000+0000
Affected Version/s	Release 6.0.0
Fix Version/s	n/a
Components	Android, iOS, LiveView
Labels	LiveView, alloy, path, relative, require
Reporter	Brenton House
Assignee	Feon Sua Xin Miao
Created	2016-11-23T18:58:11.000+0000
Updated	2018-08-06T17:57:34.000+0000

Description

When requiring a module using relative paths, an app will error out if LiveView is running.

var require_absolute_path = require('/test/myModule');
console.error('require_absolute_path: ' + require_absolute_path);

var require_relative_path = require('./myModule');
console.error('require_relative_path: ' + require_relative_path);

[INFO]  [LiveView] Error Evaluating /test/requireTest.js @ Line: <null>
[ERROR] Couldn't find module: ./myModule for architecture: x86_64
[ERROR] File: /test/requireTest.js
[ERROR] Line: <null>
[ERROR] SourceId: <null>
[ERROR] Backtrace:
[ERROR] undefined

Comments

Hans Knöchel 2016-11-27

[~brentonhouse] Is this only happening on 6.0.0+? And is it iOS, Android or both?
Brenton House 2016-11-28

[~hansknoechel] I will test on Android and update the ticket with findings.

Brenton House 2016-11-28

Failed on Android as well:

   [ERROR] TiHTTPClient: (TiHttpClient-12) [1738,1738] HTTP Error (java.io.IOException): 404 : Not Found
   [ERROR] TiHTTPClient: java.io.IOException: 404 : Not Found
   [ERROR] TiHTTPClient: 	at ti.modules.titanium.network.TiHTTPClient$ClientRunnable.run(TiHTTPClient.java:1217)
   [ERROR] TiHTTPClient: 	at java.lang.Thread.run(Thread.java:818)
   [INFO]  [LiveView] Error Evaluating /test/requireTest.js @ Line: undefined
   [ERROR] Error: Requested module not found: ./myModule
   [ERROR] File: /test/requireTest.js
   [ERROR] Line: undefined
   [ERROR] SourceId: undefined
   [ERROR] Backtrace:
   [ERROR] undefined

Feon Sua Xin Miao 2017-08-09

PR: https://github.com/appcelerator/liveview/pull/105
Abir Mukherjee 2017-08-17

[~ssekhri] Are you doing the FR for this one? This one should be a Studio FR.
Satyam Sekhri 2017-08-17

[~amukherjee] As soon as it is merged into studio. The merge ticket is already planned for this sprint but we would take it on priority.
Abir Mukherjee 2017-08-28

[~ssekhri] This ticket should actually be a Studio 4.10 ticket. Would you be able to move the fixVersion to the correct version?
Feon Sua Xin Miao 2017-09-06

Fixed in liveview@1.2.1.
Eric Merriman 2018-08-06

Cleaning up older fixed tickets from 2016 and earlier. If this ticket should not have been closed, please reopen it.

JSON Source