]> git.mxchange.org Git - friendica.git/commitdiff
code standards / simplifications
authorPhilipp Holzer <admin@philipp.info>
Thu, 5 Jul 2018 05:59:56 +0000 (07:59 +0200)
committerPhilipp Holzer <admin@philipp.info>
Thu, 5 Jul 2018 05:59:56 +0000 (07:59 +0200)
18 files changed:
src/Core/Cache/ArrayCache.php
src/Core/Cache/CacheDriverFactory.php
src/Core/Cache/ICacheDriver.php
src/Core/Cache/IMemoryCacheDriver.php
src/Core/Cache/MemcacheCacheDriver.php
src/Core/Cache/RedisCacheDriver.php
src/Core/Cache/TraitCompareDelete.php
src/Core/Cache/TraitCompareSet.php
src/Core/Lock.php
src/Core/Lock/AbstractLockDriver.php
src/Core/Lock/CacheLockDriver.php
src/Core/Lock/DatabaseLockDriver.php
src/Core/Lock/ILockDriver.php
src/Core/Lock/SemaphoreLockDriver.php
tests/src/Core/Lock/CacheLockDriverTest.php
tests/src/Core/Lock/DatabaseLockDriverTest.php
tests/src/Core/Lock/LockTest.php
tests/src/Core/Lock/SemaphoreLockDriverTest.php

index d4fe8bc7f7084b5a925d593ab5245fc3b7eb193f..71610c39dfab80e90aefdb96341ca49751d57db3 100644 (file)
@@ -6,7 +6,7 @@ namespace Friendica\Core\Cache;
 use Friendica\Core\Cache;
 
 /**
- * @brief Implementation of the IMemoryCacheDriver mainly for testing purpose
+ * Implementation of the IMemoryCacheDriver mainly for testing purpose
  *
  * Class ArrayCache
  *
@@ -80,4 +80,4 @@ class ArrayCache implements IMemoryCacheDriver
                        return false;
                }
        }
-}
\ No newline at end of file
+}
index 45cc17a52fbf8e7ca87665a664d32bba2c24ebb3..8fbdc1549e3cca434782cb7e8e10f9b5766a19d5 100644 (file)
@@ -9,12 +9,12 @@ use Friendica\Core\Config;
  *
  * @package Friendica\Core\Cache
  *
- * @brief A basic class to generate a CacheDriver
+ * A basic class to generate a CacheDriver
  */
 class CacheDriverFactory
 {
        /**
-        * @brief This method creates a CacheDriver for the given cache driver name
+        * This method creates a CacheDriver for the given cache driver name
         *
         * @param string $driver The name of the cache driver
         * @return ICacheDriver  The instance of the CacheDriver
index ff329f34eb25c0b2879364ea45a0a3baa62d177f..ced7b4e2168b935ab72467631df50581fd3f0e1a 100644 (file)
@@ -12,7 +12,7 @@ use Friendica\Core\Cache;
 interface ICacheDriver
 {
        /**
-        * @brief Fetches cached data according to the key
+        * Fetches cached data according to the key
         *
         * @param string $key The key to the cached data
         *
@@ -21,18 +21,18 @@ interface ICacheDriver
        public function get($key);
 
        /**
-        * @brief Stores data in the cache identified by the key. The input $value can have multiple formats.
+        * Stores data in the cache identified by the key. The input $value can have multiple formats.
         *
         * @param string  $key      The cache key
         * @param mixed   $value    The value to store
-        * @param integer $ttl           The cache lifespan, must be one of the Cache constants
+        * @param integer $ttl The cache lifespan, must be one of the Cache constants
         *
         * @return bool
         */
        public function set($key, $value, $ttl = Cache::FIVE_MINUTES);
 
        /**
-        * @brief Delete a key from the cache
+        * Delete a key from the cache
         *
         * @param string $key      The cache key
         *
@@ -41,7 +41,7 @@ interface ICacheDriver
        public function delete($key);
 
        /**
-        * @brief Remove outdated data from the cache
+        * Remove outdated data from the cache
         *
         * @return bool
         */
index 7843ca7b5fb86ac65bad8a47fe01fe5794409b3b..a50e2d1d48398bf25cd7f09ffc1a89aea1b36d74 100644 (file)
@@ -4,7 +4,7 @@ namespace Friendica\Core\Cache;
 use Friendica\Core\Cache;
 
 /**
- * @brief This interface defines methods for Memory-Caches only
+ * This interface defines methods for Memory-Caches only
  *
  * Interface IMemoryCacheDriver
  *
@@ -13,7 +13,7 @@ use Friendica\Core\Cache;
 interface IMemoryCacheDriver extends ICacheDriver
 {
        /**
-        * @brief Sets a value if it's not already stored
+        * Sets a value if it's not already stored
         *
         * @param string $key      The cache key
         * @param mixed  $value    The old value we know from the cache
@@ -23,7 +23,7 @@ interface IMemoryCacheDriver extends ICacheDriver
        public function add($key, $value, $ttl = Cache::FIVE_MINUTES);
 
        /**
-        * @brief Compares if the old value is set and sets the new value
+        * Compares if the old value is set and sets the new value
         *
         * @param string $key         The cache key
         * @param mixed  $oldValue    The old value we know from the cache
@@ -35,11 +35,11 @@ interface IMemoryCacheDriver extends ICacheDriver
        public function compareSet($key, $oldValue, $newValue, $ttl = Cache::FIVE_MINUTES);
 
        /**
-        * @brief Compares if the old value is set and removes it
+        * Compares if the old value is set and removes it
         *
         * @param string $key          The cache key
         * @param mixed  $value        The old value we know and want to delete
         * @return bool
         */
        public function compareDelete($key, $value);
-}
\ No newline at end of file
+}
index 0b1ca3cec960bc6bfc526eaecdbe35dc2583b1a9..8eb45d907ea45cb42e36e349f921193e03a214ff 100644 (file)
@@ -90,6 +90,9 @@ class MemcacheCacheDriver extends BaseObject implements IMemoryCacheDriver
                return $this->memcache->delete($key);
        }
 
