Completed
Push — master ( 45fcfc...6e6327 )
by Lukas
13:40
created
apps/provisioning_api/lib/Controller/GroupsController.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -72,10 +72,10 @@  discard block
 block discarded – undo
72 72
 	 */
73 73
 	public function getGroups($search = '', $limit = null, $offset = null) {
74 74
 		if ($limit !== null) {
75
-			$limit = (int)$limit;
75
+			$limit = (int) $limit;
76 76
 		}
77 77
 		if ($offset !== null) {
78
-			$offset = (int)$offset;
78
+			$offset = (int) $offset;
79 79
 		}
80 80
 
81 81
 		$groups = $this->groupManager->search($search, $limit, $offset);
@@ -100,21 +100,21 @@  discard block
 block discarded – undo
100 100
 		$user = $this->userSession->getUser();
101 101
 
102 102
 		// Check the group exists
103
-		if(!$this->groupManager->groupExists($groupId)) {
103
+		if (!$this->groupManager->groupExists($groupId)) {
104 104
 			throw new OCSException('The requested group could not be found', \OCP\API::RESPOND_NOT_FOUND);
105 105
 		}
106 106
 
107 107
 		$isSubadminOfGroup = false;
108 108
 		$group = $this->groupManager->get($groupId);
109 109
 		if ($group !== null) {
110
-			$isSubadminOfGroup =$this->groupManager->getSubAdmin()->isSubAdminofGroup($user, $group);
110
+			$isSubadminOfGroup = $this->groupManager->getSubAdmin()->isSubAdminofGroup($user, $group);
111 111
 		}
112 112
 
113 113
 		// Check subadmin has access to this group
114
-		if($this->groupManager->isAdmin($user->getUID())
114
+		if ($this->groupManager->isAdmin($user->getUID())
115 115
 		   || $isSubadminOfGroup) {
116 116
 			$users = $this->groupManager->get($groupId)->getUsers();
117
-			$users =  array_map(function($user) {
117
+			$users = array_map(function($user) {
118 118
 				/** @var IUser $user */
119 119
 				return $user->getUID();
120 120
 			}, $users);
@@ -136,12 +136,12 @@  discard block
 block discarded – undo
136 136
 	 */
137 137
 	public function addGroup($groupid) {
138 138
 		// Validate name
139
-		if(empty($groupid)){
139
+		if (empty($groupid)) {
140 140
 			\OCP\Util::writeLog('provisioning_api', 'Group name not supplied', \OCP\Util::ERROR);
141 141
 			throw new OCSException('Invalid group name', 101);
142 142
 		}
143 143
 		// Check if it exists
144
-		if($this->groupManager->groupExists($groupid)){
144
+		if ($this->groupManager->groupExists($groupid)) {
145 145
 			throw new OCSException('', 102);
146 146
 		}
147 147
 		$this->groupManager->createGroup($groupid);
@@ -157,9 +157,9 @@  discard block
 block discarded – undo
157 157
 	 */
158 158
 	public function deleteGroup($groupId) {
159 159
 		// Check it exists
160
-		if(!$this->groupManager->groupExists($groupId)){
160
+		if (!$this->groupManager->groupExists($groupId)) {
161 161
 			throw new OCSException('', 101);
162
-		} else if($groupId === 'admin' || !$this->groupManager->get($groupId)->delete()){
162
+		} else if ($groupId === 'admin' || !$this->groupManager->get($groupId)->delete()) {
163 163
 			// Cannot delete admin group
164 164
 			throw new OCSException('', 102);
165 165
 		}
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
 	public function getSubAdminsOfGroup($groupId) {
176 176
 		// Check group exists
177 177
 		$targetGroup = $this->groupManager->get($groupId);
178
-		if($targetGroup === null) {
178
+		if ($targetGroup === null) {
179 179
 			throw new OCSException('Group does not exist', 101);
180 180
 		}
181 181
 
Please login to merge, or discard this patch.
apps/updatenotification/templates/admin.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
 	$currentChannel = $_['currentChannel'];
15 15
 ?>
16 16
 <form id="oca_updatenotification_section" class="followupsection">
17
-	<?php if($isNewVersionAvailable === true) { ?>
17
+	<?php if ($isNewVersionAvailable === true) { ?>
18 18
 		<strong><?php p($l->t('A new version is available: %s', [$newVersionString])); ?></strong>
19 19
 		<?php if ($_['updaterEnabled']) { ?>
20 20
 			<input type="button" id="oca_updatenotification_button" value="<?php p($l->t('Open updater')) ?>">
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 		<label for="release-channel"><?php p($l->t('Update channel:')) ?></label>
32 32
 		<select id="release-channel">
33 33
 			<option value="<?php p($currentChannel); ?>"><?php p($currentChannel); ?></option>
34
-			<?php foreach ($channels as $channel => $channelTitle){ ?>
34
+			<?php foreach ($channels as $channel => $channelTitle) { ?>
35 35
 				<option value="<?php p($channelTitle) ?>">
36 36
 					<?php p($channelTitle) ?>
37 37
 				</option>
Please login to merge, or discard this patch.
apps/updatenotification/lib/Notification/Notifier.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
 			$notification->setParsedSubject($l->t('Update to %1$s is available.', [$parameters['version']]));
92 92
 
93 93
 			if ($this->isAdmin()) {
94
-				$notification->setLink($this->url->linkToRouteAbsolute('settings.AdminSettings.index') . '#updater');
94
+				$notification->setLink($this->url->linkToRouteAbsolute('settings.AdminSettings.index').'#updater');
95 95
 			}
96 96
 		} else {
97 97
 			$appInfo = $this->getAppInfo($notification->getObjectType());
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
 				]);
112 112
 
113 113
 			if ($this->isAdmin()) {
114
-				$notification->setLink($this->url->linkToRouteAbsolute('settings.AppSettings.viewApps') . '#app-' . $notification->getObjectType());
114
+				$notification->setLink($this->url->linkToRouteAbsolute('settings.AppSettings.viewApps').'#app-'.$notification->getObjectType());
115 115
 			}
116 116
 		}
117 117
 
Please login to merge, or discard this patch.
apps/updatenotification/lib/Controller/AdminController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -99,7 +99,7 @@
 block discarded – undo
99 99
 		$currentChannel = \OCP\Util::getChannel();
100 100
 
101 101
 		// Remove the currently used channel from the channels list
102
-		if(($key = array_search($currentChannel, $channels)) !== false) {
102
+		if (($key = array_search($currentChannel, $channels)) !== false) {
103 103
 			unset($channels[$key]);
104 104
 		}
105 105
 		$updateState = $this->updateChecker->getUpdateState();
Please login to merge, or discard this patch.
apps/updatenotification/lib/UpdateChecker.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -43,14 +43,14 @@  discard block
 block discarded – undo
43 43
 		$data = $this->updater->check();
44 44
 		$result = [];
45 45
 
46
-		if(isset($data['version']) && $data['version'] !== '' && $data['version'] !== []) {
46
+		if (isset($data['version']) && $data['version'] !== '' && $data['version'] !== []) {
47 47
 			$result['updateAvailable'] = true;
48 48
 			$result['updateVersion'] = $data['versionstring'];
49 49
 			$result['updaterEnabled'] = $data['autoupdater'] === '1';
50
-			if(substr($data['web'], 0, 8) === 'https://') {
50
+			if (substr($data['web'], 0, 8) === 'https://') {
51 51
 				$result['updateLink'] = $data['web'];
52 52
 			}
53
-			if(substr($data['url'], 0, 8) === 'https://') {
53
+			if (substr($data['url'], 0, 8) === 'https://') {
54 54
 				$result['downloadLink'] = $data['url'];
55 55
 			}
56 56
 
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
 	 * @param array $data
65 65
 	 */
66 66
 	public function getJavaScript(array $data) {
67
-		$data['array']['oc_updateState'] =  json_encode([
67
+		$data['array']['oc_updateState'] = json_encode([
68 68
 			'updateAvailable' => true,
69 69
 			'updateVersion' => $this->getUpdateState()['updateVersion'],
70 70
 			'updateLink' => isset($this->getUpdateState()['updateLink']) ? $this->getUpdateState()['updateLink'] : '',
Please login to merge, or discard this patch.
apps/updatenotification/lib/AppInfo/Application.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@
 block discarded – undo
30 30
 use OCP\AppFramework\IAppContainer;
31 31
 
32 32
 class Application extends App {
33
-	public function __construct (array $urlParams = array()) {
33
+	public function __construct(array $urlParams = array()) {
34 34
 		parent::__construct('updatenotification', $urlParams);
35 35
 		$container = $this->getContainer();
36 36
 
Please login to merge, or discard this patch.
apps/updatenotification/lib/ResetTokenBackgroundJob.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@
 block discarded – undo
69 69
 	 */
70 70
 	protected function run($argument) {
71 71
 		// Delete old tokens after 2 days
72
-		if($this->timeFactory->getTime() - $this->config->getAppValue('core', 'updater.secret.created', $this->timeFactory->getTime()) >= 172800) {
72
+		if ($this->timeFactory->getTime() - $this->config->getAppValue('core', 'updater.secret.created', $this->timeFactory->getTime()) >= 172800) {
73 73
 			$this->config->deleteSystemValue('updater.secret');
74 74
 		}
75 75
 	}
Please login to merge, or discard this patch.
apps/updatenotification/appinfo/app.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
  *
23 23
  */
24 24
 
25
-if(\OC::$server->getConfig()->getSystemValue('updatechecker', true) === true) {
25
+if (\OC::$server->getConfig()->getSystemValue('updatechecker', true) === true) {
26 26
 	$updater = new \OC\Updater\VersionCheck(
27 27
 		\OC::$server->getHTTPClientService(),
28 28
 		\OC::$server->getConfig()
@@ -32,10 +32,10 @@  discard block
 block discarded – undo
32 32
 	);
33 33
 
34 34
 	$userObject = \OC::$server->getUserSession()->getUser();
35
-	if($userObject !== null) {
36
-		if(\OC::$server->getGroupManager()->isAdmin($userObject->getUID()) &&
35
+	if ($userObject !== null) {
36
+		if (\OC::$server->getGroupManager()->isAdmin($userObject->getUID()) &&
37 37
 			!\OC::$server->getAppManager()->isEnabledForUser('notifications')) {
38
-			if($updateChecker->getUpdateState() !== []) {
38
+			if ($updateChecker->getUpdateState() !== []) {
39 39
 				\OCP\Util::addScript('updatenotification', 'notification');
40 40
 				OC_Hook::connect('\OCP\Config', 'js', $updateChecker, 'getJavaScript');
41 41
 			}
Please login to merge, or discard this patch.
apps/theming/lib/Controller/IconController.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -91,13 +91,13 @@  discard block
 block discarded – undo
91 91
 	 */
92 92
 	public function getThemedIcon($app, $image) {
93 93
 		try {
94
-			$iconFile = $this->imageManager->getCachedImage("icon-" . $app . '-' . str_replace("/","_",$image));
94
+			$iconFile = $this->imageManager->getCachedImage("icon-".$app.'-'.str_replace("/", "_", $image));
95 95
 		} catch (NotFoundException $exception) {
96 96
 			$icon = $this->iconBuilder->colorSvg($app, $image);
97 97
 			if ($icon === false || $icon === "") {
98 98
 				return new NotFoundResponse();
99 99
 			}
100
-			$iconFile = $this->imageManager->setCachedImage("icon-" . $app . '-' . str_replace("/","_",$image), $icon);
100
+			$iconFile = $this->imageManager->setCachedImage("icon-".$app.'-'.str_replace("/", "_", $image), $icon);
101 101
 		}
102 102
 		if ($iconFile !== false) {
103 103
 			$response = new FileDisplayResponse($iconFile, Http::STATUS_OK, ['Content-Type' => 'image/svg+xml']);
@@ -125,10 +125,10 @@  discard block
 block discarded – undo
125 125
 	public function getFavicon($app = "core") {
126 126
 		if ($this->themingDefaults->shouldReplaceIcons()) {
127 127
 			try {
128
-				$iconFile = $this->imageManager->getCachedImage('favIcon-' . $app);
128
+				$iconFile = $this->imageManager->getCachedImage('favIcon-'.$app);
129 129
 			} catch (NotFoundException $exception) {
130 130
 				$icon = $this->iconBuilder->getFavicon($app);
131
-				$iconFile = $this->imageManager->setCachedImage('favIcon-' . $app, $icon);
131
+				$iconFile = $this->imageManager->setCachedImage('favIcon-'.$app, $icon);
132 132
 			}
133 133
 			if ($iconFile !== false) {
134 134
 				$response = new FileDisplayResponse($iconFile, Http::STATUS_OK, ['Content-Type' => 'image/x-icon']);
@@ -156,10 +156,10 @@  discard block
 block discarded – undo
156 156
 	public function getTouchIcon($app = "core") {
157 157
 		if ($this->themingDefaults->shouldReplaceIcons()) {
158 158
 			try {
159
-				$iconFile = $this->imageManager->getCachedImage('touchIcon-' . $app);
159
+				$iconFile = $this->imageManager->getCachedImage('touchIcon-'.$app);
160 160
 			} catch (NotFoundException $exception) {
161 161
 				$icon = $this->iconBuilder->getTouchIcon($app);
162
-				$iconFile = $this->imageManager->setCachedImage('touchIcon-' . $app, $icon);
162
+				$iconFile = $this->imageManager->setCachedImage('touchIcon-'.$app, $icon);
163 163
 			}
164 164
 			if ($iconFile !== false) {
165 165
 				$response = new FileDisplayResponse($iconFile, Http::STATUS_OK, ['Content-Type' => 'image/png']);
Please login to merge, or discard this patch.