Hi Mike, thank you very much for your comment, especially that part that says: "If someone decides in the future to enforce this in the class by putting the keyword into the class definition, your code will no longer be able to inherit." If it ever happens, at least I am forewarned. PM Poon
On Mon, Aug 17, 2009 at 6:44 AM, Michael Daly <[email protected]>wrote: > Ekompute .info wrote: > > Hi Mike, I amended the following parts: > > if( $this->iscontent ) { > > $subjpage = $this->mTitle->getSubjectPage(); > > $talkpage = $this->mTitle->getTalkPage(); > > [...] > > 'useful_links', $this->mTitle->getNamespace() == 112 && > > !$prevent_active_tabs,'', true); > > } > > So it would appear that you should override: > > function buildContentActionUrls() {...} > > in skinname.php (where skinname is whatever skin you're using - e.g. > monobook.php). Of course, if you're using a standard skin, then copy > and rename it to your own skin first otherwise you'll risk losing the > changes if you upgrade to a newer version and the skin gets overwritten. > > Note that this is a private function in SkinTemplate.php according to > the comments but not explicitly using the PHP visibility keyword. If > someone decides in the future to enforce this in the class by putting > the keyword into the class definition, your code will no longer be able > to inherit. Since this is not enforced, you'll probably be ok. > > Mike > > > > _______________________________________________ > MediaWiki-l mailing list > [email protected] > https://lists.wikimedia.org/mailman/listinfo/mediawiki-l > -- PM Poon * http://dummipedia.org The 5-Minute Concise Encyclopedia * http://dummipedia.com The Dummipedia Amazon Online Store _______________________________________________ MediaWiki-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-l
