Welterkj has uploaded a new change for review.

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

Change subject: This change modifies the GraphViz extension to make graph image 
upload optional instead of automatic/required.
......................................................................

This change modifies the GraphViz extension to make graph image
upload optional instead of automatic/required.

Users have complained about needing to reload pages containing
graphs after editing the graphs (associated message: “Graph
image source changed. Reload page to display updated graph
image”).  This behavior is because:
- graph images are created by a parser hook function which...
- uses the ImageMap extension to render graph images as HTML
which...
- requires that graph images have been uploaded prior to
rendering which...
- causes an exception if the upload is done from within a
parser hook function so...
- the upload is done as a separate step after parsing is
complete which...
- requires the page reload to see the graph image rendered
by ImageMap

I’ve considered a few alternative remedies and have selected the
last one below:
- modify the MW core to allow uploads from within parser
functions (not backwards compatible)
- trickery such as custom upload (too complicated,
bad engineering)
- refactor extensions/ImageMap/ImageMap_body.php to allow
specialization by subclasses (requires co-operation/collaboration
with the ImageMap extension maintainer-- no luck with this in the
past)
- modify a copy of ImageMap_body.php to work on non-uploaded local
files and make uploading optional (I dislike the code duplication
but this seems to be the best alternative)

Change-Id: I2b171c8414d4df10a8e1cb0bab1f6974bc5f6463
---
A COPYING
A GraphVizImageMap.php
2 files changed, 732 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/GraphViz 
refs/changes/59/164159/1

