[TIMOB-856] Android KS: 'Change' in Scrollable view test crashes, and 'Add' does nothing
| GitHub Issue | n/a |
|---|---|
| Type | Bug |
| Priority | Medium |
| Status | Closed |
| Resolution | Fixed |
| Resolution Date | 2011-04-17T01:54:27.000+0000 |
| Affected Version/s | n/a |
| Fix Version/s | Release 1.3.0 |
| Components | Android |
| Labels | android, defect, regression |
| Reporter | Bill Dawson |
| Assignee | Don Thorp |
| Created | 2011-04-15T02:37:56.000+0000 |
| Updated | 2011-04-17T01:54:27.000+0000 |
Description
Trace shows the crash (when clicking 'Change') to be NullPointerException in TiUIScrollableView.setViews
(from [cd13a5bce95f72b251c15e3aa8b8bb802607d77e]) [#856 state:open] I put a patch in to prevent the crash, but bigger problems are afoot. I'm going to move this to 1.4.0 as it's too dangerous to fix the night before the release. The real problem lies in KrollObject. The getter is being added and it returns the value of the object. Then the setter method is added and it wipes out the returned value. Causing the array to be zero length. http://github.com/appcelerator/titanium_mobile/commit/cd13a5bce95f72b251c15e3aa8b8bb802607d77e"> http://github.com/appcelerator/titanium_mobile/commit/cd13a5bce95f7...
Moving back to 1.3.0. Marshall's fix of the regression for other issues. Allows me to continue fairly safely. I was probably too tired to decide to just move it.
(from [0d6e9f6a7ee43756a2e1cba46ed869492ea4528e]) [#856 state:resolved] Jump actually jumps, add is re-enabled, Scrolling works as expected http://github.com/appcelerator/titanium_mobile/commit/0d6e9f6a7ee43756a2e1cba46ed869492ea4528e"> http://github.com/appcelerator/titanium_mobile/commit/0d6e9f6a7ee43...