nextcloud / server

apps/files_versions/appinfo/routes.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 /** @var $this \OCP\Route\IRouter */
 $this->create('core_ajax_versions_preview', '/preview')->action(
 function() {
-	require_once __DIR__ . '/../ajax/preview.php';
+	require_once __DIR__.'/../ajax/preview.php';
 });
 
 $this->create('files_versions_download', 'download.php')

apps/files_versions/download.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 OCP\JSON::checkLoggedIn();
 
 $file = $_GET['file'];
-$revision=(int)$_GET['revision'];
+$revision = (int) $_GET['revision'];
 
 list($uid, $filename) = OCA\Files_Versions\Storage::getUidAndFilename($file);
 

core/templates/internalaltmail.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 print_unescaped($l->t("Hey there,\n\njust letting you know that %s shared %s with you.\nView it: %s\n\n", array($_['user_displayname'], $_['filename'], $_['link'])));
-if ( isset($_['expiration']) ) {
+if (isset($_['expiration'])) {
 	print_unescaped($l->t("The share will expire on %s.", array($_['expiration'])));
 	print_unescaped("\n\n");
 }
@@ -9,5 +9,5 @@ 
 ?>
 
 --
-<?php p($theme->getName() . ' - ' . $theme->getSlogan()); ?>
+<?php p($theme->getName().' - '.$theme->getSlogan()); ?>
 <?php print_unescaped("\n".$theme->getBaseUrl());

core/templates/internalmail.php

Spacing +4 added lines, -4 removed lines

@@ -2,8 +2,8 @@ 
 <tr><td>
 <table cellspacing="0" cellpadding="0" border="0" width="600px">
 <tr>
-<td bgcolor="<?php p($theme->getMailHeaderColor());?>" width="20px">&nbsp;</td>
-<td bgcolor="<?php p($theme->getMailHeaderColor());?>">
+<td bgcolor="<?php p($theme->getMailHeaderColor()); ?>" width="20px">&nbsp;</td>
+<td bgcolor="<?php p($theme->getMailHeaderColor()); ?>">
 <img src="<?php p(\OC::$server->getURLGenerator()->getAbsoluteURL(image_path('', 'logo-mail.gif'))); ?>" alt="<?php p($theme->getName()); ?>"/>
 </td>
 </tr>
@@ -13,7 +13,7 @@ 
 <td style="font-weight:normal; font-size:0.8em; line-height:1.2em; font-family:verdana,'arial',sans;">
 <?php
 print_unescaped($l->t('Hey there,<br><br>just letting you know that %s shared <strong>%s</strong> with you.<br><a href="%s">View it!</a><br><br>', array($_['user_displayname'], $_['filename'], $_['link'])));
-if ( isset($_['expiration']) ) {
+if (isset($_['expiration'])) {
 	p($l->t("The share will expire on %s.", array($_['expiration'])));
 	print_unescaped('<br><br>');
 }
@@ -28,7 +28,7 @@ 
 <td style="font-weight:normal; font-size:0.8em; line-height:1.2em; font-family:verdana,'arial',sans;">--<br>
 <?php p($theme->getName()); ?> -
 <?php p($theme->getSlogan()); ?>
-<br><a href="<?php p($theme->getBaseUrl()); ?>"><?php p($theme->getBaseUrl());?></a>
+<br><a href="<?php p($theme->getBaseUrl()); ?>"><?php p($theme->getBaseUrl()); ?></a>
 </td>
 </tr>
 <tr>

core/ajax/preview.php

Spacing +5 added lines, -5 removed lines

@@ -27,12 +27,12 @@
 \OC_Util::checkLoggedIn();
 \OC::$server->getSession()->close();
 
-$file = array_key_exists('file', $_GET) ? (string)$_GET['file'] : '';
-$maxX = array_key_exists('x', $_GET) ? (int)$_GET['x'] : '32';
-$maxY = array_key_exists('y', $_GET) ? (int)$_GET['y'] : '32';
-$scalingUp = array_key_exists('scalingup', $_GET) ? (bool)$_GET['scalingup'] : true;
+$file = array_key_exists('file', $_GET) ? (string) $_GET['file'] : '';
+$maxX = array_key_exists('x', $_GET) ? (int) $_GET['x'] : '32';
+$maxY = array_key_exists('y', $_GET) ? (int) $_GET['y'] : '32';
+$scalingUp = array_key_exists('scalingup', $_GET) ? (bool) $_GET['scalingup'] : true;
 $keepAspect = array_key_exists('a', $_GET) ? true : false;
-$always = array_key_exists('forceIcon', $_GET) ? (bool)$_GET['forceIcon'] : true;
+$always = array_key_exists('forceIcon', $_GET) ? (bool) $_GET['forceIcon'] : true;
 $mode = array_key_exists('mode', $_GET) ? $_GET['mode'] : 'fill';
 
 if ($file === '') {

core/ajax/appconfig.php

Spacing +13 added lines, -13 removed lines

@@ -26,44 +26,44 @@
 OC_Util::checkAdminUser();
 OCP\JSON::callCheck();
 
-$action=isset($_POST['action'])?$_POST['action']:$_GET['action'];
+$action = isset($_POST['action']) ? $_POST['action'] : $_GET['action'];
 
-if(isset($_POST['app']) || isset($_GET['app'])) {
-	$app=OC_App::cleanAppId(isset($_POST['app'])? (string)$_POST['app']: (string)$_GET['app']);
+if (isset($_POST['app']) || isset($_GET['app'])) {
+	$app = OC_App::cleanAppId(isset($_POST['app']) ? (string) $_POST['app'] : (string) $_GET['app']);
 }
 
 // An admin should not be able to add remote and public services
 // on its own. This should only be possible programmatically.
 // This change is due the fact that an admin may not be expected 
 // to execute arbitrary code in every environment.
-if($app === 'core' && isset($_POST['key']) &&(substr((string)$_POST['key'],0,7) === 'remote_' || substr((string)$_POST['key'],0,7) === 'public_')) {
+if ($app === 'core' && isset($_POST['key']) && (substr((string) $_POST['key'], 0, 7) === 'remote_' || substr((string) $_POST['key'], 0, 7) === 'public_')) {
 	OC_JSON::error(array('data' => array('message' => 'Unexpected error!')));
 	return;
 }
 
-$result=false;
+$result = false;
 $appConfig = \OC::$server->getAppConfig();
-switch($action) {
+switch ($action) {
 	case 'getValue':
-		$result=$appConfig->getValue($app, (string)$_GET['key'], (string)$_GET['defaultValue']);
+		$result = $appConfig->getValue($app, (string) $_GET['key'], (string) $_GET['defaultValue']);
 		break;
 	case 'setValue':
-		$result=$appConfig->setValue($app, (string)$_POST['key'], (string)$_POST['value']);
+		$result = $appConfig->setValue($app, (string) $_POST['key'], (string) $_POST['value']);
 		break;
 	case 'getApps':
-		$result=$appConfig->getApps();
+		$result = $appConfig->getApps();
 		break;
 	case 'getKeys':
-		$result=$appConfig->getKeys($app);
+		$result = $appConfig->getKeys($app);
 		break;
 	case 'hasKey':
-		$result=$appConfig->hasKey($app, (string)$_GET['key']);
+		$result = $appConfig->hasKey($app, (string) $_GET['key']);
 		break;
 	case 'deleteKey':
-		$result=$appConfig->deleteKey($app, (string)$_POST['key']);
+		$result = $appConfig->deleteKey($app, (string) $_POST['key']);
 		break;
 	case 'deleteApp':
-		$result=$appConfig->deleteApp($app);
+		$result = $appConfig->deleteApp($app);
 		break;
 }
 OC_JSON::success(array('data'=>$result));

core/js/config.php

Spacing +52 added lines, -52 removed lines

@@ -33,7 +33,7 @@ 
  *
  */
 
-if(!\OC::$server->getRequest()->passesStrictCookieCheck()) {
+if (!\OC::$server->getRequest()->passesStrictCookieCheck()) {
 	die();
 }
 
@@ -52,13 +52,13 @@ 
 
 // Get the config
 $apps_paths = array();
-foreach(OC_App::getEnabledApps() as $app) {
+foreach (OC_App::getEnabledApps() as $app) {
 	$apps_paths[$app] = OC_App::getAppWebPath($app);
 }
 
 $config = \OC::$server->getConfig();
 $value = $config->getAppValue('core', 'shareapi_default_expire_date', 'no');
-$defaultExpireDateEnabled = ($value === 'yes') ? true :false;
+$defaultExpireDateEnabled = ($value === 'yes') ? true : false;
 $defaultExpireDate = $enforceDefaultExpireDate = null;
 if ($defaultExpireDateEnabled) {
 	$defaultExpireDate = (int) $config->getAppValue('core', 'shareapi_expire_after_n_days', '7');
@@ -69,8 +69,8 @@ 
 
 $countOfDataLocation = 0;
 
-$dataLocation = str_replace(OC::$SERVERROOT .'/', '', $config->getSystemValue('datadirectory', ''), $countOfDataLocation);
-if($countOfDataLocation !== 1 || !OC_User::isAdminUser(OC_User::getUser())){
+$dataLocation = str_replace(OC::$SERVERROOT.'/', '', $config->getSystemValue('datadirectory', ''), $countOfDataLocation);
+if ($countOfDataLocation !== 1 || !OC_User::isAdminUser(OC_User::getUser())) {
 	$dataLocation = false;
 }
 
@@ -83,70 +83,70 @@ 
 	"datepickerFormatDate" => json_encode($l->l('jsdate', null)),
 	"dayNames" =>  json_encode(
 		array(
-			(string)$l->t('Sunday'),
-			(string)$l->t('Monday'),
-			(string)$l->t('Tuesday'),
-			(string)$l->t('Wednesday'),
-			(string)$l->t('Thursday'),
-			(string)$l->t('Friday'),
-			(string)$l->t('Saturday')
+			(string) $l->t('Sunday'),
+			(string) $l->t('Monday'),
+			(string) $l->t('Tuesday'),
+			(string) $l->t('Wednesday'),
+			(string) $l->t('Thursday'),
+			(string) $l->t('Friday'),
+			(string) $l->t('Saturday')
 		)
 	),
 	"dayNamesShort" =>  json_encode(
 		array(
-			(string)$l->t('Sun.'),
-			(string)$l->t('Mon.'),
-			(string)$l->t('Tue.'),
-			(string)$l->t('Wed.'),
-			(string)$l->t('Thu.'),
-			(string)$l->t('Fri.'),
-			(string)$l->t('Sat.')
+			(string) $l->t('Sun.'),
+			(string) $l->t('Mon.'),
+			(string) $l->t('Tue.'),
+			(string) $l->t('Wed.'),
+			(string) $l->t('Thu.'),
+			(string) $l->t('Fri.'),
+			(string) $l->t('Sat.')
 		)
 	),
 	"dayNamesMin" =>  json_encode(
 		array(
-			(string)$l->t('Su'),
-			(string)$l->t('Mo'),
-			(string)$l->t('Tu'),
-			(string)$l->t('We'),
-			(string)$l->t('Th'),
-			(string)$l->t('Fr'),
-			(string)$l->t('Sa')
+			(string) $l->t('Su'),
+			(string) $l->t('Mo'),
+			(string) $l->t('Tu'),
+			(string) $l->t('We'),
+			(string) $l->t('Th'),
+			(string) $l->t('Fr'),
+			(string) $l->t('Sa')
 		)
 	),
 	"monthNames" => json_encode(
 		array(
-			(string)$l->t('January'),
-			(string)$l->t('February'),
-			(string)$l->t('March'),
-			(string)$l->t('April'),
-			(string)$l->t('May'),
-			(string)$l->t('June'),
-			(string)$l->t('July'),
-			(string)$l->t('August'),
-			(string)$l->t('September'),
-			(string)$l->t('October'),
-			(string)$l->t('November'),
-			(string)$l->t('December')
+			(string) $l->t('January'),
+			(string) $l->t('February'),
+			(string) $l->t('March'),
+			(string) $l->t('April'),
+			(string) $l->t('May'),
+			(string) $l->t('June'),
+			(string) $l->t('July'),
+			(string) $l->t('August'),
+			(string) $l->t('September'),
+			(string) $l->t('October'),
+			(string) $l->t('November'),
+			(string) $l->t('December')
 		)
 	),
 	"monthNamesShort" => json_encode(
 		array(
-			(string)$l->t('Jan.'),
-			(string)$l->t('Feb.'),
-			(string)$l->t('Mar.'),
-			(string)$l->t('Apr.'),
-			(string)$l->t('May.'),
-			(string)$l->t('Jun.'),
-			(string)$l->t('Jul.'),
-			(string)$l->t('Aug.'),
-			(string)$l->t('Sep.'),
-			(string)$l->t('Oct.'),
-			(string)$l->t('Nov.'),
-			(string)$l->t('Dec.')
+			(string) $l->t('Jan.'),
+			(string) $l->t('Feb.'),
+			(string) $l->t('Mar.'),
+			(string) $l->t('Apr.'),
+			(string) $l->t('May.'),
+			(string) $l->t('Jun.'),
+			(string) $l->t('Jul.'),
+			(string) $l->t('Aug.'),
+			(string) $l->t('Sep.'),
+			(string) $l->t('Oct.'),
+			(string) $l->t('Nov.'),
+			(string) $l->t('Dec.')
 		)
 	),
-	"firstDay" => json_encode($l->l('firstday', null)) ,
+	"firstDay" => json_encode($l->l('firstday', null)),
 	"oc_config" => json_encode(
 		array(
 			'session_lifetime'	=> min(\OCP\Config::getSystemValue('session_lifetime', OC::$server->getIniWrapper()->getNumeric('session.gc_maxlifetime')), OC::$server->getIniWrapper()->getNumeric('session.gc_maxlifetime')),
@@ -195,5 +195,5 @@ 
 
 // Echo it
 foreach ($array as  $setting => $value) {
-	echo("var ". $setting ."=".$value.";\n");
+	echo("var ".$setting."=".$value.";\n");
 }

lib/private/legacy/l10n.php

Spacing +9 added lines, -9 removed lines

@@ -167,12 +167,12 @@ 
 	public function n($text_singular, $text_plural, $count, $parameters = array()) {
 		$this->init();
 		$identifier = "_${text_singular}_::_${text_plural}_";
-		if( array_key_exists($identifier, $this->translations)) {
-			return new OC_L10N_String( $this, $identifier, $parameters, $count );
-		}else{
-			if($count === 1) {
+		if (array_key_exists($identifier, $this->translations)) {
+			return new OC_L10N_String($this, $identifier, $parameters, $count);
+		} else {
+			if ($count === 1) {
 				return new OC_L10N_String($this, $text_singular, $parameters, $count);
-			}else{
+			} else {
 				return new OC_L10N_String($this, $text_plural, $parameters, $count);
 			}
 		}
@@ -235,9 +235,9 @@ 
 
 		$this->init();
 		$value = new DateTime();
-		if($data instanceof DateTime) {
+		if ($data instanceof DateTime) {
 			$value = $data;
-		} elseif(is_string($data) && !is_numeric($data)) {
+		} elseif (is_string($data) && !is_numeric($data)) {
 			$data = strtotime($data);
 			$value->setTimestamp($data);
 		} else {
@@ -249,7 +249,7 @@ 
 
 		$options = array_merge(array('width' => 'long'), $options);
 		$width = $options['width'];
-		switch($type) {
+		switch ($type) {
 			case 'date':
 				return Punic\Calendar::formatDate($value, $width, $locale);
 			case 'datetime':
@@ -327,7 +327,7 @@ 
 	 * @return array an array of available languages
 	 * @deprecated 9.0.0 Use \OC::$server->getL10NFactory()->findAvailableLanguages() instead
 	 */
-	public static function findAvailableLanguages($app=null) {
+	public static function findAvailableLanguages($app = null) {
 		return \OC::$server->getL10NFactory()->findAvailableLanguages($app);
 	}
 

settings/templates/users/part.userlist.php

Spacing +16 added lines, -16 removed lines

@@ -1,17 +1,17 @@ 
-<table id="userlist" class="hascontrols grid" data-groups="<?php p($_['allGroups']);?>">
+<table id="userlist" class="hascontrols grid" data-groups="<?php p($_['allGroups']); ?>">
 	<thead>
 		<tr>
 		<?php if ($_['enableAvatars']): ?>
 			<th id="headerAvatar" scope="col"></th>
 		<?php endif; ?>
 			<th id="headerName" scope="col"><?php p($l->t('Username'))?></th>
-			<th id="headerDisplayName" scope="col"><?php p($l->t( 'Full Name' )); ?></th>
-			<th id="headerPassword" scope="col"><?php p($l->t( 'Password' )); ?></th>
-			<th class="mailAddress" scope="col"><?php p($l->t( 'Email' )); ?></th>
-			<th id="headerGroups" scope="col"><?php p($l->t( 'Groups' )); ?></th>
-		<?php if(is_array($_['subadmins']) || $_['subadmins']): ?>
+			<th id="headerDisplayName" scope="col"><?php p($l->t('Full Name')); ?></th>
+			<th id="headerPassword" scope="col"><?php p($l->t('Password')); ?></th>
+			<th class="mailAddress" scope="col"><?php p($l->t('Email')); ?></th>
+			<th id="headerGroups" scope="col"><?php p($l->t('Groups')); ?></th>
+		<?php if (is_array($_['subadmins']) || $_['subadmins']): ?>
 			<th id="headerSubAdmins" scope="col"><?php p($l->t('Group Admin for')); ?></th>
-		<?php endif;?>
+		<?php endif; ?>
 			<th id="headerQuota" scope="col"><?php p($l->t('Quota')); ?></th>
 			<th class="storageLocation" scope="col"><?php p($l->t('Storage Location')); ?></th>
 			<th class="userBackend" scope="col"><?php p($l->t('User Backend')); ?></th>
@@ -41,26 +41,26 @@ 
 			<td class="groups"><div class="groupsListContainer multiselect button"
 				><span class="title groupsList"></span><span class="icon-triangle-s"></span></div>
 			</td>
-		<?php if(is_array($_['subadmins']) || $_['subadmins']): ?>
+		<?php if (is_array($_['subadmins']) || $_['subadmins']): ?>
 			<td class="subadmins"><div class="groupsListContainer multiselect button"
 				><span class="title groupsList"></span><span class="icon-triangle-s"></span></div>
 			</td>
-		<?php endif;?>
+		<?php endif; ?>
 			<td class="quota">
 				<select class="quota-user" data-inputtitle="<?php p($l->t('Please enter storage quota (ex: "512 MB" or "12 GB")')) ?>">
 					<option	value='default'>
-						<?php p($l->t('Default'));?>
+						<?php p($l->t('Default')); ?>
 					</option>
 					<option value='none'>
-						<?php p($l->t('Unlimited'));?>
+						<?php p($l->t('Unlimited')); ?>
 					</option>
-					<?php foreach($_['quota_preset'] as $preset):?>
-						<option value='<?php p($preset);?>'>
-							<?php p($preset);?>
+					<?php foreach ($_['quota_preset'] as $preset):?>
+						<option value='<?php p($preset); ?>'>
+							<?php p($preset); ?>
 						</option>
-					<?php endforeach;?>
+					<?php endforeach; ?>
 					<option value='other' data-new>
-						<?php p($l->t('Other'));?> ...
+						<?php p($l->t('Other')); ?> ...
 					</option>
 				</select>
 			</td>