John Erling Blad has submitted this change and it was merged.

Change subject: Make permission checks via EditEntity.
......................................................................


Make permission checks via EditEntity.

Refactored permission checks in EntityContent, integrated them into
EditEntity, wroite test cases to make sure permissions are always
checked and also work for items that have not yet been saved.

Change-Id: I01ef4b6860a5679eb775e681bff2aa2c527de8bb
---
M repo/includes/EditEntity.php
M repo/includes/content/EntityContent.php
M repo/includes/content/ItemContent.php
M repo/tests/phpunit/includes/EditEntityTest.php
M repo/tests/phpunit/includes/api/ApiModifyItemBase.php
M repo/tests/phpunit/includes/api/ApiPermissionsTest.php
M repo/tests/phpunit/includes/content/ItemContentTest.php
7 files changed, 419 insertions(+), 107 deletions(-)

Approvals:
  John Erling Blad: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I01ef4b6860a5679eb775e681bff2aa2c527de8bb
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Daniel Kinzler <daniel.kinz...@wikimedia.de>
Gerrit-Reviewer: Daniel Kinzler <daniel.kinz...@wikimedia.de>
Gerrit-Reviewer: Jeroen De Dauw <jeroended...@gmail.com>
Gerrit-Reviewer: John Erling Blad <john.b...@wikimedia.de>
Gerrit-Reviewer: Tobias Gritschacher <tobias.gritschac...@wikimedia.de>

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

Reply via email to