Page MenuHomePhabricator

Mark some strings for translation
ClosedPublic

Authored by joshuaspence on Jun 8 2015, 8:20 AM.
Tags
None
Referenced Files
F14033145: D13200.diff
Sat, Nov 9, 4:52 PM
F14028148: D13200.id31902.diff
Fri, Nov 8, 12:01 PM
F14022352: D13200.diff
Wed, Nov 6, 4:50 PM
F14012289: D13200.id31965.diff
Fri, Nov 1, 10:42 AM
F13985161: D13200.id31902.diff
Sun, Oct 20, 6:31 PM
F13985040: D13200.diff
Sun, Oct 20, 5:41 PM
Unknown Object (File)
Oct 1 2024, 10:46 PM
Unknown Object (File)
Sep 30 2024, 4:58 PM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rPf47e69c01594: Mark some strings for translation
Summary

Add some more phtizations.

Test Plan

Eyeball it.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6618
Build 6640: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Add some missing translations.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
joshuaspence retitled this revision from Add some missing translations to Mark some strings for translation.Jun 8 2015, 8:22 AM
joshuaspence edited edge metadata.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/audit/editor/PhabricatorAuditEditor.php
612

I think this should be untranslated: it's read by client software to thread mail, not human users.

src/applications/macro/editor/PhabricatorMacroEditor.php
158

Same reasoning.

src/infrastructure/customfield/field/PhabricatorCustomFieldAttachment.php
24

Errant }.

src/infrastructure/testing/PhabricatorTestCase.php
187

Completely nitpicking, but maybe use %s, since $seed isn't necessarily numeric in the future, even though it is today.

This revision is now accepted and ready to land.Jun 8 2015, 8:51 PM

I hit this exception whilst running the unit tests, not sure why (although I haven't looked into it yet):

EXCEPTION (PhutilAggregateException): Multiple exceptions were raised during test execution.
    - Execution: PhabricatorApplicationTransactionStructureException: Attempting to apply a transaction (of class "PhabricatorUserTransaction", with type "core:edge") which has not been constructed correctly: Transaction has type "core:edge", but that transaction type is not supported by this editor (PhabricatorUserProfileEditor).
    - Shutdown: Exception: Process exited with an open transaction! The transaction will be implicitly rolled back. Calls to openTransaction() must always be paired with a call to saveTransaction() or killTransaction().
#0 /home/joshua/workspace/github.com/phacility/arcanist/src/unit/engine/PhutilUnitTestEngine.php(65): PhutilTestCase->run()
#1 /home/joshua/workspace/github.com/phacility/arcanist/src/workflow/ArcanistUnitWorkflow.php(186): PhutilUnitTestEngine->run()
#2 /home/joshua/workspace/github.com/phacility/arcanist/src/workflow/ArcanistDiffWorkflow.php(1334): ArcanistUnitWorkflow->run()
#3 /home/joshua/workspace/github.com/phacility/arcanist/src/workflow/ArcanistDiffWorkflow.php(1213): ArcanistDiffWorkflow->runUnit()
#4 /home/joshua/workspace/github.com/phacility/arcanist/src/workflow/ArcanistDiffWorkflow.php(474): ArcanistDiffWorkflow->runLintUnit()
#5 /home/joshua/workspace/github.com/phacility/arcanist/scripts/arcanist.php(382): ArcanistDiffWorkflow->run()
#6 {main}

That's unrelated, and my fault.

This revision was automatically updated to reflect the committed changes.