Completed
Pull Request — master (#8824)
by John
17:51
created
settings/templates/settings.php 2 patches
Indentation   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -13,7 +13,6 @@  discard block
 block discarded – undo
13 13
  * $tmpl = new OC_Template('settings', 'settings', 'user');
14 14
  * $tmpl->assign('serverData', $serverData);
15 15
  * $tmpl->printPage();
16
-
17 16
  */
18 17
 
19 18
 script('settings', 'main');
@@ -21,7 +20,7 @@  discard block
 block discarded – undo
21 20
 
22 21
 // Did we have some data to inject ?
23 22
 if(is_array($_['serverData'])) {
24
-	$serverData = json_encode($_['serverData']);
23
+    $serverData = json_encode($_['serverData']);
25 24
 ?>
26 25
 <span id="serverData" data-server="<?php p($serverData);?>"></span>
27 26
 <?php } ?>
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -20,8 +20,8 @@
 block discarded – undo
20 20
 style('settings', 'settings');
21 21
 
22 22
 // Did we have some data to inject ?
23
-if(is_array($_['serverData'])) {
23
+if (is_array($_['serverData'])) {
24 24
 	$serverData = json_encode($_['serverData']);
25 25
 ?>
26
-<span id="serverData" data-server="<?php p($serverData);?>"></span>
26
+<span id="serverData" data-server="<?php p($serverData); ?>"></span>
27 27
 <?php } ?>
Please login to merge, or discard this patch.
settings/main.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -20,5 +20,5 @@
 block discarded – undo
20 20
  *
21 21
  */
22 22
 
23
- $tmpl = new OC_Template("settings", "settings", "user");
24
- $tmpl->printPage();
23
+    $tmpl = new OC_Template("settings", "settings", "user");
24
+    $tmpl->printPage();
Please login to merge, or discard this patch.
settings/users.php 2 patches
Indentation   +40 added lines, -40 removed lines patch added patch discarded remove patch
@@ -49,18 +49,18 @@  discard block
 block discarded – undo
49 49
 $config = \OC::$server->getConfig();
50 50
 
51 51
 if ($config->getSystemValue('sort_groups_by_name', false)) {
52
-	$sortGroupsBy = \OC\Group\MetaData::SORT_GROUPNAME;
52
+    $sortGroupsBy = \OC\Group\MetaData::SORT_GROUPNAME;
53 53
 } else {
54
-	$isLDAPUsed = false;
55
-	if ($appManager->isEnabledForUser('user_ldap')) {
56
-		$isLDAPUsed =
57
-			$groupManager->isBackendUsed('\OCA\User_LDAP\Group_LDAP')
58
-			|| $groupManager->isBackendUsed('\OCA\User_LDAP\Group_Proxy');
59
-		if ($isLDAPUsed) {
60
-			// LDAP user count can be slow, so we sort by group name here
61
-			$sortGroupsBy = \OC\Group\MetaData::SORT_GROUPNAME;
62
-		}
63
-	}
54
+    $isLDAPUsed = false;
55
+    if ($appManager->isEnabledForUser('user_ldap')) {
56
+        $isLDAPUsed =
57
+            $groupManager->isBackendUsed('\OCA\User_LDAP\Group_LDAP')
58
+            || $groupManager->isBackendUsed('\OCA\User_LDAP\Group_Proxy');
59
+        if ($isLDAPUsed) {
60
+            // LDAP user count can be slow, so we sort by group name here
61
+            $sortGroupsBy = \OC\Group\MetaData::SORT_GROUPNAME;
62
+        }
63
+    }
64 64
 }
65 65
 
66 66
 $uid = \OC_User::getUser();
@@ -69,62 +69,62 @@  discard block
 block discarded – undo
69 69
 $isDisabled = true;
70 70
 $user = $userManager->get($uid);
71 71
 if ($user) {
72
-	$isDisabled = !$user->isEnabled();
72
+    $isDisabled = !$user->isEnabled();
73 73
 }
74 74
 
75 75
 $groupsInfo = new \OC\Group\MetaData(
76
-	$uid,
77
-	$isAdmin,
78
-	$groupManager,
79
-	\OC::$server->getUserSession()
76
+    $uid,
77
+    $isAdmin,
78
+    $groupManager,
79
+    \OC::$server->getUserSession()
80 80
 );
81 81
 
82 82
 $groupsInfo->setSorting($sortGroupsBy);
83 83
 list($adminGroup, $groups) = $groupsInfo->get();
84 84
 
85 85
 $recoveryAdminEnabled = $appManager->isEnabledForUser('encryption') &&
86
-					    $config->getAppValue( 'encryption', 'recoveryAdminEnabled', '0');
86
+                        $config->getAppValue( 'encryption', 'recoveryAdminEnabled', '0');
87 87
 
88 88
 if($isAdmin) {
89
-	$subAdmins = \OC::$server->getGroupManager()->getSubAdmin()->getAllSubAdmins();
90
-	// New class returns IUser[] so convert back
91
-	$result = [];
92
-	foreach ($subAdmins as $subAdmin) {
93
-		$result[] = [
94
-			'gid' => $subAdmin['group']->getGID(),
95
-			'uid' => $subAdmin['user']->getUID(),
96
-		];
97
-	}
98
-	$subAdmins = $result;
89
+    $subAdmins = \OC::$server->getGroupManager()->getSubAdmin()->getAllSubAdmins();
90
+    // New class returns IUser[] so convert back
91
+    $result = [];
92
+    foreach ($subAdmins as $subAdmin) {
93
+        $result[] = [
94
+            'gid' => $subAdmin['group']->getGID(),
95
+            'uid' => $subAdmin['user']->getUID(),
96
+        ];
97
+    }
98
+    $subAdmins = $result;
99 99
 }else{
100
-	/* Retrieve group IDs from $groups array, so we can pass that information into OC_Group::displayNamesInGroups() */
101
-	$gids = array();
102
-	foreach($groups as $group) {
103
-		if (isset($group['id'])) {
104
-			$gids[] = $group['id'];
105
-		}
106
-	}
107
-	$subAdmins = false;
100
+    /* Retrieve group IDs from $groups array, so we can pass that information into OC_Group::displayNamesInGroups() */
101
+    $gids = array();
102
+    foreach($groups as $group) {
103
+        if (isset($group['id'])) {
104
+            $gids[] = $group['id'];
105
+        }
106
+    }
107
+    $subAdmins = false;
108 108
 }
109 109
 
110 110
 $disabledUsers = $isLDAPUsed ? 0 : $userManager->countDisabledUsers();
111 111
 $disabledUsersGroup = [
112
-	'id' => '_disabledUsers',
113
-	'name' => 'Disabled users',
114
-	'usercount' => $disabledUsers
112
+    'id' => '_disabledUsers',
113
+    'name' => 'Disabled users',
114
+    'usercount' => $disabledUsers
115 115
 ];
116 116
 
117 117
 // load preset quotas
118 118
 $quotaPreset=$config->getAppValue('files', 'quota_preset', '1 GB, 5 GB, 10 GB');
119 119
 $quotaPreset=explode(',', $quotaPreset);
120 120
 foreach($quotaPreset as &$preset) {
121
-	$preset=trim($preset);
121
+    $preset=trim($preset);
122 122
 }
123 123
 $quotaPreset=array_diff($quotaPreset, array('default', 'none'));
124 124
 
125 125
 $defaultQuota=$config->getAppValue('files', 'default_quota', 'none');
126 126
 $defaultQuotaIsUserDefined=array_search($defaultQuota, $quotaPreset)===false
127
-	&& array_search($defaultQuota, array('none', 'default'))===false;
127
+    && array_search($defaultQuota, array('none', 'default'))===false;
128 128
 
129 129
 \OC::$server->getEventDispatcher()->dispatch('OC\Settings\Users::loadAdditionalScripts');
130 130
 
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -83,9 +83,9 @@  discard block
 block discarded – undo
83 83
 list($adminGroup, $groups) = $groupsInfo->get();
84 84
 
85 85
 $recoveryAdminEnabled = $appManager->isEnabledForUser('encryption') &&
86
-					    $config->getAppValue( 'encryption', 'recoveryAdminEnabled', '0');
86
+					    $config->getAppValue('encryption', 'recoveryAdminEnabled', '0');
87 87
 
88
-if($isAdmin) {
88
+if ($isAdmin) {
89 89
 	$subAdmins = \OC::$server->getGroupManager()->getSubAdmin()->getAllSubAdmins();
90 90
 	// New class returns IUser[] so convert back
91 91
 	$result = [];
@@ -96,10 +96,10 @@  discard block
 block discarded – undo
96 96
 		];
97 97
 	}
98 98
 	$subAdmins = $result;
99
-}else{
99
+} else {
100 100
 	/* Retrieve group IDs from $groups array, so we can pass that information into OC_Group::displayNamesInGroups() */
101 101
 	$gids = array();
102
-	foreach($groups as $group) {
102
+	foreach ($groups as $group) {
103 103
 		if (isset($group['id'])) {
104 104
 			$gids[] = $group['id'];
105 105
 		}
@@ -115,16 +115,16 @@  discard block
 block discarded – undo
115 115
 ];
116 116
 
117 117
 // load preset quotas
118
-$quotaPreset=$config->getAppValue('files', 'quota_preset', '1 GB, 5 GB, 10 GB');
119
-$quotaPreset=explode(',', $quotaPreset);
120
-foreach($quotaPreset as &$preset) {
121
-	$preset=trim($preset);
118
+$quotaPreset = $config->getAppValue('files', 'quota_preset', '1 GB, 5 GB, 10 GB');
119
+$quotaPreset = explode(',', $quotaPreset);
120
+foreach ($quotaPreset as &$preset) {
121
+	$preset = trim($preset);
122 122
 }
123
-$quotaPreset=array_diff($quotaPreset, array('default', 'none'));
123
+$quotaPreset = array_diff($quotaPreset, array('default', 'none'));
124 124
 
125
-$defaultQuota=$config->getAppValue('files', 'default_quota', 'none');
126
-$defaultQuotaIsUserDefined=array_search($defaultQuota, $quotaPreset)===false
127
-	&& array_search($defaultQuota, array('none', 'default'))===false;
125
+$defaultQuota = $config->getAppValue('files', 'default_quota', 'none');
126
+$defaultQuotaIsUserDefined = array_search($defaultQuota, $quotaPreset) === false
127
+	&& array_search($defaultQuota, array('none', 'default')) === false;
128 128
 
129 129
 \OC::$server->getEventDispatcher()->dispatch('OC\Settings\Users::loadAdditionalScripts');
130 130
 
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
 $serverData['subadmingroups'] = $groups;
138 138
 $serverData['disabledUsersGroup'] = $disabledUsersGroup;
139 139
 // Various data
140
-$serverData['isAdmin'] = (int)$isAdmin;
140
+$serverData['isAdmin'] = (int) $isAdmin;
141 141
 $serverData['subadmins'] = $subAdmins;
142 142
 $serverData['sortGroups'] = $sortGroupsBy;
143 143
 $serverData['quotaPreset'] = $quotaPreset;
Please login to merge, or discard this patch.
settings/routes.php 1 patch
Indentation   +52 added lines, -52 removed lines patch added patch discarded remove patch
@@ -38,76 +38,76 @@
 block discarded – undo
38 38
 
39 39
 $application = new Application();
40 40
 $application->registerRoutes($this, [
41
-	'resources' => [
42
-		'users' => ['url' => '/settings/users/users'],
43
-		'AuthSettings' => ['url' => '/settings/personal/authtokens'],
44
-	],
45
-	'routes' => [
46
-		['name' => 'MailSettings#setMailSettings', 'url' => '/settings/admin/mailsettings', 'verb' => 'POST'],
47
-		['name' => 'MailSettings#storeCredentials', 'url' => '/settings/admin/mailsettings/credentials', 'verb' => 'POST'],
48
-		['name' => 'MailSettings#sendTestMail', 'url' => '/settings/admin/mailtest', 'verb' => 'POST'],
49
-		['name' => 'Encryption#startMigration', 'url' => '/settings/admin/startmigration', 'verb' => 'POST'],
50
-		['name' => 'AppSettings#listCategories', 'url' => '/settings/apps/categories', 'verb' => 'GET'],
51
-		['name' => 'AppSettings#viewApps', 'url' => '/settings/apps', 'verb' => 'GET'],
52
-		['name' => 'AppSettings#listApps', 'url' => '/settings/apps/list', 'verb' => 'GET'],
53
-		['name' => 'SecuritySettings#trustedDomains', 'url' => '/settings/admin/security/trustedDomains', 'verb' => 'POST'],
54
-		['name' => 'Users#setDisplayName', 'url' => '/settings/users/{id}/displayName', 'verb' => 'PUT'],
55
-		['name' => 'Users#setEMailAddress', 'url' => '/settings/users/{id}/mailAddress', 'verb' => 'PUT'],
56
-		['name' => 'Users#setUserSettings', 'url' => '/settings/users/{username}/settings', 'verb' => 'PUT'],
57
-		['name' => 'Users#getVerificationCode', 'url' => '/settings/users/{account}/verify', 'verb' => 'GET'],
58
-		['name' => 'Users#setEnabled', 'url' => '/settings/users/{id}/setEnabled', 'verb' => 'POST'],
59
-		['name' => 'Users#stats', 'url' => '/settings/users/stats', 'verb' => 'GET'],
60
-		['name' => 'LogSettings#setLogLevel', 'url' => '/settings/admin/log/level', 'verb' => 'POST'],
61
-		['name' => 'LogSettings#getEntries', 'url' => '/settings/admin/log/entries', 'verb' => 'GET'],
62
-		['name' => 'LogSettings#download', 'url' => '/settings/admin/log/download', 'verb' => 'GET'],
63
-		['name' => 'CheckSetup#check', 'url' => '/settings/ajax/checksetup', 'verb' => 'GET'],
64
-		['name' => 'CheckSetup#getFailedIntegrityCheckFiles', 'url' => '/settings/integrity/failed', 'verb' => 'GET'],
65
-		['name' => 'CheckSetup#rescanFailedIntegrityCheck', 'url' => '/settings/integrity/rescan', 'verb' => 'GET'],
66
-		['name' => 'Certificate#addPersonalRootCertificate', 'url' => '/settings/personal/certificate', 'verb' => 'POST'],
67
-		['name' => 'Certificate#removePersonalRootCertificate', 'url' => '/settings/personal/certificate/{certificateIdentifier}', 'verb' => 'DELETE'],
68
-		['name' => 'Certificate#addSystemRootCertificate', 'url' => '/settings/admin/certificate', 'verb' => 'POST'],
69
-		['name' => 'Certificate#removeSystemRootCertificate', 'url' => '/settings/admin/certificate/{certificateIdentifier}', 'verb' => 'DELETE'],
70
-		['name' => 'PersonalSettings#index', 'url' => '/settings/user/{section}', 'verb' => 'GET', 'defaults' => ['section' => 'personal-info']],
71
-		['name' => 'AdminSettings#index', 'url' => '/settings/admin/{section}', 'verb' => 'GET', 'defaults' => ['section' => 'server']],
72
-		['name' => 'AdminSettings#form', 'url' => '/settings/admin/{section}', 'verb' => 'GET'],
73
-		['name' => 'ChangePassword#changePersonalPassword', 'url' => '/settings/personal/changepassword', 'verb' => 'POST'],
74
-		['name' => 'ChangePassword#changeUserPassword', 'url' => '/settings/users/changepassword', 'verb' => 'POST'],
75
-		['name' => 'Groups#index', 'url' => '/settings/users/groups', 'verb' => 'GET'],
76
-		['name' => 'Groups#show', 'url' => '/settings/users/groups/{id}', 'requirements' => ['id' => '[^?]*'], 'verb' => 'GET'],
77
-		['name' => 'Groups#create', 'url' => '/settings/users/groups', 'verb' => 'POST'],
78
-		['name' => 'Groups#update', 'url' => '/settings/users/groups/{id}', 'requirements' => ['id' => '[^?]*'], 'verb' => 'PUT'],
79
-		['name' => 'Groups#destroy', 'url' => '/settings/users/groups/{id}', 'requirements' => ['id' => '[^?]*'], 'verb' => 'DELETE'],
80
-	]
41
+    'resources' => [
42
+        'users' => ['url' => '/settings/users/users'],
43
+        'AuthSettings' => ['url' => '/settings/personal/authtokens'],
44
+    ],
45
+    'routes' => [
46
+        ['name' => 'MailSettings#setMailSettings', 'url' => '/settings/admin/mailsettings', 'verb' => 'POST'],
47
+        ['name' => 'MailSettings#storeCredentials', 'url' => '/settings/admin/mailsettings/credentials', 'verb' => 'POST'],
48
+        ['name' => 'MailSettings#sendTestMail', 'url' => '/settings/admin/mailtest', 'verb' => 'POST'],
49
+        ['name' => 'Encryption#startMigration', 'url' => '/settings/admin/startmigration', 'verb' => 'POST'],
50
+        ['name' => 'AppSettings#listCategories', 'url' => '/settings/apps/categories', 'verb' => 'GET'],
51
+        ['name' => 'AppSettings#viewApps', 'url' => '/settings/apps', 'verb' => 'GET'],
52
+        ['name' => 'AppSettings#listApps', 'url' => '/settings/apps/list', 'verb' => 'GET'],
53
+        ['name' => 'SecuritySettings#trustedDomains', 'url' => '/settings/admin/security/trustedDomains', 'verb' => 'POST'],
54
+        ['name' => 'Users#setDisplayName', 'url' => '/settings/users/{id}/displayName', 'verb' => 'PUT'],
55
+        ['name' => 'Users#setEMailAddress', 'url' => '/settings/users/{id}/mailAddress', 'verb' => 'PUT'],
56
+        ['name' => 'Users#setUserSettings', 'url' => '/settings/users/{username}/settings', 'verb' => 'PUT'],
57
+        ['name' => 'Users#getVerificationCode', 'url' => '/settings/users/{account}/verify', 'verb' => 'GET'],
58
+        ['name' => 'Users#setEnabled', 'url' => '/settings/users/{id}/setEnabled', 'verb' => 'POST'],
59
+        ['name' => 'Users#stats', 'url' => '/settings/users/stats', 'verb' => 'GET'],
60
+        ['name' => 'LogSettings#setLogLevel', 'url' => '/settings/admin/log/level', 'verb' => 'POST'],
61
+        ['name' => 'LogSettings#getEntries', 'url' => '/settings/admin/log/entries', 'verb' => 'GET'],
62
+        ['name' => 'LogSettings#download', 'url' => '/settings/admin/log/download', 'verb' => 'GET'],
63
+        ['name' => 'CheckSetup#check', 'url' => '/settings/ajax/checksetup', 'verb' => 'GET'],
64
+        ['name' => 'CheckSetup#getFailedIntegrityCheckFiles', 'url' => '/settings/integrity/failed', 'verb' => 'GET'],
65
+        ['name' => 'CheckSetup#rescanFailedIntegrityCheck', 'url' => '/settings/integrity/rescan', 'verb' => 'GET'],
66
+        ['name' => 'Certificate#addPersonalRootCertificate', 'url' => '/settings/personal/certificate', 'verb' => 'POST'],
67
+        ['name' => 'Certificate#removePersonalRootCertificate', 'url' => '/settings/personal/certificate/{certificateIdentifier}', 'verb' => 'DELETE'],
68
+        ['name' => 'Certificate#addSystemRootCertificate', 'url' => '/settings/admin/certificate', 'verb' => 'POST'],
69
+        ['name' => 'Certificate#removeSystemRootCertificate', 'url' => '/settings/admin/certificate/{certificateIdentifier}', 'verb' => 'DELETE'],
70
+        ['name' => 'PersonalSettings#index', 'url' => '/settings/user/{section}', 'verb' => 'GET', 'defaults' => ['section' => 'personal-info']],
71
+        ['name' => 'AdminSettings#index', 'url' => '/settings/admin/{section}', 'verb' => 'GET', 'defaults' => ['section' => 'server']],
72
+        ['name' => 'AdminSettings#form', 'url' => '/settings/admin/{section}', 'verb' => 'GET'],
73
+        ['name' => 'ChangePassword#changePersonalPassword', 'url' => '/settings/personal/changepassword', 'verb' => 'POST'],
74
+        ['name' => 'ChangePassword#changeUserPassword', 'url' => '/settings/users/changepassword', 'verb' => 'POST'],
75
+        ['name' => 'Groups#index', 'url' => '/settings/users/groups', 'verb' => 'GET'],
76
+        ['name' => 'Groups#show', 'url' => '/settings/users/groups/{id}', 'requirements' => ['id' => '[^?]*'], 'verb' => 'GET'],
77
+        ['name' => 'Groups#create', 'url' => '/settings/users/groups', 'verb' => 'POST'],
78
+        ['name' => 'Groups#update', 'url' => '/settings/users/groups/{id}', 'requirements' => ['id' => '[^?]*'], 'verb' => 'PUT'],
79
+        ['name' => 'Groups#destroy', 'url' => '/settings/users/groups/{id}', 'requirements' => ['id' => '[^?]*'], 'verb' => 'DELETE'],
80
+    ]
81 81
 ]);
82 82
 
83 83
 /** @var $this \OCP\Route\IRouter */
84 84
 
85 85
 // Settings pages
86 86
 $this->create('settings_help', '/settings/help')
87
-	->actionInclude('settings/help.php');
87
+    ->actionInclude('settings/help.php');
88 88
 $this->create('settings_users', '/settings/users')
89
-	->actionInclude('settings/users.php');
89
+    ->actionInclude('settings/users.php');
90 90
 // Settings ajax actions
91 91
 // users
92 92
 $this->create('settings_ajax_setquota', '/settings/ajax/setquota.php')
93
-	->actionInclude('settings/ajax/setquota.php');
93
+    ->actionInclude('settings/ajax/setquota.php');
94 94
 $this->create('settings_ajax_togglegroups', '/settings/ajax/togglegroups.php')
95
-	->actionInclude('settings/ajax/togglegroups.php');
95
+    ->actionInclude('settings/ajax/togglegroups.php');
96 96
 $this->create('settings_ajax_togglesubadmins', '/settings/ajax/togglesubadmins.php')
97
-	->actionInclude('settings/ajax/togglesubadmins.php');
97
+    ->actionInclude('settings/ajax/togglesubadmins.php');
98 98
 $this->create('settings_ajax_changegorupname', '/settings/ajax/changegroupname.php')
99
-	->actionInclude('settings/ajax/changegroupname.php');
99
+    ->actionInclude('settings/ajax/changegroupname.php');
100 100
 // apps
101 101
 $this->create('settings_ajax_enableapp', '/settings/ajax/enableapp.php')
102
-	->actionInclude('settings/ajax/enableapp.php');
102
+    ->actionInclude('settings/ajax/enableapp.php');
103 103
 $this->create('settings_ajax_disableapp', '/settings/ajax/disableapp.php')
104
-	->actionInclude('settings/ajax/disableapp.php');
104
+    ->actionInclude('settings/ajax/disableapp.php');
105 105
 $this->create('settings_ajax_updateapp', '/settings/ajax/updateapp.php')
106
-	->actionInclude('settings/ajax/updateapp.php');
106
+    ->actionInclude('settings/ajax/updateapp.php');
107 107
 $this->create('settings_ajax_uninstallapp', '/settings/ajax/uninstallapp.php')
108
-	->actionInclude('settings/ajax/uninstallapp.php');
108
+    ->actionInclude('settings/ajax/uninstallapp.php');
109 109
 $this->create('settings_ajax_navigationdetect', '/settings/ajax/navigationdetect.php')
110
-	->actionInclude('settings/ajax/navigationdetect.php');
110
+    ->actionInclude('settings/ajax/navigationdetect.php');
111 111
 // admin
112 112
 $this->create('settings_ajax_excludegroups', '/settings/ajax/excludegroups.php')
113
-	->actionInclude('settings/ajax/excludegroups.php');
113
+    ->actionInclude('settings/ajax/excludegroups.php');
Please login to merge, or discard this patch.