diff --git a/COPYING b/COPYING
new file mode 100644
index 0000000..d159169
--- /dev/null
+++ b/COPYING
@@ -0,0 +1,339 @@
+                    GNU GENERAL PUBLIC LICENSE
+                       Version 2, June 1991
+
+ Copyright (C) 1989, 1991 Free Software Foundation, Inc.,
+ 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ Everyone is permitted to copy and distribute verbatim copies
+ of this license document, but changing it is not allowed.
+
+                            Preamble
+
+  The licenses for most software are designed to take away your
+freedom to share and change it.  By contrast, the GNU General Public
+License is intended to guarantee your freedom to share and change free
+software--to make sure the software is free for all its users.  This
+General Public License applies to most of the Free Software
+Foundation's software and to any other program whose authors commit to
+using it.  (Some other Free Software Foundation software is covered by
+the GNU Lesser General Public License instead.)  You can apply it to
+your programs, too.
+
+  When we speak of free software, we are referring to freedom, not
+price.  Our General Public Licenses are designed to make sure that you
+have the freedom to distribute copies of free software (and charge for
+this service if you wish), that you receive source code or can get it
+if you want it, that you can change the software or use pieces of it
+in new free programs; and that you know you can do these things.
+
+  To protect your rights, we need to make restrictions that forbid
+anyone to deny you these rights or to ask you to surrender the rights.
+These restrictions translate to certain responsibilities for you if you
+distribute copies of the software, or if you modify it.
+
+  For example, if you distribute copies of such a program, whether
+gratis or for a fee, you must give the recipients all the rights that
+you have.  You must make sure that they, too, receive or can get the
+source code.  And you must show them these terms so they know their
+rights.
+
+  We protect your rights with two steps: (1) copyright the software, and
+(2) offer you this license which gives you legal permission to copy,
+distribute and/or modify the software.
+
+  Also, for each author's protection and ours, we want to make certain
+that everyone understands that there is no warranty for this free
+software.  If the software is modified by someone else and passed on, we
+want its recipients to know that what they have is not the original, so
+that any problems introduced by others will not reflect on the original
+authors' reputations.
+
+  Finally, any free program is threatened constantly by software
+patents.  We wish to avoid the danger that redistributors of a free
+program will individually obtain patent licenses, in effect making the
+program proprietary.  To prevent this, we have made it clear that any
+patent must be licensed for everyone's free use or not licensed at all.
+
+  The precise terms and conditions for copying, distribution and
+modification follow.
+
+                    GNU GENERAL PUBLIC LICENSE
+   TERMS AND CONDITIONS FOR COPYING, DISTRIBUTION AND MODIFICATION
+
+  0. This License applies to any program or other work which contains
+a notice placed by the copyright holder saying it may be distributed
+under the terms of this General Public License.  The "Program", below,
+refers to any such program or work, and a "work based on the Program"
+means either the Program or any derivative work under copyright law:
+that is to say, a work containing the Program or a portion of it,
+either verbatim or with modifications and/or translated into another
+language.  (Hereinafter, translation is included without limitation in
+the term "modification".)  Each licensee is addressed as "you".
+
+Activities other than copying, distribution and modification are not
+covered by this License; they are outside its scope.  The act of
+running the Program is not restricted, and the output from the Program
+is covered only if its contents constitute a work based on the
+Program (independent of having been made by running the Program).
+Whether that is true depends on what the Program does.
+
+  1. You may copy and distribute verbatim copies of the Program's
+source code as you receive it, in any medium, provided that you
+conspicuously and appropriately publish on each copy an appropriate
+copyright notice and disclaimer of warranty; keep intact all the
+notices that refer to this License and to the absence of any warranty;
+and give any other recipients of the Program a copy of this License
+along with the Program.
+
+You may charge a fee for the physical act of transferring a copy, and
+you may at your option offer warranty protection in exchange for a fee.
+
+  2. You may modify your copy or copies of the Program or any portion
+of it, thus forming a work based on the Program, and copy and
+distribute such modifications or work under the terms of Section 1
+above, provided that you also meet all of these conditions:
+
+    a) You must cause the modified files to carry prominent notices
+    stating that you changed the files and the date of any change.
+
+    b) You must cause any work that you distribute or publish, that in
+    whole or in part contains or is derived from the Program or any
+    part thereof, to be licensed as a whole at no charge to all third
+    parties under the terms of this License.
+
+    c) If the modified program normally reads commands interactively
+    when run, you must cause it, when started running for such
+    interactive use in the most ordinary way, to print or display an
+    announcement including an appropriate copyright notice and a
+    notice that there is no warranty (or else, saying that you provide
+    a warranty) and that users may redistribute the program under
+    these conditions, and telling the user how to view a copy of this
+    License.  (Exception: if the Program itself is interactive but
+    does not normally print such an announcement, your work based on
+    the Program is not required to print an announcement.)
+
+These requirements apply to the modified work as a whole.  If
+identifiable sections of that work are not derived from the Program,
+and can be reasonably considered independent and separate works in
+themselves, then this License, and its terms, do not apply to those
+sections when you distribute them as separate works.  But when you
+distribute the same sections as part of a whole which is a work based
+on the Program, the distribution of the whole must be on the terms of
+this License, whose permissions for other licensees extend to the
+entire whole, and thus to each and every part regardless of who wrote it.
+
+Thus, it is not the intent of this section to claim rights or contest
+your rights to work written entirely by you; rather, the intent is to
+exercise the right to control the distribution of derivative or
+collective works based on the Program.
+
+In addition, mere aggregation of another work not based on the Program
+with the Program (or with a work based on the Program) on a volume of
+a storage or distribution medium does not bring the other work under
+the scope of this License.
+
+  3. You may copy and distribute the Program (or a work based on it,
+under Section 2) in object code or executable form under the terms of
+Sections 1 and 2 above provided that you also do one of the following:
+
+    a) Accompany it with the complete corresponding machine-readable
+    source code, which must be distributed under the terms of Sections
+    1 and 2 above on a medium customarily used for software interchange; or,
+
+    b) Accompany it with a written offer, valid for at least three
+    years, to give any third party, for a charge no more than your
+    cost of physically performing source distribution, a complete
+    machine-readable copy of the corresponding source code, to be
+    distributed under the terms of Sections 1 and 2 above on a medium
+    customarily used for software interchange; or,
+
+    c) Accompany it with the information you received as to the offer
+    to distribute corresponding source code.  (This alternative is
+    allowed only for noncommercial distribution and only if you
+    received the program in object code or executable form with such
+    an offer, in accord with Subsection b above.)
+
+The source code for a work means the preferred form of the work for
+making modifications to it.  For an executable work, complete source
+code means all the source code for all modules it contains, plus any
+associated interface definition files, plus the scripts used to
+control compilation and installation of the executable.  However, as a
+special exception, the source code distributed need not include
+anything that is normally distributed (in either source or binary
+form) with the major components (compiler, kernel, and so on) of the
+operating system on which the executable runs, unless that component
+itself accompanies the executable.
+
+If distribution of executable or object code is made by offering
+access to copy from a designated place, then offering equivalent
+access to copy the source code from the same place counts as
+distribution of the source code, even though third parties are not
+compelled to copy the source along with the object code.
+
+  4. You may not copy, modify, sublicense, or distribute the Program
+except as expressly provided under this License.  Any attempt
+otherwise to copy, modify, sublicense or distribute the Program is
+void, and will automatically terminate your rights under this License.
+However, parties who have received copies, or rights, from you under
+this License will not have their licenses terminated so long as such
+parties remain in full compliance.
+
+  5. You are not required to accept this License, since you have not
+signed it.  However, nothing else grants you permission to modify or
+distribute the Program or its derivative works.  These actions are
+prohibited by law if you do not accept this License.  Therefore, by
+modifying or distributing the Program (or any work based on the
+Program), you indicate your acceptance of this License to do so, and
+all its terms and conditions for copying, distributing or modifying
+the Program or works based on it.
+
+  6. Each time you redistribute the Program (or any work based on the
+Program), the recipient automatically receives a license from the
+original licensor to copy, distribute or modify the Program subject to
+these terms and conditions.  You may not impose any further
+restrictions on the recipients' exercise of the rights granted herein.
+You are not responsible for enforcing compliance by third parties to
+this License.
+
+  7. If, as a consequence of a court judgment or allegation of patent
+infringement or for any other reason (not limited to patent issues),
+conditions are imposed on you (whether by court order, agreement or
+otherwise) that contradict the conditions of this License, they do not
+excuse you from the conditions of this License.  If you cannot
+distribute so as to satisfy simultaneously your obligations under this
+License and any other pertinent obligations, then as a consequence you
+may not distribute the Program at all.  For example, if a patent
+license would not permit royalty-free redistribution of the Program by
+all those who receive copies directly or indirectly through you, then
+the only way you could satisfy both it and this License would be to
+refrain entirely from distribution of the Program.
+
+If any portion of this section is held invalid or unenforceable under
+any particular circumstance, the balance of the section is intended to
+apply and the section as a whole is intended to apply in other
+circumstances.
+
+It is not the purpose of this section to induce you to infringe any
+patents or other property right claims or to contest validity of any
+such claims; this section has the sole purpose of protecting the
+integrity of the free software distribution system, which is
+implemented by public license practices.  Many people have made
+generous contributions to the wide range of software distributed
+through that system in reliance on consistent application of that
+system; it is up to the author/donor to decide if he or she is willing
+to distribute software through any other system and a licensee cannot
+impose that choice.
+
+This section is intended to make thoroughly clear what is believed to
+be a consequence of the rest of this License.
+
+  8. If the distribution and/or use of the Program is restricted in
+certain countries either by patents or by copyrighted interfaces, the
+original copyright holder who places the Program under this License
+may add an explicit geographical distribution limitation excluding
+those countries, so that distribution is permitted only in or among
+countries not thus excluded.  In such case, this License incorporates
+the limitation as if written in the body of this License.
+
+  9. The Free Software Foundation may publish revised and/or new versions
+of the General Public License from time to time.  Such new versions will
+be similar in spirit to the present version, but may differ in detail to
+address new problems or concerns.
+
+Each version is given a distinguishing version number.  If the Program
+specifies a version number of this License which applies to it and "any
+later version", you have the option of following the terms and conditions
+either of that version or of any later version published by the Free
+Software Foundation.  If the Program does not specify a version number of
+this License, you may choose any version ever published by the Free Software
+Foundation.
+
+  10. If you wish to incorporate parts of the Program into other free
+programs whose distribution conditions are different, write to the author
+to ask for permission.  For software which is copyrighted by the Free
+Software Foundation, write to the Free Software Foundation; we sometimes
+make exceptions for this.  Our decision will be guided by the two goals
+of preserving the free status of all derivatives of our free software and
+of promoting the sharing and reuse of software generally.
+
+                            NO WARRANTY
+
+  11. BECAUSE THE PROGRAM IS LICENSED FREE OF CHARGE, THERE IS NO WARRANTY
+FOR THE PROGRAM, TO THE EXTENT PERMITTED BY APPLICABLE LAW.  EXCEPT WHEN
+OTHERWISE STATED IN WRITING THE COPYRIGHT HOLDERS AND/OR OTHER PARTIES
+PROVIDE THE PROGRAM "AS IS" WITHOUT WARRANTY OF ANY KIND, EITHER EXPRESSED
+OR IMPLIED, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
+MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.  THE ENTIRE RISK AS
+TO THE QUALITY AND PERFORMANCE OF THE PROGRAM IS WITH YOU.  SHOULD THE
+PROGRAM PROVE DEFECTIVE, YOU ASSUME THE COST OF ALL NECESSARY SERVICING,
+REPAIR OR CORRECTION.
+
+  12. IN NO EVENT UNLESS REQUIRED BY APPLICABLE LAW OR AGREED TO IN WRITING
+WILL ANY COPYRIGHT HOLDER, OR ANY OTHER PARTY WHO MAY MODIFY AND/OR
+REDISTRIBUTE THE PROGRAM AS PERMITTED ABOVE, BE LIABLE TO YOU FOR DAMAGES,
+INCLUDING ANY GENERAL, SPECIAL, INCIDENTAL OR CONSEQUENTIAL DAMAGES ARISING
+OUT OF THE USE OR INABILITY TO USE THE PROGRAM (INCLUDING BUT NOT LIMITED
+TO LOSS OF DATA OR DATA BEING RENDERED INACCURATE OR LOSSES SUSTAINED BY
+YOU OR THIRD PARTIES OR A FAILURE OF THE PROGRAM TO OPERATE WITH ANY OTHER
+PROGRAMS), EVEN IF SUCH HOLDER OR OTHER PARTY HAS BEEN ADVISED OF THE
+POSSIBILITY OF SUCH DAMAGES.
+
+                     END OF TERMS AND CONDITIONS
+
+            How to Apply These Terms to Your New Programs
+
+  If you develop a new program, and you want it to be of the greatest
+possible use to the public, the best way to achieve this is to make it
+free software which everyone can redistribute and change under these terms.
+
+  To do so, attach the following notices to the program.  It is safest
+to attach them to the start of each source file to most effectively
+convey the exclusion of warranty; and each file should have at least
+the "copyright" line and a pointer to where the full notice is found.
+
+    <one line to give the program's name and a brief idea of what it does.>
+    Copyright (C) <year>  <name of author>
+
+    This program is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 2 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License along
+    with this program; if not, write to the Free Software Foundation, Inc.,
+    51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+
+Also add information on how to contact you by electronic and paper mail.
+
+If the program is interactive, make it output a short notice like this
+when it starts in an interactive mode:
+
+    Gnomovision version 69, Copyright (C) year name of author
+    Gnomovision comes with ABSOLUTELY NO WARRANTY; for details type `show w'.
+    This is free software, and you are welcome to redistribute it
+    under certain conditions; type `show c' for details.
+
+The hypothetical commands `show w' and `show c' should show the appropriate
+parts of the General Public License.  Of course, the commands you use may
+be called something other than `show w' and `show c'; they could even be
+mouse-clicks or menu items--whatever suits your program.
+
+You should also get your employer (if you work as a programmer) or your
+school, if any, to sign a "copyright disclaimer" for the program, if
+necessary.  Here is a sample; alter the names:
+
+  Yoyodyne, Inc., hereby disclaims all copyright interest in the program
+  `Gnomovision' (which makes passes at compilers) written by James Hacker.
+
+  <signature of Ty Coon>, 1 April 1989
+  Ty Coon, President of Vice
+
+This General Public License does not permit incorporating your program into
+proprietary programs.  If your program is a subroutine library, you may
+consider it more useful to permit linking proprietary applications with the
+library.  If this is what you want to do, use the GNU Lesser General
+Public License instead of this License.
diff --git a/GraphVizImageMap.php b/GraphVizImageMap.php
new file mode 100644
index 0000000..7b53285
--- /dev/null
+++ b/GraphVizImageMap.php
@@ -0,0 +1,393 @@
+<?php
+/**
+ * Main file for extension ImageMap.
+ *
+ * @file
+ * @ingroup Extensions
+ *
+ * Syntax:
+ * <imagemap>
+ * Image:Foo.jpg | 100px | picture of a foo
+ *
+ * rect    0  0  50 50  [[Foo type A]]
+ * circle  50 50 20     [[Foo type B]]
+ *
+ * desc bottom-left
+ * </imagemap>
+ *
+ * Coordinates are relative to the source image, not the thumbnail
+ *
+ */
+
+class ImageMap {
+       static public $id = 0;
+
+       const TOP_RIGHT = 0;
+       const BOTTOM_RIGHT = 1;
+       const BOTTOM_LEFT = 2;
+       const TOP_LEFT = 3;
+       const NONE = 4;
+
+       /**
+        * @param $input
+        * @param $params
+        * @param $parser Parser
+        * @return array|mixed|string
+        */
+       public static function render( $input, $params, $parser ) {
+               global $wgExtensionAssetsPath, $wgUrlProtocols, 
$wgNoFollowLinks;
+
+               $lines = explode( "\n", $input );
+
+               $first = true;
+               $lineNum = 0;
+               $mapHTML = '';
+               $links = array();
+
+               # Define canonical desc types to allow i18n of 
'imagemap_desc_types'
+               $descTypesCanonical = 'top-right, bottom-right, bottom-left, 
top-left, none';
+               $descType = self::BOTTOM_RIGHT;
+               $defaultLinkAttribs = false;
+               $realmap = true;
+               $extLinks = array();
+               foreach ( $lines as $line ) {
+                       ++$lineNum;
+                       $externLink = false;
+
+                       $line = trim( $line );
+                       if ( $line == '' || $line[0] == '#' ) {
+                               continue;
+                       }
+
+                       if ( $first ) {
+                               $first = false;
+
+                               # The first line should have an image 
specification on it
+                               # Extract it and render the HTML
+                               $bits = explode( '|', $line, 2 );
+                               if ( count( $bits ) == 1 ) {
+                                       $image = $bits[0];
+                                       $options = '';
+                               } else {
+                                       list( $image, $options ) = $bits;
+                               }
+                               $imageTitle = Title::newFromText( $image );
+                               if ( !$imageTitle || 
$imageTitle->getNamespace() != NS_IMAGE ) {
+                                       return self::error( 'imagemap_no_image' 
);
+                               }
+                               if ( wfIsBadImage( $imageTitle->getDBkey(), 
$parser->mTitle ) ) {
+                                       return self::error( 
'imagemap_bad_image' );
+                               }
+                               // Parse the options so we can use links and 
the like in the caption
+                               $parsedOptions = $parser->recursiveTagParse( 
$options );
+                               $imageHTML = $parser->makeImage( $imageTitle, 
$parsedOptions );
+                               $parser->replaceLinkHolders( $imageHTML );
+                               $imageHTML = $parser->mStripState->unstripBoth( 
$imageHTML );
+                               $imageHTML = 
Sanitizer::normalizeCharReferences( $imageHTML );
+
+                               $domDoc = new DOMDocument();
+                               wfSuppressWarnings();
+                               $ok = $domDoc->loadXML( $imageHTML );
+                               wfRestoreWarnings();
+                               if ( !$ok ) {
+                                       return self::error( 
'imagemap_invalid_image' );
+                               }
+                               $xpath = new DOMXPath( $domDoc );
+                               $imgs = $xpath->query( '//img' );
+                               if ( !$imgs->length ) {
+                                       return self::error( 
'imagemap_invalid_image' );
+                               }
+                               $imageNode = $imgs->item( 0 );
+                               $thumbWidth = $imageNode->getAttribute( 'width' 
);
+                               $thumbHeight = $imageNode->getAttribute( 
'height' );
+
+                               $imageObj = wfFindFile( $imageTitle );
+                               if ( !$imageObj || !$imageObj->exists() ) {
+                                       return self::error( 
'imagemap_invalid_image' );
+                               }
+                               # Add the linear dimensions to avoid inaccuracy 
in the scale
+                               # factor when one is much larger than the other
+                               # (sx+sy)/(x+y) = s
+                               $denominator = $imageObj->getWidth() + 
$imageObj->getHeight();
+                               $numerator = $thumbWidth + $thumbHeight;
+                               if ( $denominator <= 0 || $numerator <= 0 ) {
+                                       return self::error( 
'imagemap_invalid_image' );
+                               }
+                               $scale = $numerator / $denominator;
+                               continue;
+                       }
+
+                       # Handle desc spec
+                       $cmd = strtok( $line, " \t" );
+                       if ( $cmd == 'desc' ) {
+                               $typesText = wfMessage( 'imagemap_desc_types' 
)->inContentLanguage()->text();
+                               if ( $descTypesCanonical != $typesText ) {
+                                       // i18n desc types exists
+                                       $typesText = $descTypesCanonical . ', ' 
. $typesText;
+                               }
+                               $types = array_map( 'trim', explode( ',', 
$typesText ) );
+                               $type = trim( strtok( '' ) );
+                               $descType = array_search( $type, $types );
+                               if ( $descType > 4 ) {
+                                       // A localized descType is used. 
Subtract 5 to reach the canonical desc type.
+                                       $descType = $descType - 5;
+                               }
+                               if ( $descType === false || $descType < 0 ) { 
// <0? In theory never, but paranoia...
+                                       return self::error( 
'imagemap_invalid_desc', $typesText );
+                               }
+                               continue;
+                       }
+
+                       # Find the link
+                       $link = trim( strstr( $line, '[' ) );
+                       $m = array();
+                       if ( preg_match( '/^ \[\[  ([^|]*+)  \|  ([^\]]*+)  
\]\] \w* $ /x', $link, $m ) ) {
+                               $title = Title::newFromText( $m[1] );
+                               $alt = trim( $m[2] );
+                       } elseif ( preg_match( '/^ \[\[  ([^\]]*+) \]\] \w* $ 
/x', $link, $m ) ) {
+                               $title = Title::newFromText( $m[1] );
+                               if ( is_null( $title ) ) {
+                                       return self::error( 
'imagemap_invalid_title', $lineNum );
+                               }
+                               $alt = $title->getFullText();
+                       } elseif ( in_array( substr( $link, 1, strpos( $link, 
'//' ) + 1 ), $wgUrlProtocols ) || in_array( substr( $link, 1, strpos( $link, 
':' ) ), $wgUrlProtocols ) ) {
+                               if ( preg_match( '/^ \[  ([^\s]*+)  \s  
([^\]]*+)  \] \w* $ /x', $link, $m ) ) {
+                                       $title = $m[1];
+                                       $alt = trim( $m[2] );
+                                       $externLink = true;
+                               } elseif ( preg_match( '/^ \[  ([^\]]*+) \] \w* 
$ /x', $link, $m ) ) {
+                                       $title = $alt = trim( $m[1] );
+                                       $externLink = true;
+                               }
+                       } else {
+                               return self::error( 'imagemap_no_link', 
$lineNum );
+                       }
+                       if ( !$title ) {
+                               return self::error( 'imagemap_invalid_title', 
$lineNum );
+                       }
+
+                       $shapeSpec = substr( $line, 0, -strlen( $link ) );
+
+                       # Tokenize shape spec
+                       $shape = strtok( $shapeSpec, " \t" );
+                       switch ( $shape ) {
+                               case 'default':
+                                       $coords = array();
+                                       break;
+                               case 'rect':
+                                       $coords = self::tokenizeCoords( 4, 
$lineNum );
+                                       if ( !is_array( $coords ) ) {
+                                               return $coords;
+                                       }
+                                       break;
+                               case 'circle':
+                                       $coords = self::tokenizeCoords( 3, 
$lineNum );
+                                       if ( !is_array( $coords ) ) {
+                                               return $coords;
+                                       }
+                                       break;
+                               case 'poly':
+                                       $coords = array();
+                                       $coord = strtok( " \t" );
+                                       while ( $coord !== false ) {
+                                               $coords[] = $coord;
+                                               $coord = strtok( " \t" );
+                                       }
+                                       if ( !count( $coords ) ) {
+                                               return self::error( 
'imagemap_missing_coord', $lineNum );
+                                       }
+                                       if ( count( $coords ) % 2 !== 0 ) {
+                                               return self::error( 
'imagemap_poly_odd', $lineNum );
+                                       }
+                                       break;
+                               default:
+                                       return self::error( 
'imagemap_unrecognised_shape', $lineNum );
+                       }
+
+                       # Scale the coords using the size of the source image
+                       foreach ( $coords as $i => $c ) {
+                               $coords[$i] = intval( round( $c * $scale ) );
+                       }
+
+                       # Construct the area tag
+                       $attribs = array();
+                       if ( $externLink ) {
+                               $attribs['href'] = $title;
+                               $attribs['class'] = 'plainlinks';
+                               if ( $wgNoFollowLinks ) {
+                                       $attribs['rel'] = 'nofollow';
+                               }
+                       } elseif ( $title->getFragment() != '' && 
$title->getPrefixedDBkey() == '' ) {
+                               # XXX: kluge to handle [[#Fragment]] links, 
should really fix getLocalURL()
+                               # in Title.php to return an empty string in 
this case
+                               $attribs['href'] = $title->getFragmentForURL();
+                       } else {
+                               $attribs['href'] = $title->getLocalURL() . 
$title->getFragmentForURL();
+                       }
+                       if ( $shape != 'default' ) {
+                               $attribs['shape'] = $shape;
+                       }
+                       if ( $coords ) {
+                               $attribs['coords'] = implode( ',', $coords );
+                       }
+                       if ( $alt != '' ) {
+                               if ( $shape != 'default' ) {
+                                       $attribs['alt'] = $alt;
+                               }
+                               $attribs['title'] = $alt;
+                       }
+                       if ( $shape == 'default' ) {
+                               $defaultLinkAttribs = $attribs;
+                       } else {
+                               $mapHTML .= Xml::element( 'area', $attribs ) . 
"\n";
+                       }
+                       if ( $externLink ) {
+                               $extLinks[] = $title;
+                       } else {
+                               $links[] = $title;
+                       }
+               }
+
+               if ( $first ) {
+                       return self::error( 'imagemap_no_image' );
+               }
+
+               if ( $mapHTML == '' ) {
+                       // no areas defined, default only. It's not a real 
imagemap, so we do not need some tags
+                       $realmap = false;
+               }
+
+               if ( $realmap ) {
+                       # Construct the map
+                       # Add random number to avoid breaking cached HTML 
fragments that are
+                       # later joined together on the one page (bug 16471)
+                       $mapName = "ImageMap_" . ++self::$id . '_' . mt_rand( 
0, 0x7fffffff );
+                       $mapHTML = "<map name=\"$mapName\">\n$mapHTML</map>\n";
+
+                       # Alter the image tag
+                       $imageNode->setAttribute( 'usemap', "#$mapName" );
+               }
+
+               # Add a surrounding div, remove the default link to the 
description page
+               $anchor = $imageNode->parentNode;
+               $parent = $anchor->parentNode;
+               $div = $parent->insertBefore( new DOMElement( 'div' ), $anchor 
);
+               $div->setAttribute( 'class', 'noresize' );
+               if ( $defaultLinkAttribs ) {
+                       $defaultAnchor = $div->appendChild( new DOMElement( 'a' 
) );
+                       foreach ( $defaultLinkAttribs as $name => $value ) {
+                               $defaultAnchor->setAttribute( $name, $value );
+                       }
+                       $imageParent = $defaultAnchor;
+               } else {
+                       $imageParent = $div;
+               }
+
+               # Add the map HTML to the div
+               # We used to add it before the div, but that made tidy unhappy
+               if ( $mapHTML != '' ) {
+                       $mapDoc = new DOMDocument();
+                       $mapDoc->loadXML( $mapHTML );
+                       $mapNode = $domDoc->importNode( 
$mapDoc->documentElement, true );
+                       $div->appendChild( $mapNode );
+               }
+
+               $imageParent->appendChild( $imageNode->cloneNode( true ) );
+               $parent->removeChild( $anchor );
+
+               # Determine whether a "magnify" link is present
+               $xpath = new DOMXPath( $domDoc );
+               $magnify = $xpath->query( '//div[@class="magnify"]' );
+               if ( !$magnify->length && $descType != self::NONE ) {
+                       # Add image description link
+                       if ( $descType == self::TOP_LEFT || $descType == 
self::BOTTOM_LEFT ) {
+                               $marginLeft = 0;
+                       } else {
+                               $marginLeft = $thumbWidth - 20;
+                       }
+                       if ( $descType == self::TOP_LEFT || $descType == 
self::TOP_RIGHT ) {
+                               $marginTop = -$thumbHeight;
+                               // 1px hack for IE, to stop it poking out the 
top
+                               $marginTop += 1;
+                       } else {
+                               $marginTop = -20;
+                       }
+                       $div->setAttribute( 'style', "height: {$thumbHeight}px; 
width: {$thumbWidth}px; " );
+                       $descWrapper = $div->appendChild( new DOMElement( 'div' 
) );
+                       $descWrapper->setAttribute( 'style',
+                               "margin-left: {$marginLeft}px; " .
+                                       "margin-top: {$marginTop}px; " .
+                                       "text-align: left;"
+                       );
+
+                       $descAnchor = $descWrapper->appendChild( new 
DOMElement( 'a' ) );
+                       $descAnchor->setAttribute( 'href', 
$imageTitle->getLocalURL() );
+                       $descAnchor->setAttribute(
+                               'title',
+                               wfMessage( 'imagemap_description' 
)->inContentLanguage()->text()
+                       );
+                       $descImg = $descAnchor->appendChild( new DOMElement( 
'img' ) );
+                       $descImg->setAttribute(
+                               'alt',
+                               wfMessage( 'imagemap_description' 
)->inContentLanguage()->text()
+                       );
+                       $descImg->setAttribute( 'src', 
"$wgExtensionAssetsPath/ImageMap/desc-20.png" );
+                       $descImg->setAttribute( 'style', 'border: none;' );
+               }
+
+               # Output the result
+               # We use saveXML() not saveHTML() because then we get 
XHTML-compliant output.
+               # The disadvantage is that we have to strip out the DTD
+               $output = preg_replace( '/<\?xml[^?]*\?>/', '', 
$domDoc->saveXML() );
+
+               # Register links
+               foreach ( $links as $title ) {
+                       if ( $title->isExternal() || $title->getNamespace() == 
NS_SPECIAL ) {
+                               // Don't register special or interwiki links...
+                       } elseif ( $title->getNamespace() == NS_MEDIA ) {
+                               // Regular Media: links are recorded as image 
usages
+                               $parser->mOutput->addImage( $title->getDBkey() 
);
+                       } else {
+                               // Plain ol' link
+                               $parser->mOutput->addLink( $title );
+                       }
+               }
+               if ( isset( $extLinks ) ) {
+                       foreach ( $extLinks as $title ) {
+                               $parser->mOutput->addExternalLink( $title );
+                       }
+               }
+               # Armour output against broken parser
+               $output = str_replace( "\n", '', $output );
+               return $output;
+       }
+
+       /**
+        * @param $count int
+        * @param $lineNum int|string
+        * @return array|string
+        */
+       static function tokenizeCoords( $count, $lineNum ) {
+               $coords = array();
+               for ( $i = 0; $i < $count; $i++ ) {
+                       $coord = strtok( " \t" );
+                       if ( $coord === false ) {
+                               return self::error( 'imagemap_missing_coord', 
$lineNum );
+                       }
+                       if ( !is_numeric( $coord ) || $coord > 1e9 || $coord < 
0 ) {
+                               return self::error( 'imagemap_invalid_coord', 
$lineNum );
+                       }
+                       $coords[$i] = $coord;
+               }
+               return $coords;
+       }
+
+       /**
+        * @param $name string
+        * @param $line string|int|bool
+        * @return string
+        */
+       static function error( $name, $line = false ) {
+               return '<p class="error">' . wfMessage( $name, $line )->text() 
. '</p>';
+       }
+}

-- 
To view, visit https://gerrit.wikimedia.org/r/164159
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2b171c8414d4df10a8e1cb0bab1f6974bc5f6463
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/GraphViz
Gerrit-Branch: master
Gerrit-Owner: Welterkj <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to