Added documentation and renamed function

This commit is contained in:
Michael 2017-06-06 17:25:28 +00:00
parent f4652d6e50
commit 286b2da7a9
2 changed files with 28 additions and 28 deletions

View File

@ -1,11 +1,11 @@
Table locks Table locks
=========== ===========
| Field | Description | Type | Null | Key | Default | Extra | | Field | Description | Type | Null | Key | Default | Extra |
|---------|------------------|--------------|------|-----|---------------------|----------------| |---------|------------------|------------------|------|-----|---------------------|----------------|
| id | sequential ID | int(11) | NO | PRI | NULL | auto_increment | | id | sequential ID | int(11) | NO | PRI | NULL | auto_increment |
| name | | varchar(128) | NO | | | | | name | | varchar(128) | NO | | | |
| locked | | tinyint(1) | NO | | 0 | | | locked | | tinyint(1) | NO | | 0 | |
| created | | datetime | YES | | 0001-01-01 00:00:00 | | | pid | | int(10) unsigned | NO | | 0 | |
Return to [database documentation](help/database) Return to [database documentation](help/database)

View File

@ -18,30 +18,30 @@ use dbm;
*/ */
class Lock { class Lock {
/** /**
* @brief Check for memcache and open a connection if configured * @brief Check for memcache and open a connection if configured
* *
* @return object|boolean The memcache object - or "false" if not successful * @return object|boolean The memcache object - or "false" if not successful
*/ */
public static function memcache() { private static function connect_memcache() {
if (!function_exists('memcache_connect')) { if (!function_exists('memcache_connect')) {
return false; return false;
} }
if (!Config::get('system', 'memcache')) { if (!Config::get('system', 'memcache')) {
return false; return false;
} }
$memcache_host = Config::get('system', 'memcache_host', '127.0.0.1'); $memcache_host = Config::get('system', 'memcache_host', '127.0.0.1');
$memcache_port = Config::get('system', 'memcache_port', 11211); $memcache_port = Config::get('system', 'memcache_port', 11211);
$memcache = new Memcache; $memcache = new Memcache;
if (!$memcache->connect($memcache_host, $memcache_port)) { if (!$memcache->connect($memcache_host, $memcache_port)) {
return false; return false;
} }
return $memcache; return $memcache;
} }
/** /**
* @brief Sets a lock for a given name * @brief Sets a lock for a given name
@ -55,7 +55,7 @@ class Lock {
$got_lock = false; $got_lock = false;
$start = time(); $start = time();
$memcache = self::memcache(); $memcache = self::connect_memcache();
if (is_object($memcache)) { if (is_object($memcache)) {
$wait_sec = 0.2; $wait_sec = 0.2;
$cachekey = get_app()->get_hostname().";lock:".$fn_name; $cachekey = get_app()->get_hostname().";lock:".$fn_name;
@ -126,7 +126,7 @@ class Lock {
* @param string $fn_name Name of the lock * @param string $fn_name Name of the lock
*/ */
public static function remove($fn_name) { public static function remove($fn_name) {
$memcache = self::memcache(); $memcache = self::connect_memcache();
if (is_object($memcache)) { if (is_object($memcache)) {
$cachekey = get_app()->get_hostname().";lock:".$fn_name; $cachekey = get_app()->get_hostname().";lock:".$fn_name;
$lock = $memcache->get($cachekey); $lock = $memcache->get($cachekey);
@ -147,7 +147,7 @@ class Lock {
* @brief Removes all lock that were set by us * @brief Removes all lock that were set by us
*/ */
public static function removeAll() { public static function removeAll() {
$memcache = self::memcache(); $memcache = self::connect_memcache();
if (is_object($memcache)) { if (is_object($memcache)) {
// We cannot delete all cache entries, but this doesn't matter with memcache // We cannot delete all cache entries, but this doesn't matter with memcache
return; return;