Using the same condition for share button

(cherry picked from commit db45cbb)
femiwiki · Jun 11, 2021 · e6d6390 · e6d6390
1 parent b60136f
commit e6d6390
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/includes/SkinFemiwiki.php b/includes/SkinFemiwiki.php
@@ -27,7 +27,7 @@ public function __construct( $options = [] ) {
 		if ( $this->getUser()->isLoggedIn() ) {
 			$options['scripts'][] = 'skins.femiwiki.notifications';
 		}
-		if ( !$this->getTitle()->isSpecialPage() ) {
+		if ( $this->shouldShowShare() ) {
 			$options['scripts'][] = 'skins.femiwiki.share';
 		}
 		parent::__construct( $options );
@@ -89,11 +89,18 @@ protected function getSidebar() {
 		return [ $sidebar, $toolbox ];
 	}
 
+	/**
+	 * @return bool
+	 */
+	protected function shouldShowShare() {
+		return $this->getOutput()->getTitle()->getArticleID() !== 0;
+	}
+
 	/**
 	 * @return string|null
 	 */
 	protected function getShare() {
-		if ( $this->getOutput()->getTitle()->getArticleID() === 0 ) {
+		if ( !$this->shouldShowShare() ) {
 			return null;
 		}
 		return new ButtonWidget( [