jenkins-bot has submitted this change and it was merged.

Change subject: Move EventLogging schema registration into extension.json
......................................................................


Move EventLogging schema registration into extension.json

Change-Id: I9dea6584d5e811d761542f207a5f63357479d644
Depends-On: I01aa2887a9b98c2ef52cacc3a892390a72b4a3ba
---
M Popups.hooks.php
M extension.json
2 files changed, 3 insertions(+), 10 deletions(-)

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



diff --git a/Popups.hooks.php b/Popups.hooks.php
index 2743b02..1852669 100644
--- a/Popups.hooks.php
+++ b/Popups.hooks.php
@@ -55,13 +55,6 @@
        }
 
        /**
-        * @param array $schemas
-        */
-       public static function onEventLoggingRegisterSchemas( array &$schemas ) 
{
-               $schemas['Popups'] = 11625443;
-       }
-
-       /**
         * @param ResourceLoader $rl
         * @return bool
         */
diff --git a/extension.json b/extension.json
index 56111f6..3f9e846 100644
--- a/extension.json
+++ b/extension.json
@@ -24,9 +24,6 @@
                "ResourceLoaderTestModules": [
                        "PopupsHooks::onResourceLoaderTestModules"
                ],
-               "EventLoggingRegisterSchemas": [
-                       "PopupsHooks::onEventLoggingRegisterSchemas"
-               ],
                "ResourceLoaderRegisterModules": [
                        "PopupsHooks::onResourceLoaderRegisterModules"
                ],
@@ -39,6 +36,9 @@
                        "i18n"
                ]
        },
+       "EventLoggingSchemas": {
+               "Popups": 11625443
+       },
        "config": {
                "@PopupsBetaFeature": "@var bool: Whether the extension should 
be enabled as an opt-in beta feature. If true, the BetaFeatures extension must 
be installed. False by default.",
                "PopupsBetaFeature": false,

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9dea6584d5e811d761542f207a5f63357479d644
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/Popups
Gerrit-Branch: master
Gerrit-Owner: Legoktm <[email protected]>
Gerrit-Reviewer: Prtksxna <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to