See bug 735094.
Also c.f. bug 735357 for inspiration (not that the fix is that complicated...)
Duplicate of this bug: 736009
Created attachment 606155 [details] [diff] [review] Fix v1
Created attachment 606244 [details] [diff] [review] Fix v1 Sorry, the patch was corrupted and wouldn´t apply.
Attachment #606244 - Attachment is patch: true
Comment on attachment 606244 [details] [diff] [review] Fix v1 I would actually prefer if we const'ed those values, e.g.: const READONLY = "readonly"; etc. Looks good otherwise!
Attachment #606244 - Flags: review?(philipp) → feedback+
Created attachment 606669 [details] [diff] [review] Fix v2 Yes, I thought about the constants, but the ContactDB patch didn´t use constants, so I chose to also don´t use it. Anyway, I also prefer this way.
Attachment #606669 - Flags: review?(philipp) → review+
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
You need to log in before you can comment on or make changes to this bug.