Basic caching for dir result

src/cachingshare.php

@@ -0,0 +1,61 @@
+<?php
+/**
+ * Copyright (c) 2013 Robin Appelman <icewind@owncloud.com>
+ * This file is licensed under the Affero General Public License version 3 or
+ * later.
+ * See the COPYING-README file.
+ */
+
+namespace SMB;
+
+/**
+ * Class CachingShare
+ *
+ * caches metadata from share
+ *
+ * @package SMB
+ */
+class CachingShare extends Share {
+	private $dirCache = array();
+
+	private function clear($path = null) {
+		if (is_null($path)) {
+			$this->dirCache = array();
+		} else {
+			unset($this->dirCache[$path]);
+		}
+	}
+
+	public function dir($path) {
+		if (!isset($this->dirCache[$path])) {
+			$this->dirCache[$path] = parent::dir($path);
+		}
+		return $this->dirCache[$path];
+	}
+
+	public function mkdir($path) {
+		$this->clear(dirname($path));
+		return parent::mkdir($path);
+	}
+
+	public function rmdir($path) {
+		$this->clear(dirname($path));
+		return parent::rmdir($path);
+	}
+
+	public function del($path) {
+		$this->clear(dirname($path));
+		return parent::del($path);
+	}
+
+	public function put($source, $target) {
+		$this->clear(dirname($target));
+		return parent::put($source, $target);
+	}
+
+	public function rename($from, $to) {
+		$this->clear(dirname($from));
+		$this->clear(dirname($to));
+		return parent::rename($from, $to);
+	}
+}

src/server.php

@@ -12,6 +12,9 @@ class Server {
 	const CLIENT = 'smbclient';
 	const LOCALE = 'en_US.UTF-8';
 
+	const CACHING_ENABLED = true;
+	const CACHING_DISABLED = true;
+
 	/**
 	 * @var string $host
 	 */
@@ -27,15 +30,21 @@ class Server {
 	 */
 	private $password;
 
+	/**
+	 * @var bool $caching
+	 */
+	private $caching;
+
 	/**
 	 * @param string $host
 	 * @param string $user
 	 * @param string $password
 	 */
-	public function __construct($host, $user, $password) {
+	public function __construct($host, $user, $password, $caching = self::CACHING_ENABLED) {
 		$this->host = $host;
 		$this->user = $user;
 		$this->password = $password;
+		$this->caching = $caching;
 	}
 
 	/**
@@ -55,7 +64,7 @@ class Server {
 	/**
 	 * @return string
 	 */
-	public function getPassword(){
+	public function getPassword() {
 		return $this->password;
 	}
 
@@ -74,7 +83,7 @@ class Server {
 		$shareNames = $cmd->run(null);
 		$shares = array();
 		foreach ($shareNames as $name => $description) {
-			$shares[] = new Share($this, $name);
+			$shares[] = $this->getShare($name);
 		}
 		return $shares;
 	}
@@ -84,6 +93,10 @@ class Server {
 	 * @return Share
 	 */
 	public function getShare($name) {
+		if ($this->caching === self::CACHING_ENABLED) {
+			return new CachingShare($this, $name);
+		} else {
 			return new Share($this, $name);
 		}
+	}
 }