Page MenuHomePhabricator

Language::uc/lc return type correctness and perf review
Open, Needs TriagePublic

Description

From @aaron:

In some cases these methods would previously not return strings if passed things like
integers or floats which should be harmless. Note that ord() on an integer or
float internally casts the value to a string to get the first octet code.

Event Timeline

Krinkle created this task.Jul 29 2019, 7:26 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 29 2019, 7:26 PM

Change 522011 had a related patch set uploaded (by Krinkle; owner: Aaron Schulz):
[mediawiki/core@master] language: optimize uc(), lc(), and (uc|lc)first() and enforce return type

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

Tagging CPT for interest and possible CR to help while I'm keeping the lights one as one-man show (vacations etc.)