[MediaWiki-commits] [Gerrit] mediawiki...Kartographer[master]: build: Updating mediawiki/mediawiki-codesniffer to 0.11.0

2017-08-10 Thread jenkins-bot (Code Review) 
jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/371327 )

Change subject: build: Updating mediawiki/mediawiki-codesniffer to 0.11.0
..


build: Updating mediawiki/mediawiki-codesniffer to 0.11.0

Change-Id: I99174ef505a2536e969da311fca6e13f555dc833
---
M composer.json
M includes/ApiSanitizeMapData.php
M tests/phpunit/ApiSanitizeMapDataTest.php
3 files changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Legoktm: Looks good to me, approved
  Yurik: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/composer.json b/composer.json
index f7e5ecc..c11bc98 100644
--- a/composer.json
+++ b/composer.json
@@ -5,7 +5,7 @@
"require-dev": {
"jakub-onderka/php-parallel-lint": "0.9.2",
"jakub-onderka/php-console-highlighter": "0.3.2",
-   "mediawiki/mediawiki-codesniffer": "0.10.1"
+   "mediawiki/mediawiki-codesniffer": "0.11.0"
},
"scripts": {
"fix": "phpcbf",
diff --git a/includes/ApiSanitizeMapData.php b/includes/ApiSanitizeMapData.php
index 452da5e..b3593b0 100644
--- a/includes/ApiSanitizeMapData.php
+++ b/includes/ApiSanitizeMapData.php
@@ -60,7 +60,7 @@
$simpleStyle = new SimpleStyleParser( $parser, null );
$status = $simpleStyle->parse( $text );
if ( !$status->isOK() ) {
-   $error =  $status->getHTML( false, false, 
$this->getLanguage() );
+   $error = $status->getHTML( false, false, 
$this->getLanguage() );
$this->getResult()->addValue( null, 
$this->getModuleName(), [ 'error' => $error ] );
} else {
$data = $status->getValue();
diff --git a/tests/phpunit/ApiSanitizeMapDataTest.php 
b/tests/phpunit/ApiSanitizeMapDataTest.php
index 1a58d1e..cfafe99 100644
--- a/tests/phpunit/ApiSanitizeMapDataTest.php
+++ b/tests/phpunit/ApiSanitizeMapDataTest.php
@@ -28,7 +28,7 @@
public function test( $title, $json, $isValid, $text ) {
$result = $this->makeRequest( $title, $json );
 
-   $data =$result->getResultData();
+   $data = $result->getResultData();
$this->assertArrayHasKey( 'sanitize-mapdata', $data );
$data = $data['sanitize-mapdata'];
if ( $isValid ) {

-- 
To view, visit https://gerrit.wikimedia.org/r/371327
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I99174ef505a2536e969da311fca6e13f555dc833
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Kartographer
Gerrit-Branch: master
Gerrit-Owner: Legoktm 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: MaxSem 
Gerrit-Reviewer: Yurik 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

[MediaWiki-commits] [Gerrit] mediawiki...Kartographer[master]: build: Updating mediawiki/mediawiki-codesniffer to 0.11.0

2017-08-10 Thread Legoktm (Code Review) 
Legoktm has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/371327 )

Change subject: build: Updating mediawiki/mediawiki-codesniffer to 0.11.0
..

build: Updating mediawiki/mediawiki-codesniffer to 0.11.0

Change-Id: I99174ef505a2536e969da311fca6e13f555dc833
---
M composer.json
M includes/ApiSanitizeMapData.php
M tests/phpunit/ApiSanitizeMapDataTest.php
3 files changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Kartographer 
refs/changes/27/371327/1

diff --git a/composer.json b/composer.json
index f7e5ecc..c11bc98 100644
--- a/composer.json
+++ b/composer.json
@@ -5,7 +5,7 @@
"require-dev": {
"jakub-onderka/php-parallel-lint": "0.9.2",
"jakub-onderka/php-console-highlighter": "0.3.2",
-   "mediawiki/mediawiki-codesniffer": "0.10.1"
+   "mediawiki/mediawiki-codesniffer": "0.11.0"
},
"scripts": {
"fix": "phpcbf",
diff --git a/includes/ApiSanitizeMapData.php b/includes/ApiSanitizeMapData.php
index 452da5e..b3593b0 100644
--- a/includes/ApiSanitizeMapData.php
+++ b/includes/ApiSanitizeMapData.php
@@ -60,7 +60,7 @@
$simpleStyle = new SimpleStyleParser( $parser, null );
$status = $simpleStyle->parse( $text );
if ( !$status->isOK() ) {
-   $error =  $status->getHTML( false, false, 
$this->getLanguage() );
+   $error = $status->getHTML( false, false, 
$this->getLanguage() );
$this->getResult()->addValue( null, 
$this->getModuleName(), [ 'error' => $error ] );
} else {
$data = $status->getValue();
diff --git a/tests/phpunit/ApiSanitizeMapDataTest.php 
b/tests/phpunit/ApiSanitizeMapDataTest.php
index 1a58d1e..cfafe99 100644
--- a/tests/phpunit/ApiSanitizeMapDataTest.php
+++ b/tests/phpunit/ApiSanitizeMapDataTest.php
@@ -28,7 +28,7 @@
public function test( $title, $json, $isValid, $text ) {
$result = $this->makeRequest( $title, $json );
 
-   $data =$result->getResultData();
+   $data = $result->getResultData();
$this->assertArrayHasKey( 'sanitize-mapdata', $data );
$data = $data['sanitize-mapdata'];
if ( $isValid ) {

-- 
To view, visit https://gerrit.wikimedia.org/r/371327
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I99174ef505a2536e969da311fca6e13f555dc833
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Kartographer
Gerrit-Branch: master
Gerrit-Owner: Legoktm 
Gerrit-Reviewer: Legoktm 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits