[MediaWiki-commits] [Gerrit] Don't return page title from DB if not marked for translation - change (mediawiki...Translate)

[Kunalgrover05 (Code Review)]

Kunalgrover05 has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/141263

Change subject: Don't return page title from DB if not marked for translation
......................................................................

Don't return page title from DB if not marked for translation

Change-Id: I28d101e15b6f45d6e83b23f6d307495632c7500d
---
M tag/TranslatablePage.php
1 file changed, 6 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Translate 
refs/changes/63/141263/1

diff --git a/tag/TranslatablePage.php b/tag/TranslatablePage.php
index 7d7b077..3f8d182 100644
--- a/tag/TranslatablePage.php
+++ b/tag/TranslatablePage.php
@@ -214,8 +214,13 @@
         */
        public function getPageDisplayTitle( $code ) {
                $section = str_replace( ' ', '_', $this->displayTitle );
-               $page = $this->getTitle()->getPrefixedDBKey();
 
+               // Don't display page title from DB if not marked for 
translation
+               if ( !in_array( $section, $this->getSections() ) ) {
+                       return false;
+               }
+
+               $page = $this->getTitle()->getPrefixedDBKey();
                return $this->getMessageGroup()->getMessage( "$page/$section", 
$code );
        }
 

-- 
To view, visit https://gerrit.wikimedia.org/r/141263
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I28d101e15b6f45d6e83b23f6d307495632c7500d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Translate
Gerrit-Branch: master
Gerrit-Owner: Kunalgrover05 <kunalgrove...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits