[MediaWiki-commits] [Gerrit] mediawiki...PageTriage[master]: PageTriageHooks: Use modifyExtensionTable() now that it exists

2017-11-21 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/391153 )

Change subject: PageTriageHooks: Use modifyExtensionTable() now that it exists
..


PageTriageHooks: Use modifyExtensionTable() now that it exists

Depends-On: I20368bf3c007a01718513a435de24907dc0aaf81
Change-Id: I65e647669cb11c18e092645c27e36655d51dc699
---
M PageTriage.hooks.php
1 file changed, 3 insertions(+), 5 deletions(-)

Approvals:
  jenkins-bot: Verified
  Jforrester: Looks good to me, approved



diff --git a/PageTriage.hooks.php b/PageTriage.hooks.php
index 2f395a5..debd516 100644
--- a/PageTriage.hooks.php
+++ b/PageTriage.hooks.php
@@ -839,12 +839,10 @@
'ptrl_comment',
$base . '/PageTriageLogPatch_Drop_ptrl_comment.sql'
);
-   $updater->addExtensionUpdate( [
-   'modifyTable',
+   $updater->modifyExtensionTable(
'pagetriage_tags',
-   $base . '/PageTriageTagsPatch.sql',
-   true
-   ] );
+   $base . '/PageTriageTagsPatch.sql'
+   );
 
return true;
}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I65e647669cb11c18e092645c27e36655d51dc699
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/PageTriage
Gerrit-Branch: master
Gerrit-Owner: Catrope 
Gerrit-Reviewer: Jforrester 
Gerrit-Reviewer: jenkins-bot <>

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


[MediaWiki-commits] [Gerrit] mediawiki...PageTriage[master]: PageTriageHooks: Use modifyExtensionTable() now that it exists

2017-11-13 Thread Catrope (Code Review)
Catrope has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/391153 )

Change subject: PageTriageHooks: Use modifyExtensionTable() now that it exists
..

PageTriageHooks: Use modifyExtensionTable() now that it exists

Depends-On: I20368bf3c007a01718513a435de24907dc0aaf81
Change-Id: I65e647669cb11c18e092645c27e36655d51dc699
---
M PageTriage.hooks.php
1 file changed, 3 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/PageTriage 
refs/changes/53/391153/1

diff --git a/PageTriage.hooks.php b/PageTriage.hooks.php
index 2f395a5..debd516 100644
--- a/PageTriage.hooks.php
+++ b/PageTriage.hooks.php
@@ -839,12 +839,10 @@
'ptrl_comment',
$base . '/PageTriageLogPatch_Drop_ptrl_comment.sql'
);
-   $updater->addExtensionUpdate( [
-   'modifyTable',
+   $updater->modifyExtensionTable(
'pagetriage_tags',
-   $base . '/PageTriageTagsPatch.sql',
-   true
-   ] );
+   $base . '/PageTriageTagsPatch.sql'
+   );
 
return true;
}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I65e647669cb11c18e092645c27e36655d51dc699
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PageTriage
Gerrit-Branch: master
Gerrit-Owner: Catrope 

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