Hi,
I'm looking for some input on a fix I wrote for an SMW/GraphViz interop bug
(phabricator T75073 <https://phabricator.wikimedia.org/T75073>).

To recap the bug:


   1. The GraphViz extension uploaded a file (a graph image) from the
   context of a Parser::parse call.
   2. The upload triggered the NewRevisionFromEditComplete hook.
   3. The Semantic MediaWiki extension NewRevisionFromEditComplete hook
   eventually called Parser::parse (indirectly) which causes the reported
   Invalid Marker exception.

My fix is to disable all hooks while GraphViz is uploading a file.  The
only hook on the upload code path is NewRevisionFromEditComplete but for
simplicity and future-proofing I disable all hooks.

My question is, what is the consequence to an installation with SMW
installed if the NewRevisionFromEditComplete hook does not run for images
uploaded by GraphViz?

Thanks in advance for your help,

Keith
------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=157005751&iu=/4140/ostg.clktrk
_______________________________________________
Semediawiki-devel mailing list
Semediawiki-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/semediawiki-devel

Reply via email to