archive/phabricator
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Fix reverting commit language

Browse Source 
Summary: I think the language here is inverted? On a commit which is reverted, the text states `$author added a reverted commit: $commit`. In reality, `$commit` is the //reverting// commit, not the //reverted// commit.

Test Plan: Think about English.

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D13067
This commit is contained in:
Joshua Spence
2015-06-01 09:54:30 +10:00
parent 65d9c0ea24
commit ebde9357c7
2 changed files with 12 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/applications/diffusion/edge/DiffusionCommitRevertedByCommitEdgeType.php
View File

@@ -19,7 +19,7 @@ final class DiffusionCommitRevertedByCommitEdgeType
$add_edges) {
return pht(
'%s added %s reverted commit(s): %s.',
'%s added %s reverting commit(s): %s.',
$actor,
$add_count,
$add_edges);
@@ -31,7 +31,7 @@ final class DiffusionCommitRevertedByCommitEdgeType
$rem_edges) {
return pht(
'%s removed %s reverted commit(s): %s.',
'%s removed %s reverting commit(s): %s.',
$actor,
$rem_count,
$rem_edges);
@@ -46,7 +46,7 @@ final class DiffusionCommitRevertedByCommitEdgeType
$rem_edges) {
return pht(
'%s edited reverted commit(s), added %s: %s; removed %s: %s.',
'%s edited reverting commit(s), added %s: %s; removed %s: %s.',
$actor,
$add_count,
$add_edges,
@@ -61,7 +61,7 @@ final class DiffusionCommitRevertedByCommitEdgeType
$add_edges) {
return pht(
'%s added %s reverted commit(s) for %s: %s.',
'%s added %s reverting commit(s) for %s: %s.',
$actor,
$add_count,
$object,
@@ -75,7 +75,7 @@ final class DiffusionCommitRevertedByCommitEdgeType
$rem_edges) {
return pht(
'%s removed %s reverted commit(s) for %s: %s.',
'%s removed %s reverting commit(s) for %s: %s.',
$actor,
$rem_count,
$object,
@@ -92,7 +92,7 @@ final class DiffusionCommitRevertedByCommitEdgeType
$rem_edges) {
return pht(
'%s edited reverted commit(s) for %s, added %s: %s; removed %s: %s.',
'%s edited reverting commit(s) for %s, added %s: %s; removed %s: %s.',
$actor,
$object,
$add_count,

12
src/applications/diffusion/edge/DiffusionCommitRevertsCommitEdgeType.php
View File

@@ -22,7 +22,7 @@ final class DiffusionCommitRevertsCommitEdgeType extends PhabricatorEdgeType {
$add_edges) {
return pht(
'%s added %s reverting commit(s): %s.',
'%s added %s reverted commit(s): %s.',
$actor,
$add_count,
$add_edges);
@@ -34,7 +34,7 @@ final class DiffusionCommitRevertsCommitEdgeType extends PhabricatorEdgeType {
$rem_edges) {
return pht(
'%s removed %s reverting commit(s): %s.',
'%s removed %s reverted commit(s): %s.',
$actor,
$rem_count,
$rem_edges);
@@ -49,7 +49,7 @@ final class DiffusionCommitRevertsCommitEdgeType extends PhabricatorEdgeType {
$rem_edges) {
return pht(
'%s edited reverting commit(s), added %s: %s; removed %s: %s.',
'%s edited reverted commit(s), added %s: %s; removed %s: %s.',
$actor,
$add_count,
$add_edges,
@@ -64,7 +64,7 @@ final class DiffusionCommitRevertsCommitEdgeType extends PhabricatorEdgeType {
$add_edges) {
return pht(
'%s added %s reverting commit(s) for %s: %s.',
'%s added %s reverted commit(s) for %s: %s.',
$actor,
$add_count,
$object,
@@ -78,7 +78,7 @@ final class DiffusionCommitRevertsCommitEdgeType extends PhabricatorEdgeType {
$rem_edges) {
return pht(
'%s removed %s reverting commit(s) for %s: %s.',
'%s removed %s reverted commit(s) for %s: %s.',
$actor,
$rem_count,
$object,
@@ -95,7 +95,7 @@ final class DiffusionCommitRevertsCommitEdgeType extends PhabricatorEdgeType {
$rem_edges) {
return pht(
'%s edited reverting commit(s) for %s, added %s: %s; removed %s: %s.',
'%s edited reverted commit(s) for %s, added %s: %s; removed %s: %s.',
$actor,
$object,
$add_count,