Page MenuHomePhabricator

Make the meme cache case-sensitive
ClosedPublic

Authored by epriestley on Jul 26 2018, 4:46 PM.
Tags
None
Referenced Files
F13178282: D19537.diff
Wed, May 8, 8:19 PM
Unknown Object (File)
Sun, May 5, 7:21 PM
Unknown Object (File)
Tue, Apr 30, 9:54 AM
Unknown Object (File)
Thu, Apr 25, 1:53 AM
Unknown Object (File)
Apr 7 2024, 3:52 AM
Unknown Object (File)
Mar 31 2024, 4:36 AM
Unknown Object (File)
Feb 14 2024, 9:05 PM
Unknown Object (File)
Feb 12 2024, 8:22 PM
Subscribers
None

Details

Summary

Fixes T13172. At one point we always capitalized all the text, and the cache uses capitalized text.

However, we stopped capitalizing the text at some point. Modern memes are more more subtle than old memes, and when we eventualy add support for things like "explodey brain" we'll certainly want to support mixed case.

Practically, this stops you from changing the capitalization of a cached meme. Get rid of the cache transform.

Test Plan

none lul

(I don't have gd installed locally and buiding it requires building libjpeg and libpng or giving up and using brew. I'l vet this in production.)

Diff Detail

Repository
rP Phabricator
Branch
meme1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20512
Build 27857: Run Core Tests
Build 27856: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jul 26 2018, 7:09 PM
This revision was automatically updated to reflect the committed changes.