Completed
Push — master ( ca4932...cc779e )
by
unknown
02:22
created
src/ConstraintParameterRenderer.php 1 patch
Spacing   +30 added lines, -30 removed lines patch added patch discarded remove patch
@@ -71,20 +71,20 @@  discard block
 block discarded – undo
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,23 +95,23 @@  discard block
 block discarded – undo
95 95
 	 *
96 96
 	 * @return string 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
-		$valueFormatter = function ( $value ) {
104
-			return $this->formatValue( $value );
103
+		$valueFormatter = function($value) {
104
+			return $this->formatValue($value);
105 105
 		};
106 106
 
107 107
 		$formattedParameters = [];
108
-		foreach ( $parameters as $parameterName => $parameterValue ) {
109
-			$formattedParameterValues = implode( ', ',
110
-				$this->limitArrayLength( array_map( $valueFormatter, $parameterValue ) ) );
111
-			$formattedParameters[] = sprintf( '%s: %s', $parameterName, $formattedParameterValues );
108
+		foreach ($parameters as $parameterName => $parameterValue) {
109
+			$formattedParameterValues = implode(', ',
110
+				$this->limitArrayLength(array_map($valueFormatter, $parameterValue)));
111
+			$formattedParameters[] = sprintf('%s: %s', $parameterName, $formattedParameterValues);
112 112
 		}
113 113
 
114
-		return implode( '; ', $formattedParameters );
114
+		return implode('; ', $formattedParameters);
115 115
 	}
116 116
 
117 117
 	/**
@@ -121,10 +121,10 @@  discard block
 block discarded – undo
121 121
 	 *
122 122
 	 * @return array
123 123
 	 */
124
-	private function limitArrayLength( array $array ) {
125
-		if ( count( $array ) > self::MAX_PARAMETER_ARRAY_LENGTH ) {
126
-			$array = array_slice( $array, 0, self::MAX_PARAMETER_ARRAY_LENGTH );
127
-			array_push( $array, '...' );
124
+	private function limitArrayLength(array $array) {
125
+		if (count($array) > self::MAX_PARAMETER_ARRAY_LENGTH) {
126
+			$array = array_slice($array, 0, self::MAX_PARAMETER_ARRAY_LENGTH);
127
+			array_push($array, '...');
128 128
 		}
129 129
 
130 130
 		return $array;
@@ -134,16 +134,16 @@  discard block
 block discarded – undo
134 134
 	 * @param DataValue $value
135 135
 	 * @return string HTML
136 136
 	 */
137
-	public function formatDataValue( DataValue $value ) {
138
-		return $this->dataValueFormatter->format( $value );
137
+	public function formatDataValue(DataValue $value) {
138
+		return $this->dataValueFormatter->format($value);
139 139
 	}
140 140
 
141 141
 	/**
142 142
 	 * @param EntityId $entityId
143 143
 	 * @return string HTML
144 144
 	 */
145
-	public function formatEntityId( EntityId $entityId ) {
146
-		return $this->entityIdLabelFormatter->formatEntityId( $entityId );
145
+	public function formatEntityId(EntityId $entityId) {
146
+		return $this->entityIdLabelFormatter->formatEntityId($entityId);
147 147
 	}
148 148
 
149 149
 	/**
@@ -152,17 +152,17 @@  discard block
 block discarded – undo
152 152
 	 * @param ItemIdSnakValue $value
153 153
 	 * @return string HTML
154 154
 	 */
155
-	public function formatItemIdSnakValue( ItemIdSnakValue $value ) {
156
-		switch ( true ) {
155
+	public function formatItemIdSnakValue(ItemIdSnakValue $value) {
156
+		switch (true) {
157 157
 			case $value->isValue():
158
-				return $this->formatEntityId( $value->getItemId() );
158
+				return $this->formatEntityId($value->getItemId());
159 159
 			case $value->isSomeValue():
160 160
 				return $this->messageLocalizer
161
-					->msg( 'wikibase-snakview-snaktypeselector-somevalue' )
161
+					->msg('wikibase-snakview-snaktypeselector-somevalue')
162 162
 					->escaped();
163 163
 			case $value->isNoValue():
164 164
 				return $this->messageLocalizer
165
-					->msg( 'wikibase-snakview-snaktypeselector-novalue' )
165
+					->msg('wikibase-snakview-snaktypeselector-novalue')
166 166
 					->escaped();
167 167
 		}
168 168
 	}
Please login to merge, or discard this patch.