buggnokii - Bugs: bug #20901, unable to write postal address

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #20901: unable to write postal address

Submitter:  Baurzhan Ismagulov <ibr>
Submitted:  Sat 25 Aug 2007 06:24:58 PM UTC
Votes: 1
 
Category:  gnokii Severity:  3 - Normal
Item Group:  minor malfunction Status:  In Progress
Privacy:  Public Assigned to:  None
Open/Closed:  Open

Sun 10 Jul 2011 01:04:11 PM UTC, comment #5: 

Reading empty parts from a file (like when PO Box is missing) is fixed in
http://git.savannah.gnu.org/cgit/gnokii.git/commit/?id=dba2bd21c5bde1578cc713f759bb20271ca2fa4f

Writing to phone still doesn't work (tested with Nokia 5300).

Daniele Forsi <dforsi>
Group administrator
Fri 22 Oct 2010 06:00:31 PM UTC, comment #4: 

A line like
ADR;TYPE=HOME,PREF:POBOX;Extended;Street;City;State;01234;Country
fills the entry->address.* fields, but does not add anything to the entry->subentries array. (this is code in common/vcard.c)

In common/phones/nk6510.c, only fields from subentries are added. I suppose:
 a) entry->address.* fields should be added as subentries
 b) or nk6510.c should take care to translate the entry->address.* fields to the phone's format.
I am using a 6230i, which only supports a single postal address subentry. So if the postal address is split among several subentries they should either be merged, or a single entry could be created directly from the entry->address.* fields.

Right now I have implemented adding the subentries as Pobox, Zipcode etc. in common/vcard.c but the code in common/phones/nk6510.c ignores those and simply creates a phone entry from the entry->address.* fields. Since this seems doing the same thing in 2 places I would like to ask for feedback here.

Felix Engel <fcengel>
Sun 11 Jul 2010 01:55:52 PM UTC, comment #3: 

These are two different issues.

Writing the raw entry as per the first post
echo "Abc;;ME;357;5;9;0;0;Abc" |gnokii --writephonebook -o
doesn't fail now but you read back:
Abc;;ME;357;5
(gnokii 0.6.29 and Nokia 5300)

Reading ADR from vCards hits a bug in our use of sscanf() that doesn't match empty fields like :; or ;;
In this line
ADR;TYPE=HOME,PREF:;Extended;Street;City;State;01234;Country
the "PO Box" field is missing, however it still doesn't work even after adding it.

Daniele Forsi <dforsi>
Group administrator
Mon 15 Feb 2010 01:51:28 PM UTC, comment #2: 

Ping - Is there any way to get gnokii to write "ADR" vCard properties to Nokia 6300? I've even built a validator to make sure that they are formatted according to RFC 2426, but none of them are imported.

Victor Engmark <l0b0>
Wed 19 Mar 2008 12:23:35 PM UTC, comment #1: 

Ditto for vCards on Nokia 6300 - I can get addresses:
$ gnokii --getphonebook ME 1 -v
GNOKII Version 0.6.18
BEGIN:VCARD
VERSION:3.0
...
ADR;TYPE=HOME,PREF:;Extended;Street;City;State;01234;Country
...

I tried copying that line verbatim into an existing vCard, but it's just ignored when importing.

Victor Engmark <l0b0>
Sat 25 Aug 2007 06:24:58 PM UTC, original submission:  

The following works:

echo "Abc;;ME;357;5" |gnokii --writephonebook -o
Write Succeeded: memory type: Internal memory, loc: 357, name: Abc, number:

Adding the address manually and reading also does:

gnokii --getphonebook ME 357 -r
Abc;;ME;357;5;9;0;0;Abc

The following doesn't:

echo "Abc;;ME;357;5;9;0;0;Abc" |gnokii --writephonebook -o
Write FAILED (Data format is not valid.): memory type: Internal memory, loc: 357, name: Abc, number:

Baurzhan Ismagulov <ibr>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #13797:  a added by ibr (2KiB - application/octet-stream)
file #13798:  .gnokiirc added by ibr (285B - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by fcengel (Posted a comment)
  • -email is unavailable- added by dforsi (Posted a comment)
  • -email is unavailable- added by l0b0 (Posted a comment)
  • -email is unavailable- added by l0b0
  • -email is unavailable- added by l0b0 (Voted in favor of this item)
  • -email is unavailable- added by ibr (Submitted the item)
  •  

    There is 1 vote so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2010-10-22 fcengel Attached File- Added 0001-Factor-out-address-handling-in-nk6510.c-to.patch, #21759
        Attached File- Added 0002-add-separate-address-fields-as-subentries-to-a-phone.patch, #21760
    2010-07-11 dforsi StatusNone In Progress
    2008-03-19 l0b0 Carbon-Copy- Added -email is unavailable-
        Carbon-Copy- Added l0b0
    2007-08-25 ibr Attached File- Added a, #13797
        Attached File- Added .gnokiirc, #13798

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code