Mailing-List: contact issues-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
Date: Mon, 24 Feb 2014 21:17:19 +0000 (UTC)
From: "Andrew Grieve (JIRA)" <jira@apache.org>
To: issues@cordova.apache.org
Message-ID: <JIRA.12668932.1379384215142.95476.1393276639630@arcas>
In-Reply-To: <JIRA.12668932.1379384215142@arcas>
References: <JIRA.12668932.1379384215142@arcas>
Subject: [jira] [Updated] (CB-4849) Inconsistent constructors for the
 Contact objects
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/CB-4849?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Grieve updated CB-4849:
------------------------------

    Component/s:     (was: CordovaJS)

> Inconsistent constructors for the Contact objects
> -------------------------------------------------
>
>                 Key: CB-4849
>                 URL: https://issues.apache.org/jira/browse/CB-4849
>             Project: Apache Cordova
>          Issue Type: Improvement
>          Components: Plugin Contacts
>    Affects Versions: 2.9.0
>            Reporter: Peter
>            Assignee: Andrew Grieve
>            Priority: Minor
>
> It is curious that the ContactField constructor ensures that members are converted to strings as per the documentation, but the other Contact constructors like ContactOrganization, ContactAddress do not follow the same code pattern. Is there some reason for the difference?
> For example,
> {code}
> var ContactField = function(type, value, pref) {
>     this.id = null;
>     this.type = (type && type.toString()) || null;
>     this.value = (value && value.toString()) || null;
>     this.pref = (typeof pref != 'undefined' ? pref : false);
> };
> {code}
> vs.
> {code}
> var ContactAddress = function(pref, type, formatted, streetAddress, locality, region, postalCode, country) {
>     this.id = null;
>     this.pref = (typeof pref != 'undefined' ? pref : false);
>     this.type = type || null;
>     this.formatted = formatted || null;
>     this.streetAddress = streetAddress || null;
>     this.locality = locality || null;
>     this.region = region || null;
>     this.postalCode = postalCode || null;
>     this.country = country || null;
> };
> {code}
> It means (for example) that user can create a ContactField for a phone numbers passing a value of 123456 and it will be coverted OK to '123456'. But if user passed numeric 1234 as a ContactAddress postalCode the assigned member will have the wrong type.


--
This message was sent by Atlassian JIRA
(v6.1.5#6160)