r97216 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:< r97215‎ | r97216 | r97217 >
Date:22:30, 15 September 2011
Author:yaron
Status:ok (Comments)
Tags:
Comment:
Added new hook, 'SIOHandler::updateData', from patch by Keith Fligg
Modified paths:
  • /trunk/extensions/SemanticInternalObjects/SemanticInternalObjects_body.php (modified) (history)

Diff [purge]

Index: trunk/extensions/SemanticInternalObjects/SemanticInternalObjects_body.php
——@@ -382,6 +382,7 @@
383383 $allAtts2Inserts = array_merge( $allAtts2Inserts, $upAtts2 );
384384 $allText2Inserts = array_merge( $allText2Inserts, $upText2 );
385385 $allCoordsInserts = array_merge( $allCoordsInserts, $upCoords );
 386+ wfRunHooks( 'SIOHandler::updateData', array( $internalObject ) );
386387 }
387388
388389 // now save everything to the database, in a single transaction

Comments

#Comment by Nikerabbit (talk | contribs)   05:16, 16 September 2011

Does SIO have a documentation of what hooks it introduces?

#Comment by Yaron Koren (talk | contribs)   05:35, 16 September 2011

That is SIO's first and only hook. Is there a general recommendation on how hooks defined in extensions should be documented?

#Comment by Nikerabbit (talk | contribs)   06:02, 16 September 2011

Not that I am aware of. I just added first hook to Translate extension myself and wondered the same thing.

Status & tagging log