Page MenuHomePhabricator
Create Task
Maniphest T250566

Replace PageContent(Insert|Save)Complete hooks
Closed, ResolvedPublic
Actions

Assigned To
DannyS712
Authored By
DannyS712
Apr 18 2020, 6:03 PM
Tags
Referenced Files
None
Subscribers
Aklapper
DannyS712
Ostrzyciel
• Pchelolo

Description

PageContentInsertComplete is called when a page is newly created
PageContentSaveComplete is called when a page revision is saved; if it was newly created both are called

Suggest replacing these with a single hook: PageSaveComplete, and having whether it is new included as a parameter.

This opportunity can also be used to clean up the hook implementations:

  • PageContentInsertComplete passes three parameters by reference for legacy reasons
  • PageContentSaveComplete passes four parameters by reference for legacy reasons
  • Both hooks include two parameters that are no longer used, isWatch and section
  • Both hooks pass a flags parameter, as well as a boolean isMinor that is simply $flags & EDIT_MINOR. The new hook would no longer have a minor parameter, and the flags would be checked.

To see if a page is newly created, use $flags & EDIT_NEW.

Deployed extensions to be updated:

Core hard deprecation: https://gerrit.wikimedia.org/r/c/mediawiki/core/+/608676

Details

SubjectRepoBranchLines +/-
Hard deprecate the `TitleMoveComplete` and `PageContent(Insert|Save)Complete` hooksmediawiki/coremaster+18 -8
Update hooks to use `PageSaveComplete`mediawiki/extensions/ConfirmEditmaster+16 -13
Update hooks to use `PageSaveComplete` hook in MW 1.35+mediawiki/extensions/TheWikipediaLibrarymaster+89 -34
Update hooks to use `PageSaveComplete` hook in MW 1.35+mediawiki/extensions/Gadgetsmaster+74 -17
Update hooks to use `PageSaveComplete`mediawiki/extensions/FlaggedRevsmaster+43 -39
Update hooks to use `PageSaveComplete` hook in MW 1.35+mediawiki/extensions/Cognatemaster+51 -1
Update hooks to use `PageSaveComplete`mediawiki/extensions/Echomaster+24 -40
Update hooks to use `PageSaveComplete`mediawiki/extensions/Jademaster+94 -143
Remove use of `PageContentSaveComplete` hook in MW 1.35+mediawiki/extensions/Translatemaster+146 -9
Update hooks to use `PageSaveComplete`mediawiki/extensions/UploadWizardmaster+17 -12
Update hooks to use `PageSaveComplete`mediawiki/extensions/WikimediaEditorTasksmaster+23 -33
Update hooks to use `PageSaveComplete`mediawiki/extensions/LiquidThreadsmaster+17 -21
Update hooks to use `PageSaveComplete` hook in MW 1.35+mediawiki/extensions/FeaturedFeedsmaster+22 -2
Update hooks to use `PageSaveComplete`mediawiki/extensions/EventBusmaster+33 -71
Update hooks to use `PageSaveComplete`mediawiki/extensions/WikimediaEventsmaster+18 -17
Update hooks to use `PageSaveComplete`mediawiki/extensions/MachineVisionmaster+19 -32
Update hooks to use `PageSaveComplete`mediawiki/extensions/ProofreadPagemaster+5 -3
Update hooks to use `PageSaveComplete`mediawiki/extensions/JsonConfigmaster+4 -5
Update hooks to use `PageSaveComplete`mediawiki/extensions/SpamBlacklistmaster+15 -21
Update hooks to use `PageSaveComplete`mediawiki/extensions/TitleBlacklistmaster+16 -8
Update hooks to use `PageSaveComplete`mediawiki/extensions/AbuseFiltermaster+27 -16
Replace PageContentInsertComplete with PageSaveCompletemediawiki/extensions/GlobalUserPagemaster+5 -3
Hooks: Update to use `PageMoveComplete` and `PageSaveComplete`mediawiki/extensions/PageTriagemaster+54 -53
Add `PageSaveComplete` hook to replace `PageContent(Insert|Save)Complete`mediawiki/coremaster+77 -7
Show related patches Customize query in gerrit

Related Objects
Search...

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
DannyS712 mentioned this in T249434: Replace hooks using Revision objects with RevisionRecord.Apr 18 2020, 6:03 PM
gerritbot added a comment.Jun 1 2020, 1:31 AM

