jenkins-bot has submitted this change and it was merged.
Change subject: Remove not needed state
......................................................................
Remove not needed state
Change-Id: Ia611f09b29019c07d531bafa98142cb706b1b3e3
---
M repo/includes/store/sql/SqlChangeStore.php
1 file changed, 5 insertions(+), 11 deletions(-)
Approvals:
Hoo man: Looks good to me, approved
Daniel Kinzler: Looks good to me, approved
Addshore: Looks good to me, approved
jenkins-bot: Verified
diff --git a/repo/includes/store/sql/SqlChangeStore.php
b/repo/includes/store/sql/SqlChangeStore.php
index dde9d3f..dbb8a92 100644
--- a/repo/includes/store/sql/SqlChangeStore.php
+++ b/repo/includes/store/sql/SqlChangeStore.php
@@ -82,19 +82,13 @@
private function getValues( ChangeRow $change ) {
$fields = $change->getFields();
- $time = isset( $fields['time'] ) ? $fields['time'] :
wfTimestampNow();
- $objectId = isset( $fields['object_id'] ) ?
$fields['object_id'] : '';
- $revId = isset( $fields['revision_id'] ) ?
$fields['revision_id'] : '0';
- $userId = isset( $fields['user_id'] ) ? $fields['user_id'] :
'0';
- $info = $change->serializeInfo( $fields['info'] );
-
return array(
'change_type' => $fields['type'],
- 'change_time' => $time,
- 'change_object_id' => $objectId,
- 'change_revision_id' => $revId,
- 'change_user_id' => $userId,
- 'change_info' => $info
+ 'change_time' => isset( $fields['time'] ) ?
$fields['time'] : wfTimestampNow(),
+ 'change_object_id' => isset( $fields['object_id'] ) ?
$fields['object_id'] : '',
+ 'change_revision_id' => isset( $fields['revision_id'] )
? $fields['revision_id'] : '0',
+ 'change_user_id' => isset( $fields['user_id'] ) ?
$fields['user_id'] : '0',
+ 'change_info' => $change->serializeInfo(
$fields['info'] )
);
}
--
To view, visit https://gerrit.wikimedia.org/r/245602
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia611f09b29019c07d531bafa98142cb706b1b3e3
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Jeroen De Dauw <[email protected]>
Gerrit-Reviewer: Addshore <[email protected]>
Gerrit-Reviewer: Daniel Kinzler <[email protected]>
Gerrit-Reviewer: Hoo man <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
