From: Ilmari Karonen <vyznev@users.mediawiki.org>
Date: Sat, 13 Dec 2008 04:35:48 +0000 (+0000)
Subject: Remove  parameter from FileRepo::findFiles(), it was fubar and nobody was using it... 
X-Git-Tag: 1.31.0-rc.0~44001
X-Git-Url: https://git.cyclocoop.org/%7B%24www_url%7Dadmin/compta/comptes/?a=commitdiff_plain;h=b74934a6f31615f9281eb9341dd7db32b7da3ec4;p=lhc%2Fweb%2Fwiklou.git

Remove  parameter from FileRepo::findFiles(), it was fubar and nobody was using it anyway.  (What we really need is some way to pass timestamps to that function, which probably requires further rethinking of the interface.)
---

diff --git a/includes/filerepo/FileRepo.php b/includes/filerepo/FileRepo.php
index fdd9757c97..1f2a4cd882 100644
--- a/includes/filerepo/FileRepo.php
+++ b/includes/filerepo/FileRepo.php
@@ -133,10 +133,10 @@ abstract class FileRepo {
 	 * @param array $titles, an array of titles
 	 * @param int $flags
 	 */
-	function findFiles( $titles, $flags ) {
+	function findFiles( $titles ) {
 		$result = array();
 		foreach ( $titles as $index => $title ) {
-			$file = $this->findFile( $title, $flags );
+			$file = $this->findFile( $title );
 			if ( $file )
 				$result[$file->getTitle()->getDBkey()] = $file;
 		}
diff --git a/includes/filerepo/LocalRepo.php b/includes/filerepo/LocalRepo.php
index ee03e32de9..5eb1a11cc2 100644
--- a/includes/filerepo/LocalRepo.php
+++ b/includes/filerepo/LocalRepo.php
@@ -164,8 +164,7 @@ class LocalRepo extends FSRepo {
 	/*
 	 * Find many files using one query
 	 */
-	function findFiles( $titles, $flags ) {
-		// FIXME: Comply with $flags
+	function findFiles( $titles ) {
 	 	// FIXME: Only accepts a $titles array where the keys are the sanitized
 	 	// file names.