jsgoupil / quickbooks-sync

Sync Quickbooks Desktop
MIT License
87 stars 40 forks source link

Add zero support to GUIDTYPE #27

Closed tofer closed 5 years ago

tofer commented 5 years ago

Updated GUIDTYPE to support Intuit's "magic code" that uses a 0 for OwnerID instead of the normal hexadecimal format {00000000-0000-0000-0000-000000000000}. This is required when dealing with data extensions to reference custom fields instead of private data.

Only constructing using new GUIDTYPE("0") or parsed XML values of 0 will change the ToString() output of GUIDTYPE to keep as much backwards compatibility with existing code as possible, as hopefully verified by the added unit tests.

jsgoupil commented 5 years ago

Thanks, I'll fix some braces and ship it tomorrow.