Skip to content

Commit 42bdfd2

Browse files
authored
Merge pull request #229 from wmde/arraysyntax
Switch all arrays to short syntax
2 parents b1da10e + e55bda2 commit 42bdfd2

56 files changed

Lines changed: 1567 additions & 1571 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

phpcs.xml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,5 @@
88
</properties>
99
</rule>
1010

11-
<rule ref="Generic.Arrays.DisallowLongArraySyntax.Found">
12-
<exclude name="Generic.Arrays.DisallowLongArraySyntax.Found" />
13-
</rule>
14-
1511
<file>.</file>
1612
</ruleset>

src/DeserializerFactory.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,10 @@ public function __construct(
5959
* @return DispatchableDeserializer
6060
*/
6161
public function newEntityDeserializer() {
62-
return new DispatchingDeserializer( array(
62+
return new DispatchingDeserializer( [
6363
$this->newItemDeserializer(),
6464
$this->newPropertyDeserializer()
65-
) );
65+
] );
6666
}
6767

6868
/**

src/Deserializers/AliasGroupListDeserializer.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ private function getDeserialized( array $serialization ) {
5858
* @return AliasGroup
5959
*/
6060
private function deserializeAliasGroup( array $serialization, $languageCode ) {
61-
$aliases = array();
61+
$aliases = [];
6262

6363
foreach ( $serialization as $aliasSerialization ) {
6464
$this->assertIsValidAliasSerialization( $aliasSerialization, $languageCode );

src/Deserializers/SiteLinkDeserializer.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,11 +58,11 @@ private function getDeserialized( array $serialization ) {
5858

5959
private function getDeserializeBadges( array $serialization ) {
6060
if ( !array_key_exists( 'badges', $serialization ) ) {
61-
return array();
61+
return [];
6262
}
6363
$this->assertBadgesIsArray( $serialization );
6464

65-
$badges = array();
65+
$badges = [];
6666
foreach ( $serialization['badges'] as $badgeSerialization ) {
6767
$badges[] = $this->deserializeItemId( $badgeSerialization );
6868
}

src/Deserializers/SnakDeserializer.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ private function hasSnakType( $serialization ) {
6464
}
6565

6666
private function hasCorrectSnakType( $serialization ) {
67-
return in_array( $serialization['snaktype'], array( 'novalue', 'somevalue', 'value' ) );
67+
return in_array( $serialization['snaktype'], [ 'novalue', 'somevalue', 'value' ] );
6868
}
6969

7070
/**

src/SerializerFactory.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,10 +99,10 @@ private function shouldSerializeReferenceSnaksWithHash() {
9999
* @return Serializer
100100
*/
101101
public function newEntitySerializer() {
102-
return new DispatchingSerializer( array(
102+
return new DispatchingSerializer( [
103103
$this->newItemSerializer(),
104104
$this->newPropertySerializer()
105-
) );
105+
] );
106106
}
107107

108108
/**

src/Serializers/AliasGroupListSerializer.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ private function assertIsSerializerFor( $object ) {
5959
* @return array[]
6060
*/
6161
private function getSerialized( AliasGroupList $aliasGroupList ) {
62-
$serialization = array();
62+
$serialization = [];
6363

6464
foreach ( $aliasGroupList->getIterator() as $aliasGroup ) {
6565
$serialization[$aliasGroup->getLanguageCode()] = $this->aliasGroupSerializer->serialize( $aliasGroup );

src/Serializers/AliasGroupSerializer.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,14 +40,14 @@ private function assertIsSerializerFor( $object ) {
4040
* @return array[]
4141
*/
4242
private function getSerialized( AliasGroup $aliasGroup ) {
43-
$serialization = array();
43+
$serialization = [];
4444
$language = $aliasGroup->getLanguageCode();
4545

4646
foreach ( $aliasGroup->getAliases() as $value ) {
47-
$result = array(
47+
$result = [
4848
'language' => $language,
4949
'value' => $value
50-
);
50+
];
5151

5252
if ( $aliasGroup instanceof AliasGroupFallback ) {
5353
$result['language'] = $aliasGroup->getActualLanguageCode();

src/Serializers/ItemSerializer.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,9 +95,9 @@ public function serialize( $object ) {
9595
}
9696

9797
private function getSerialized( Item $item ) {
98-
$serialization = array(
98+
$serialization = [
9999
'type' => $item->getType()
100-
);
100+
];
101101

102102
$this->addIdToSerialization( $item, $serialization );
103103
$this->addTermsToSerialization( $item, $serialization );
@@ -128,7 +128,7 @@ private function addStatementListToSerialization( Item $item, array &$serializat
128128
}
129129

130130
private function addSiteLinksToSerialization( Item $item, array &$serialization ) {
131-
$serialization['sitelinks'] = array();
131+
$serialization['sitelinks'] = [];
132132

133133
foreach ( $item->getSiteLinkList()->toArray() as $siteLink ) {
134134
$serialization['sitelinks'][$siteLink->getSiteId()] = $this->siteLinkSerializer->serialize( $siteLink );

src/Serializers/PropertySerializer.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,10 +79,10 @@ public function serialize( $object ) {
7979
}
8080

8181
private function getSerialized( Property $property ) {
82-
$serialization = array(
82+
$serialization = [
8383
'type' => $property->getType(),
8484
'datatype' => $property->getDataTypeId(),
85-
);
85+
];
8686

8787
$this->addIdToSerialization( $property, $serialization );
8888
$this->addTermsToSerialization( $property, $serialization );

0 commit comments

Comments
 (0)