Page MenuHomePhabricator

Use PhutilClassMapQuery
ClosedPublic

Authored by joshuaspence on Jun 25 2015, 12:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 11, 7:46 PM
Unknown Object (File)
Sun, Dec 29, 7:41 AM
Unknown Object (File)
Thu, Dec 26, 6:17 PM
Unknown Object (File)
Thu, Dec 26, 4:54 PM
Unknown Object (File)
Tue, Dec 24, 3:19 PM
Unknown Object (File)
Dec 20 2024, 5:37 PM
Unknown Object (File)
Dec 20 2024, 11:52 AM
Unknown Object (File)
Dec 13 2024, 11:44 PM
Subscribers

Details

Summary

Use PhutilClassMapQuery where appropriate.

Test Plan

Not quite sure.

Diff Detail

Repository
rPHU libphutil
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6996
Build 7048: [Placeholder Plan] Wait for 30 Seconds
Build 7047: arc lint + arc unit

Event Timeline

joshuaspence retitled this revision from to Use PhutilClassMapQuery.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

This is not quite correct: translations do not have unique locale codes, and it's OK for a bunch of translations to translate the same locale.

The use case here is if you develop a third-party application like "SuperCountdown", you might want to translate it into BritishEnglish. You can add a translation file to your application and distribute it with the rest of your code, with just your application's strings, and it will supplement the existing main translation file.

This should be OK without the setUniqueMethod() call.

src/internationalization/PhutilTranslation.php
72–74

This part is potentially combining the maps for a bunch of third-party code.

This revision now requires changes to proceed.Jun 25 2015, 12:15 PM
joshuaspence edited edge metadata.

Retain original behavior

epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 25 2015, 12:22 PM
This revision was automatically updated to reflect the committed changes.