grommunio / grommunio-web

plugins/files/php/plugin.files.php

Switch Indentation +24 added lines, -24 removed lines

@@ -45,37 +45,37 @@
 
 			case 'server.index.load.custom':
 				switch ($data['name']) {
-				case 'files_get_recipients':
-					RecipientHandler::doGetRecipients();
-					break;
+					case 'files_get_recipients':
+						RecipientHandler::doGetRecipients();
+						break;
 
-				case 'download_file':
-					DownloadHandler::doDownload();
-					break;
+					case 'download_file':
+						DownloadHandler::doDownload();
+						break;
 
-				case 'upload_file':
-					UploadHandler::doUpload();
-					break;
+					case 'upload_file':
+						UploadHandler::doUpload();
+						break;
 
-				case 'form':
-					if (isset($_GET['backend'])) {
-						$backend = urldecode($_GET["backend"]);
-					}
-					else {
-						$backend = '';
-					}
-					$backendstore = Files\Backend\BackendStore::getInstance();
+					case 'form':
+						if (isset($_GET['backend'])) {
+							$backend = urldecode($_GET["backend"]);
+						}
+						else {
+							$backend = '';
+						}
+						$backendstore = Files\Backend\BackendStore::getInstance();
 
-					if ($backendstore->backendExists($backend)) {
-						$backendInstance = $backendstore->getInstanceOfBackend($backend);
-						$formdata = $backendInstance->getFormConfig();
+						if ($backendstore->backendExists($backend)) {
+							$backendInstance = $backendstore->getInstanceOfBackend($backend);
+							$formdata = $backendInstance->getFormConfig();
 
-						exit($formdata);
-					}
+							exit($formdata);
+						}
 
-						exit("Specified backend does not exist!");
+							exit("Specified backend does not exist!");
 
-					break;
+						break;
 			}
 			break;
 		}

plugins/smime/php/lib/X509.php

Switch Indentation +25 added lines, -25 removed lines

@@ -15,31 +15,31 @@
 		$tag = $this->peek();
 
 		switch ($tag) {
-		case 0:
-			$res['otherName'] = $this->oid(-6);
-			break;
-
-		case 1:
-			$res['rfc822Name'] = $this->next(-22);
-			break;
-
-		case 2:
-			$res['dNSName'] = $this->next(-22);
-			break;
-
-		case 4:
-			$this->next(4);
-			$res['directoryName'] = $this->name();
-			$res['directoryName_'] = $this->nameasstring($res['directoryName']);
-			break;
-
-		case 6:
-			$res['uniformResourceIdentifier'] = $this->next(-22);
-			break;
-
-		default:
-			throw new \Exception("Unsupported GeneralName: {$tag}");
-#                trigger_error("Unsupported GeneralName: $tag", E_USER_ERROR);
+			case 0:
+				$res['otherName'] = $this->oid(-6);
+				break;
+
+			case 1:
+				$res['rfc822Name'] = $this->next(-22);
+				break;
+
+			case 2:
+				$res['dNSName'] = $this->next(-22);
+				break;
+
+			case 4:
+				$this->next(4);
+				$res['directoryName'] = $this->name();
+				$res['directoryName_'] = $this->nameasstring($res['directoryName']);
+				break;
+
+			case 6:
+				$res['uniformResourceIdentifier'] = $this->next(-22);
+				break;
+
+			default:
+				throw new \Exception("Unsupported GeneralName: {$tag}");
+	#                trigger_error("Unsupported GeneralName: $tag", E_USER_ERROR);
 		}
 
 		return $res;

plugins/smime/php/class.pluginsmimemodule.php

Braces +3 added lines, -1 removed lines

