From 17dd5d0816e500d0818a07bc9ca72b52abce40b3 Mon Sep 17 00:00:00 2001 From: Robin McCorkell Date: Fri, 9 Jan 2015 13:12:21 +0000 Subject: [PATCH 1/2] Add Null memcacher --- lib/private/memcache/factory.php | 4 ++-- lib/private/memcache/null.php | 35 ++++++++++++++++++++++++++++++++ 2 files changed, 37 insertions(+), 2 deletions(-) create mode 100644 lib/private/memcache/null.php diff --git a/lib/private/memcache/factory.php b/lib/private/memcache/factory.php index dba9e8a0e0..1e663eecfe 100644 --- a/lib/private/memcache/factory.php +++ b/lib/private/memcache/factory.php @@ -24,7 +24,7 @@ class Factory implements ICacheFactory { } /** - * get a cache instance, will return null if no backend is available + * get a cache instance, or Null backend if no backend available * * @param string $prefix * @return \OC\Memcache\Cache @@ -42,7 +42,7 @@ class Factory implements ICacheFactory { } elseif (Memcached::isAvailable()) { return new Memcached($prefix); } else { - return null; + return new Null($prefix); } } diff --git a/lib/private/memcache/null.php b/lib/private/memcache/null.php new file mode 100644 index 0000000000..62cd060aac --- /dev/null +++ b/lib/private/memcache/null.php @@ -0,0 +1,35 @@ + + * This file is licensed under the Affero General Public License version 3 or + * later. + * See the COPYING-README file. + */ + +namespace OC\Memcache; + +class Null extends Cache { + public function get($key) { + return null; + } + + public function set($key, $value, $ttl = 0) { + return true; + } + + public function hasKey($key) { + return false; + } + + public function remove($key) { + return true; + } + + public function clear($prefix = '') { + return true; + } + + static public function isAvailable() { + return true; + } +} From c29138311614cba8bc4699a0afa485aefe621b6b Mon Sep 17 00:00:00 2001 From: Robin McCorkell Date: Fri, 9 Jan 2015 13:13:02 +0000 Subject: [PATCH 2/2] Memcache binary executable searching It's slow, this makes it fast! --- lib/private/helper.php | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/lib/private/helper.php b/lib/private/helper.php index 8e9b7d3b6f..86cc0850bc 100644 --- a/lib/private/helper.php +++ b/lib/private/helper.php @@ -882,13 +882,19 @@ class OC_Helper { * @return null|string */ public static function findBinaryPath($program) { + $memcache = \OC::$server->getMemCacheFactory()->create('findBinaryPath'); + if ($memcache->hasKey($program)) { + return $memcache->get($program); + } + $result = null; if (!\OC_Util::runningOnWindows() && self::is_function_enabled('exec')) { exec('command -v ' . escapeshellarg($program) . ' 2> /dev/null', $output, $returnCode); if ($returnCode === 0 && count($output) > 0) { - return escapeshellcmd($output[0]); + $result = escapeshellcmd($output[0]); } } - return null; + $memcache->set($program, $result, 3600); + return $result; } /**