Change 601099 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/core@master] Add PageSaveComplete hook to replace PageContent(Insert|Save)Complete

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

gerritbot added a project: Patch-For-Review.Jun 1 2020, 1:31 AM
DannyS712 moved this task from Next to In progress on the User-DannyS712 board.Jun 1 2020, 1:44 AM
gerritbot added a comment.Jun 10 2020, 9:02 PM

Change 601099 merged by jenkins-bot:
[mediawiki/core@master] Add PageSaveComplete hook to replace PageContent(Insert|Save)Complete

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

gerritbot added a comment.Jun 10 2020, 9:04 PM

Change 604509 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/PageTriage@master] Hooks: Update to use PageMoveComplete and PageSaveComplete

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

ReleaseTaggerBot added a project: MW-1.35-notes (1.35.0-wmf.37; 2020-06-16).Jun 10 2020, 10:00 PM
Ostrzyciel subscribed.Jun 14 2020, 3:23 PM

Please have a look at this patch. I'm wrapping up the implementation of the EditResult class which would provide the info already given by the last two parameters of this hook. My next task will be modifying hooks in the PageUpdater to pass the EditResult to extensions, so I would have to add a parameter to PageSaveComplete that makes some of this redundant.

I'll be doing this before the end of June. Could we possibly amend the definition of this hook before 1.35 release? I'd remove the $originalRevId and $undidRevId parameters in favour of an EditResult. That would certainly make for a much cleaner hook definition :)

gerritbot added a comment.Jun 15 2020, 9:43 PM

Change 604509 merged by jenkins-bot:
[mediawiki/extensions/PageTriage@master] Hooks: Update to use PageMoveComplete and PageSaveComplete

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

gerritbot added a comment.Jun 15 2020, 9:48 PM

Change 605679 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/GlobalUserPage@master] Replace PageContentInsertComplete with PageSaveComplete

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

gerritbot added a comment.Jun 16 2020, 12:53 AM

Change 605679 merged by jenkins-bot:
[mediawiki/extensions/GlobalUserPage@master] Replace PageContentInsertComplete with PageSaveComplete

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

Maintenance_bot removed a project: Patch-For-Review.Jun 16 2020, 1:10 AM
gerritbot added a comment.Jun 16 2020, 4:03 AM

Change 605724 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/Echo@master] Update hooks to use PageSaveComplete

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

gerritbot added a project: Patch-For-Review.Jun 16 2020, 4:03 AM
gerritbot added a comment.Jun 16 2020, 4:12 AM

Change 605725 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/AbuseFilter@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 16 2020, 4:19 AM

Change 605726 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/JsonConfig@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 16 2020, 4:22 AM

Change 605727 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/MachineVision@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 16 2020, 4:25 AM

Change 605728 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/ProofreadPage@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 16 2020, 4:29 AM

Change 605729 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/SpamBlacklist@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 16 2020, 4:30 AM

Change 605730 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/TitleBlacklist@master] Update hooks to use PageSaveComplete

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

Ostrzyciel added a comment.Jun 16 2020, 8:10 AM

Umm, sure, ignoring my comment and deploying the hook to all extensions en masse seems like a great idea.

It will be very funny for people to read docs referencing this hook. "Introduced in 1.35 with this set of parameters, also changed in 1.35 to add another parameter that makes two other params redundant."

DannyS712 added a comment.Jun 16 2020, 8:37 AM
In T250566#6227253, @Ostrzyciel wrote:

Umm, sure, ignoring my comment and deploying the hook to all extensions en masse seems like a great idea.

It will be very funny for people to read docs referencing this hook. "Introduced in 1.35 with this set of parameters, also changed in 1.35 to add another parameter that makes two other params redundant."

I'm sorry, I didn't see your comment earlier. Given the urgency to try and finish deprecating the Revision class before 1.35 is cut, I'd rather not wait for EditResult, sorry

Ostrzyciel added a comment.Jun 16 2020, 8:49 AM
In T250566#6227355, @DannyS712 wrote:

I'm sorry, I didn't see your comment earlier. Given the urgency to try and finish deprecating the Revision class before 1.35 is cut, I'd rather not wait for EditResult, sorry

Then I am to be sorry for assuming bad intent! :D

I thought I should be able to finish it this week (my mentor promised me a CR today), and changing the hooks shouldn't be too complicated, assuming they are not deployed yet, that's why I suggested it. Nevertheless, if this is an urgent matter and you are afraid it could be stalled by me (which is possible), I can just add a parameter later.

gerritbot added a comment.Jun 16 2020, 5:16 PM

Change 605725 merged by jenkins-bot:
[mediawiki/extensions/AbuseFilter@master] Update hooks to use PageSaveComplete

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

ReleaseTaggerBot edited projects, added MW-1.35-notes (1.35.0-wmf.38; 2020-06-23); removed MW-1.35-notes (1.35.0-wmf.37; 2020-06-16).Jun 16 2020, 6:00 PM
DannyS712 added projects: AbuseFilter, PageTriage, GlobalUserPage, Jade, Cognate, MediaWiki-extensions-Translate, ConfirmEdit (CAPTCHA extension), Notifications, Event-Platform, MediaWiki-extensions-FlaggedRevs, MediaWiki-extensions-Gadgets, JsonConfig, MediaWiki-extensions-LiquidThreads, MachineVision, ProofreadPage, SpamBlacklist, The-Wikipedia-Library, TitleBlacklist, UploadWizard, WikimediaEditorTasks, MediaWiki-extensions-WikimediaEvents, MediaWiki-extensions-FeaturedFeeds.Jun 18 2020, 1:08 PM
DannyS712 updated the task description. (Show Details)
Restricted Application added projects: Growth-Team, Analytics, Product-Infrastructure-Team-Backlog-Deprecated, Structured-Data-Backlog. · View Herald TranscriptJun 18 2020, 1:08 PM
• fdans edited projects, added Analytics-Radar; removed Analytics.Jun 18 2020, 3:48 PM
gerritbot added a comment.Jun 22 2020, 9:45 AM

Change 606976 had a related patch set uploaded (by Ostrzyciel; owner: Ostrzyciel):
[mediawiki/core@master] [WIP] Modify PageSaveComplete hook to use EditResult

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

gerritbot added a comment.Jun 22 2020, 9:54 AM

Change 606980 had a related patch set uploaded (by Ostrzyciel; owner: Ostrzyciel):
[mediawiki/extensions/AbuseFilter@master] Update PageSaveComplete hook to use EditResult

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

gerritbot added a comment.Jun 22 2020, 9:59 AM

Change 606981 had a related patch set uploaded (by Ostrzyciel; owner: Ostrzyciel):
[mediawiki/extensions/PageTriage@master] Update PageSaveComplete hook to use EditResult

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

DannyS712 added a comment.Jun 22 2020, 2:13 PM

@Ostrzyciel the work to use EditResult shouldn't be tagged as part of this task

Ostrzyciel added a comment.Jun 22 2020, 2:33 PM
In T250566#6244591, @DannyS712 wrote:

@Ostrzyciel the work to use EditResult shouldn't be tagged as part of this task

Alright, I'll remove it. I thought it's relevant as these patches are about this hook.

DannyS712 added a comment.Jun 22 2020, 2:35 PM
In T250566#6244668, @Ostrzyciel wrote:
In T250566#6244591, @DannyS712 wrote:

@Ostrzyciel the work to use EditResult shouldn't be tagged as part of this task

Alright, I'll remove it. I thought it's relevant as these patches are about this hook.

This is specifically about replacing the old hooks, not improvements to the new hook; thanks

gerritbot added a comment.Jun 23 2020, 7:34 PM

Change 607352 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/ConfirmEdit@master] Update hooks to use PageSaveComplete

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

DannyS712 updated the task description. (Show Details)Jun 23 2020, 7:38 PM
gerritbot added a comment.Jun 24 2020, 1:38 AM

Change 605730 merged by jenkins-bot:
[mediawiki/extensions/TitleBlacklist@master] Update hooks to use PageSaveComplete

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

ReleaseTaggerBot edited projects, added MW-1.35-notes (1.35.0-wmf.39; 2020-06-30); removed MW-1.35-notes (1.35.0-wmf.38; 2020-06-23).Jun 24 2020, 2:00 AM
gerritbot added a comment.Jun 24 2020, 2:01 AM

Change 605729 merged by jenkins-bot:
[mediawiki/extensions/SpamBlacklist@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 24 2020, 2:17 AM

Change 605726 merged by jenkins-bot:
[mediawiki/extensions/JsonConfig@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 24 2020, 2:17 AM

Change 605728 merged by jenkins-bot:
[mediawiki/extensions/ProofreadPage@master] Update hooks to use PageSaveComplete

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

DannyS712 updated the task description. (Show Details)Jun 24 2020, 2:25 AM
LGoto moved this task from Needs triage to Tracking on the Product-Infrastructure-Team-Backlog-Deprecated board.Jun 24 2020, 3:49 PM
gerritbot added a comment.Jun 24 2020, 8:57 PM

Change 605727 merged by jenkins-bot:
[mediawiki/extensions/MachineVision@master] Update hooks to use PageSaveComplete

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

DannyS712 mentioned this in T246284: Hard deprecate the Revision class.Jun 25 2020, 3:45 AM
DannyS712 updated the task description. (Show Details)
gerritbot added a comment.Jun 25 2020, 3:54 AM

Change 607656 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/Translate@master] Remove use of PageContentSaveComplete hook in MW 1.35+

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

gerritbot added a comment.Jun 25 2020, 5:55 AM

Change 607661 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/Jade@master] Update hooks to use PageSaveComplete

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

DannyS712 updated the task description. (Show Details)Jun 25 2020, 6:20 AM
gerritbot added a comment.Jun 25 2020, 7:08 AM

Change 607716 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/FlaggedRevs@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 25 2020, 4:27 PM

Change 607830 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/EventBus@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 25 2020, 4:33 PM

Change 607831 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/LiquidThreads@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 25 2020, 4:39 PM

Change 607833 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/UploadWizard@master] Update hooks to use PageSaveComplete

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

DannyS712 updated the task description. (Show Details)Jun 25 2020, 4:45 PM
gerritbot added a comment.Jun 25 2020, 4:46 PM

Change 607835 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/WikimediaEvents@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 25 2020, 4:56 PM

Change 607836 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/WikimediaEditorTasks@master] Update hooks to use PageSaveComplete

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

DannyS712 updated the task description. (Show Details)Jun 25 2020, 4:59 PM
gerritbot added a comment.Jun 25 2020, 5:10 PM

Change 607837 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/Gadgets@master] Update hooks to use PageSaveComplete hook in MW 1.35+

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

gerritbot added a comment.Jun 25 2020, 5:21 PM

Change 607835 merged by jenkins-bot:
[mediawiki/extensions/WikimediaEvents@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 25 2020, 5:23 PM

Change 607840 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/Cognate@master] Update hooks to use PageSaveComplete hook in MW 1.35+

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

DannyS712 updated the task description. (Show Details)Jun 25 2020, 5:29 PM
gerritbot added a comment.Jun 25 2020, 5:33 PM

Change 607842 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/TheWikipediaLibrary@master] Update hooks to use PageSaveComplete hook in MW 1.35+

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

gerritbot added a comment.Jun 25 2020, 5:41 PM

Change 607844 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/FeaturedFeeds@master] Update hooks to use PageSaveComplete hook in MW 1.35+

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

DannyS712 mentioned this in T256403: Tests failing for the master branch of TheWikipediaLibrary.Jun 25 2020, 5:48 PM
DannyS712 updated the task description. (Show Details)Jun 25 2020, 5:50 PM
DannyS712 mentioned this in T250023: Replace TitleMoveComplet(e|ing) hooks.Jun 25 2020, 6:27 PM
gerritbot added a comment.Jun 25 2020, 7:40 PM

Change 607830 merged by jenkins-bot:
[mediawiki/extensions/EventBus@master] Update hooks to use PageSaveComplete

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

DannyS712 mentioned this in T256423: Tests failing for the master branch of ConfirmEdit.Jun 25 2020, 8:17 PM
gerritbot added a comment.Jun 25 2020, 11:05 PM

Change 607844 merged by jenkins-bot:
[mediawiki/extensions/FeaturedFeeds@master] Update hooks to use PageSaveComplete hook in MW 1.35+

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

gerritbot added a comment.Jun 25 2020, 11:05 PM