@@ -25,9 +25,11 @@
 	 * @return bool true on success or false on failure
 	 */
 	public function execute() {
-		foreach ($this->data as $actionType => $actionData) try {
+		foreach ($this->data as $actionType => $actionData) {
+			try {
 			if (!isset($actionType))
 				continue;
+		}
 			switch ($actionType) {
 			case 'certificate':
 				$data = $this->verifyCertificate($actionData);

Switch Indentation +50 added lines, -50 removed lines

@@ -29,62 +29,62 @@
 			if (!isset($actionType))
 				continue;
 			switch ($actionType) {
-			case 'certificate':
-				$data = $this->verifyCertificate($actionData);
-				$response = [
-					'type' => 3,
-					'status' => $data['status'],
-					'message' => $data['message'],
-					'data' => $data['data'],
-				];
-				$this->addActionData('certificate', $response);
-				$GLOBALS['bus']->addData($this->getResponseData());
-				break;
+				case 'certificate':
+					$data = $this->verifyCertificate($actionData);
+					$response = [
+						'type' => 3,
+						'status' => $data['status'],
+						'message' => $data['message'],
+						'data' => $data['data'],
+					];
+					$this->addActionData('certificate', $response);
+					$GLOBALS['bus']->addData($this->getResponseData());
+					break;
 
-			case 'passphrase':
-				$data = $this->verifyPassphrase($actionData);
-				$response = [
-					'type' => 3,
-					'status' => $data['status'],
-				];
-				$this->addActionData('passphrase', $response);
-				$GLOBALS['bus']->addData($this->getResponseData());
-				break;
+				case 'passphrase':
+					$data = $this->verifyPassphrase($actionData);
+					$response = [
+						'type' => 3,
+						'status' => $data['status'],
+					];
+					$this->addActionData('passphrase', $response);
+					$GLOBALS['bus']->addData($this->getResponseData());
+					break;
 
-			case 'changepassphrase':
-				$data = $this->changePassphrase($actionData);
-				if ($data === CHANGE_PASSPHRASE_SUCCESS) {
-					// Reset cached passphrase.
-					$encryptionStore = EncryptionStore::getInstance();
-					withPHPSession(function () use ($encryptionStore) {
-						$encryptionStore->add('smime', '');
-					});
-				}
-				$response = [
-					'type' => 3,
-					'code' => $data,
-				];
-				$this->addActionData('changepassphrase', $response);
-				$GLOBALS['bus']->addData($this->getResponseData());
-				break;
+				case 'changepassphrase':
+					$data = $this->changePassphrase($actionData);
+					if ($data === CHANGE_PASSPHRASE_SUCCESS) {
+						// Reset cached passphrase.
+						$encryptionStore = EncryptionStore::getInstance();
+						withPHPSession(function () use ($encryptionStore) {
+							$encryptionStore->add('smime', '');
+						});
+					}
+					$response = [
+						'type' => 3,
+						'code' => $data,
+					];
+					$this->addActionData('changepassphrase', $response);
+					$GLOBALS['bus']->addData($this->getResponseData());
+					break;
 
-			case 'list':
-				$data = $this->getPublicCertificates();
-				$this->addActionData('list', $data);
-				$GLOBALS['bus']->addData($this->getResponseData());
-				break;
+				case 'list':
+					$data = $this->getPublicCertificates();
+					$this->addActionData('list', $data);
+					$GLOBALS['bus']->addData($this->getResponseData());
+					break;
 
-			case 'delete':
-				// FIXME: handle multiple deletes? Separate function?
-				$entryid = $actionData['entryid'];
-				$root = mapi_msgstore_openentry($this->store);
-				mapi_folder_deletemessages($root, [hex2bin($entryid)]);
+				case 'delete':
+					// FIXME: handle multiple deletes? Separate function?
+					$entryid = $actionData['entryid'];
+					$root = mapi_msgstore_openentry($this->store);
+					mapi_folder_deletemessages($root, [hex2bin($entryid)]);
 
-				$this->sendFeedback(true);
-				break;
+					$this->sendFeedback(true);
+					break;
 
-			default:
-				$this->handleUnknownActionType($actionType);
+				default:
+					$this->handleUnknownActionType($actionType);
 			}
 		}
 		catch (Exception $e) {

plugins/smime/php/class.certificate.php

Switch Indentation +6 added lines, -6 removed lines

@@ -33,14 +33,14 @@
 		}
 
 		switch ($this->status) {
-		case 'good':
-			return OCSP_CERT_STATUS_GOOD;
+			case 'good':
+				return OCSP_CERT_STATUS_GOOD;
 
-		case 'revoked':
-			return OCSP_CERT_STATUS_REVOKED;
+			case 'revoked':
+				return OCSP_CERT_STATUS_REVOKED;
 
-		default:
-			return OCSP_CERT_STATUS_UNKOWN;
+			default:
+				return OCSP_CERT_STATUS_UNKOWN;
 		}
 	}
 }

plugins/smime/php/util.php

Braces +3 added lines, -2 removed lines

@@ -98,8 +98,9 @@
 	// Get messages from certificates
 	foreach ($privateCerts as $privateCert) {
 		$privateCertMessage = mapi_msgstore_openentry($store, $privateCert[PR_ENTRYID]);
-		if ($privateCertMessage === false)
-			continue;
+		if ($privateCertMessage === false) {
+					continue;
+		}
 		$pkcs12 = "";
 		$certs = [];
 		// Read pkcs12 cert from message

server/includes/templates/login.php

Braces +2 added lines, -1 removed lines

@@ -96,7 +96,8 @@
 		<div class="disclaimer">
 			<?php include '/etc/grommunio-web/disclaimer.html'; ?>
 		</div>
-		<?php } elseif (file_exists('disclaimer.html')) { ?>
+		<?php }
+elseif (file_exists('disclaimer.html')) { ?>
 		<div class="disclaimer">
 			<?php include 'disclaimer.html'; ?>
 		</div>

plugins/kendox/php/class.kendox-module.php

Braces +2 added lines, -1 removed lines

@@ -186,7 +186,8 @@
 		// Read message properties
 		try {
 			$messageProps = mapi_getprops($this->mapiMessage, [PR_SUBJECT, PR_MESSAGE_CLASS]);
-		} catch (Exception $ex) {
+		}
+		catch (Exception $ex) {
 			$this->logErrorAndThrow("Error on getting MAPI message properties", $ex);
 		}
 

plugins/kendox/php/kendox-client/class.kendox-client.php

Braces +21 added lines, -17 removed lines

@@ -25,9 +25,10 @@ 
          */

         public $ConnectionId = null;

 

-        function __construct($serviceEndpoint)

-        {

-            if (!str_ends_with($serviceEndpoint, '/')) $serviceEndpoint .= '/';

+        function __construct($serviceEndpoint) {
+            if (!str_ends_with($serviceEndpoint, '/')) {
+            	$serviceEndpoint .= '/';
+            }
             $this->ServiceEndpoint = $serviceEndpoint;

         }

 

@@ -40,7 +41,7 @@ 
          * 

          * @return bool

          */

-        public function loginWithToken($pfxFile, $pfxPassword, $userName) {

+        public function loginWithToken($pfxFile, $pfxPassword, $userName) {
             try {

                 $issuer = gethostname();

                 $tokenGenerator = new \Kendox\TokenGenerator($issuer, $pfxFile, $pfxPassword);

@@ -53,7 +54,8 @@ 
                 $result = $this->post("Authentication/LogonWithToken", $logonParameters);

                 $this->ConnectionId = $result->LogonWithTokenResult->ConnectionId;

                 return true;

-            } catch(\Exception $ex) {

+            }
+            catch(\Exception $ex) {

                 throw new \Exception("Token-Login failed: ".$ex->getMessage());

             }

         }

@@ -63,7 +65,7 @@ 
          * 

          * @return bool

          */

-        public function logout() {

+        public function logout() {
             try {

                 $logoutParameters = [

                     "connectionId" => $this->ConnectionId

@@ -71,7 +73,8 @@ 
                 $result = $this->post("Authentication/Logout", $logoutParameters);

                 $this->ConnectionId = null;

                 return true;

-            } catch(\Exception $ex) {

+            }
+            catch(\Exception $ex) {

                 throw new \Exception("Token-Login failed: ".$ex->getMessage());

             }

         }

@@ -85,7 +88,7 @@ 
          * 

          * @return array The data result as an array

          */

-        public function userTableQuery($userTableName, $whereClauseElements, $addColumnHeaders) {

+        public function userTableQuery($userTableName, $whereClauseElements, $addColumnHeaders) {
             try {

                 $parameters = [

                     "connectionId" => $this->ConnectionId,

@@ -94,9 +97,12 @@ 
                     "addColumnHeaders" => $addColumnHeaders

                 ];

                 $result = $this->post("UserTable/UserTableGetRecords", $parameters);

-                if (!isset($result->UserTableGetRecordsResult)) throw new \Exception("Unexpected result");

+                if (!isset($result->UserTableGetRecordsResult)) {
+                	throw new \Exception("Unexpected result");
+                }
                 return $result->UserTableGetRecordsResult;

-            } catch(\Exception $ex) {

+            }
+            catch(\Exception $ex) {

                 throw new \Exception("User table query failed: ".$ex->getMessage());

             }

         }

@@ -105,7 +111,7 @@ 
          * Uploading a file

          * @param string $file Path and file name of file to upload

          */

-        public function uploadFile($file) {

+        public function uploadFile($file) {
             $content = file_get_contents($file);

             return $this->uploadContent($content);

         }

@@ -114,12 +120,12 @@ 
          * Uploading a stream of data

          * @param Stream $stream Stream of content to upload

          */

-        public function uploadStream($stream) {

+        public function uploadStream($stream) {
             $content = stream_get_contents($stream);

             return $this->uploadContent($content);

         }

 

-        private function uploadContent($content) {

+        private function uploadContent($content) {
             $base64 = base64_encode($content);

             $uploadParameters = [

                 "connectionId" => $this->ConnectionId,

@@ -137,8 +143,7 @@ 
          * 

          * @return object Returns object with data. If service returns an error an exception will be thrown with detailed information

          */

-        private function post($path, $data)

-        {

+        private function post($path, $data) {
             $ch = curl_init();

             //curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);

             curl_setopt($ch, CURLOPT_URL, $this->ServiceEndpoint.$path);

@@ -154,8 +159,7 @@ 
 

         }

         

-        private function handleJsonResult($json)

-        {

+        private function handleJsonResult($json) {
             if ($json === FALSE) {

                 throw new \Exception("No valid JSON has been returned from service.");

             }

plugins/kendox/php/kendox-client/class.kendox-token-generator.php

Braces +23 added lines, -18 removed lines

@@ -44,8 +44,7 @@ 
     private $PfxFile, /**
      * Password for PFX-File (Certificate)
      */
-    private $PfxPassword)

-    {

+    private $PfxPassword) {
         $this->loadCertificateFromPfx();        

     }

 

@@ -56,8 +55,7 @@ 
      * 

      * @return string Token in XML format

      */

-    public function generateToken($userEMail)

-    {

+    public function generateToken($userEMail) {
         try {

             $now = new \DateTime("now", new \DateTimeZone("utc"));

             $guid = $this->createGUID();

@@ -73,7 +71,8 @@ 
                 xmlwriter_end_element($writer);                

             xmlwriter_end_element($writer);

             return xmlwriter_output_memory($writer);

-        } catch(\Exception $ex) {

+        }
+        catch(\Exception $ex) {

             throw new \Exception("Generating token failed: ".$ex->getMessage());

         }

     }   

@@ -81,21 +80,27 @@ 
     /**

      * Loads the X509-certificate from PFX-File

      */

-    private function loadCertificateFromPfx()

-    {

-        if ($this->PfxFile == null) throw new \Exception("No PFX-File available.");

-        if (!file_exists($this->PfxFile)) throw new \Exception("PFX-File not found.");

-        if (empty($this->PfxPassword)) throw new \Exception("Password not set for PFX-File.");

+    private function loadCertificateFromPfx() {
+        if ($this->PfxFile == null) {
+        	throw new \Exception("No PFX-File available.");
+        }
+        if (!file_exists($this->PfxFile)) {
+        	throw new \Exception("PFX-File not found.");
+        }
+        if (empty($this->PfxPassword)) {
+        	throw new \Exception("Password not set for PFX-File.");
+        }
         $pfxContent = file_get_contents($this->PfxFile);

         $results = [];

         $read = openssl_pkcs12_read($pfxContent, $results, $this->PfxPassword);

-        if ($read == false) throw new \Exception("Error on reading PFX-File: ".openssl_error_string());

+        if ($read == false) {
+        	throw new \Exception("Error on reading PFX-File: ".openssl_error_string());
+        }
         $this->Certificate = $results['pkey'].$results['cert'];

         $this->CertPrivateKey = $results['pkey'];

     }

 

-    private function writeSignedInfo($writer, $userEMail, $time, $uniqueId)

-    {

+    private function writeSignedInfo($writer, $userEMail, $time, $uniqueId) {
         $utcTime = $time->format('Y-m-d H:i:s');

         $utcTime = str_replace(" ", "T", $utcTime)."Z";

         xmlwriter_start_element($writer, "SignedInfo");

@@ -136,8 +141,7 @@ 
      * 

      * @return string

      */

-    function SignXmlString($signedInfoXml)

-    {

+    function SignXmlString($signedInfoXml) {
         try {

             $data = iconv('utf-8', 'utf-16le', $signedInfoXml);    

             $privateKey = \phpseclib3\Crypt\RSA::loadFormat('PKCS8', $this->CertPrivateKey)

@@ -145,7 +149,8 @@ 
                             ->withHash('sha512');

             $base64 = base64_encode($privateKey->sign($data));

             return $base64;

-        } catch (\Exception $ex) {

+        }
+        catch (\Exception $ex) {

             throw new \Exception("XML signing failed: ".$ex->getMessage());

         }

     }

@@ -154,10 +159,10 @@ 
      * Creates a unique ID

      * @return string

      */

-    private function createGUID(){

+    private function createGUID() {
         if (function_exists('com_create_guid')){

             return com_create_guid();

-        }

+        }
         else {

             mt_srand((double)microtime()*10000);

             $charid = strtoupper(md5(uniqid(random_int(0, mt_getrandmax()), true)));