@@ -80,7 +80,7 @@ discard block |
||
| 80 | 80 | /** |
| 81 | 81 | * @inheritdoc |
| 82 | 82 | */ |
| 83 | - public function get ($size = 64) { |
|
| 83 | + public function get($size = 64) { |
|
| 84 | 84 | try { |
| 85 | 85 | $file = $this->getFile($size); |
| 86 | 86 | } catch (NotFoundException $e) { |
@@ -110,9 +110,9 @@ discard block |
||
| 110 | 110 | * @throws NotSquareException if the image is not square |
| 111 | 111 | * @return void |
| 112 | 112 | */ |
| 113 | - public function set ($data) { |
|
| 113 | + public function set($data) { |
|
| 114 | 114 | |
| 115 | - if($data instanceOf IImage) { |
|
| 115 | + if ($data instanceOf IImage) { |
|
| 116 | 116 | $img = $data; |
| 117 | 117 | $data = $img->data(); |
| 118 | 118 | } else { |
@@ -143,12 +143,12 @@ discard block |
||
| 143 | 143 | * remove the users avatar |
| 144 | 144 | * @return void |
| 145 | 145 | */ |
| 146 | - public function remove () { |
|
| 146 | + public function remove() { |
|
| 147 | 147 | $regex = '/^avatar\.([0-9]+\.)?(jpg|png)$/'; |
| 148 | 148 | $avatars = $this->folder->getDirectoryListing(); |
| 149 | 149 | |
| 150 | 150 | $this->config->setUserValue($this->user->getUID(), 'avatar', 'version', |
| 151 | - (int)$this->config->getUserValue($this->user->getUID(), 'avatar', 'version', 0) + 1); |
|
| 151 | + (int) $this->config->getUserValue($this->user->getUID(), 'avatar', 'version', 0) + 1); |
|
| 152 | 152 | |
| 153 | 153 | foreach ($avatars as $avatar) { |
| 154 | 154 | if (preg_match($regex, $avatar->getName())) { |
@@ -165,9 +165,9 @@ discard block |
||
| 165 | 165 | $ext = $this->getExtension(); |
| 166 | 166 | |
| 167 | 167 | if ($size === -1) { |
| 168 | - $path = 'avatar.' . $ext; |
|
| 168 | + $path = 'avatar.'.$ext; |
|
| 169 | 169 | } else { |
| 170 | - $path = 'avatar.' . $size . '.' . $ext; |
|
| 170 | + $path = 'avatar.'.$size.'.'.$ext; |
|
| 171 | 171 | } |
| 172 | 172 | |
| 173 | 173 | try { |
@@ -179,7 +179,7 @@ discard block |
||
| 179 | 179 | |
| 180 | 180 | $avatar = new OC_Image(); |
| 181 | 181 | /** @var ISimpleFile $file */ |
| 182 | - $file = $this->folder->getFile('avatar.' . $ext); |
|
| 182 | + $file = $this->folder->getFile('avatar.'.$ext); |
|
| 183 | 183 | $avatar->loadFromData($file->getContent()); |
| 184 | 184 | if ($size !== -1) { |
| 185 | 185 | $avatar->resize($size); |
@@ -188,7 +188,7 @@ discard block |
||
| 188 | 188 | $file = $this->folder->newFile($path); |
| 189 | 189 | $file->putContent($avatar->data()); |
| 190 | 190 | } catch (NotPermittedException $e) { |
| 191 | - $this->logger->error('Failed to save avatar for ' . $this->user->getUID()); |
|
| 191 | + $this->logger->error('Failed to save avatar for '.$this->user->getUID()); |
|
| 192 | 192 | } |
| 193 | 193 | } |
| 194 | 194 | |
@@ -37,14 +37,14 @@ discard block |
||
| 37 | 37 | /** |
| 38 | 38 | * actions that user backends can define |
| 39 | 39 | */ |
| 40 | - const CREATE_USER = 1; // 1 << 0 |
|
| 41 | - const SET_PASSWORD = 16; // 1 << 4 |
|
| 42 | - const CHECK_PASSWORD = 256; // 1 << 8 |
|
| 43 | - const GET_HOME = 4096; // 1 << 12 |
|
| 44 | - const GET_DISPLAYNAME = 65536; // 1 << 16 |
|
| 45 | - const SET_DISPLAYNAME = 1048576; // 1 << 20 |
|
| 46 | - const PROVIDE_AVATAR = 16777216; // 1 << 24 |
|
| 47 | - const COUNT_USERS = 268435456; // 1 << 28 |
|
| 40 | + const CREATE_USER = 1; // 1 << 0 |
|
| 41 | + const SET_PASSWORD = 16; // 1 << 4 |
|
| 42 | + const CHECK_PASSWORD = 256; // 1 << 8 |
|
| 43 | + const GET_HOME = 4096; // 1 << 12 |
|
| 44 | + const GET_DISPLAYNAME = 65536; // 1 << 16 |
|
| 45 | + const SET_DISPLAYNAME = 1048576; // 1 << 20 |
|
| 46 | + const PROVIDE_AVATAR = 16777216; // 1 << 24 |
|
| 47 | + const COUNT_USERS = 268435456; // 1 << 28 |
|
| 48 | 48 | |
| 49 | 49 | protected $possibleActions = array( |
| 50 | 50 | self::CREATE_USER => 'createUser', |
@@ -66,8 +66,8 @@ discard block |
||
| 66 | 66 | */ |
| 67 | 67 | public function getSupportedActions() { |
| 68 | 68 | $actions = 0; |
| 69 | - foreach($this->possibleActions AS $action => $methodName) { |
|
| 70 | - if(method_exists($this, $methodName)) { |
|
| 69 | + foreach ($this->possibleActions AS $action => $methodName) { |
|
| 70 | + if (method_exists($this, $methodName)) { |
|
| 71 | 71 | $actions |= $action; |
| 72 | 72 | } |
| 73 | 73 | } |
@@ -84,7 +84,7 @@ discard block |
||
| 84 | 84 | * compared with self::CREATE_USER etc. |
| 85 | 85 | */ |
| 86 | 86 | public function implementsActions($actions) { |
| 87 | - return (bool)($this->getSupportedActions() & $actions); |
|
| 87 | + return (bool) ($this->getSupportedActions() & $actions); |
|
| 88 | 88 | } |
| 89 | 89 | |
| 90 | 90 | /** |
@@ -94,7 +94,7 @@ discard block |
||
| 94 | 94 | * |
| 95 | 95 | * Deletes a user |
| 96 | 96 | */ |
| 97 | - public function deleteUser( $uid ) { |
|
| 97 | + public function deleteUser($uid) { |
|
| 98 | 98 | return false; |
| 99 | 99 | } |
| 100 | 100 | |
@@ -148,7 +148,7 @@ discard block |
||
| 148 | 148 | public function getDisplayNames($search = '', $limit = null, $offset = null) { |
| 149 | 149 | $displayNames = array(); |
| 150 | 150 | $users = $this->getUsers($search, $limit, $offset); |
| 151 | - foreach ( $users as $user) { |
|
| 151 | + foreach ($users as $user) { |
|
| 152 | 152 | $displayNames[$user] = $user; |
| 153 | 153 | } |
| 154 | 154 | return $displayNames; |
@@ -64,7 +64,7 @@ discard block |
||
| 64 | 64 | try { |
| 65 | 65 | return new \DateTimeZone($timeZone); |
| 66 | 66 | } catch (\Exception $e) { |
| 67 | - \OCP\Util::writeLog('datetimezone', 'Failed to created DateTimeZone "' . $timeZone . "'", \OCP\Util::DEBUG); |
|
| 67 | + \OCP\Util::writeLog('datetimezone', 'Failed to created DateTimeZone "'.$timeZone."'", \OCP\Util::DEBUG); |
|
| 68 | 68 | return new \DateTimeZone($this->getDefaultTimeZone()); |
| 69 | 69 | } |
| 70 | 70 | } |
@@ -85,9 +85,9 @@ discard block |
||
| 85 | 85 | // so a positive offset means negative timeZone |
| 86 | 86 | // and the other way around. |
| 87 | 87 | if ($offset > 0) { |
| 88 | - $timeZone = 'Etc/GMT-' . $offset; |
|
| 88 | + $timeZone = 'Etc/GMT-'.$offset; |
|
| 89 | 89 | } else { |
| 90 | - $timeZone = 'Etc/GMT+' . abs($offset); |
|
| 90 | + $timeZone = 'Etc/GMT+'.abs($offset); |
|
| 91 | 91 | } |
| 92 | 92 | |
| 93 | 93 | return new \DateTimeZone($timeZone); |
@@ -109,7 +109,7 @@ discard block |
||
| 109 | 109 | } |
| 110 | 110 | |
| 111 | 111 | // No timezone found, fallback to UTC |
| 112 | - \OCP\Util::writeLog('datetimezone', 'Failed to find DateTimeZone for offset "' . $offset . "'", \OCP\Util::DEBUG); |
|
| 112 | + \OCP\Util::writeLog('datetimezone', 'Failed to find DateTimeZone for offset "'.$offset."'", \OCP\Util::DEBUG); |
|
| 113 | 113 | return new \DateTimeZone($this->getDefaultTimeZone()); |
| 114 | 114 | } |
| 115 | 115 | } |
@@ -50,12 +50,12 @@ discard block |
||
| 50 | 50 | * @return array An array of Tag objects. |
| 51 | 51 | */ |
| 52 | 52 | public function loadTags($owners, $type) { |
| 53 | - if(!is_array($owners)) { |
|
| 53 | + if (!is_array($owners)) { |
|
| 54 | 54 | $owners = array($owners); |
| 55 | 55 | } |
| 56 | 56 | |
| 57 | - $sql = 'SELECT `id`, `uid`, `type`, `category` FROM `' . $this->getTableName() . '` ' |
|
| 58 | - . 'WHERE `uid` IN (' . str_repeat('?,', count($owners)-1) . '?) AND `type` = ? ORDER BY `category`'; |
|
| 57 | + $sql = 'SELECT `id`, `uid`, `type`, `category` FROM `'.$this->getTableName().'` ' |
|
| 58 | + . 'WHERE `uid` IN ('.str_repeat('?,', count($owners) - 1).'?) AND `type` = ? ORDER BY `category`'; |
|
| 59 | 59 | return $this->findEntities($sql, array_merge($owners, array($type))); |
| 60 | 60 | } |
| 61 | 61 | |
@@ -66,7 +66,7 @@ discard block |
||
| 66 | 66 | * @return bool |
| 67 | 67 | */ |
| 68 | 68 | public function tagExists($tag) { |
| 69 | - $sql = 'SELECT `id`, `uid`, `type`, `category` FROM `' . $this->getTableName() . '` ' |
|
| 69 | + $sql = 'SELECT `id`, `uid`, `type`, `category` FROM `'.$this->getTableName().'` ' |
|
| 70 | 70 | . 'WHERE `uid` = ? AND `type` = ? AND `category` = ?'; |
| 71 | 71 | try { |
| 72 | 72 | $this->findEntity($sql, array($tag->getOwner(), $tag->getType(), $tag->getName())); |
@@ -62,7 +62,7 @@ discard block |
||
| 62 | 62 | * @todo migrate existing database columns to the correct names |
| 63 | 63 | * to be able to drop this direct mapping |
| 64 | 64 | */ |
| 65 | - public function columnToProperty($columnName){ |
|
| 65 | + public function columnToProperty($columnName) { |
|
| 66 | 66 | if ($columnName === 'category') { |
| 67 | 67 | return 'name'; |
| 68 | 68 | } elseif ($columnName === 'uid') { |
@@ -78,7 +78,7 @@ discard block |
||
| 78 | 78 | * @param string $property the name of the property |
| 79 | 79 | * @return string the column name |
| 80 | 80 | */ |
| 81 | - public function propertyToColumn($property){ |
|
| 81 | + public function propertyToColumn($property) { |
|
| 82 | 82 | if ($property === 'name') { |
| 83 | 83 | return 'category'; |
| 84 | 84 | } elseif ($property === 'owner') { |
@@ -52,7 +52,7 @@ |
||
| 52 | 52 | */ |
| 53 | 53 | public function generate($name, $parameters = array(), $absolute = false) { |
| 54 | 54 | asort($parameters); |
| 55 | - $key = $this->context->getHost() . '#' . $this->context->getBaseUrl() . $name . sha1(json_encode($parameters)) . intval($absolute); |
|
| 55 | + $key = $this->context->getHost().'#'.$this->context->getBaseUrl().$name.sha1(json_encode($parameters)).intval($absolute); |
|
| 56 | 56 | $cachedKey = $this->cache->get($key); |
| 57 | 57 | if ($cachedKey) { |
| 58 | 58 | return $cachedKey; |
@@ -41,10 +41,10 @@ discard block |
||
| 41 | 41 | public function search($pattern, $searchProperties = array(), $options = array()) { |
| 42 | 42 | $this->loadAddressBooks(); |
| 43 | 43 | $result = array(); |
| 44 | - foreach($this->addressBooks as $addressBook) { |
|
| 44 | + foreach ($this->addressBooks as $addressBook) { |
|
| 45 | 45 | $r = $addressBook->search($pattern, $searchProperties, $options); |
| 46 | 46 | $contacts = array(); |
| 47 | - foreach($r as $c){ |
|
| 47 | + foreach ($r as $c) { |
|
| 48 | 48 | $c['addressbook-key'] = $addressBook->getKey(); |
| 49 | 49 | $contacts[] = $c; |
| 50 | 50 | } |
@@ -124,7 +124,7 @@ discard block |
||
| 124 | 124 | public function getAddressBooks() { |
| 125 | 125 | $this->loadAddressBooks(); |
| 126 | 126 | $result = array(); |
| 127 | - foreach($this->addressBooks as $addressBook) { |
|
| 127 | + foreach ($this->addressBooks as $addressBook) { |
|
| 128 | 128 | $result[$addressBook->getKey()] = $addressBook->getDisplayName(); |
| 129 | 129 | } |
| 130 | 130 | |
@@ -181,7 +181,7 @@ discard block |
||
| 181 | 181 | */ |
| 182 | 182 | protected function loadAddressBooks() |
| 183 | 183 | { |
| 184 | - foreach($this->addressBookLoaders as $callable) { |
|
| 184 | + foreach ($this->addressBookLoaders as $callable) { |
|
| 185 | 185 | $callable($this); |
| 186 | 186 | } |
| 187 | 187 | $this->addressBookLoaders = array(); |
@@ -57,7 +57,7 @@ discard block |
||
| 57 | 57 | */ |
| 58 | 58 | public function execute($jobList, ILogger $logger = null) { |
| 59 | 59 | // add an interval of 15 mins |
| 60 | - $this->setInterval(15*60); |
|
| 60 | + $this->setInterval(15 * 60); |
|
| 61 | 61 | |
| 62 | 62 | $this->jobList = $jobList; |
| 63 | 63 | $this->logger = $logger; |
@@ -90,7 +90,7 @@ discard block |
||
| 90 | 90 | try { |
| 91 | 91 | $repair->addStep($step); |
| 92 | 92 | } catch (\Exception $ex) { |
| 93 | - $this->logger->logException($ex,[ |
|
| 93 | + $this->logger->logException($ex, [ |
|
| 94 | 94 | 'app' => 'migration' |
| 95 | 95 | ]); |
| 96 | 96 | |
@@ -55,7 +55,7 @@ |
||
| 55 | 55 | if ($command instanceof ICommand) { |
| 56 | 56 | // ensure the command can be serialized |
| 57 | 57 | $serialized = serialize($command); |
| 58 | - if(strlen($serialized) > 4000) { |
|
| 58 | + if (strlen($serialized) > 4000) { |
|
| 59 | 59 | throw new \InvalidArgumentException('Trying to push a command which serialized form can not be stored in the database (>4000 character)'); |
| 60 | 60 | } |
| 61 | 61 | $unserialized = unserialize($serialized); |