Jeroen De Dauw has submitted this change and it was merged.

Change subject: Implemented by-property grouping in API serialization
......................................................................


Implemented by-property grouping in API serialization

Also improved tests for statement and claim aggregates since the serialization 
based one broke for no good reason

Change-Id: Iedf7d4085b10dd5f8e501691d2e605641ef1b4b5
---
M lib/includes/ByPropertyIdArray.php
M lib/includes/HashableObjectStorage.php
M lib/tests/phpunit/claim/ClaimAggregateTest.php
M lib/tests/phpunit/statement/StatementAggregateTest.php
M repo/includes/api/serializers/SnakSerializer.php
M repo/includes/api/serializers/SnaksSerializer.php
M repo/includes/api/serializers/StatementsSerializer.php
M repo/tests/phpunit/includes/api/serializers/SnakSerializerTest.php
M repo/tests/phpunit/includes/api/serializers/SnaksSerializerTest.php
M repo/tests/phpunit/includes/api/serializers/StatementsSerializerTest.php
10 files changed, 133 insertions(+), 37 deletions(-)

Approvals:
  Denny Vrandecic: Looks good to me, approved
  Jeroen De Dauw: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iedf7d4085b10dd5f8e501691d2e605641ef1b4b5
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Jeroen De Dauw <[email protected]>
Gerrit-Reviewer: Aude <[email protected]>
Gerrit-Reviewer: Daniel Kinzler <[email protected]>
Gerrit-Reviewer: Daniel Werner <[email protected]>
Gerrit-Reviewer: Denny Vrandecic <[email protected]>
Gerrit-Reviewer: Jens Ohlig <[email protected]>
Gerrit-Reviewer: Jeroen De Dauw <[email protected]>

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

Reply via email to