Closed Bug 702137 Opened 10 years ago Closed 10 years ago
Photo Handler saves wrong photoname
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:8.0) Gecko/20100101 Firefox/8.0 Build ID: 20111104165243 Steps to reproduce: the onSave method of onSave of genericPhotoHandler in abCardOverlay.js has the statement "aCard.setProperty("PhotoName", null);". this actually stores the string "null" in the card, which is wrong. the statemant should be "aCard.setProperty("PhotoName", "");". additionally it should also clear the photoURI: "aCard.setProperty("PhotoName", null);"
Assignee: nobody → samuel.mueller
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Comment on attachment 574168 [details] [diff] [review] patch fixing the problem Requesting review on your behalf (you can do that yourself if a follow-up patch should be necessary).
Attachment #574168 - Flags: review?(mbanner)
Attachment #574168 - Flags: review?(mbanner) → review?(mconley)
Comment on attachment 574168 [details] [diff] [review] patch fixing the problem Looks good to me.
Attachment #574168 - Flags: review?(mconley) → review+
Thanks Mike. Samuel, assuming that this doesn't need any further reviews, you can now set "checkin-needed" in the "Keywords" field, then someone will push it on the comm-central repository and your fix will show up in the trunk nightly builds.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 11.0
You need to log in before you can comment on or make changes to this bug.