Page MenuHomePhabricator

Remove use of User::isBlockedGlobally and User::getGlobalBlock in UploadWizard
Closed, ResolvedPublic2 Estimated Story Points

Description

Following T257701: Add global blocks into CompositeBlocks rather than treating them separately
Remove usage of User::isBlockedGlobally and User::getGlobalBlock in UploadWizard

Usage of User::isBlockedGlobally found here
Usage of User::getGlobalBlock found here

Details

Event Timeline

TThoabala removed TThoabala as the assignee of this task.

Change 855653 had a related patch set uploaded (by AGueyte; author: AGueyte):

[mediawiki/extensions/UploadWizard@master] Remove GlobalBlocking calls

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

Tchanders updated the task description. (Show Details)

Change 855653 merged by jenkins-bot:

[mediawiki/extensions/UploadWizard@master] Remove GlobalBlocking calls

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

I tested by going to Special:UploadWizard as a logged out user with a global block. I saw the global block message. When I removed the global block, I could use Special:UploadWizard.

@AGueyte  I validated that I was receiving a block message when testing global block IP, range, XFF, local sitewide, and composite blocks as seen in the screenshots below for Special:UploadWizard. Please let me know if there is any other testing that you want me to do. If not, is it ok for me to move this to Done?

Test Environment: Local

Browser: Chrome

Global Block IP

T318896_UploadWizard_GB_IP_Chrome.png (614×1 px, 192 KB)

Global Block IP Range

T318896_UploadWizard_GB_IPRange_Chrome.png (577×2 px, 201 KB)

Global Block IP XFF

T318896_UploadWizard_GB_IPXFF_Chrome.png (633×1 px, 201 KB)

Local Block Sitewide

T318896_UploadWizard_GB_Sitewide_Chrome.png (639×2 px, 214 KB)

Composite Block

T318896_UploadWizard_GB_CompositeBlock_Chrome.png (659×2 px, 209 KB)

@AGueyte  I validated that I was receiving a block message when testing global block IP, range, XFF, local sitewide, and composite blocks as seen in the screenshots below for Special:UploadWizard. Please let me know if there is any other testing that you want me to do. If not, is it ok for me to move this to Done?

Test Environment: Local

Browser: Chrome

Global Block IP

T318896_UploadWizard_GB_IP_Chrome.png (614×1 px, 192 KB)

Global Block IP Range

T318896_UploadWizard_GB_IPRange_Chrome.png (577×2 px, 201 KB)

Global Block IP XFF

T318896_UploadWizard_GB_IPXFF_Chrome.png (633×1 px, 201 KB)

Local Block Sitewide

T318896_UploadWizard_GB_Sitewide_Chrome.png (639×2 px, 214 KB)

Composite Block

T318896_UploadWizard_GB_CompositeBlock_Chrome.png (659×2 px, 209 KB)

All good for me, thank you @GMikesell-WMF!