#55 Incorrect case sensitivity

Alpha (0.9.x)
closed-fixed
vCard (11)
5
2010-02-06
2010-01-29
D. P.
No

Artifact 2639393 claims that the VCard parser is now case insensitive (as the VCard Spec requires). However, testing with a Vcard file generated by citadel (see attached vcard file) with a lot of lower case entries does not work.

Debugging showed that VcardBuilder:158 checks for "BEGIN:VCARD" using a regular expression that does not allow lower case (eg. "begin:vcard") to be used here. The VCard 2.1 Spec (http://www.imc.org/pdi/vcard-21.txt) states that "All literal property names are valid as upper, lower, or mixed case", so having a lower case "begin:vcard" is valid.

Please extend the regular expression to allow for all-upper, all-lower or moxed case "BEGIN:VCARD" and "END:VCARD".

Discussion

  • D. P.

    D. P. - 2010-01-29

    VCard with lower case entries

     
  • D. P.

    D. P. - 2010-02-01

    Patch to make the parser case-insensitive for "[BEGIN|END]:VCARD"

     
  • D. P.

    D. P. - 2010-02-01

    Apparently I'm not able to use sf.net correctly - this should have been a bug report rather than a patch. But oh well, I created and applied a trivial patch. I couldn't test this though as I don't have a maven installation available.

     
  • Ben Fortuna

    Ben Fortuna - 2010-02-06
    • milestone: --> Alpha (0.9.x)
    • assigned_to: nobody --> fortuna
    • status: open --> closed-fixed
     
  • Ben Fortuna

    Ben Fortuna - 2010-02-06

    Patch applied. Changes now available in CVS.

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks