Closed Bug 439498 Opened 16 years ago Closed 16 years ago

Its possible to attempt to delete mailing lists from OS X Address book via TB

Categories

(MailNews Core :: Address Book, defect)

All
macOS
defect
Not set
major

Tracking

(Not tracked)

VERIFIED FIXED
mozilla1.9

People

(Reporter: hwaara, Assigned: standard8)

Details

Attachments

(1 file)

If I select a mailing list (which is from the OS X address book) and click the delete button, it doesn't work and I get this:

An error occurred executing the button_delete command
[Exception... "Component returned failure code: 0x80004001 (NS_ERROR_NOT_IMPLEMENTED) [nsIAbManager.deleteAddressBook]"  nsresult: "0x80004001 (NS_ERROR_NOT_IMPLEMENTED)"  location: "JS frame :: chrome://messenger/content/addressbook/abCommon.js :: AbDeleteDirectory :: line 334"  data: no]
The problem here is that we're not disabling the relevant button if the address book is read-only.
Flags: wanted-thunderbird3+
Summary: Cannot delete (OS X address book) mailing list from TB → Its possible to attempt to delete mailing lists from OS X Address book via TB
Attached patch The fixSplinter Review
Here's the fix, quite easy as it turned out, when checking cmd_delete or button_delete just ensure the directory does or doesn't have the opWrite flag set.
Assignee: nobody → bugzilla
Status: NEW → ASSIGNED
Attachment #325310 - Flags: superreview?(bienvenu)
Attachment #325310 - Flags: review?(bienvenu)
Attachment #325310 - Flags: superreview?(bienvenu)
Attachment #325310 - Flags: superreview+
Attachment #325310 - Flags: review?(bienvenu)
Attachment #325310 - Flags: review+
Checked in -> fixed. For testers, ensure that Mac Mailing lists don't have the delete option available, and that the delete option remains available on address books (apart from PAB & CAB) and other types of mailing lists.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9
Verified with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.0.2pre) Gecko/2008070703 Thunderbird/3.0a2pre ID:2008070703
Status: RESOLVED → VERIFIED
Hardware: PC → All
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: