Last modified: 2010-05-15 15:42:50 UTC
The relevant sections of code are as follows: $content_actions['edit'] is set after this check in buildContentActionUrls(): if ( $this->mTitle->quickUserCan( 'edit' ) && ( $this->mTitle->exists() || $this->mTitle->quickUserCan( 'create' ) ) ) { However, it is used in this fashion in outputPage(): // XXX: attach this from javascript, same with section editing if($this->iseditable && $wgUser->getOption("editondblclick") ) { $tpl->set('body_ondblclick', 'document.location = "' .$content_actions['edit']['href'] .'";'); } When a user has "editondblclick" enabled, but views a protected page, $content_actions['edit'] will be used before initialization.
Fixed in r22713; using the direct URL generator func instead of trying to rip it out of the array. On protected page, the 'edit' link goes into a 'viewsource' tab instead, so wasn't getting picked up as expected.
*** Bug 6832 has been marked as a duplicate of this bug. ***