Apache Cordova
  1. Apache Cordova
  2. CB-1143

Indicate through the ContactError code if remove() failed because the contact is nonexistent / has already been deleted

    Details

    • Type: New Feature New Feature
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.0.0
    • Fix Version/s: None
    • Component/s: Plugin Contacts
    • Labels:

      Description

      At least with the iOS implementation, if a contact has already been deleted, calling Contact.remove() on it results in an error as expected; however, the error code is 0 (UNKNOWN_ERROR), which gives the app no clue as to what happened.

      It would be nice if the error code could indicate that removal failed because the contact has already been deleted.

      Suggested fix: Add an error code, NONEXISTENT, that would be used to indicate that the device has no contact with the supplied ID.

        Activity

        Steve Gill made changes -
        Assignee Steve Gill [ stevegill ]
        Filip Maj made changes -
        Assignee Filip Maj [ filmaj ] Steve Gill [ stevegill ]
        Component/s Plugin Contacts [ 12320652 ]
        Component/s CordovaJS [ 12317202 ]
        Hide
        Filip Maj added a comment -

        Assigned to plugin contacnts.

        Show
        Filip Maj added a comment - Assigned to plugin contacnts.
        Filip Maj made changes -
        Field Original Value New Value
        Priority Major [ 3 ] Minor [ 4 ]
        Labels contacts audit contacts
        Hide
        Filip Maj added a comment -

        Dropping priority, tagging as "audit" for future reevaluation. See the Core API Audit wiki page for details.

        Show
        Filip Maj added a comment - Dropping priority, tagging as "audit" for future reevaluation. See the Core API Audit wiki page for details.
        Daniel Trebbien created issue -

          People

          • Assignee:
            Unassigned
            Reporter:
            Daniel Trebbien
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development