jenkins-bot has submitted this change and it was merged. Change subject: Hygiene: Add support to Collection model for hidden status ......................................................................
Hygiene: Add support to Collection model for hidden status Change-Id: Ib7e0a55e2c5c0b8f7ad40299990a1aa4d0c8cabe --- M includes/models/Collection.php M includes/models/CollectionBase.php M includes/models/CollectionsList.php 3 files changed, 29 insertions(+), 0 deletions(-) Approvals: Jhernandez: Looks good to me, approved jenkins-bot: Verified diff --git a/includes/models/Collection.php b/includes/models/Collection.php index 04687d6..adcbd58 100644 --- a/includes/models/Collection.php +++ b/includes/models/Collection.php @@ -186,6 +186,9 @@ $owner = User::newFromName( $list['owner'] ); $collection = new Collection( $id, $owner, $list['label'], $list['description'], $list['public'], $image ); + if ( $list['perm'] === 'hidden' ) { + $collection->setHidden(); + } } } diff --git a/includes/models/CollectionBase.php b/includes/models/CollectionBase.php index ccbf6d2..944109f 100644 --- a/includes/models/CollectionBase.php +++ b/includes/models/CollectionBase.php @@ -47,6 +47,13 @@ protected $public; /** + * Whether collection has been hidden + * + * @var bool + */ + protected $hidden; + + /** * Image that represents the collection. * * @var File @@ -117,6 +124,22 @@ } /** + * Marks the collection as hidden. + */ + public function setHidden() { + $this->hidden = true; + } + + /** + * Returns if the list has been hidden by an admin. + * + * @return boolean + */ + public function isHidden() { + return $this->hidden; + } + + /** * Returns items count * * @return int count of items in collection diff --git a/includes/models/CollectionsList.php b/includes/models/CollectionsList.php index 3f5a56b..4ba464d 100644 --- a/includes/models/CollectionsList.php +++ b/includes/models/CollectionsList.php @@ -154,6 +154,9 @@ $info = new models\CollectionInfo( $list['id'], $user, $list['label'], $list['description'], $list['public'], $image ); $info->setCount( $list['count'] ); + if ( $list['perm'] === 'hidden' ) { + $info->setHidden(); + } if ( $memberTitle ) { $info->setMember( $memberTitle, $list['title'] ); } -- To view, visit https://gerrit.wikimedia.org/r/202286 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib7e0a55e2c5c0b8f7ad40299990a1aa4d0c8cabe Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Gather Gerrit-Branch: master Gerrit-Owner: Jdlrobson <jrob...@wikimedia.org> Gerrit-Reviewer: Jhernandez <jhernan...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits