Titanium JIRA Archive
Titanium SDK/CLI (TIMOB)

[TIMOB-1035] Android: Properties API always expects default value

GitHub Issuen/a
TypeBug
PriorityLow
StatusClosed
ResolutionFixed
Resolution Date2011-04-17T01:55:01.000+0000
Affected Version/sn/a
Fix Version/sRelease 1.4.0
ComponentsAndroid
Labelsandroid, defect
ReporterDon Thorp
AssigneeMarshall Culpepper
Created2011-04-15T02:42:01.000+0000
Updated2011-04-17T01:55:01.000+0000

Description

The current implementation always expects a default for each getXXXX() method instead of treating as a default. getXXXX() methods should acccept and Object[] and deal with the optional param correctly as documented in the API docs.

Comments

  1. Don Thorp 2011-04-15

    The java implementation needs to change to be getXXXX(Object[] args); Inside the method you need to verify that it has at least one argument the "key" and optionally a default value.

  2. Matthew Lenton 2011-04-15

    Don, can you point toward the particular API docs as referenced in the initial ticket detail?

    I think I know what you mean though just need a little more detail in determining the scope for this ticket.

  3. Don Thorp 2011-04-15

    Ti.App.Properties. Specifically PropertiesModule.java

  4. Dasher 2011-04-15

    I needed this for something else I'm working on - http://github.com/dasher/titanium_mobile/commit/8f7cb0a3bf8283bc461048920bb4196b28a508a9"> http://github.com/dasher/titanium_mobile/commit/8f7cb0a3bf8283bc461...

  5. Jeff Haynie 2011-04-15

    (from [66472312aeb03c97bfdeb32d50c252347173e3ac]) Allowed the properties API to function without needing a default to be passed & added a helper to TiProperties to detect if a list exists or not [#1035 state:resolved] http://github.com/appcelerator/titanium_mobile/commit/66472312aeb03c97bfdeb32d50c252347173e3ac"> http://github.com/appcelerator/titanium_mobile/commit/66472312aeb03...

JSON Source