+       /**
+        * (@inheritdoc)
+        */
        public function clear()
        {
                return $this->memcache->flush();
index d23fa2697bfe9b2398a4fdd5d29e780b4c732a28..25c18aa6b636e401176349d29f820927edc4ba66 100644 (file)
@@ -84,12 +84,7 @@ class RedisCacheDriver extends BaseObject implements IMemoryCacheDriver
 
 
        /**
-        * @brief Sets a value if it's not already stored
-        *
-        * @param string $key The cache key
-        * @param mixed $value The old value we know from the cache
-        * @param int    $ttl      The cache lifespan, must be one of the Cache constants
-        * @return bool
+        * (@inheritdoc)
         */
        public function add($key, $value, $ttl = Cache::FIVE_MINUTES)
        {
@@ -101,13 +96,7 @@ class RedisCacheDriver extends BaseObject implements IMemoryCacheDriver
        }
 
        /**
-        * @brief Compares if the old value is set and sets the new value
-        *
-        * @param string $key The cache key
-        * @param mixed $oldValue The old value we know
-        * @param mixed $newValue The new value we want to set
-        * @param int    $ttl      The cache lifespan, must be one of the Cache constants
-        * @return bool
+        * (@inheritdoc)
         */
        public function compareSet($key, $oldValue, $newValue, $ttl = Cache::FIVE_MINUTES)
        {
@@ -133,11 +122,7 @@ class RedisCacheDriver extends BaseObject implements IMemoryCacheDriver
                return false;
        }
        /**
-        * @brief Compares if the old value is set and removes it
-        *
-        * @param string $key The cache key
-        * @param mixed $value The old value we know and want to delete
-        * @return bool
+        * (@inheritdoc)
         */
        public function compareDelete($key, $value)
        {
index 898e39aecc7d7cd9731d203082d9f2ebb5c1358b..ef59f69cd17f4fd1a5d868337d4d517720480b68 100644 (file)
@@ -7,7 +7,7 @@ use Friendica\Core\Cache;
 /**
  * Trait TraitCompareSetDelete
  *
- * @brief This Trait is to compensate non native "exclusive" sets/deletes in caches
+ * This Trait is to compensate non native "exclusive" sets/deletes in caches
  *
  * @package Friendica\Core\Cache
  */
@@ -22,7 +22,7 @@ trait TraitCompareDelete
        abstract public function add($key, $value, $ttl = Cache::FIVE_MINUTES);
 
        /**
-        * @brief NonNative - Compares if the old value is set and removes it
+        * NonNative - Compares if the old value is set and removes it
         *
         * @param string $key          The cache key
         * @param mixed  $value        The old value we know and want to delete
@@ -42,4 +42,4 @@ trait TraitCompareDelete
                        return false;
                }
        }
-}
\ No newline at end of file
+}
index 55193b7567f0ecbb5736367fcefe894d74cab82e..77a6028355c41e4f0bb6d25ab6a023ad7b14ddbf 100644 (file)
@@ -7,7 +7,7 @@ use Friendica\Core\Cache;
 /**
  * Trait TraitCompareSetDelete
  *
- * @brief This Trait is to compensate non native "exclusive" sets/deletes in caches
+ * This Trait is to compensate non native "exclusive" sets/deletes in caches
  *
  * @package Friendica\Core\Cache
  */
@@ -22,7 +22,7 @@ trait TraitCompareSet
        abstract public function add($key, $value, $ttl = Cache::FIVE_MINUTES);
 
        /**
-        * @brief NonNative - Compares if the old value is set and sets the new value
+        * NonNative - Compares if the old value is set and sets the new value
         *
         * @param string $key         The cache key
         * @param mixed  $oldValue    The old value we know from the cache
@@ -45,4 +45,4 @@ trait TraitCompareSet
                        return false;
                }
        }
-}
\ No newline at end of file
+}
index 7235c64a982aa3a9e2ec1e6f35f013652655a9ea..47a1f9b4fe3090f9d7ea3aa95455478e1691bcad 100644 (file)
@@ -29,21 +29,9 @@ class Lock
                try {
                        switch ($lock_driver) {
                                case 'memcache':
-                                       $cache_driver = CacheDriverFactory::create('memcache');
-                                       if ($cache_driver instanceof IMemoryCacheDriver) {
-                                               self::$driver = new Lock\CacheLockDriver($cache_driver);
-                                       }
-                                       break;
-
                                case 'memcached':
-                                       $cache_driver = CacheDriverFactory::create('memcached');
-                                       if ($cache_driver instanceof IMemoryCacheDriver) {
-                                               self::$driver = new Lock\CacheLockDriver($cache_driver);
-                                       }
-                                       break;
-
                                case 'redis':
-                                       $cache_driver = CacheDriverFactory::create('redis');
+                                       $cache_driver = CacheDriverFactory::create($lock_driver);
                                        if ($cache_driver instanceof IMemoryCacheDriver) {
                                                self::$driver = new Lock\CacheLockDriver($cache_driver);
                                        }
@@ -129,7 +117,7 @@ class Lock
         */
        public static function acquireLock($key, $timeout = 120)
        {
-               return self::getDriver()->acquireLock($key, $timeout);
+               return self::getDriver()->acquire($key, $timeout);
        }
 
        /**
@@ -140,7 +128,7 @@ class Lock
         */
        public static function releaseLock($key)
        {
-               return self::getDriver()->releaseLock($key);
+               return self::getDriver()->release($key);
        }
 
        /**
index 09549c50bf9540d7448598f86bb7529f554df6fd..53597d45fc9c316e243031f18011301ec9405140 100644 (file)
@@ -8,7 +8,7 @@ use Friendica\BaseObject;
  *
  * @package Friendica\Core\Lock
  *
- * @brief Basic class for Locking with common functions (local acquired locks, releaseAll, ..)
+ * Basic class for Locking with common functions (local acquired locks, releaseAll, ..)
  */
 abstract class AbstractLockDriver extends BaseObject implements ILockDriver
 {
@@ -18,7 +18,7 @@ abstract class AbstractLockDriver extends BaseObject implements ILockDriver
        protected $acquiredLocks = [];
 
        /**
-        * @brief Check if we've locally acquired a lock
+        * Check if we've locally acquired a lock
         *
         * @param string key The Name of the lock
         * @return bool      Returns true if the lock is set
@@ -28,7 +28,7 @@ abstract class AbstractLockDriver extends BaseObject implements ILockDriver
        }
 
        /**
-        * @brief Mark a locally acquired lock
+        * Mark a locally acquired lock
         *
         * @param string $key The Name of the lock
         */
@@ -37,7 +37,7 @@ abstract class AbstractLockDriver extends BaseObject implements ILockDriver
        }
 
        /**
-        * @brief Mark a release of a locally acquired lock
+        * Mark a release of a locally acquired lock
         *
         * @param string $key The Name of the lock
         */
@@ -46,13 +46,13 @@ abstract class AbstractLockDriver extends BaseObject implements ILockDriver
        }
 
        /**
-        * @brief Releases all lock that were set by us
+        * Releases all lock that were set by us
         *
         * @return void
         */
        public function releaseAll() {
                foreach ($this->acquiredLocks as $acquiredLock => $hasLock) {
-                       $this->releaseLock($acquiredLock);
+                       $this->release($acquiredLock);
                }
        }
 }
