Jsahleen has submitted this change and it was merged.
Change subject: Throw exception if old revtag_type is still present
......................................................................
Throw exception if old revtag_type is still present
Even though it has been long deprecated, there has been no visible
indication if it is present. This forces admins to remove the table.
BC code will be removed in few months.
Bug: T60273
Change-Id: I3d9ba7a48ecf2df6b021903af52f072719a66e7b
---
M utils/RevTag.php
1 file changed, 4 insertions(+), 2 deletions(-)
Approvals:
Jsahleen: Looks good to me, approved
jenkins-bot: Verified
diff --git a/utils/RevTag.php b/utils/RevTag.php
index c918e57..7779a75 100644
--- a/utils/RevTag.php
+++ b/utils/RevTag.php
@@ -4,7 +4,6 @@
*
* @file
* @author Niklas Laxström
- * @copyright Copyright © 2011 Niklas Laxström
* @license GPL-2.0+
*/
@@ -25,7 +24,10 @@
} else {
$dbr = wfGetDB( DB_SLAVE );
if ( $dbr->tableExists( 'revtag_type' ) ) {
- return self::$schema = 1;
+ throw new Exception(
+ "Please run migrate-schema2.php. " .
+
"https://phabricator.wikimedia.org/T60273"
+ );
} else {
return self::$schema = 2;
}
--
To view, visit https://gerrit.wikimedia.org/r/187956
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3d9ba7a48ecf2df6b021903af52f072719a66e7b
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/Translate
Gerrit-Branch: master
Gerrit-Owner: Nikerabbit <[email protected]>
Gerrit-Reviewer: Jsahleen <[email protected]>
Gerrit-Reviewer: Siebrand <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits