Page MenuHomePhabricator

Move classes under api folder to namespace
Closed, ResolvedPublic8 Estimated Story Points

Description

Move the following classes to namespace:

In addition to moving the classes to namespace:

  • Add dependency injection
  • Remove usage of deprecated methods
  • Use strict types
  • Check for external users of the class / methods

Details

SubjectRepoBranchLines +/-
mediawiki/extensions/Translatemaster+1 -3
mediawiki/extensions/Translatemaster+55 -53
mediawiki/extensions/Translatemaster+1 -2
mediawiki/extensions/Translatewmf/1.39.0-wmf.22+3 -4
mediawiki/extensions/Translatemaster+7 -7
mediawiki/extensions/Translatemaster+15 -17
mediawiki/extensions/Translatemaster+14 -14
mediawiki/extensions/Translatemaster+37 -31
mediawiki/extensions/Translatemaster+3 -4
mediawiki/extensions/Translatemaster+48 -20
mediawiki/extensions/Translatemaster+100 -42
mediawiki/extensions/Translatemaster+130 -103
mediawiki/extensions/Translatewmf/1.39.0-wmf.21+353 -301
mediawiki/extensions/Translatemaster+27 -21
mediawiki/extensions/Translatemaster+58 -39
mediawiki/extensions/Translatemaster+20 -12
mediawiki/extensions/Translatemaster+55 -22
mediawiki/extensions/Translatemaster+78 -47
mediawiki/extensions/Translatemaster+353 -301
mediawiki/extensions/Translatemaster+86 -68
mediawiki/extensions/Translatemaster+40 -34
mediawiki/extensions/Translatemaster+140 -90
mediawiki/extensions/Translatemaster+73 -58
Show related patches Customize query in gerrit

Event Timeline

abi_ triaged this task as Medium priority.
abi_ created this task.
abi_ updated the task description. (Show Details)
abi_ set the point value for this task to 8.
abi_ edited subscribers, added: Wangombe; removed: awight.

Change 810937 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] ReviewTranslationActiveApi: Move to namespace and add strict types

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

Change 811209 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] SearchTranslationsActionApi: Add strict types and move to namespace

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

Change 811588 had a related patch set uploaded (by Wangombe; author: Wangombe):

[mediawiki/extensions/Translate@master] GroupReviewActionApi: Add strict types and move to namespace

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

Change 811968 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] CheckTranslationActionApi: Add strict types and move to namespace

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

Change 812030 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] QueryStatsActionApi: Add strict types and move to namespace

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

Change 812237 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] QueryLanguageStatsActionApi: Add strict types and move to namespace

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

Change 812238 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] QueryMessageGroupStatsActionApi: Add strict types and move to namespace

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

Change 812858 had a related patch set uploaded (by Wangombe; author: Wangombe):

[mediawiki/extensions/Translate@master] QueryManageMessageGroupsActionApi: Add strict types; move to namespace

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

Change 812906 had a related patch set uploaded (by Wangombe; author: Wangombe):

[mediawiki/extensions/Translate@master] Add MessageGroupReview as a service

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

Change 813237 had a related patch set uploaded (by Wangombe; author: Wangombe):

[mediawiki/extensions/Translate@master] ManageMessageGroupsActionApi: Add strict types; move to namespace

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

Change 813610 had a related patch set uploaded (by Wangombe; author: Wangombe):

[mediawiki/extensions/Translate@master] QueryMessageCollectionActionApi: dd strict types and move to namespace

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

Change 813626 had a related patch set uploaded (by Wangombe; author: Wangombe):

[mediawiki/extensions/Translate@master] QueryMessageGroupsActionApi: Add strict types; move to namespace

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

Change 811588 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] GroupReviewActionApi: Add strict types and move to namespace

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

Change 812858 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] QueryManageMessageGroupsActionApi: Add strict types; move to namespace

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

Change 812906 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Use MessageGroupReview as a service

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

Change 813237 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] ManageMessageGroupsActionApi: Add strict types; move to namespace

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

Change 810937 merged by Wangombe:

