search

cozy / cozy-vcard

Cozy-vcard provides vcard Parser and Objects
Other
29 stars 17 forks source link

fix cases of composed line where datapoint is not initialized #17

Closed mizrachiran closed 9 years ago

mizrachiran commented 9 years ago

This small change fix bug #16 (multiple vcards import cause an error). Was verified carefully with multiple sets of vcards. all UTs pass.

clochix commented 9 years ago

Thanks a lot @mizrachiran ! @jacquarg or @aenario, could you have a look at this please ?

frankrousseau commented 9 years ago

@mizrachiran Could you add tests (btw thank you for the fix!) ?

mizrachiran commented 9 years ago

Hi @frankrousseau,

I added a test, it doesn't check much but it will function as a gate to ensure this functionality.

mizrachiran commented 9 years ago

I would appreciate if you will release an update to the npm package once you merge this change.

Thanks!

frankrousseau commented 9 years ago

Great thank you!

frankrousseau commented 9 years ago

Published as 0.2.14