From 9f6e0b9a934719634098793907c4cf5a86b5cfd5 Mon Sep 17 00:00:00 2001
From:  <mako@atdot.cc>
Date: Mon, 11 Aug 2008 19:45:25 -0400
Subject: [PATCH 1/1] made some changes getting ready to release it

---
 EditImage.i18n.php |  3 ++-
 EditImage.php      |  2 --
 EditImage_body.php | 33 +++++++++++++++++----------------
 3 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/EditImage.i18n.php b/EditImage.i18n.php
index f19659c..4731333 100644
--- a/EditImage.i18n.php
+++ b/EditImage.i18n.php
@@ -2,6 +2,7 @@
 $messages = array();
 $messages['en'] = array( 
     'editimage' => 'Edit Image',
-    'cropimage' => 'Crop Image'
+    'cropimage' => 'Crop Image',
+    'editimage-instructions' => 'Use your mouse to select an area in the image below and then click "Crop Image" to crop the image to the size.'
 );
 ?>
diff --git a/EditImage.php b/EditImage.php
index af64e8d..10e51e0 100644
--- a/EditImage.php
+++ b/EditImage.php
@@ -38,8 +38,6 @@ $wgExtensionCredits['specialpage'][] = array(
     'description' => 'This extension provides a way for Mediawiki users to edit images from the web.'
 );
 
-
-
 function wfAddActionContentHook( &$content_actions ) {
     global $wgRequest, $wgRequest, $wgTitle;
     
diff --git a/EditImage_body.php b/EditImage_body.php
index 6315b85..294a07d 100644
--- a/EditImage_body.php
+++ b/EditImage_body.php
@@ -18,7 +18,7 @@ function resizeImage ( $srcpath, $x1, $y1, $x2, $y2) {
     # specify white background color, will be used for transparent images
     # in internet explorer/windows instead of default black.
     $cmd  =  wfescapeshellarg($wgImageMagickConvertCommand) .
-        " -background white ".
+        " -background white " .
         wfescapeshellarg($srcpath) .
         // coalesce is needed to scale animated gifs properly (bug 1017).
         ' -coalesce ' .
@@ -41,10 +41,7 @@ function uploadNewFile ($old_file, $new_filename) {
     $form->mSessionKey     = false;
     $form->mStashed        = false;
     $form->mUploadClicked  = true;
-    //$form->mReUpload       = true;
 
-    // PHP won't auto-cleanup the file
-    $form->mRemoveTempFile = file_exists( $local_file);
     $form->execute();
 }
  
@@ -59,13 +56,9 @@ class EditImage extends SpecialPage {
 
         global $wgContLang;
         global $wgUser;
-        # add the javascript
-        global $wgJsMimeType, $wgScriptPath ;
-        $wgOut->addScript( "<script type=\"{$wgJsMimeType}\" src=\"$wgScriptPath/extensions/EditImage/lib/prototype.js\"></script>\n");
-        $wgOut->addScript( "<script type=\"{$wgJsMimeType}\" src=\"$wgScriptPath/extensions/EditImage/lib/scriptaculous.js\"></script>\n");
-        $wgOut->addScript( "<script type=\"{$wgJsMimeType}\" src=\"$wgScriptPath/extensions/EditImage/cropper.js\"></script>\n");
-        $wgOut->addScript( "<script type=\"{$wgJsMimeType}\" src=\"$wgScriptPath/extensions/EditImage/EditImage.js\"></script>\n");
-
+        # globals for javascript
+        global $wgJsMimeType, $wgScriptPath; 
+        
         # try to find the image that the user entered
         $image = $wgRequest->getText('image');
         $title = Title::newFromText( $image, NS_IMAGE );
@@ -84,10 +77,21 @@ class EditImage extends SpecialPage {
                 $resized_fn = resizeImage($file->getFullPath(), $x1, $y1, $x2, $y2);
                 uploadNewFile($file, $resized_fn);
 
+                # delete the file if it still exists
+                if (file_exists($resized_fn)) { unlink($resized_fn); }
+
             } else {
 
                 if ( $file && $file->exists() ) {
-                    $wgOut->addHTML("<p>Use your mouse to select the new area on the image below. When you are done, press crop.</p>\n");
+                    # add the javascript
+                    $wgOut->addScript( "<script type=\"{$wgJsMimeType}\" src=\"$wgScriptPath/extensions/EditImage/lib/prototype.js\"></script>\n");
+                    $wgOut->addScript( "<script type=\"{$wgJsMimeType}\" src=\"$wgScriptPath/extensions/EditImage/lib/scriptaculous.js\"></script>\n");
+                    $wgOut->addScript( "<script type=\"{$wgJsMimeType}\" src=\"$wgScriptPath/extensions/EditImage/cropper.js\"></script>\n");
+                    $wgOut->addScript( "<script type=\"{$wgJsMimeType}\" src=\"$wgScriptPath/extensions/EditImage/EditImage.js\"></script>\n");
+
+                    $instructions = wfMsg('editimage-instructions');
+                    $crop_text = wfMsg('cropimage');
+                    $wgOut->addHTML("<p>{$instructions}</p>\n");
 
                     # add image
                     $wgOut->addHTML("<div><img src=\"{$file->getUrl()}\" alt=\"source image\" id=\"sourceImage\" /></div>\n");
@@ -119,18 +123,15 @@ class EditImage extends SpecialPage {
                                        </td>
                                      </tr><table>
 
-                                     <p><input type=\"submit\" value=\"Crop Image\" /></p>
+                                     <p><input type=\"submit\" value=\"{$crop_text}\" /></p>
                                      </form>");
 
                 } else {
                     $wgOut->setStatusCode( 404 );
                 }
             }
-
         }
-
     }
-
 }
 
 ?>
-- 
2.39.5