@@ -71,20 +71,20 @@ discard block |
||
| 71 | 71 | * |
| 72 | 72 | * @return string HTML |
| 73 | 73 | */ |
| 74 | - public function formatValue( $value ) { |
|
| 75 | - if ( is_string( $value ) ) { |
|
| 74 | + public function formatValue($value) { |
|
| 75 | + if (is_string($value)) { |
|
| 76 | 76 | // Cases like 'Format' 'pattern' or 'minimum'/'maximum' values, which we have stored as |
| 77 | 77 | // strings |
| 78 | - return htmlspecialchars( $value ); |
|
| 79 | - } elseif ( $value instanceof EntityId ) { |
|
| 78 | + return htmlspecialchars($value); |
|
| 79 | + } elseif ($value instanceof EntityId) { |
|
| 80 | 80 | // Cases like 'Conflicts with' 'property', to which we can link |
| 81 | - return $this->formatEntityId( $value ); |
|
| 82 | - } elseif ( $value instanceof ItemIdSnakValue ) { |
|
| 81 | + return $this->formatEntityId($value); |
|
| 82 | + } elseif ($value instanceof ItemIdSnakValue) { |
|
| 83 | 83 | // Cases like EntityId but can also be somevalue or novalue |
| 84 | - return $this->formatItemIdSnakValue( $value ); |
|
| 84 | + return $this->formatItemIdSnakValue($value); |
|
| 85 | 85 | } else { |
| 86 | 86 | // Cases where we format a DataValue |
| 87 | - return $this->formatDataValue( $value ); |
|
| 87 | + return $this->formatDataValue($value); |
|
| 88 | 88 | } |
| 89 | 89 | } |
| 90 | 90 | |
@@ -95,19 +95,19 @@ discard block |
||
| 95 | 95 | * |
| 96 | 96 | * @return string|null HTML |
| 97 | 97 | */ |
| 98 | - public function formatParameters( $parameters ) { |
|
| 99 | - if ( $parameters === null || $parameters === [] ) { |
|
| 98 | + public function formatParameters($parameters) { |
|
| 99 | + if ($parameters === null || $parameters === []) { |
|
| 100 | 100 | return null; |
| 101 | 101 | } |
| 102 | 102 | |
| 103 | 103 | $formattedParameters = []; |
| 104 | - foreach ( $parameters as $parameterName => $parameterValue ) { |
|
| 105 | - $formattedParameterValues = implode( ', ', |
|
| 106 | - $this->limitArrayLength( array_map( [ $this, 'formatValue' ], $parameterValue ) ) ); |
|
| 107 | - $formattedParameters[] = sprintf( '%s: %s', $parameterName, $formattedParameterValues ); |
|
| 104 | + foreach ($parameters as $parameterName => $parameterValue) { |
|
| 105 | + $formattedParameterValues = implode(', ', |
|
| 106 | + $this->limitArrayLength(array_map([$this, 'formatValue'], $parameterValue))); |
|
| 107 | + $formattedParameters[] = sprintf('%s: %s', $parameterName, $formattedParameterValues); |
|
| 108 | 108 | } |
| 109 | 109 | |
| 110 | - return implode( '; ', $formattedParameters ); |
|
| 110 | + return implode('; ', $formattedParameters); |
|
| 111 | 111 | } |
| 112 | 112 | |
| 113 | 113 | /** |
@@ -117,10 +117,10 @@ discard block |
||
| 117 | 117 | * |
| 118 | 118 | * @return string[] |
| 119 | 119 | */ |
| 120 | - private function limitArrayLength( array $array ) { |
|
| 121 | - if ( count( $array ) > self::MAX_PARAMETER_ARRAY_LENGTH ) { |
|
| 122 | - $array = array_slice( $array, 0, self::MAX_PARAMETER_ARRAY_LENGTH ); |
|
| 123 | - array_push( $array, '...' ); |
|
| 120 | + private function limitArrayLength(array $array) { |
|
| 121 | + if (count($array) > self::MAX_PARAMETER_ARRAY_LENGTH) { |
|
| 122 | + $array = array_slice($array, 0, self::MAX_PARAMETER_ARRAY_LENGTH); |
|
| 123 | + array_push($array, '...'); |
|
| 124 | 124 | } |
| 125 | 125 | |
| 126 | 126 | return $array; |
@@ -130,16 +130,16 @@ discard block |
||
| 130 | 130 | * @param DataValue $value |
| 131 | 131 | * @return string HTML |
| 132 | 132 | */ |
| 133 | - public function formatDataValue( DataValue $value ) { |
|
| 134 | - return $this->dataValueFormatter->format( $value ); |
|
| 133 | + public function formatDataValue(DataValue $value) { |
|
| 134 | + return $this->dataValueFormatter->format($value); |
|
| 135 | 135 | } |
| 136 | 136 | |
| 137 | 137 | /** |
| 138 | 138 | * @param EntityId $entityId |
| 139 | 139 | * @return string HTML |
| 140 | 140 | */ |
| 141 | - public function formatEntityId( EntityId $entityId ) { |
|
| 142 | - return $this->entityIdLabelFormatter->formatEntityId( $entityId ); |
|
| 141 | + public function formatEntityId(EntityId $entityId) { |
|
| 142 | + return $this->entityIdLabelFormatter->formatEntityId($entityId); |
|
| 143 | 143 | } |
| 144 | 144 | |
| 145 | 145 | /** |
@@ -148,17 +148,17 @@ discard block |
||
| 148 | 148 | * @param ItemIdSnakValue $value |
| 149 | 149 | * @return string HTML |
| 150 | 150 | */ |
| 151 | - public function formatItemIdSnakValue( ItemIdSnakValue $value ) { |
|
| 152 | - switch ( true ) { |
|
| 151 | + public function formatItemIdSnakValue(ItemIdSnakValue $value) { |
|
| 152 | + switch (true) { |
|
| 153 | 153 | case $value->isValue(): |
| 154 | - return $this->formatEntityId( $value->getItemId() ); |
|
| 154 | + return $this->formatEntityId($value->getItemId()); |
|
| 155 | 155 | case $value->isSomeValue(): |
| 156 | 156 | return $this->messageLocalizer |
| 157 | - ->msg( 'wikibase-snakview-snaktypeselector-somevalue' ) |
|
| 157 | + ->msg('wikibase-snakview-snaktypeselector-somevalue') |
|
| 158 | 158 | ->escaped(); |
| 159 | 159 | case $value->isNoValue(): |
| 160 | 160 | return $this->messageLocalizer |
| 161 | - ->msg( 'wikibase-snakview-snaktypeselector-novalue' ) |
|
| 161 | + ->msg('wikibase-snakview-snaktypeselector-novalue') |
|
| 162 | 162 | ->escaped(); |
| 163 | 163 | } |
| 164 | 164 | } |