Change Details

Here's a list of the issues found so far after the protection form has been converted to OOUI (T235424): - Prefilling the `mwProtect-reason` input field leaves the field blank. - Prefilling the `mwProtect-edit-expires` input field also populates the `mwProtect-reason` input field with the same value. - Prefilling the `mwProtect-level-move` select field also checks the `mwProtect-cascade` checkbox. - Prefilling any field also checks the `mwProtectUnchained` checkbox. None of this should happen.

Here's a list of the issues found so far after the protection form has been converted to OOUI (T235424): - Prefilling the `mwProtect-reason` input field leaves the field blank. - Prefilling the `mwProtect-edit-expires` input field also populates the `mwProtect-reason` input field with the same value. - Prefilling the `mwProtect-level-move` select field also checks the `mwProtect-cascade` checkbox. - Prefilling any field also checks the `mwProtectUnchained` checkbox. None of this should happen. # Developer notes Previously it was possible to prefill all fields in the form using query string parameters e.g. http://localhost:8888/w/index.php?title=Map_test&action=protect&mwProtectExpirySelection-move=othertime&mwProtect-level-edit=sysop&wpProtectExpirySelection-edit=othertime&mwProtect-expiry-edit=5%20weeks&mwProtect-level-move=sysop&wpProtectExpirySelection-move=1%20month&mwProtect-cascade=1&mwProtect-reason=hello&mwProtectWatch= resulted in {F34110382} now: {F34110384} # QA On a wiki where you have permission privileges - [ ] On [[ http://localhost:8888/w/index.php?title=Map_test&action=protect&mwProtectExpirySelection-move=othertime&mwProtect-level-edit=sysop&wpProtectExpirySelection-edit=othertime&mwProtect-expiry-edit=5%20weeks&mwProtect-level-move=sysop&wpProtectExpirySelection-move=1%20month&mwProtect-cascade=1&mwProtect-reason=hello&mwProtectWatch= | URL ]] "other reason" should be hello and cascade should be checked - [ ] On [[ https://it.wikipedia.org/wiki/Peneoenanthe_pulverulenta?&action=protect&mwProtectExpirySelection-move=othertime&mwProtect-level-edit=sysop&wpProtectExpirySelection-edit=othertime&mwProtect-expiry-edit=5%20weeks&mwProtect-level-move=sysop&wpProtectExpirySelection-move=1%20month | URL ]] "cascade" should NOT be checked - [ ] On [[ http://localhost:8888/w/index.php?title=Map_test&action=protect&mwProtect-reason=hello| URL ]] the reason field should be prefilled with "hello" - [ ] On [[ http://localhost:8888/w/index.php?title=Map_test&action=protect&mwProtect-level-edit=sysop&mwProtect-reason=hello | URL ]] Unlock further protect options should not be checked

Here's a list of the issues found so far after the protection form has been converted to OOUI (T235424): - Prefilling the `mwProtect-reason` input field leaves the field blank. - Prefilling the `mwProtect-edit-expires` input field also populates the `mwProtect-reason` input field with the same value. - Prefilling the `mwProtect-level-move` select field also checks the `mwProtect-cascade` checkbox. - Prefilling any field also checks the `mwProtectUnchained` checkbox. None of this should happen. # Developer notes Previously it was possible to prefill all fields in the form using query string parameters e.g. http://localhost:8888/w/index.php?title=Map_test&action=protect&mwProtectExpirySelection-move=othertime&mwProtect-level-edit=sysop&wpProtectExpirySelection-edit=othertime&mwProtect-expiry-edit=5%20weeks&mwProtect-level-move=sysop&wpProtectExpirySelection-move=1%20month&mwProtect-cascade=1&mwProtect-reason=hello&mwProtectWatch= resulted in {F34110382} now: {F34110384} # QA On a wiki where you have permission privileges - [ ] On [[ http://localhost:8888/w/index.php?title=Map_test&action=protect&mwProtectExpirySelection-move=othertime&mwProtect-level-edit=sysop&wpProtectExpirySelection-edit=othertime&mwProtect-expiry-edit=5%20weeks&mwProtect-level-move=sysop&wpProtectExpirySelection-move=1%20month&mwProtect-cascade=1&mwProtect-reason=hello&mwProtectWatch= | URL ]] "other reason" should be hello and cascade should be checked - [ ] On [[ https://it.wikipedia.org/wiki/Peneoenanthe_pulverulenta?&action=protect&mwProtectExpirySelection-move=othertime&mwProtect-level-edit=sysop&wpProtectExpirySelection-edit=othertime&mwProtect-expiry-edit=5%20weeks&mwProtect-level-move=sysop&wpProtectExpirySelection-move=1%20month | URL ]] "cascade" should NOT be checked - [ ] On [[ http://localhost:8888/w/index.php?title=Map_test&action=protect&mwProtect-reason=hello| URL ]] the reason field should be prefilled with "hello" - [ ] On [[ http://localhost:8888/w/index.php?title=Map_test&action=protect&mwProtect-level-edit=sysop&mwProtect-reason=hello | URL ]] Unlock further protect options should not be checked