@@ -26,14 +26,14 @@ |
||
26 | 26 | use OCP\AutoloadNotAllowedException; |
27 | 27 | |
28 | 28 | class RegularJob extends \OC\BackgroundJob\Job { |
29 | - public function run($argument) { |
|
30 | - try { |
|
31 | - if (is_callable($argument)) { |
|
32 | - call_user_func($argument); |
|
33 | - } |
|
34 | - } catch (AutoloadNotAllowedException $e) { |
|
35 | - // job is from a disabled app, ignore |
|
36 | - return null; |
|
37 | - } |
|
38 | - } |
|
29 | + public function run($argument) { |
|
30 | + try { |
|
31 | + if (is_callable($argument)) { |
|
32 | + call_user_func($argument); |
|
33 | + } |
|
34 | + } catch (AutoloadNotAllowedException $e) { |
|
35 | + // job is from a disabled app, ignore |
|
36 | + return null; |
|
37 | + } |
|
38 | + } |
|
39 | 39 | } |
@@ -24,12 +24,12 @@ |
||
24 | 24 | namespace OC\BackgroundJob\Legacy; |
25 | 25 | |
26 | 26 | class QueuedJob extends \OC\BackgroundJob\QueuedJob { |
27 | - public function run($argument) { |
|
28 | - $class = $argument['klass']; |
|
29 | - $method = $argument['method']; |
|
30 | - $parameters = $argument['parameters']; |
|
31 | - if (is_callable(array($class, $method))) { |
|
32 | - call_user_func(array($class, $method), $parameters); |
|
33 | - } |
|
34 | - } |
|
27 | + public function run($argument) { |
|
28 | + $class = $argument['klass']; |
|
29 | + $method = $argument['method']; |
|
30 | + $parameters = $argument['parameters']; |
|
31 | + if (is_callable(array($class, $method))) { |
|
32 | + call_user_func(array($class, $method), $parameters); |
|
33 | + } |
|
34 | + } |
|
35 | 35 | } |
@@ -26,67 +26,67 @@ |
||
26 | 26 | use OCP\Settings\IIconSection; |
27 | 27 | |
28 | 28 | class Section implements IIconSection { |
29 | - /** @var string */ |
|
30 | - private $id; |
|
31 | - /** @var string */ |
|
32 | - private $name; |
|
33 | - /** @var int */ |
|
34 | - private $priority; |
|
35 | - /** @var string */ |
|
36 | - private $icon; |
|
29 | + /** @var string */ |
|
30 | + private $id; |
|
31 | + /** @var string */ |
|
32 | + private $name; |
|
33 | + /** @var int */ |
|
34 | + private $priority; |
|
35 | + /** @var string */ |
|
36 | + private $icon; |
|
37 | 37 | |
38 | - /** |
|
39 | - * @param string $id |
|
40 | - * @param string $name |
|
41 | - * @param int $priority |
|
42 | - * @param string $icon |
|
43 | - */ |
|
44 | - public function __construct($id, $name, $priority, $icon = '') { |
|
45 | - $this->id = $id; |
|
46 | - $this->name = $name; |
|
47 | - $this->priority = $priority; |
|
48 | - $this->icon = $icon; |
|
49 | - } |
|
38 | + /** |
|
39 | + * @param string $id |
|
40 | + * @param string $name |
|
41 | + * @param int $priority |
|
42 | + * @param string $icon |
|
43 | + */ |
|
44 | + public function __construct($id, $name, $priority, $icon = '') { |
|
45 | + $this->id = $id; |
|
46 | + $this->name = $name; |
|
47 | + $this->priority = $priority; |
|
48 | + $this->icon = $icon; |
|
49 | + } |
|
50 | 50 | |
51 | - /** |
|
52 | - * returns the ID of the section. It is supposed to be a lower case string, |
|
53 | - * e.g. 'ldap' |
|
54 | - * |
|
55 | - * @returns string |
|
56 | - */ |
|
57 | - public function getID() { |
|
58 | - return $this->id; |
|
59 | - } |
|
51 | + /** |
|
52 | + * returns the ID of the section. It is supposed to be a lower case string, |
|
53 | + * e.g. 'ldap' |
|
54 | + * |
|
55 | + * @returns string |
|
56 | + */ |
|
57 | + public function getID() { |
|
58 | + return $this->id; |
|
59 | + } |
|
60 | 60 | |
61 | - /** |
|
62 | - * returns the translated name as it should be displayed, e.g. 'LDAP / AD |
|
63 | - * integration'. Use the L10N service to translate it. |
|
64 | - * |
|
65 | - * @return string |
|
66 | - */ |
|
67 | - public function getName() { |
|
68 | - return $this->name; |
|
69 | - } |
|
61 | + /** |
|
62 | + * returns the translated name as it should be displayed, e.g. 'LDAP / AD |
|
63 | + * integration'. Use the L10N service to translate it. |
|
64 | + * |
|
65 | + * @return string |
|
66 | + */ |
|
67 | + public function getName() { |
|
68 | + return $this->name; |
|
69 | + } |
|
70 | 70 | |
71 | - /** |
|
72 | - * @return int whether the form should be rather on the top or bottom of |
|
73 | - * the settings navigation. The sections are arranged in ascending order of |
|
74 | - * the priority values. It is required to return a value between 0 and 99. |
|
75 | - * |
|
76 | - * E.g.: 70 |
|
77 | - */ |
|
78 | - public function getPriority() { |
|
79 | - return $this->priority; |
|
80 | - } |
|
71 | + /** |
|
72 | + * @return int whether the form should be rather on the top or bottom of |
|
73 | + * the settings navigation. The sections are arranged in ascending order of |
|
74 | + * the priority values. It is required to return a value between 0 and 99. |
|
75 | + * |
|
76 | + * E.g.: 70 |
|
77 | + */ |
|
78 | + public function getPriority() { |
|
79 | + return $this->priority; |
|
80 | + } |
|
81 | 81 | |
82 | - /** |
|
83 | - * returns the relative path to an 16*16 icon describing the section. |
|
84 | - * e.g. '/core/img/places/files.svg' |
|
85 | - * |
|
86 | - * @returns string |
|
87 | - * @since 12 |
|
88 | - */ |
|
89 | - public function getIcon() { |
|
90 | - return $this->icon; |
|
91 | - } |
|
82 | + /** |
|
83 | + * returns the relative path to an 16*16 icon describing the section. |
|
84 | + * e.g. '/core/img/places/files.svg' |
|
85 | + * |
|
86 | + * @returns string |
|
87 | + * @since 12 |
|
88 | + */ |
|
89 | + public function getIcon() { |
|
90 | + return $this->icon; |
|
91 | + } |
|
92 | 92 | } |
@@ -80,7 +80,7 @@ |
||
80 | 80 | $fh = \OC\Files\Filesystem::fopen($file, 'r'); |
81 | 81 | $this->addFileFromStream($fh, $internalDir . $filename, $filesize, $fileTime); |
82 | 82 | fclose($fh); |
83 | - }elseif(\OC\Files\Filesystem::is_dir($file)) { |
|
83 | + } elseif(\OC\Files\Filesystem::is_dir($file)) { |
|
84 | 84 | $this->addDirRecursive($file, $internalDir); |
85 | 85 | } |
86 | 86 | } |
@@ -29,135 +29,135 @@ |
||
29 | 29 | use ZipStreamer\ZipStreamer; |
30 | 30 | |
31 | 31 | class Streamer { |
32 | - // array of regexp. Matching user agents will get tar instead of zip |
|
33 | - private $preferTarFor = [ '/macintosh|mac os x/i' ]; |
|
32 | + // array of regexp. Matching user agents will get tar instead of zip |
|
33 | + private $preferTarFor = [ '/macintosh|mac os x/i' ]; |
|
34 | 34 | |
35 | - // streamer instance |
|
36 | - private $streamerInstance; |
|
35 | + // streamer instance |
|
36 | + private $streamerInstance; |
|
37 | 37 | |
38 | - /** |
|
39 | - * Streamer constructor. |
|
40 | - * |
|
41 | - * @param IRequest $request |
|
42 | - * @param int $size The size of the files in bytes |
|
43 | - * @param int $numberOfFiles The number of files (and directories) that will |
|
44 | - * be included in the streamed file |
|
45 | - */ |
|
46 | - public function __construct(IRequest $request, int $size, int $numberOfFiles){ |
|
38 | + /** |
|
39 | + * Streamer constructor. |
|
40 | + * |
|
41 | + * @param IRequest $request |
|
42 | + * @param int $size The size of the files in bytes |
|
43 | + * @param int $numberOfFiles The number of files (and directories) that will |
|
44 | + * be included in the streamed file |
|
45 | + */ |
|
46 | + public function __construct(IRequest $request, int $size, int $numberOfFiles){ |
|
47 | 47 | |
48 | - /** |
|
49 | - * zip32 constraints for a basic (without compression, volumes nor |
|
50 | - * encryption) zip file according to the Zip specification: |
|
51 | - * - No file size is larger than 4 bytes (file size < 4294967296); see |
|
52 | - * 4.4.9 uncompressed size |
|
53 | - * - The size of all files plus their local headers is not larger than |
|
54 | - * 4 bytes; see 4.4.16 relative offset of local header and 4.4.24 |
|
55 | - * offset of start of central directory with respect to the starting |
|
56 | - * disk number |
|
57 | - * - The total number of entries (files and directories) in the zip file |
|
58 | - * is not larger than 2 bytes (number of entries < 65536); see 4.4.22 |
|
59 | - * total number of entries in the central dir |
|
60 | - * - The size of the central directory is not larger than 4 bytes; see |
|
61 | - * 4.4.23 size of the central directory |
|
62 | - * |
|
63 | - * Due to all that, zip32 is used if the size is below 4GB and there are |
|
64 | - * less than 65536 files; the margin between 4*1000^3 and 4*1024^3 |
|
65 | - * should give enough room for the extra zip metadata. Technically, it |
|
66 | - * would still be possible to create an invalid zip32 file (for example, |
|
67 | - * a zip file from files smaller than 4GB with a central directory |
|
68 | - * larger than 4GiB), but it should not happen in the real world. |
|
69 | - */ |
|
70 | - if ($size < 4 * 1000 * 1000 * 1000 && $numberOfFiles < 65536) { |
|
71 | - $this->streamerInstance = new ZipStreamer(['zip64' => false]); |
|
72 | - } else if ($request->isUserAgent($this->preferTarFor)) { |
|
73 | - $this->streamerInstance = new TarStreamer(); |
|
74 | - } else { |
|
75 | - $this->streamerInstance = new ZipStreamer(['zip64' => PHP_INT_SIZE !== 4]); |
|
76 | - } |
|
77 | - } |
|
48 | + /** |
|
49 | + * zip32 constraints for a basic (without compression, volumes nor |
|
50 | + * encryption) zip file according to the Zip specification: |
|
51 | + * - No file size is larger than 4 bytes (file size < 4294967296); see |
|
52 | + * 4.4.9 uncompressed size |
|
53 | + * - The size of all files plus their local headers is not larger than |
|
54 | + * 4 bytes; see 4.4.16 relative offset of local header and 4.4.24 |
|
55 | + * offset of start of central directory with respect to the starting |
|
56 | + * disk number |
|
57 | + * - The total number of entries (files and directories) in the zip file |
|
58 | + * is not larger than 2 bytes (number of entries < 65536); see 4.4.22 |
|
59 | + * total number of entries in the central dir |
|
60 | + * - The size of the central directory is not larger than 4 bytes; see |
|
61 | + * 4.4.23 size of the central directory |
|
62 | + * |
|
63 | + * Due to all that, zip32 is used if the size is below 4GB and there are |
|
64 | + * less than 65536 files; the margin between 4*1000^3 and 4*1024^3 |
|
65 | + * should give enough room for the extra zip metadata. Technically, it |
|
66 | + * would still be possible to create an invalid zip32 file (for example, |
|
67 | + * a zip file from files smaller than 4GB with a central directory |
|
68 | + * larger than 4GiB), but it should not happen in the real world. |
|
69 | + */ |
|
70 | + if ($size < 4 * 1000 * 1000 * 1000 && $numberOfFiles < 65536) { |
|
71 | + $this->streamerInstance = new ZipStreamer(['zip64' => false]); |
|
72 | + } else if ($request->isUserAgent($this->preferTarFor)) { |
|
73 | + $this->streamerInstance = new TarStreamer(); |
|
74 | + } else { |
|
75 | + $this->streamerInstance = new ZipStreamer(['zip64' => PHP_INT_SIZE !== 4]); |
|
76 | + } |
|
77 | + } |
|
78 | 78 | |
79 | - /** |
|
80 | - * Send HTTP headers |
|
81 | - * @param string $name |
|
82 | - */ |
|
83 | - public function sendHeaders($name){ |
|
84 | - $extension = $this->streamerInstance instanceof ZipStreamer ? '.zip' : '.tar'; |
|
85 | - $fullName = $name . $extension; |
|
86 | - $this->streamerInstance->sendHeaders($fullName); |
|
87 | - } |
|
79 | + /** |
|
80 | + * Send HTTP headers |
|
81 | + * @param string $name |
|
82 | + */ |
|
83 | + public function sendHeaders($name){ |
|
84 | + $extension = $this->streamerInstance instanceof ZipStreamer ? '.zip' : '.tar'; |
|
85 | + $fullName = $name . $extension; |
|
86 | + $this->streamerInstance->sendHeaders($fullName); |
|
87 | + } |
|
88 | 88 | |
89 | - /** |
|
90 | - * Stream directory recursively |
|
91 | - * @param string $dir |
|
92 | - * @param string $internalDir |
|
93 | - */ |
|
94 | - public function addDirRecursive($dir, $internalDir='') { |
|
95 | - $dirname = basename($dir); |
|
96 | - $rootDir = $internalDir . $dirname; |
|
97 | - if (!empty($rootDir)) { |
|
98 | - $this->streamerInstance->addEmptyDir($rootDir); |
|
99 | - } |
|
100 | - $internalDir .= $dirname . '/'; |
|
101 | - // prevent absolute dirs |
|
102 | - $internalDir = ltrim($internalDir, '/'); |
|
89 | + /** |
|
90 | + * Stream directory recursively |
|
91 | + * @param string $dir |
|
92 | + * @param string $internalDir |
|
93 | + */ |
|
94 | + public function addDirRecursive($dir, $internalDir='') { |
|
95 | + $dirname = basename($dir); |
|
96 | + $rootDir = $internalDir . $dirname; |
|
97 | + if (!empty($rootDir)) { |
|
98 | + $this->streamerInstance->addEmptyDir($rootDir); |
|
99 | + } |
|
100 | + $internalDir .= $dirname . '/'; |
|
101 | + // prevent absolute dirs |
|
102 | + $internalDir = ltrim($internalDir, '/'); |
|
103 | 103 | |
104 | - $files= \OC\Files\Filesystem::getDirectoryContent($dir); |
|
105 | - foreach($files as $file) { |
|
106 | - $filename = $file['name']; |
|
107 | - $file = $dir . '/' . $filename; |
|
108 | - if(\OC\Files\Filesystem::is_file($file)) { |
|
109 | - $filesize = \OC\Files\Filesystem::filesize($file); |
|
110 | - $fileTime = \OC\Files\Filesystem::filemtime($file); |
|
111 | - $fh = \OC\Files\Filesystem::fopen($file, 'r'); |
|
112 | - $this->addFileFromStream($fh, $internalDir . $filename, $filesize, $fileTime); |
|
113 | - fclose($fh); |
|
114 | - }elseif(\OC\Files\Filesystem::is_dir($file)) { |
|
115 | - $this->addDirRecursive($file, $internalDir); |
|
116 | - } |
|
117 | - } |
|
118 | - } |
|
104 | + $files= \OC\Files\Filesystem::getDirectoryContent($dir); |
|
105 | + foreach($files as $file) { |
|
106 | + $filename = $file['name']; |
|
107 | + $file = $dir . '/' . $filename; |
|
108 | + if(\OC\Files\Filesystem::is_file($file)) { |
|
109 | + $filesize = \OC\Files\Filesystem::filesize($file); |
|
110 | + $fileTime = \OC\Files\Filesystem::filemtime($file); |
|
111 | + $fh = \OC\Files\Filesystem::fopen($file, 'r'); |
|
112 | + $this->addFileFromStream($fh, $internalDir . $filename, $filesize, $fileTime); |
|
113 | + fclose($fh); |
|
114 | + }elseif(\OC\Files\Filesystem::is_dir($file)) { |
|
115 | + $this->addDirRecursive($file, $internalDir); |
|
116 | + } |
|
117 | + } |
|
118 | + } |
|
119 | 119 | |
120 | - /** |
|
121 | - * Add a file to the archive at the specified location and file name. |
|
122 | - * |
|
123 | - * @param string $stream Stream to read data from |
|
124 | - * @param string $internalName Filepath and name to be used in the archive. |
|
125 | - * @param int $size Filesize |
|
126 | - * @param int|bool $time File mtime as int, or false |
|
127 | - * @return bool $success |
|
128 | - */ |
|
129 | - public function addFileFromStream($stream, $internalName, $size, $time) { |
|
130 | - $options = []; |
|
131 | - if ($time) { |
|
132 | - $options = [ |
|
133 | - 'timestamp' => $time |
|
134 | - ]; |
|
135 | - } |
|
120 | + /** |
|
121 | + * Add a file to the archive at the specified location and file name. |
|
122 | + * |
|
123 | + * @param string $stream Stream to read data from |
|
124 | + * @param string $internalName Filepath and name to be used in the archive. |
|
125 | + * @param int $size Filesize |
|
126 | + * @param int|bool $time File mtime as int, or false |
|
127 | + * @return bool $success |
|
128 | + */ |
|
129 | + public function addFileFromStream($stream, $internalName, $size, $time) { |
|
130 | + $options = []; |
|
131 | + if ($time) { |
|
132 | + $options = [ |
|
133 | + 'timestamp' => $time |
|
134 | + ]; |
|
135 | + } |
|
136 | 136 | |
137 | - if ($this->streamerInstance instanceof ZipStreamer) { |
|
138 | - return $this->streamerInstance->addFileFromStream($stream, $internalName, $options); |
|
139 | - } else { |
|
140 | - return $this->streamerInstance->addFileFromStream($stream, $internalName, $size, $options); |
|
141 | - } |
|
142 | - } |
|
137 | + if ($this->streamerInstance instanceof ZipStreamer) { |
|
138 | + return $this->streamerInstance->addFileFromStream($stream, $internalName, $options); |
|
139 | + } else { |
|
140 | + return $this->streamerInstance->addFileFromStream($stream, $internalName, $size, $options); |
|
141 | + } |
|
142 | + } |
|
143 | 143 | |
144 | - /** |
|
145 | - * Add an empty directory entry to the archive. |
|
146 | - * |
|
147 | - * @param string $dirName Directory Path and name to be added to the archive. |
|
148 | - * @return bool $success |
|
149 | - */ |
|
150 | - public function addEmptyDir($dirName){ |
|
151 | - return $this->streamerInstance->addEmptyDir($dirName); |
|
152 | - } |
|
144 | + /** |
|
145 | + * Add an empty directory entry to the archive. |
|
146 | + * |
|
147 | + * @param string $dirName Directory Path and name to be added to the archive. |
|
148 | + * @return bool $success |
|
149 | + */ |
|
150 | + public function addEmptyDir($dirName){ |
|
151 | + return $this->streamerInstance->addEmptyDir($dirName); |
|
152 | + } |
|
153 | 153 | |
154 | - /** |
|
155 | - * Close the archive. |
|
156 | - * A closed archive can no longer have new files added to it. After |
|
157 | - * closing, the file is completely written to the output stream. |
|
158 | - * @return bool $success |
|
159 | - */ |
|
160 | - public function finalize(){ |
|
161 | - return $this->streamerInstance->finalize(); |
|
162 | - } |
|
154 | + /** |
|
155 | + * Close the archive. |
|
156 | + * A closed archive can no longer have new files added to it. After |
|
157 | + * closing, the file is completely written to the output stream. |
|
158 | + * @return bool $success |
|
159 | + */ |
|
160 | + public function finalize(){ |
|
161 | + return $this->streamerInstance->finalize(); |
|
162 | + } |
|
163 | 163 | } |
@@ -30,7 +30,7 @@ discard block |
||
30 | 30 | |
31 | 31 | class Streamer { |
32 | 32 | // array of regexp. Matching user agents will get tar instead of zip |
33 | - private $preferTarFor = [ '/macintosh|mac os x/i' ]; |
|
33 | + private $preferTarFor = ['/macintosh|mac os x/i']; |
|
34 | 34 | |
35 | 35 | // streamer instance |
36 | 36 | private $streamerInstance; |
@@ -43,7 +43,7 @@ discard block |
||
43 | 43 | * @param int $numberOfFiles The number of files (and directories) that will |
44 | 44 | * be included in the streamed file |
45 | 45 | */ |
46 | - public function __construct(IRequest $request, int $size, int $numberOfFiles){ |
|
46 | + public function __construct(IRequest $request, int $size, int $numberOfFiles) { |
|
47 | 47 | |
48 | 48 | /** |
49 | 49 | * zip32 constraints for a basic (without compression, volumes nor |
@@ -80,9 +80,9 @@ discard block |
||
80 | 80 | * Send HTTP headers |
81 | 81 | * @param string $name |
82 | 82 | */ |
83 | - public function sendHeaders($name){ |
|
83 | + public function sendHeaders($name) { |
|
84 | 84 | $extension = $this->streamerInstance instanceof ZipStreamer ? '.zip' : '.tar'; |
85 | - $fullName = $name . $extension; |
|
85 | + $fullName = $name.$extension; |
|
86 | 86 | $this->streamerInstance->sendHeaders($fullName); |
87 | 87 | } |
88 | 88 | |
@@ -91,27 +91,27 @@ discard block |
||
91 | 91 | * @param string $dir |
92 | 92 | * @param string $internalDir |
93 | 93 | */ |
94 | - public function addDirRecursive($dir, $internalDir='') { |
|
94 | + public function addDirRecursive($dir, $internalDir = '') { |
|
95 | 95 | $dirname = basename($dir); |
96 | - $rootDir = $internalDir . $dirname; |
|
96 | + $rootDir = $internalDir.$dirname; |
|
97 | 97 | if (!empty($rootDir)) { |
98 | 98 | $this->streamerInstance->addEmptyDir($rootDir); |
99 | 99 | } |
100 | - $internalDir .= $dirname . '/'; |
|
100 | + $internalDir .= $dirname.'/'; |
|
101 | 101 | // prevent absolute dirs |
102 | 102 | $internalDir = ltrim($internalDir, '/'); |
103 | 103 | |
104 | - $files= \OC\Files\Filesystem::getDirectoryContent($dir); |
|
105 | - foreach($files as $file) { |
|
104 | + $files = \OC\Files\Filesystem::getDirectoryContent($dir); |
|
105 | + foreach ($files as $file) { |
|
106 | 106 | $filename = $file['name']; |
107 | - $file = $dir . '/' . $filename; |
|
108 | - if(\OC\Files\Filesystem::is_file($file)) { |
|
107 | + $file = $dir.'/'.$filename; |
|
108 | + if (\OC\Files\Filesystem::is_file($file)) { |
|
109 | 109 | $filesize = \OC\Files\Filesystem::filesize($file); |
110 | 110 | $fileTime = \OC\Files\Filesystem::filemtime($file); |
111 | 111 | $fh = \OC\Files\Filesystem::fopen($file, 'r'); |
112 | - $this->addFileFromStream($fh, $internalDir . $filename, $filesize, $fileTime); |
|
112 | + $this->addFileFromStream($fh, $internalDir.$filename, $filesize, $fileTime); |
|
113 | 113 | fclose($fh); |
114 | - }elseif(\OC\Files\Filesystem::is_dir($file)) { |
|
114 | + }elseif (\OC\Files\Filesystem::is_dir($file)) { |
|
115 | 115 | $this->addDirRecursive($file, $internalDir); |
116 | 116 | } |
117 | 117 | } |
@@ -147,7 +147,7 @@ discard block |
||
147 | 147 | * @param string $dirName Directory Path and name to be added to the archive. |
148 | 148 | * @return bool $success |
149 | 149 | */ |
150 | - public function addEmptyDir($dirName){ |
|
150 | + public function addEmptyDir($dirName) { |
|
151 | 151 | return $this->streamerInstance->addEmptyDir($dirName); |
152 | 152 | } |
153 | 153 | |
@@ -157,7 +157,7 @@ discard block |
||
157 | 157 | * closing, the file is completely written to the output stream. |
158 | 158 | * @return bool $success |
159 | 159 | */ |
160 | - public function finalize(){ |
|
160 | + public function finalize() { |
|
161 | 161 | return $this->streamerInstance->finalize(); |
162 | 162 | } |
163 | 163 | } |
@@ -23,35 +23,35 @@ |
||
23 | 23 | namespace OC\Memcache; |
24 | 24 | |
25 | 25 | trait CASTrait { |
26 | - abstract public function get($key); |
|
26 | + abstract public function get($key); |
|
27 | 27 | |
28 | - abstract public function set($key, $value, $ttl = 0); |
|
28 | + abstract public function set($key, $value, $ttl = 0); |
|
29 | 29 | |
30 | - abstract public function remove($key); |
|
30 | + abstract public function remove($key); |
|
31 | 31 | |
32 | - abstract public function add($key, $value, $ttl = 0); |
|
32 | + abstract public function add($key, $value, $ttl = 0); |
|
33 | 33 | |
34 | - /** |
|
35 | - * Compare and set |
|
36 | - * |
|
37 | - * @param string $key |
|
38 | - * @param mixed $old |
|
39 | - * @param mixed $new |
|
40 | - * @return bool |
|
41 | - */ |
|
42 | - public function cas($key, $old, $new) { |
|
43 | - //no native cas, emulate with locking |
|
44 | - if ($this->add($key . '_lock', true)) { |
|
45 | - if ($this->get($key) === $old) { |
|
46 | - $this->set($key, $new); |
|
47 | - $this->remove($key . '_lock'); |
|
48 | - return true; |
|
49 | - } else { |
|
50 | - $this->remove($key . '_lock'); |
|
51 | - return false; |
|
52 | - } |
|
53 | - } else { |
|
54 | - return false; |
|
55 | - } |
|
56 | - } |
|
34 | + /** |
|
35 | + * Compare and set |
|
36 | + * |
|
37 | + * @param string $key |
|
38 | + * @param mixed $old |
|
39 | + * @param mixed $new |
|
40 | + * @return bool |
|
41 | + */ |
|
42 | + public function cas($key, $old, $new) { |
|
43 | + //no native cas, emulate with locking |
|
44 | + if ($this->add($key . '_lock', true)) { |
|
45 | + if ($this->get($key) === $old) { |
|
46 | + $this->set($key, $new); |
|
47 | + $this->remove($key . '_lock'); |
|
48 | + return true; |
|
49 | + } else { |
|
50 | + $this->remove($key . '_lock'); |
|
51 | + return false; |
|
52 | + } |
|
53 | + } else { |
|
54 | + return false; |
|
55 | + } |
|
56 | + } |
|
57 | 57 | } |
@@ -41,13 +41,13 @@ |
||
41 | 41 | */ |
42 | 42 | public function cas($key, $old, $new) { |
43 | 43 | //no native cas, emulate with locking |
44 | - if ($this->add($key . '_lock', true)) { |
|
44 | + if ($this->add($key.'_lock', true)) { |
|
45 | 45 | if ($this->get($key) === $old) { |
46 | 46 | $this->set($key, $new); |
47 | - $this->remove($key . '_lock'); |
|
47 | + $this->remove($key.'_lock'); |
|
48 | 48 | return true; |
49 | 49 | } else { |
50 | - $this->remove($key . '_lock'); |
|
50 | + $this->remove($key.'_lock'); |
|
51 | 51 | return false; |
52 | 52 | } |
53 | 53 | } else { |
@@ -26,72 +26,72 @@ |
||
26 | 26 | namespace OC\Memcache; |
27 | 27 | |
28 | 28 | abstract class Cache implements \ArrayAccess, \OCP\ICache { |
29 | - /** |
|
30 | - * @var string $prefix |
|
31 | - */ |
|
32 | - protected $prefix; |
|
29 | + /** |
|
30 | + * @var string $prefix |
|
31 | + */ |
|
32 | + protected $prefix; |
|
33 | 33 | |
34 | - /** |
|
35 | - * @param string $prefix |
|
36 | - */ |
|
37 | - public function __construct($prefix = '') { |
|
38 | - $this->prefix = $prefix; |
|
39 | - } |
|
34 | + /** |
|
35 | + * @param string $prefix |
|
36 | + */ |
|
37 | + public function __construct($prefix = '') { |
|
38 | + $this->prefix = $prefix; |
|
39 | + } |
|
40 | 40 | |
41 | - /** |
|
42 | - * @return string Prefix used for caching purposes |
|
43 | - */ |
|
44 | - public function getPrefix() { |
|
45 | - return $this->prefix; |
|
46 | - } |
|
41 | + /** |
|
42 | + * @return string Prefix used for caching purposes |
|
43 | + */ |
|
44 | + public function getPrefix() { |
|
45 | + return $this->prefix; |
|
46 | + } |
|
47 | 47 | |
48 | - /** |
|
49 | - * @param string $key |
|
50 | - * @return mixed |
|
51 | - */ |
|
52 | - abstract public function get($key); |
|
48 | + /** |
|
49 | + * @param string $key |
|
50 | + * @return mixed |
|
51 | + */ |
|
52 | + abstract public function get($key); |
|
53 | 53 | |
54 | - /** |
|
55 | - * @param string $key |
|
56 | - * @param mixed $value |
|
57 | - * @param int $ttl |
|
58 | - * @return mixed |
|
59 | - */ |
|
60 | - abstract public function set($key, $value, $ttl = 0); |
|
54 | + /** |
|
55 | + * @param string $key |
|
56 | + * @param mixed $value |
|
57 | + * @param int $ttl |
|
58 | + * @return mixed |
|
59 | + */ |
|
60 | + abstract public function set($key, $value, $ttl = 0); |
|
61 | 61 | |
62 | - /** |
|
63 | - * @param string $key |
|
64 | - * @return mixed |
|
65 | - */ |
|
66 | - abstract public function hasKey($key); |
|
62 | + /** |
|
63 | + * @param string $key |
|
64 | + * @return mixed |
|
65 | + */ |
|
66 | + abstract public function hasKey($key); |
|
67 | 67 | |
68 | - /** |
|
69 | - * @param string $key |
|
70 | - * @return mixed |
|
71 | - */ |
|
72 | - abstract public function remove($key); |
|
68 | + /** |
|
69 | + * @param string $key |
|
70 | + * @return mixed |
|
71 | + */ |
|
72 | + abstract public function remove($key); |
|
73 | 73 | |
74 | - /** |
|
75 | - * @param string $prefix |
|
76 | - * @return mixed |
|
77 | - */ |
|
78 | - abstract public function clear($prefix = ''); |
|
74 | + /** |
|
75 | + * @param string $prefix |
|
76 | + * @return mixed |
|
77 | + */ |
|
78 | + abstract public function clear($prefix = ''); |
|
79 | 79 | |
80 | - //implement the ArrayAccess interface |
|
80 | + //implement the ArrayAccess interface |
|
81 | 81 | |
82 | - public function offsetExists($offset) { |
|
83 | - return $this->hasKey($offset); |
|
84 | - } |
|
82 | + public function offsetExists($offset) { |
|
83 | + return $this->hasKey($offset); |
|
84 | + } |
|
85 | 85 | |
86 | - public function offsetSet($offset, $value) { |
|
87 | - $this->set($offset, $value); |
|
88 | - } |
|
86 | + public function offsetSet($offset, $value) { |
|
87 | + $this->set($offset, $value); |
|
88 | + } |
|
89 | 89 | |
90 | - public function offsetGet($offset) { |
|
91 | - return $this->get($offset); |
|
92 | - } |
|
90 | + public function offsetGet($offset) { |
|
91 | + return $this->get($offset); |
|
92 | + } |
|
93 | 93 | |
94 | - public function offsetUnset($offset) { |
|
95 | - $this->remove($offset); |
|
96 | - } |
|
94 | + public function offsetUnset($offset) { |
|
95 | + $this->remove($offset); |
|
96 | + } |
|
97 | 97 | } |
@@ -37,7 +37,7 @@ discard block |
||
37 | 37 | use CADTrait; |
38 | 38 | |
39 | 39 | public function get($key) { |
40 | - $result = apcu_fetch($this->getPrefix() . $key, $success); |
|
40 | + $result = apcu_fetch($this->getPrefix().$key, $success); |
|
41 | 41 | if (!$success) { |
42 | 42 | return null; |
43 | 43 | } |
@@ -45,24 +45,24 @@ discard block |
||
45 | 45 | } |
46 | 46 | |
47 | 47 | public function set($key, $value, $ttl = 0) { |
48 | - return apcu_store($this->getPrefix() . $key, $value, $ttl); |
|
48 | + return apcu_store($this->getPrefix().$key, $value, $ttl); |
|
49 | 49 | } |
50 | 50 | |
51 | 51 | public function hasKey($key) { |
52 | - return apcu_exists($this->getPrefix() . $key); |
|
52 | + return apcu_exists($this->getPrefix().$key); |
|
53 | 53 | } |
54 | 54 | |
55 | 55 | public function remove($key) { |
56 | - return apcu_delete($this->getPrefix() . $key); |
|
56 | + return apcu_delete($this->getPrefix().$key); |
|
57 | 57 | } |
58 | 58 | |
59 | 59 | public function clear($prefix = '') { |
60 | - $ns = $this->getPrefix() . $prefix; |
|
60 | + $ns = $this->getPrefix().$prefix; |
|
61 | 61 | $ns = preg_quote($ns, '/'); |
62 | - if(class_exists('\APCIterator')) { |
|
63 | - $iter = new \APCIterator('user', '/^' . $ns . '/', APC_ITER_KEY); |
|
62 | + if (class_exists('\APCIterator')) { |
|
63 | + $iter = new \APCIterator('user', '/^'.$ns.'/', APC_ITER_KEY); |
|
64 | 64 | } else { |
65 | - $iter = new \APCUIterator('/^' . $ns . '/', APC_ITER_KEY); |
|
65 | + $iter = new \APCUIterator('/^'.$ns.'/', APC_ITER_KEY); |
|
66 | 66 | } |
67 | 67 | return apcu_delete($iter); |
68 | 68 | } |
@@ -76,7 +76,7 @@ discard block |
||
76 | 76 | * @return bool |
77 | 77 | */ |
78 | 78 | public function add($key, $value, $ttl = 0) { |
79 | - return apcu_add($this->getPrefix() . $key, $value, $ttl); |
|
79 | + return apcu_add($this->getPrefix().$key, $value, $ttl); |
|
80 | 80 | } |
81 | 81 | |
82 | 82 | /** |
@@ -100,8 +100,8 @@ discard block |
||
100 | 100 | * see https://github.com/krakjoe/apcu/issues/183#issuecomment-244038221 |
101 | 101 | * for details |
102 | 102 | */ |
103 | - return apcu_exists($this->getPrefix() . $key) |
|
104 | - ? apcu_inc($this->getPrefix() . $key, $step) |
|
103 | + return apcu_exists($this->getPrefix().$key) |
|
104 | + ? apcu_inc($this->getPrefix().$key, $step) |
|
105 | 105 | : false; |
106 | 106 | } |
107 | 107 | |
@@ -125,8 +125,8 @@ discard block |
||
125 | 125 | * see https://github.com/krakjoe/apcu/issues/183#issuecomment-244038221 |
126 | 126 | * for details |
127 | 127 | */ |
128 | - return apcu_exists($this->getPrefix() . $key) |
|
129 | - ? apcu_dec($this->getPrefix() . $key, $step) |
|
128 | + return apcu_exists($this->getPrefix().$key) |
|
129 | + ? apcu_dec($this->getPrefix().$key, $step) |
|
130 | 130 | : false; |
131 | 131 | } |
132 | 132 | |
@@ -141,7 +141,7 @@ discard block |
||
141 | 141 | public function cas($key, $old, $new) { |
142 | 142 | // apc only does cas for ints |
143 | 143 | if (is_int($old) and is_int($new)) { |
144 | - return apcu_cas($this->getPrefix() . $key, $old, $new); |
|
144 | + return apcu_cas($this->getPrefix().$key, $old, $new); |
|
145 | 145 | } else { |
146 | 146 | return $this->casEmulated($key, $old, $new); |
147 | 147 | } |
@@ -30,140 +30,140 @@ |
||
30 | 30 | use OCP\IMemcache; |
31 | 31 | |
32 | 32 | class APCu extends Cache implements IMemcache { |
33 | - use CASTrait { |
|
34 | - cas as casEmulated; |
|
35 | - } |
|
33 | + use CASTrait { |
|
34 | + cas as casEmulated; |
|
35 | + } |
|
36 | 36 | |
37 | - use CADTrait; |
|
37 | + use CADTrait; |
|
38 | 38 | |
39 | - public function get($key) { |
|
40 | - $result = apcu_fetch($this->getPrefix() . $key, $success); |
|
41 | - if (!$success) { |
|
42 | - return null; |
|
43 | - } |
|
44 | - return $result; |
|
45 | - } |
|
39 | + public function get($key) { |
|
40 | + $result = apcu_fetch($this->getPrefix() . $key, $success); |
|
41 | + if (!$success) { |
|
42 | + return null; |
|
43 | + } |
|
44 | + return $result; |
|
45 | + } |
|
46 | 46 | |
47 | - public function set($key, $value, $ttl = 0) { |
|
48 | - return apcu_store($this->getPrefix() . $key, $value, $ttl); |
|
49 | - } |
|
47 | + public function set($key, $value, $ttl = 0) { |
|
48 | + return apcu_store($this->getPrefix() . $key, $value, $ttl); |
|
49 | + } |
|
50 | 50 | |
51 | - public function hasKey($key) { |
|
52 | - return apcu_exists($this->getPrefix() . $key); |
|
53 | - } |
|
51 | + public function hasKey($key) { |
|
52 | + return apcu_exists($this->getPrefix() . $key); |
|
53 | + } |
|
54 | 54 | |
55 | - public function remove($key) { |
|
56 | - return apcu_delete($this->getPrefix() . $key); |
|
57 | - } |
|
55 | + public function remove($key) { |
|
56 | + return apcu_delete($this->getPrefix() . $key); |
|
57 | + } |
|
58 | 58 | |
59 | - public function clear($prefix = '') { |
|
60 | - $ns = $this->getPrefix() . $prefix; |
|
61 | - $ns = preg_quote($ns, '/'); |
|
62 | - if(class_exists('\APCIterator')) { |
|
63 | - $iter = new \APCIterator('user', '/^' . $ns . '/', APC_ITER_KEY); |
|
64 | - } else { |
|
65 | - $iter = new \APCUIterator('/^' . $ns . '/', APC_ITER_KEY); |
|
66 | - } |
|
67 | - return apcu_delete($iter); |
|
68 | - } |
|
59 | + public function clear($prefix = '') { |
|
60 | + $ns = $this->getPrefix() . $prefix; |
|
61 | + $ns = preg_quote($ns, '/'); |
|
62 | + if(class_exists('\APCIterator')) { |
|
63 | + $iter = new \APCIterator('user', '/^' . $ns . '/', APC_ITER_KEY); |
|
64 | + } else { |
|
65 | + $iter = new \APCUIterator('/^' . $ns . '/', APC_ITER_KEY); |
|
66 | + } |
|
67 | + return apcu_delete($iter); |
|
68 | + } |
|
69 | 69 | |
70 | - /** |
|
71 | - * Set a value in the cache if it's not already stored |
|
72 | - * |
|
73 | - * @param string $key |
|
74 | - * @param mixed $value |
|
75 | - * @param int $ttl Time To Live in seconds. Defaults to 60*60*24 |
|
76 | - * @return bool |
|
77 | - */ |
|
78 | - public function add($key, $value, $ttl = 0) { |
|
79 | - return apcu_add($this->getPrefix() . $key, $value, $ttl); |
|
80 | - } |
|
70 | + /** |
|
71 | + * Set a value in the cache if it's not already stored |
|
72 | + * |
|
73 | + * @param string $key |
|
74 | + * @param mixed $value |
|
75 | + * @param int $ttl Time To Live in seconds. Defaults to 60*60*24 |
|
76 | + * @return bool |
|
77 | + */ |
|
78 | + public function add($key, $value, $ttl = 0) { |
|
79 | + return apcu_add($this->getPrefix() . $key, $value, $ttl); |
|
80 | + } |
|
81 | 81 | |
82 | - /** |
|
83 | - * Increase a stored number |
|
84 | - * |
|
85 | - * @param string $key |
|
86 | - * @param int $step |
|
87 | - * @return int | bool |
|
88 | - */ |
|
89 | - public function inc($key, $step = 1) { |
|
90 | - $this->add($key, 0); |
|
91 | - /** |
|
92 | - * TODO - hack around a PHP 7 specific issue in APCu |
|
93 | - * |
|
94 | - * on PHP 7 the apcu_inc method on a non-existing object will increment |
|
95 | - * "0" and result in "1" as value - therefore we check for existence |
|
96 | - * first |
|
97 | - * |
|
98 | - * on PHP 5.6 this is not the case |
|
99 | - * |
|
100 | - * see https://github.com/krakjoe/apcu/issues/183#issuecomment-244038221 |
|
101 | - * for details |
|
102 | - */ |
|
103 | - return apcu_exists($this->getPrefix() . $key) |
|
104 | - ? apcu_inc($this->getPrefix() . $key, $step) |
|
105 | - : false; |
|
106 | - } |
|
82 | + /** |
|
83 | + * Increase a stored number |
|
84 | + * |
|
85 | + * @param string $key |
|
86 | + * @param int $step |
|
87 | + * @return int | bool |
|
88 | + */ |
|
89 | + public function inc($key, $step = 1) { |
|
90 | + $this->add($key, 0); |
|
91 | + /** |
|
92 | + * TODO - hack around a PHP 7 specific issue in APCu |
|
93 | + * |
|
94 | + * on PHP 7 the apcu_inc method on a non-existing object will increment |
|
95 | + * "0" and result in "1" as value - therefore we check for existence |
|
96 | + * first |
|
97 | + * |
|
98 | + * on PHP 5.6 this is not the case |
|
99 | + * |
|
100 | + * see https://github.com/krakjoe/apcu/issues/183#issuecomment-244038221 |
|
101 | + * for details |
|
102 | + */ |
|
103 | + return apcu_exists($this->getPrefix() . $key) |
|
104 | + ? apcu_inc($this->getPrefix() . $key, $step) |
|
105 | + : false; |
|
106 | + } |
|
107 | 107 | |
108 | - /** |
|
109 | - * Decrease a stored number |
|
110 | - * |
|
111 | - * @param string $key |
|
112 | - * @param int $step |
|
113 | - * @return int | bool |
|
114 | - */ |
|
115 | - public function dec($key, $step = 1) { |
|
116 | - /** |
|
117 | - * TODO - hack around a PHP 7 specific issue in APCu |
|
118 | - * |
|
119 | - * on PHP 7 the apcu_dec method on a non-existing object will decrement |
|
120 | - * "0" and result in "-1" as value - therefore we check for existence |
|
121 | - * first |
|
122 | - * |
|
123 | - * on PHP 5.6 this is not the case |
|
124 | - * |
|
125 | - * see https://github.com/krakjoe/apcu/issues/183#issuecomment-244038221 |
|
126 | - * for details |
|
127 | - */ |
|
128 | - return apcu_exists($this->getPrefix() . $key) |
|
129 | - ? apcu_dec($this->getPrefix() . $key, $step) |
|
130 | - : false; |
|
131 | - } |
|
108 | + /** |
|
109 | + * Decrease a stored number |
|
110 | + * |
|
111 | + * @param string $key |
|
112 | + * @param int $step |
|
113 | + * @return int | bool |
|
114 | + */ |
|
115 | + public function dec($key, $step = 1) { |
|
116 | + /** |
|
117 | + * TODO - hack around a PHP 7 specific issue in APCu |
|
118 | + * |
|
119 | + * on PHP 7 the apcu_dec method on a non-existing object will decrement |
|
120 | + * "0" and result in "-1" as value - therefore we check for existence |
|
121 | + * first |
|
122 | + * |
|
123 | + * on PHP 5.6 this is not the case |
|
124 | + * |
|
125 | + * see https://github.com/krakjoe/apcu/issues/183#issuecomment-244038221 |
|
126 | + * for details |
|
127 | + */ |
|
128 | + return apcu_exists($this->getPrefix() . $key) |
|
129 | + ? apcu_dec($this->getPrefix() . $key, $step) |
|
130 | + : false; |
|
131 | + } |
|
132 | 132 | |
133 | - /** |
|
134 | - * Compare and set |
|
135 | - * |
|
136 | - * @param string $key |
|
137 | - * @param mixed $old |
|
138 | - * @param mixed $new |
|
139 | - * @return bool |
|
140 | - */ |
|
141 | - public function cas($key, $old, $new) { |
|
142 | - // apc only does cas for ints |
|
143 | - if (is_int($old) and is_int($new)) { |
|
144 | - return apcu_cas($this->getPrefix() . $key, $old, $new); |
|
145 | - } else { |
|
146 | - return $this->casEmulated($key, $old, $new); |
|
147 | - } |
|
148 | - } |
|
133 | + /** |
|
134 | + * Compare and set |
|
135 | + * |
|
136 | + * @param string $key |
|
137 | + * @param mixed $old |
|
138 | + * @param mixed $new |
|
139 | + * @return bool |
|
140 | + */ |
|
141 | + public function cas($key, $old, $new) { |
|
142 | + // apc only does cas for ints |
|
143 | + if (is_int($old) and is_int($new)) { |
|
144 | + return apcu_cas($this->getPrefix() . $key, $old, $new); |
|
145 | + } else { |
|
146 | + return $this->casEmulated($key, $old, $new); |
|
147 | + } |
|
148 | + } |
|
149 | 149 | |
150 | - /** |
|
151 | - * @return bool |
|
152 | - */ |
|
153 | - static public function isAvailable() { |
|
154 | - if (!extension_loaded('apcu')) { |
|
155 | - return false; |
|
156 | - } elseif (!\OC::$server->getIniWrapper()->getBool('apc.enabled')) { |
|
157 | - return false; |
|
158 | - } elseif (!\OC::$server->getIniWrapper()->getBool('apc.enable_cli') && \OC::$CLI) { |
|
159 | - return false; |
|
160 | - } elseif ( |
|
161 | - version_compare(phpversion('apc') ?: '0.0.0', '4.0.6') === -1 && |
|
162 | - version_compare(phpversion('apcu') ?: '0.0.0', '5.1.0') === -1 |
|
163 | - ) { |
|
164 | - return false; |
|
165 | - } else { |
|
166 | - return true; |
|
167 | - } |
|
168 | - } |
|
150 | + /** |
|
151 | + * @return bool |
|
152 | + */ |
|
153 | + static public function isAvailable() { |
|
154 | + if (!extension_loaded('apcu')) { |
|
155 | + return false; |
|
156 | + } elseif (!\OC::$server->getIniWrapper()->getBool('apc.enabled')) { |
|
157 | + return false; |
|
158 | + } elseif (!\OC::$server->getIniWrapper()->getBool('apc.enable_cli') && \OC::$CLI) { |
|
159 | + return false; |
|
160 | + } elseif ( |
|
161 | + version_compare(phpversion('apc') ?: '0.0.0', '4.0.6') === -1 && |
|
162 | + version_compare(phpversion('apcu') ?: '0.0.0', '5.1.0') === -1 |
|
163 | + ) { |
|
164 | + return false; |
|
165 | + } else { |
|
166 | + return true; |
|
167 | + } |
|
168 | + } |
|
169 | 169 | } |
@@ -23,32 +23,32 @@ |
||
23 | 23 | namespace OC\Memcache; |
24 | 24 | |
25 | 25 | trait CADTrait { |
26 | - abstract public function get($key); |
|
26 | + abstract public function get($key); |
|
27 | 27 | |
28 | - abstract public function remove($key); |
|
28 | + abstract public function remove($key); |
|
29 | 29 | |
30 | - abstract public function add($key, $value, $ttl = 0); |
|
30 | + abstract public function add($key, $value, $ttl = 0); |
|
31 | 31 | |
32 | - /** |
|
33 | - * Compare and delete |
|
34 | - * |
|
35 | - * @param string $key |
|
36 | - * @param mixed $old |
|
37 | - * @return bool |
|
38 | - */ |
|
39 | - public function cad($key, $old) { |
|
40 | - //no native cas, emulate with locking |
|
41 | - if ($this->add($key . '_lock', true)) { |
|
42 | - if ($this->get($key) === $old) { |
|
43 | - $this->remove($key); |
|
44 | - $this->remove($key . '_lock'); |
|
45 | - return true; |
|
46 | - } else { |
|
47 | - $this->remove($key . '_lock'); |
|
48 | - return false; |
|
49 | - } |
|
50 | - } else { |
|
51 | - return false; |
|
52 | - } |
|
53 | - } |
|
32 | + /** |
|
33 | + * Compare and delete |
|
34 | + * |
|
35 | + * @param string $key |
|
36 | + * @param mixed $old |
|
37 | + * @return bool |
|
38 | + */ |
|
39 | + public function cad($key, $old) { |
|
40 | + //no native cas, emulate with locking |
|
41 | + if ($this->add($key . '_lock', true)) { |
|
42 | + if ($this->get($key) === $old) { |
|
43 | + $this->remove($key); |
|
44 | + $this->remove($key . '_lock'); |
|
45 | + return true; |
|
46 | + } else { |
|
47 | + $this->remove($key . '_lock'); |
|
48 | + return false; |
|
49 | + } |
|
50 | + } else { |
|
51 | + return false; |
|
52 | + } |
|
53 | + } |
|
54 | 54 | } |
@@ -38,13 +38,13 @@ |
||
38 | 38 | */ |
39 | 39 | public function cad($key, $old) { |
40 | 40 | //no native cas, emulate with locking |
41 | - if ($this->add($key . '_lock', true)) { |
|
41 | + if ($this->add($key.'_lock', true)) { |
|
42 | 42 | if ($this->get($key) === $old) { |
43 | 43 | $this->remove($key); |
44 | - $this->remove($key . '_lock'); |
|
44 | + $this->remove($key.'_lock'); |
|
45 | 45 | return true; |
46 | 46 | } else { |
47 | - $this->remove($key . '_lock'); |
|
47 | + $this->remove($key.'_lock'); |
|
48 | 48 | return false; |
49 | 49 | } |
50 | 50 | } else { |
@@ -27,47 +27,47 @@ |
||
27 | 27 | namespace OC\Memcache; |
28 | 28 | |
29 | 29 | class NullCache extends Cache implements \OCP\IMemcache { |
30 | - public function get($key) { |
|
31 | - return null; |
|
32 | - } |
|
30 | + public function get($key) { |
|
31 | + return null; |
|
32 | + } |
|
33 | 33 | |
34 | - public function set($key, $value, $ttl = 0) { |
|
35 | - return true; |
|
36 | - } |
|
34 | + public function set($key, $value, $ttl = 0) { |
|
35 | + return true; |
|
36 | + } |
|
37 | 37 | |
38 | - public function hasKey($key) { |
|
39 | - return false; |
|
40 | - } |
|
38 | + public function hasKey($key) { |
|
39 | + return false; |
|
40 | + } |
|
41 | 41 | |
42 | - public function remove($key) { |
|
43 | - return true; |
|
44 | - } |
|
42 | + public function remove($key) { |
|
43 | + return true; |
|
44 | + } |
|
45 | 45 | |
46 | - public function add($key, $value, $ttl = 0) { |
|
47 | - return true; |
|
48 | - } |
|
46 | + public function add($key, $value, $ttl = 0) { |
|
47 | + return true; |
|
48 | + } |
|
49 | 49 | |
50 | - public function inc($key, $step = 1) { |
|
51 | - return true; |
|
52 | - } |
|
50 | + public function inc($key, $step = 1) { |
|
51 | + return true; |
|
52 | + } |
|
53 | 53 | |
54 | - public function dec($key, $step = 1) { |
|
55 | - return true; |
|
56 | - } |
|
54 | + public function dec($key, $step = 1) { |
|
55 | + return true; |
|
56 | + } |
|
57 | 57 | |
58 | - public function cas($key, $old, $new) { |
|
59 | - return true; |
|
60 | - } |
|
58 | + public function cas($key, $old, $new) { |
|
59 | + return true; |
|
60 | + } |
|
61 | 61 | |
62 | - public function cad($key, $old) { |
|
63 | - return true; |
|
64 | - } |
|
62 | + public function cad($key, $old) { |
|
63 | + return true; |
|
64 | + } |
|
65 | 65 | |
66 | - public function clear($prefix = '') { |
|
67 | - return true; |
|
68 | - } |
|
66 | + public function clear($prefix = '') { |
|
67 | + return true; |
|
68 | + } |
|
69 | 69 | |
70 | - static public function isAvailable() { |
|
71 | - return true; |
|
72 | - } |
|
70 | + static public function isAvailable() { |
|
71 | + return true; |
|
72 | + } |
|
73 | 73 | } |