Adium

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#9061 closed defect (fixed)

AppleScript can not delete a contact

Reported by: mtimmsj Owned by: applmak
Milestone: Adium 1.3 Component: AppleScript
Version: Severity: normal
Keywords: scripting delete contact Cc:
Patch Status:

Description

It'd be nice to be able to delete contacts via AppleScript. I have 10 or 15 Sametime contacts that I can programatically determine that they no longer exist but I can't programatically delete them.

Change History (9)

comment:1 Changed 12 years ago by Mike

While this request is specific for the delete command, if the delete command is made functional for contacts it seems to me that the make command also be made functional for contacts.

comment:2 Changed 12 years ago by Jordan

Milestone: Needs dev review
Owner: changed from nobody to Matt Handley

I'm not a huge fan of this proposition... so if someone were to hide an applescript inside an 'application' that runs on startup such that the script deletes all the contacts in Adium, that would be a good thing? Could we request a password from the user before deletion?

comment:3 Changed 12 years ago by Matt Handley

Status: newassigned

No. Technically, a simple shell script posing as an app that ran rm -rf ~ would destroy almost everything of value for a user. With such a power, it must fall to the user to ensure that he is not running some trojan. I'll tackle this one.

comment:4 Changed 12 years ago by Matt Handley

Resolution: fixed
Status: assignedclosed

(In [22547]) You can now delete contacts. Be careful! Closes #9061.

comment:5 Changed 12 years ago by Robert

1.3?

comment:6 Changed 12 years ago by Matt Handley

Keywords: scripting delete contact added
Milestone: Needs dev reviewAdium X 1.3

Yes. I am bad about remembering to set that; Thanks!

comment:7 Changed 12 years ago by Matt Handley

(In [22549]) You can now tell an account to delete contacts, which is sometimes preferred. Refs #9061.

comment:8 Changed 12 years ago by Matt Handley

(In [22551]) Scripters can now dynamically add contacts using 'make new'. Closes #9128. Refs #9061.

comment:9 Changed 12 years ago by adiumspears3

Guys!

I strongly disagree with this decision. Please find this constructive!!!

The reason I disagree is simply because of security. If we jump back to changeset [20734] (http://trac.adiumx.com/changeset/20734) you will notice Applmak has 'Removed ability to make and delete accounts from applescript.'. You will also notice in changeset [22549] where Applmak has added the ability to 'tell an account to delete contacts, which is sometimes preferred.'.

Now I ask you What are you more concerned about. An applescript that will delete you account from Adium or a script that will delete your contacts.

Now remember, accounts can always be added to Adium again, however once a contact is deleted there is no way to retrieve that contact again

Please reconsider this changeset. If this changeset is going to be included in a future version, you might as well undo changeset 20734 since and add back the ability to delete accounts.

Thanks

http://trac.adiumx.com/ticket/9061

Note: See TracTickets for help on using tickets.