[mediawiki/extensions/Translate@master] ReviewTranslationActionApi: Move to namespace and add strict types

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

Change 815292 had a related patch set uploaded (by Wangombe; author: Wangombe):

[mediawiki/extensions/Translate@master] ApiAggregateGroups: Add strict types and move to namespace

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

Change 815299 had a related patch set uploaded (by Wangombe; author: Wangombe):

[mediawiki/extensions/Translate@master] QueryMessageTranslationsActionApi: Add strict types; move to namespace

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

Change 813610 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] QueryMessageCollectionActionApi: add strict types and move to namespace

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

Change 815689 had a related patch set uploaded (by Wangombe; author: Wangombe):

[mediawiki/extensions/Translate@master] TranslatorSandboxActionApi: Use strict types and add to namespace

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

Change 811209 merged by Wangombe:

[mediawiki/extensions/Translate@master] SearchTranslationsActionApi: Add strict types and move to namespace

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

Change 811968 merged by Wangombe:

[mediawiki/extensions/Translate@master] CheckTranslationActionApi: Add strict types and move to namespace

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

Change 813626 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] QueryMessageGroupsActionApi: Add strict types; move to namespace

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

Change 815703 had a related patch set uploaded (by Wangombe; author: Wangombe):

[mediawiki/extensions/Translate@master] TranslationStashActionApi: Use strict types and add to namespace

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

Change 815708 had a related patch set uploaded (by Wangombe; author: Wangombe):

[mediawiki/extensions/Translate@master] TtmServerActionApi: Use strict types and add to namespace

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

Change 815299 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] QueryMessageTranslationsActionApi: Add strict types; move to namespace

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

Change 815292 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] AggregateGroupsActionApi: Add strict types and move to namespace

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

Change 816272 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@wmf/1.39.0-wmf.21] ReviewTranslationActionApi: Move to namespace and add strict types

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

Change 816272 merged by jenkins-bot:

[mediawiki/extensions/Translate@wmf/1.39.0-wmf.21] ReviewTranslationActionApi: Move to namespace and add strict types

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

Mentioned in SAL (#wikimedia-operations) [2022-07-25T08:15:41Z] <kartik@deploy1002> Synchronized php-1.39.0-wmf.21/extensions/Translate: Backport: [[gerrit:816272|ReviewTranslationActionApi: Move to namespace and add strict types (T312008 T313608)]] (duration: 03m 09s)

Change 815708 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] TtmServerActionApi: Use strict types and add to namespace

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

Change 815689 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] TranslatorSandboxActionApi: Use strict types and add to namespace

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

Change 815703 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] TranslationStashActionApi: Use strict types and add to namespace

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

Change 816885 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] TranslationStashActionApi: Fix incorrect constructor dependencies

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

Change 816778 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@wmf/1.39.0-wmf.22] TranslationStashActionApi: Fix incorrect constructor dependencies

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

Change 816885 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] TranslationStashActionApi: Fix incorrect constructor dependencies

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

Change 812030 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] QueryStatsActionApi: Add strict types and move to namespace

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

Change 812237 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] QueryLanguageStatsActionApi: Add strict types and move to namespace

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

Change 812238 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] QueryMessageGroupStatsActionApi: Add strict types and move to namespace

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

Change 817779 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] TranslatorSandboxActionApi: Pass user object when sending email

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

Change 817779 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] TranslatorSandboxActionApi: Pass user object when sending email

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

Change 816778 abandoned by Abijeet Patro:

[mediawiki/extensions/Translate@wmf/1.39.0-wmf.22] TranslationStashActionApi: Fix incorrect constructor dependencies

Reason:

Not required.

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

Change 819435 had a related patch set uploaded (by Abijeet Patro; author: Abijeet Patro):

[mediawiki/extensions/Translate@master] Remove remaining references to api folder

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

Change 819435 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Remove remaining references to api directories

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

Change 890231 had a related patch set uploaded (by Wangombe; author: Wangombe):

[mediawiki/extensions/Translate@master] Remove temp variable from Utilities.php

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

Change 890231 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Remove temp variable from Utilities.php

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