Merge "Made RepoGroup use ProcessCacheLRU"
[lhc/web/wiklou.git] / includes / filerepo / RepoGroup.php
index 039f435..33ab8ae 100644 (file)
@@ -30,12 +30,16 @@ class RepoGroup {
        /** @var LocalRepo */
        protected $localRepo;
 
+       /** @var FileRepo[] */
        protected $foreignRepos;
 
        /** @var bool */
        protected $reposInitialised = false;
 
+       /** @var array */
        protected $localInfo;
+
+       /** @var array */
        protected $foreignInfo;
 
        /** @var ProcessCacheLRU  */
@@ -76,7 +80,7 @@ class RepoGroup {
         * It's not enough to just create a superclass ... you have
         * to get people to call into it even though all they know is RepoGroup::singleton()
         *
-        * @param $instance RepoGroup
+        * @param RepoGroup $instance
         */
        static function setSingleton( $instance ) {
                self::$instance = $instance;
@@ -87,9 +91,9 @@ class RepoGroup {
         *
         * @param array $localInfo Associative array for local repo's info
         * @param array $foreignInfo of repository info arrays.
-        *     Each info array is an associative array with the 'class' member
-        *     giving the class name. The entire array is passed to the repository
-        *     constructor as the first parameter.
+        *   Each info array is an associative array with the 'class' member
+        *   giving the class name. The entire array is passed to the repository
+        *   constructor as the first parameter.
         */
        function __construct( $localInfo, $foreignInfo ) {
                $this->localInfo = $localInfo;
@@ -103,18 +107,15 @@ class RepoGroup {
         *
         * @param $title Title|string Title object or string
         * @param array $options Associative array of options:
-        *     time:           requested time for an archived image, or false for the
-        *                     current version. An image object will be returned which was
-        *                     created at the specified time.
-        *
-        *     ignoreRedirect: If true, do not follow file redirects
-        *
-        *     private:        If true, return restricted (deleted) files if the current
-        *                     user is allowed to view them. Otherwise, such files will not
-        *                     be found.
-        *
-        *     bypassCache:    If true, do not use the process-local cache of File objects
-        * @return File object or false if it is not found
+        *   time:           requested time for an archived image, or false for the
+        *                   current version. An image object will be returned which was
+        *                   created at the specified time.
+        *   ignoreRedirect: If true, do not follow file redirects
+        *   private:        If true, return restricted (deleted) files if the current
+        *                   user is allowed to view them. Otherwise, such files will not
+        *                   be found.
+        *   bypassCache:    If true, do not use the process-local cache of File objects
+        * @return File|bool False if title is not found
         */
        function findFile( $title, $options = array() ) {
                if ( !is_array( $options ) ) {
@@ -167,10 +168,27 @@ class RepoGroup {
        }
 
        /**
-        * @param $inputItems array
+        * Search repositories for many files at once.
+        *
+        * @param array $items An array of titles, or an array of findFile() options with
+        *    the "title" option giving the title. Example:
+        *
+        *     $findItem = array( 'title' => $title, 'private' => true );
+        *     $findBatch = array( $findItem );
+        *     $repo->findFiles( $findBatch );
+        *
+        *    No title should appear in $items twice, as the result use titles as keys
+        * @param int $flags Supports:
+        *     - FileRepo::NAME_AND_TIME_ONLY : return a (search title => (title,timestamp)) map.
+        *       The search title uses the input titles; the other is the final post-redirect title.
+        *       All titles are returned as string DB keys and the inner array is associative.
+        * @return array Map of (file name => File objects) for matches
+        *
+        * @param array $inputItems
+        * @param integer $flags
         * @return array
         */
-       function findFiles( $inputItems ) {
+       function findFiles( array $inputItems, $flags = 0 ) {
                if ( !$this->reposInitialised ) {
                        $this->initialiseRepos();
                }
@@ -186,7 +204,7 @@ class RepoGroup {
                        }
                }
 
-               $images = $this->localRepo->findFiles( $items );
+               $images = $this->localRepo->findFiles( $items, $flags );
 
                foreach ( $this->foreignRepos as $repo ) {
                        // Remove found files from $items
@@ -194,7 +212,7 @@ class RepoGroup {
                                unset( $items[$name] );
                        }
 
-                       $images = array_merge( $images, $repo->findFiles( $items ) );
+                       $images = array_merge( $images, $repo->findFiles( $items, $flags ) );
                }
 
                return $images;
@@ -214,6 +232,7 @@ class RepoGroup {
                if ( $redir ) {
                        return $redir;
                }
+
                foreach ( $this->foreignRepos as $repo ) {
                        $redir = $repo->checkRedirect( $title );
                        if ( $redir ) {
@@ -274,7 +293,7 @@ class RepoGroup {
         * Find all instances of files with this keys
         *
         * @param array $hashes base 36 SHA-1 hashes
-        * @return Array of array of File objects
+        * @return array of array of File objects
         */
        function findBySha1s( array $hashes ) {
                if ( !$this->reposInitialised ) {
@@ -295,7 +314,7 @@ class RepoGroup {
 
        /**
         * Get the repo instance with a given key.
-        * @param $index string|int
+        * @param string|int $index
         * @return bool|LocalRepo
         */
        function getRepo( $index ) {
@@ -313,7 +332,7 @@ class RepoGroup {
 
        /**
         * Get the repo instance by its name
-        * @param $name string
+        * @param string $name
         * @return bool
         */
        function getRepoByName( $name ) {
@@ -343,8 +362,8 @@ class RepoGroup {
         * Call a function for each foreign repo, with the repo object as the
         * first parameter.
         *
-        * @param $callback Callback: the function to call
-        * @param array $params optional additional parameters to pass to the function
+        * @param callable $callback The function to call
+        * @param array $params Optional additional parameters to pass to the function
         * @return bool
         */
        function forEachForeignRepo( $callback, $params = array() ) {
@@ -360,7 +379,7 @@ class RepoGroup {
 
        /**
         * Does the installation have any foreign repos set up?
-        * @return Boolean
+        * @return bool
         */
        function hasForeignRepos() {
                return (bool)$this->foreignRepos;
@@ -393,7 +412,7 @@ class RepoGroup {
 
        /**
         * Split a virtual URL into repo, zone and rel parts
-        * @param $url string
+        * @param string $url
         * @throws MWException
         * @return array containing repo, zone and rel
         */
@@ -411,7 +430,7 @@ class RepoGroup {
        }
 
        /**
-        * @param $fileName string
+        * @param string $fileName
         * @return array
         */
        function getFileProps( $fileName ) {
@@ -430,7 +449,7 @@ class RepoGroup {
 
        /**
         * Clear RepoGroup process cache used for finding a file
-        * @param $title Title|null Title of the file or null to clear all files
+        * @param Title|null $title Title of the file or null to clear all files
         */
        public function clearCache( Title $title = null ) {
                if ( $title == null ) {