Page MenuHomePhabricator
Create Task
Maniphest T9945

Unable to delete owner packages, it does not implement PhabricatorDestructibleInterface
Closed, ResolvedPublic
Actions

Description

I need to remove some obsolete owner packages in order to prevent them from being added as auditors (archiving still adds them as auditors, possibly another bug?).

However, after locating its PHID and executing a destroy (./remove destroy PHID-OPKG-ik43d6ip3qvlrqazgx4s), I get the following error:

Usage Exception: Object "PHID-OPKG-ik43d6ip3qvlrqazgx4s" can not be destroyed (it does not implement PhabricatorDestructibleInterface).

So it is, at this moment, impossible to delete them.

Reproduction steps:

  1. Create a new owner package at: /owners/edit/form/default/
  2. Locate the PHID in order to destroy it (I used the owners.query api method in conduit, because I could not find it any other way)
  3. Execute a remove destroy in the terminal

I filed this bug report as suggested in Q230

Revisions and Commits

rP Phabricator
D14729rP7c98cd85fe6d Implement DestructibleInterface for Owners Packages

Event Timeline

JohnHeroHD created this task.Dec 10 2015, 7:27 AM
JohnHeroHD raised the priority of this task from to Needs Triage.
JohnHeroHD updated the task description. (Show Details)
JohnHeroHD changed the edit policy from "All Users" to "Custom Policy".
JohnHeroHD added a project: Bug Report.
JohnHeroHD added a subscriber: JohnHeroHD.
JohnHeroHD updated the task description. (Show Details)Dec 10 2015, 7:39 AM
joshuaspence updated the task description. (Show Details)Dec 10 2015, 9:12 AM
joshuaspence added a project: Owners.
joshuaspence added a subscriber: joshuaspence.
JohnHeroHD updated the task description. (Show Details)Dec 10 2015, 9:15 AM
epriestley claimed this task.Dec 10 2015, 1:15 PM
epriestley added a revision: D14729: Implement DestructibleInterface for Owners Packages.Dec 10 2015, 1:18 PM
epriestley closed this task as Resolved by committing rP7c98cd85fe6d: Implement DestructibleInterface for Owners Packages.Dec 10 2015, 3:04 PM
epriestley added a commit: rP7c98cd85fe6d: Implement DestructibleInterface for Owners Packages.
JohnHeroHD added a comment.Dec 10 2015, 4:05 PM

Thanks @epriestley, much appreciated!

I have 2 questions regarding this, if you do not mind.

  1. What will happen to audits containing the owner package as an auditor after destroying the owner package?

Will the owner simply disappear from the audit, and if he was the only one still needing to audit, and will the audit itself it be marked as completely audited? I think this will make the most sense.

I would like to make sure that I will not be messing up anything :-)

  1. Do you have an ETA when I will be able to pull this from the stable branch?
Phabricator · Built by Phacility