Description
When using the code below an error is thrown, to be in parity with other platforms an error should not be thrown and a warning should be logged, for example iOS logs
\[WARN\] ListView: Update item index is out of range
var win = Ti.UI.createWindow({ backgroundColor: 'blue' }),
section = Ti.UI.createListSection({
items: [
{ properties: { title: 'B' } },
{ properties: { title: 'A' } },
{ properties: { title: 'E' } },
{ properties: { title: 'G' } }
]
}),
listView = Ti.UI.createListView({ sections: [section] });
section.updateItemAt(4, { properties: { title: 'B' } });
win.add(listView);
win.open();
Steps to reproduce
Add the above code to an existing app.js
Build for windows platform
Actual results
An error is thrown
Expected results
To maintain parity an error should not be thrown, a warning should be logged
master: https://github.com/appcelerator/titanium_mobile_windows/pull/841
Verified fixed, console now logs the following warning:
[WARN] : ListSection::updateItemAt() index is out of range*Environment*Device: Lumia 640 LTE Operating System Name = Microsoft Windows 10 Pro Version = 10.0.14393 Architecture = 64bit # CPUs = 4 Memory = 17034395648 Node.js Node.js Version = 4.4.4 npm Version = 2.11.3 Titanium CLI CLI Version = 5.0.10 Titanium SDK SDK Version = 6.1.0.v20161202124434 Target Platform = windows