Change 607831 merged by jenkins-bot:
[mediawiki/extensions/LiquidThreads@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 25 2020, 11:05 PM

Change 607833 merged by jenkins-bot:
[mediawiki/extensions/UploadWizard@master] Update hooks to use PageSaveComplete

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

gerritbot added a comment.Jun 25 2020, 11:05 PM

Change 607836 merged by jenkins-bot:
[mediawiki/extensions/WikimediaEditorTasks@master] Update hooks to use PageSaveComplete

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

DannyS712 updated the task description. (Show Details)Jun 25 2020, 11:07 PM
DannyS712 mentioned this in rEWET093c83d1c070: Update hooks to use `PageSaveComplete`.Jun 25 2020, 11:07 PM
gerritbot added a comment.Jun 26 2020, 9:27 AM

Change 607656 merged by jenkins-bot:
[mediawiki/extensions/Translate@master] Remove use of PageContentSaveComplete hook in MW 1.35+

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

gerritbot added a comment.Jun 30 2020, 12:38 AM

Change 607661 merged by jenkins-bot:
[mediawiki/extensions/Jade@master] Update hooks to use PageSaveComplete

https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Jade/ /607661

DannyS712 updated the task description. (Show Details)Jun 30 2020, 12:42 AM
gerritbot added a comment.Jun 30 2020, 10:40 AM

Change 605724 merged by jenkins-bot:
[mediawiki/extensions/Echo@master] Update hooks to use PageSaveComplete

https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Echo/ /605724

DannyS712 updated the task description. (Show Details)Jun 30 2020, 10:42 AM
ReleaseTaggerBot edited projects, added MW-1.35-notes (1.35.0-wmf.40; 2020-07-07); removed MW-1.35-notes (1.35.0-wmf.39; 2020-06-30).Jun 30 2020, 11:00 AM
gerritbot added a comment.Jun 30 2020, 3:18 PM

Change 607840 merged by jenkins-bot:
[mediawiki/extensions/Cognate@master] Update hooks to use PageSaveComplete hook in MW 1.35+

https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Cognate/ /607840

gerritbot added a comment.Jun 30 2020, 3:22 PM

Change 607716 merged by jenkins-bot:
[mediawiki/extensions/FlaggedRevs@master] Update hooks to use PageSaveComplete

https://gerrit.wikimedia.org/r/c/mediawiki/extensions/FlaggedRevs/ /607716

gerritbot added a comment.Jun 30 2020, 3:42 PM

Change 607837 merged by jenkins-bot:
[mediawiki/extensions/Gadgets@master] Update hooks to use PageSaveComplete hook in MW 1.35+

https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Gadgets/ /607837

DannyS712 updated the task description. (Show Details)Jun 30 2020, 4:00 PM
gerritbot added a comment.Jun 30 2020, 11:49 PM

Change 607352 merged by Ppchelko:
[mediawiki/extensions/ConfirmEdit@master] Update hooks to use PageSaveComplete

https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ConfirmEdit/ /607352

gerritbot added a comment.Jun 30 2020, 11:51 PM

Change 607842 merged by Ppchelko:
[mediawiki/extensions/TheWikipediaLibrary@master] Update hooks to use PageSaveComplete hook in MW 1.35+

https://gerrit.wikimedia.org/r/c/mediawiki/extensions/TheWikipediaLibrary/ /607842

gerritbot added a comment.Jul 1 2020, 12:22 AM

Change 608676 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/core@master] Hard deprecate the TitleMoveComplete and PageContent(Insert|Save)Complete hooks

https://gerrit.wikimedia.org/r/c/mediawiki/core/ /608676

DannyS712 updated the task description. (Show Details)Jul 1 2020, 12:23 AM
DannyS712 moved this task from In progress to Awaiting review and deployment on the User-DannyS712 board.
DannyS712 updated the task description. (Show Details)
DannyS712 mentioned this in rEWPL0dd780adf78e: Update hooks to use `PageSaveComplete` hook in MW 1.35+.Jul 1 2020, 1:44 AM
gerritbot added a comment.Jul 1 2020, 2:38 AM

Change 608676 merged by jenkins-bot:
[mediawiki/core@master] Hard deprecate the TitleMoveComplete and PageContent(Insert|Save)Complete hooks

https://gerrit.wikimedia.org/r/c/mediawiki/core/ /608676

DannyS712 closed this task as Resolved.Jul 1 2020, 3:08 AM
DannyS712 removed a project: Patch-For-Review.
Reedy mentioned this in T257032: Question: Why do deprecations break CI tests? .Jul 3 2020, 1:27 PM
DannyS712 moved this task from Awaiting review and deployment to Resolved (deprecation) on the User-DannyS712 board.Jul 9 2020, 11:12 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL