For removing a keyword from a bookmark, the new API expects update to be called with an empty string keyword value, rather than null. The empty string is then stored in the DB. This makes the old API (getKeywordForBookmark) and maybe also the new API (fetch, I didn't test) return different values for keyword-less bookmarks, depending on whether they have ever had a keyword (an empty string returned if they did; null if they didn't). This breaks the test for the EditKeyword transaction if it's converted to use the update API.
4 years ago
Summary: Removing a keyword sets it to an empty string → aNewValue=undefined in onItemChange for removed keyword
Created attachment 8528390 [details] [diff] [review] patch
Assignee: nobody → mano
Status: NEW → ASSIGNED
Attachment #8528390 - Flags: review?(mak77)
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Flags: in-testsuite? → in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.