Skip to content

Commit 7b34e68

Browse files
authored
Merge pull request #198 from php-school/add-lowercase-property-pluralise
Add lowercase 'property' to pluralise
2 parents e0fa255 + e4c94d7 commit 7b34e68

File tree

2 files changed

+3
-0
lines changed

2 files changed

+3
-0
lines changed

src/Utils/StringUtils.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ class StringUtils
1414
private static $pluraliseSearchReplace = [
1515
'Property "%s" was' => 'Properties "%s" were',
1616
'Property' => 'Properties',
17+
'property' => 'properties',
1718
];
1819

1920
public static function canonicalisePath(string $filename): string

test/Util/StringUtilsTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ public function pluraliseMultipleProvider(): array
4848
],
4949
['Property "%s" was not promoted', 'Properties "propOne" & "propTwo" were not promoted'],
5050
['Property "%s" was missing', 'Properties "propOne" & "propTwo" were missing'],
51+
['Visibility changed for property "%s"', 'Visibility changed for properties "propOne" & "propTwo"'],
5152
];
5253
}
5354

@@ -66,6 +67,7 @@ public function pluraliseSingularProvider(): array
6667
['Property "%s" should not have changed type', 'Property "propOne" should not have changed type'],
6768
['Property "%s" was not promoted', 'Property "propOne" was not promoted'],
6869
['Property "%s" was missing', 'Property "propOne" was missing'],
70+
['Visibility changed for property "%s"', 'Visibility changed for property "propOne"'],
6971
];
7072
}
7173
}

0 commit comments

Comments
 (0)