mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-20 05:42:40 +01:00
Add Subscribers to Passphrase
Summary: Fixes T9078, Adds SubscribableInterface to Passphrase. Test Plan: Create a new passphrase, see myself subscribed. Subscribe to other Passphrases. Reviewers: epriestley Reviewed By: epriestley Subscribers: Korvin Maniphest Tasks: T9078 Differential Revision: https://secure.phabricator.com/D13799
This commit is contained in:
parent
e05e539a9c
commit
b34dc6164a
3 changed files with 21 additions and 0 deletions
|
@ -5189,6 +5189,7 @@ phutil_register_library_map(array(
|
|||
'PhabricatorApplicationTransactionInterface',
|
||||
'PhabricatorPolicyInterface',
|
||||
'PhabricatorFlaggableInterface',
|
||||
'PhabricatorSubscribableInterface',
|
||||
'PhabricatorDestructibleInterface',
|
||||
'PhabricatorSpacesInterface',
|
||||
),
|
||||
|
|
|
@ -193,6 +193,8 @@ final class PassphraseCredentialViewController extends PassphraseController {
|
|||
$viewer->renderHandleList($used_by_phids));
|
||||
}
|
||||
|
||||
$properties->invokeWillRenderEvent();
|
||||
|
||||
$description = $credential->getDescription();
|
||||
if (strlen($description)) {
|
||||
$properties->addSectionHeader(
|
||||
|
|
|
@ -5,6 +5,7 @@ final class PassphraseCredential extends PassphraseDAO
|
|||
PhabricatorApplicationTransactionInterface,
|
||||
PhabricatorPolicyInterface,
|
||||
PhabricatorFlaggableInterface,
|
||||
PhabricatorSubscribableInterface,
|
||||
PhabricatorDestructibleInterface,
|
||||
PhabricatorSpacesInterface {
|
||||
|
||||
|
@ -147,6 +148,23 @@ final class PassphraseCredential extends PassphraseDAO
|
|||
return null;
|
||||
}
|
||||
|
||||
|
||||
/* -( PhabricatorSubscribableInterface )----------------------------------- */
|
||||
|
||||
|
||||
public function isAutomaticallySubscribed($phid) {
|
||||
return false;
|
||||
}
|
||||
|
||||
public function shouldShowSubscribersProperty() {
|
||||
return true;
|
||||
}
|
||||
|
||||
public function shouldAllowSubscription($phid) {
|
||||
return true;
|
||||
}
|
||||
|
||||
|
||||
/* -( PhabricatorDestructibleInterface )----------------------------------- */
|
||||
|
||||
public function destroyObjectPermanently(
|
||||
|
|
Loading…
Reference in a new issue