Page MenuHomePhabricator

Refactor out 'infinity' variants
Closed, ResolvedPublic

Description

In ProtectionForm:
if ( $value == 'infinite' || $value == 'indefinite' || $value == 'infinity' ) {

In ApiProtect:
if ( in_array( $expiry[$i], array( 'infinite', 'indefinite', 'infinity', 'never' ) ) ) {

There's a few places in the code we check for these variants of infinity. They should be refactored out


Version: unspecified
Severity: minor

Details

Reference
bz66646
Related Changes in Gerrit:

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 22 2014, 3:13 AM
bzimport set Reference to bz66646.
bzimport added a subscriber: Unknown Object (MLST).

Change 140276 had a related patch set uploaded by devunt:
Refactor out 'infinity' variants

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

Change 140276 had a related patch set uploaded (by devunt):
Refactor out 'infinity' variants

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

Patch-For-Review

devunt raised the priority of this task from Lowest to Medium.
devunt set Security to None.

I think there is actually no 'debt' in this issue.

Change 140276 merged by jenkins-bot:
Refactor out 'infinity' variants

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