Page MenuHomePhabricator

[Task] Replace stuff in WikibaseRepo hooks with mechanism in core for disabling direct editing of content types
Closed, ResolvedPublic

Description

We should replace the hacks (use of ApiCheckCanExecute and mobile frontend hooks) in WikibaseRepo hooks with some mechanism in core for disabling direct editing of content types.

Event Timeline

aude claimed this task.
aude raised the priority of this task from to Needs Triage.
aude updated the task description. (Show Details)
aude added a subscriber: aude.

Change 204230 had a related patch set uploaded (by Aude):
Add supportsDirectEditing methods to ContentHandler

https://gerrit.wikimedia.org/r/204230

first step is to add the mechanism in core, which is what https://gerrit.wikimedia.org/r/204230 does

then we can remove the ApiCheckCanExecute hook stuff in Wikibase

for mobile, adapt it to use the new mechanism, probably with a method_exists check on ContentHandler, assuming they care about backwards compatibility. then we don't need the SkinMinvera hook code in Wikibase.

aude triaged this task as Medium priority.Apr 18 2015, 9:11 AM

Change 204230 merged by jenkins-bot:
Add supportsDirectEditing methods to ContentHandler

https://gerrit.wikimedia.org/r/204230

Core patch is merged. Wikibase patch for using the new logic is pending.

Patch for mobile is also still pending.

The core patch means that edits to wikibase items can no longer be undone through the API T101694

aude renamed this task from Replace stuff in WikibaseRepo hooks with mechanism in core for disabling direct editing of content types to [Task] Replace stuff in WikibaseRepo hooks with mechanism in core for disabling direct editing of content types.Aug 13 2015, 8:27 PM