index 13d912c1e2979726fff44e3c511180a5739b8410..57627acecf22a4206866b1868b3d9194857c0fa4 100644 (file)
@@ -22,15 +22,9 @@ class CacheLockDriver extends AbstractLockDriver
        }
 
        /**
-        *
-        * @brief Sets a lock for a given name
-        *
-        * @param string $key The Name of the lock
-        * @param integer $timeout Seconds until we give up
-        *
-        * @return boolean Was the lock successful?
+        * (@inheritdoc)
         */
-       public function acquireLock($key, $timeout = 120)
+       public function acquire($key, $timeout = 120)
        {
                $got_lock = false;
                $start = time();
@@ -64,11 +58,9 @@ class CacheLockDriver extends AbstractLockDriver
        }
 
        /**
-        * @brief Removes a lock if it was set by us
-        *
-        * @param string $key Name of the lock
+        * (@inheritdoc)
         */
-       public function releaseLock($key)
+       public function release($key)
        {
                $cachekey = self::getCacheKey($key);
 
@@ -77,10 +69,7 @@ class CacheLockDriver extends AbstractLockDriver
        }
 
        /**
-        * @brief Checks, if a key is currently locked to a process
-        *
-        * @param string $key The name of the lock
-        * @return bool
+        * (@inheritdoc)
         */
        public function isLocked($key)
        {
index 6f4b942a4d2a72a4caa55075298f18265646bd16..8f8e174214184423a35ff4fbdf41c0c20939186e 100644 (file)
@@ -14,7 +14,7 @@ class DatabaseLockDriver extends AbstractLockDriver
        /**
         * (@inheritdoc)
         */
-       public function acquireLock($key, $timeout = 120)
+       public function acquire($key, $timeout = 120)
        {
                $got_lock = false;
                $start = time();
@@ -55,7 +55,7 @@ class DatabaseLockDriver extends AbstractLockDriver
        /**
         * (@inheritdoc)
         */
-       public function releaseLock($key)
+       public function release($key)
        {
                dba::delete('locks', ['name' => $key, 'pid' => getmypid()]);
 
index 3fbe049d3726fd37d25dcf91d2d85240650ac372..af8a1d56aef93dd45061cea5d6ceda7ff5fe20c3 100644 (file)
@@ -10,7 +10,7 @@ namespace Friendica\Core\Lock;
 interface ILockDriver
 {
        /**
-        * @brief Checks, if a key is currently locked to a or my process
+        * Checks, if a key is currently locked to a or my process
         *
         * @param string $key           The name of the lock
         * @return bool
@@ -19,26 +19,26 @@ interface ILockDriver
 
        /**
         *
-        * @brief Acquires a lock for a given name
+        * Acquires a lock for a given name
         *
         * @param string  $key      The Name of the lock
         * @param integer $timeout  Seconds until we give up
         *
         * @return boolean Was the lock successful?
         */
-       public function acquireLock($key, $timeout = 120);
+       public function acquire($key, $timeout = 120);
 
        /**
-        * @brief Releases a lock if it was set by us
+        * Releases a lock if it was set by us
         *
         * @param string $key The Name of the lock
         *
         * @return void
         */
-       public function releaseLock($key);
+       public function release($key);
 
        /**
-        * @brief Releases all lock that were set by us
+        * Releases all lock that were set by us
         *
         * @return void
         */
index b4439743c85fcee74b2a7c54a1cfac8ef7c3219c..250a75fbfe92030d513ff33b8e55b3a7c608df25 100644 (file)
@@ -14,11 +14,7 @@ class SemaphoreLockDriver extends AbstractLockDriver
        }
 
        /**
-        * @brief Creates a semaphore key
-        *
-        * @param string $key Name of the lock
-        *
-        * @return integer the semaphore key
+        * (@inheritdoc)
         */
        private static function semaphoreKey($key)
        {
@@ -35,14 +31,9 @@ class SemaphoreLockDriver extends AbstractLockDriver
 
        /**
         *
-        * @brief Sets a lock for a given name
-        *
-        * @param string $key The Name of the lock
-        * @param integer $timeout Seconds until we give up
-        *
-        * @return boolean Was the lock successful?
+        * (@inheritdoc)
         */
-       public function acquireLock($key, $timeout = 120)
+       public function acquire($key, $timeout = 120)
        {
                self::$semaphore[$key] = sem_get(self::semaphoreKey($key));
                if (self::$semaphore[$key]) {
@@ -56,13 +47,9 @@ class SemaphoreLockDriver extends AbstractLockDriver
        }
 
        /**
-        * @brief Removes a lock if it was set by us
-        *
-        * @param string $key Name of the lock
-        *
-        * @return mixed
+        * (@inheritdoc)
         */
-       public function releaseLock($key)
+       public function release($key)
        {
                if (empty(self::$semaphore[$key])) {
                        return false;
@@ -75,10 +62,7 @@ class SemaphoreLockDriver extends AbstractLockDriver
        }
 
        /**
-        * @brief Checks, if a key is currently locked to a process
-        *
-        * @param string $key The name of the lock
-        * @return bool
+        * (@inheritdoc)
         */
        public function isLocked($key)
        {
index a089059725391c89b348bd928b8b25757575c7de..b39000e119e90ad0f1fdc225b14c6c7ffee419a0 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 
-namespace Friendica\Test\src\Core\Lock;
+namespace Friendica\Test\Core\Lock;
 
 
 use Friendica\Core\Cache\ArrayCache;
index a80ff4c37cef4dbff7c6c50f5a112b26497dca89..c6ded6e3d7a667f0cff10b6f9f1e1364ec4862bb 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 
-namespace Friendica\Test\src\Core\Lock;
+namespace Friendica\Test\Core\Lock;
 
 use dba;
 use Friendica\Core\Lock\DatabaseLockDriver;
index ec7b97a9c78d102052016cae292548e62bc942d4..d6a450c86bc3c5676541d30521d800d036e4bf10 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 
-namespace Friendica\Test\src\Core\Lock;
+namespace Friendica\Test\Core\Lock;
 
 use Friendica\App;
 use Friendica\Core\Config;
@@ -34,29 +34,29 @@ abstract class LockTest extends TestCase
        }
 
        public function testLock() {
-               $this->instance->acquireLock('foo', 1);
+               $this->instance->acquire('foo', 1);
                $this->assertTrue($this->instance->isLocked('foo'));
                $this->assertFalse($this->instance->isLocked('bar'));
        }
 
        public function testDoubleLock() {
-               $this->instance->acquireLock('foo', 1);
+               $this->instance->acquire('foo', 1);
                $this->assertTrue($this->instance->isLocked('foo'));
                // We already locked it
-               $this->assertTrue($this->instance->acquireLock('foo', 1));
+               $this->assertTrue($this->instance->acquire('foo', 1));
        }
 
        public function testReleaseLock() {
-               $this->instance->acquireLock('foo', 1);
+               $this->instance->acquire('foo', 1);
                $this->assertTrue($this->instance->isLocked('foo'));
-               $this->instance->releaseLock('foo');
+               $this->instance->release('foo');
                $this->assertFalse($this->instance->isLocked('foo'));
        }
 
        public function testReleaseAll() {
-               $this->instance->acquireLock('foo', 1);
-               $this->instance->acquireLock('bar', 1);
-               $this->instance->acquireLock('#/$%§', 1);
+               $this->instance->acquire('foo', 1);
+               $this->instance->acquire('bar', 1);
+               $this->instance->acquire('#/$%§', 1);
 
                $this->instance->releaseAll();
 
@@ -66,11 +66,11 @@ abstract class LockTest extends TestCase
        }
 
        public function testReleaseAfterUnlock() {
-               $this->instance->acquireLock('foo', 1);
-               $this->instance->acquireLock('bar', 1);
-               $this->instance->acquireLock('#/$%§', 1);
+               $this->instance->acquire('foo', 1);
+               $this->instance->acquire('bar', 1);
+               $this->instance->acquire('#/$%§', 1);
 
-               $this->instance->releaseLock('foo');
+               $this->instance->release('foo');
 
                $this->instance->releaseAll();
 
index fb7efd6584ecdb9d9d330097fabbc308ba7ad3b9..0fcf789e611a6a5a2fc455f302edebaf3f4c8744 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 
-namespace Friendica\Test\src\Core\Lock;
+namespace Friendica\Test\Core\Lock;
 
 
 use Friendica\Core\Lock\SemaphoreLockDriver;