cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ian Clelland <iclell...@chromium.org>
Subject Re: codova-mobile-spec test case contacts.spec.21
Date Mon, 15 Jul 2013 15:53:21 GMT
Sorry for bringing this up really late --

We discovered that issue as well; the code was previously implemented for
iOS, so we concluded that the test was correct, and the Android
implementation was just lacking support.

It was logged as CB-4084, and David fixed it last week for 3.0.0.

Ian


On Tue, Jul 2, 2013 at 1:22 PM, Simon MacDonald
<simon.macdonald@gmail.com>wrote:

> If you set the email address to an empty string it should be removed from
> the contact. For some crazy reason the W3C API didn't provide a method/way
> to delete an email address, phone number, etc. So the convention for
> deleting an entry like an email address is to set it to the empty string.
>
> If that is not working then you should raise a bug:
>
> https://issues.apache.org/jira/browse/CB
>
>
> Simon Mac Donald
> http://hi.im/simonmacdonald
>
>
> On Tue, Jul 2, 2013 at 2:17 AM, Smith, Peter <peters@fast.au.fujitsu.com
> >wrote:
>
> > I'm trying to understand the cordova-mobile-spec test case
> > "contacts.spec.21 update a contact".
> >
> >
> >
> > This test case clearly expects that saving over a contact email with an
> > empty string will remove that email.
> >
> >
> >
> > But I didn't see any mention of this kind of functionality in the
> > cordova documentation:
> > http://cordova.apache.org/docs/en/2.9.0/cordova_contacts_contacts.md.htm
> > l#Contacts
> >
> >
> >
> > And I don't see any Android source code (e.g. in ContactAccessorSdk5)
> > for Contacts modification which has any logic like this to handle empty
> > emails.
> >
> >
> >
> > The test fails on my Android device.
> >
> >
> >
> > My questions are:
> >
> > *        Why is there a test written to expect something which is
> > neither documented nor implemented?
> >
> > *        Or is the test actually correct and the
> > documentation/implementation which is wrong?
> >
> > *        Or is this expected behaviour on some other (not Android)
> > platform?
> >
> >
> >
> > Thanks.
> >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message