Yaron Koren has submitted this change and it was merged.
Change subject: Fix for bug in XML output
......................................................................
Fix for bug in XML output
Change-Id: Id6b3eefc0c5ba70bfec381706772f26a8f5a2eca
---
M includes/DT_PageStructure.php
1 file changed, 5 insertions(+), 5 deletions(-)
Approvals:
Yaron Koren: Verified; Looks good to me, approved
diff --git a/includes/DT_PageStructure.php b/includes/DT_PageStructure.php
index 1eea0a3..8981fd2 100644
--- a/includes/DT_PageStructure.php
+++ b/includes/DT_PageStructure.php
@@ -68,19 +68,19 @@
if ( is_numeric( $fieldName ) ) {
// add "Field" to the beginning
of the file name, since
// XML tags that are simply
numbers aren't allowed
- $fieldName = $field_str . '_' .
$fieldName;
+ $fieldTag = $field_str . '_' .
$fieldName;
} else {
- $fieldName = str_replace( ' ',
'_', trim( $fieldName ) );
+ $fieldTag = str_replace( ' ',
'_', trim( $fieldName ) );
}
$attrs = null;
} else {
- $fieldName = $field_str;
+ $fieldTag = $field_str;
$attrs = array( $name_str => $fieldName
);
}
if ( is_array( $fieldValue ) ) {
- $bodyXML .= Xml::tags( $fieldName,
$attrs, $fieldValueXML );
+ $bodyXML .= Xml::tags( $fieldTag,
$attrs, $fieldValueXML );
} else {
- $bodyXML .= Xml::element( $fieldName,
$attrs, $fieldValue );
+ $bodyXML .= Xml::element( $fieldTag,
$attrs, $fieldValue );
}
}
--
To view, visit https://gerrit.wikimedia.org/r/61052
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id6b3eefc0c5ba70bfec381706772f26a8f5a2eca
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/DataTransfer
Gerrit-Branch: master
Gerrit-Owner: Yaron Koren <[email protected]>
Gerrit-Reviewer: Yaron Koren <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits