jenkins-bot has submitted this change and it was merged.

Change subject: Parse group and inlinelabel, if available
......................................................................


Parse group and inlinelabel, if available

The LocationParser currently doesn't parse all metadata, add
code to parse Group and InlineLabel

Change-Id: Ic97af8672f9e1adbf8098eab4d8274eabae5da35
---
M includes/parsers/LocationParser.php
1 file changed, 8 insertions(+), 0 deletions(-)

Approvals:
  Jeroen De Dauw: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/parsers/LocationParser.php 
b/includes/parsers/LocationParser.php
index fa02d21..0217d5b 100644
--- a/includes/parsers/LocationParser.php
+++ b/includes/parsers/LocationParser.php
@@ -53,6 +53,14 @@
                        $location->setIcon( array_shift( $metaData ) );
                }
 
+               if ( $metaData !== array() ) {
+                       $location->setGroup( array_shift( $metaData ) );
+               }
+
+               if ( $metaData !== array() ) {
+                       $location->setInlineLabel( array_shift( $metaData ) );
+               }
+
                return $location;
        }
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic97af8672f9e1adbf8098eab4d8274eabae5da35
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Maps
Gerrit-Branch: master
Gerrit-Owner: MathiasLidal <[email protected]>
Gerrit-Reviewer: Jeroen De Dauw <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to