From: Chad Horohoe <demon@users.mediawiki.org>
Date: Tue, 18 Nov 2008 12:11:03 +0000 (+0000)
Subject: Cleanup to r43657, don't really need strlen() here.
X-Git-Tag: 1.31.0-rc.0~44285
X-Git-Url: http://git.cyclocoop.org/%7B%7B%20url_for%28%27admin_vote_add%27%29%20%7D%7D?a=commitdiff_plain;h=7b0ea1e8fd3813fa3a8b8bf8701f4ede970ee693;p=lhc%2Fweb%2Fwiklou.git

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

diff --git a/includes/filerepo/ForeignAPIRepo.php b/includes/filerepo/ForeignAPIRepo.php
index b4b08e883e..a6773e11ec 100644
--- a/includes/filerepo/ForeignAPIRepo.php
+++ b/includes/filerepo/ForeignAPIRepo.php
@@ -128,8 +128,7 @@ class ForeignAPIRepo extends FileRepo {
 			$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);