[TIMOB-9735] Android: Error Dialog Platform Parity
GitHub Issue | n/a |
---|---|
Type | Bug |
Priority | High |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2013-10-14T22:08:36.000+0000 |
Affected Version/s | Release 2.1.0 |
Fix Version/s | Sprint 2012-19 Core, 2012 Sprint 19, Release 3.2.0 |
Components | Android |
Labels | core, module_package, parity, triage |
Reporter | Dawson Toth |
Assignee | Vishal Duggal |
Created | 2012-06-22T16:17:08.000+0000 |
Updated | 2013-10-23T23:48:57.000+0000 |
Description
Parity Issue
Error dialogs are inconsistent across deploy types and platforms.Android Behavior
On Android, [error dialogs always show up](https://github.com/appcelerator/titanium_mobile/blob/master/android/runtime/v8/src/native/V8Util.cpp#L127), even in production.iOS Behavior
iOS only shows error dialogs [when the deploy type is not equal to production](https://github.com/appcelerator/titanium_mobile/blob/master/iphone/Classes/TiApp.m#L524).Mobile Web Behavior
Mobile Web [shows errors](https://github.com/appcelerator/titanium_mobile/blob/master/mobileweb/titanium/Ti.js#L306) depending on the [config](https://github.com/appcelerator/titanium_mobile/blob/master/mobileweb/src/config.js#L36).Reproduction
This will throw an error. The various platforms will handle it as they wish.
var win = Ti.UI.createWindow({
backgroundColor: '#fff'
});
win.add(fail);
win.open();
Desired Behavior
Error dialogs are consistent across deployment types. Config values are also equally exposed on all platforms.Attachments
File | Date | Size |
---|---|---|
android_device.png | 2012-12-18T22:28:26.000+0000 | 126567 |
android_production.png | 2012-12-18T22:28:26.000+0000 | 247175 |
ios_device.png | 2012-12-18T22:28:26.000+0000 | 76664 |
ios_production.png | 2012-12-18T22:28:26.000+0000 | 585479 |
iOS Simulator Screen shot Dec 18, 2012 12.17.19 PM.png | 2012-12-18T22:28:26.000+0000 | 30051 |
Spurred from this Stack Overflow question, by the way: [http://stackoverflow.com/questions/10827463/disable-runtime-error-dialog-in-android-titanium/11165347#11165347]
+1
Has this been merged into the master branch, or better yet, 2.1? I just published for production on both and I'm still seing these error dialogs.
Still same issue. When are they going to solve it? It is very annoying
Reopening. I can reproduce the original issue. Tested with: Titanium Studio, build: 3.0.1.201212181159 Titanium SDK, build: 3.0.0.GA Devices: GalaxyS3 4.0.4 Android emulator 3.0 iphone 4s 5.0.1 iOS simulator 6.0
Updated the links to the relevant source files, AFAIK. It looks like Android V8 only checks that we have an environment, then continues blissfully on its way.
@Ingo Muschenetz Please reassign to devs since the issue was reopened and original assignee was Opie Cyrus.
Tested with: Appcelerator Studio, build: 3.1.1.201306041845 Titanium SDK, build: 3.1.1.v20130605141849 Mac osx 10.8.3 Mountail Lion CLI: 3.1.1-cr Devices: Nexus 7 Android version 4.2 GalaxyS4 Android version 4.2.2 iPhone 5 iOS 6.1.4 iPad mini iOS 6.0 There is no runtime error in production.
Here's the merged PR from Opie, by the way: https://github.com/appcelerator/titanium_mobile/pull/2941
Any updates here? The merges PR seems to work by checking the Ti.App.getDeployType().
Just confirmed what olga said... there is no error dialog in production for android, which is the expected behavior. Resolving as fixed.
Verified fixed. TiSDK 3.2.0.v20131023160426 CLI 3.2.0 Titanium Studio 3.2.0.201310230548 Closing.