Revision: 43670
Author:   demon
Date:     2008-11-18 12:11:03 +0000 (Tue, 18 Nov 2008)

Log Message:
-----------
Cleanup to r43657, don't really need strlen() here.

Modified Paths:
--------------
    trunk/phase3/includes/filerepo/ForeignAPIRepo.php

Modified: trunk/phase3/includes/filerepo/ForeignAPIRepo.php
===================================================================
--- trunk/phase3/includes/filerepo/ForeignAPIRepo.php   2008-11-18 11:36:09 UTC 
(rev 43669)
+++ trunk/phase3/includes/filerepo/ForeignAPIRepo.php   2008-11-18 12:11:03 UTC 
(rev 43670)
@@ -128,8 +128,7 @@
                        $foreignUrl = $this->getThumbUrl( $name, $width, 
$height );
                        
                        // We need the same filename as the remote one :)
-                       $fileName = ltrim( substr( $foreignUrl, strrpos( 
$foreignUrl, '/' ),
-                                                               strlen ( 
$foreignUrl ) ), '/' );
+                       $fileName = ltrim( substr( $foreignUrl, strrpos( 
$foreignUrl, '/' ) ), '/' );
                        $path = 'thumb/' . $this->getHashPath( $name ) . $name 
. "/";
                        if ( !is_dir($wgUploadDirectory . '/' . $path) ) {
                                wfMkdirParents($wgUploadDirectory . '/' . 
$path);



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

Reply via email to