Page MenuHomePhabricator

Popups schema has the wrong type for popupDelay
Closed, ResolvedPublic

Description

This schema is used very sparingly now, but we discovered a bug in how the popupDelay value was stored. The refine process inferred it was an integer and we're not sure if the data in the field is correct or cast down from a meaningful number like 0 < x < 1. Please advise.

Event Timeline

phuedx claimed this task.
phuedx subscribed.

@Tbayer pointed me at this task during our 1:1 yesterday.

AFAICT it's the inferred type that is correct rather than the type specified in the schema as the underlying value is specified as a whole number of milliseconds (IIRC it should always be 500 ms).

[I've updated the type of the property to match that of the perceivedWait property](https://meta.wikimedia.org/w/index.php?title=Schema%3APopups&type=revision&diff=18904225&oldid=18791212).