GEANT / CAT

web/lib/admin/view/AbstractDialogBox.php

Spacing +2 added lines, -2 removed lines

@@ -13,13 +13,13 @@
     
     protected $action = '';
     
-    protected $params = array ();
+    protected $params = array();
     
     /**
      * 
      * @param string $action
      */
-    public function __construct($action){
+    public function __construct($action) {
         $this->action = $action;
     }
     

core/common/OutsideComm.php

Spacing +2 added lines, -2 removed lines

@@ -172,9 +172,9 @@
                 $totalFailures = 0;
                 foreach ($decoded_response['messages'] as $message) {
                     if ($message['status'] == 0) {
-                        $loggerInstance->debug(2, $message['message-id']. ": Success");
+                        $loggerInstance->debug(2, $message['message-id'] . ": Success");
                     } else {
-                        $loggerInstance->debug(2, $message['message-id']. ": Failed (failure code = ".$message['status'].")");
+                        $loggerInstance->debug(2, $message['message-id'] . ": Failed (failure code = " . $message['status'] . ")");
                         $totalFailures++;
                     }
                 }

core/diag/Telepath.php

Doc Comments +3 added lines

@@ -120,6 +120,9 @@
         return Telepath::STATUS_GOOD;
     }
 
+    /**
+     * @param string $homeFlr
+     */
     private function checkNROFlow($visitedFlr, $homeFlr) {
         // TODO: this is a stub, need eduroam OT API to query the current server status
         // APIQueryNRODirect($visitedFlr, $homeFlr);

web/lib/admin/view/ComposeEmailBox.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
      * {@inheritDoc}
      * @see \web\lib\admin\view\AbstractDialogBox::renderControls()
      */
-    protected function renderControls(){
+    protected function renderControls() {
         ?>
         <div style="position: relative; padding-bottom: 10px; width: 450px;">
              <label>Send email using client aplication:</label>

web/lib/admin/http/SendTokenBySms.php

Spacing +4 added lines, -4 removed lines

@@ -10,7 +10,7 @@ 
  * @author Zilvinas Vaira
  *
  */
-class SendTokenBySms extends AbstractInvokerCommand{
+class SendTokenBySms extends AbstractInvokerCommand {
     
     const COMMAND = "sendtokenbysms";
     const PARAM_PHONE = "phone";
@@ -38,7 +38,7 @@ 
      * @param string $commandToken
      * @param SilverbulletContext $context
      */
-    public function __construct($commandToken, $context){
+    public function __construct($commandToken, $context) {
         parent::__construct($commandToken, $context);
         $this->detailsCommand = new GetTokenEmailDetails(GetTokenEmailDetails::COMMAND, $context);
         $this->context = $context;
@@ -49,7 +49,7 @@ 
      * @param string $phone
      * @param string $content
      */
-    private function sendSMS($phone, $content){
+    private function sendSMS($phone, $content) {
         $error = "";
         $result = -1;
         try {
@@ -79,7 +79,7 @@ 
      * @see \web\lib\admin\http\AbstractCommand::execute()
      */
     public function execute() {
-        if(isset($_POST[GetTokenEmailDetails::PARAM_TOKENLINK]) && isset($_POST[SendTokenBySms::PARAM_PHONE])){
+        if (isset($_POST[GetTokenEmailDetails::PARAM_TOKENLINK]) && isset($_POST[SendTokenBySms::PARAM_PHONE])) {
             
             $invitationToken = $this->parseString($_POST[GetTokenEmailDetails::PARAM_TOKENLINK]);
             $phone = $this->parseString($_POST[SendTokenBySms::PARAM_PHONE]);

web/lib/admin/http/SendTokenByEmail.php

Spacing +4 added lines, -4 removed lines

@@ -12,7 +12,7 @@ 
  * @author Zilvinas Vaira
  *
  */
-class SendTokenByEmail extends AbstractInvokerCommand{
+class SendTokenByEmail extends AbstractInvokerCommand {
     
     const COMMAND = "sendtokenbyemail";
     
@@ -39,7 +39,7 @@ 
      * @param string $commandToken
      * @param SilverbulletContext $context
      */
-    public function __construct($commandToken, $context){
+    public function __construct($commandToken, $context) {
         parent::__construct($commandToken, $context);
         $this->mail = OutsideComm::mailHandle();
         $this->detailsCommand = new GetTokenEmailDetails(GetTokenEmailDetails::COMMAND, $context);
@@ -52,7 +52,7 @@ 
      * @see \web\lib\admin\http\AbstractCommand::execute()
      */
     public function execute() {
-        if(isset($_POST[GetTokenEmailDetails::PARAM_TOKENLINK]) && isset($_POST[ValidateEmailAddress::PARAM_ADDRESS])){
+        if (isset($_POST[GetTokenEmailDetails::PARAM_TOKENLINK]) && isset($_POST[ValidateEmailAddress::PARAM_ADDRESS])) {
             
             $invitationToken = $this->parseString($_POST[GetTokenEmailDetails::PARAM_TOKENLINK]);
             $address = $this->parseString($_POST[ValidateEmailAddress::PARAM_ADDRESS]);
@@ -64,7 +64,7 @@ 
             $this->mail->addStringAttachment($bytestream, "qr-code-invitation.png", "base64", "image/png");
             
             $this->mail->addAddress($address);
-            if($this->mail->send()) {
+            if ($this->mail->send()) {
                 $this->storeInfoMessage(sprintf(_("Email message has been sent successfuly to '%s'!"), $address));
             } else {
                 $this->storeErrorMessage(sprintf(_("Email message could not be sent to '%s'. Mailer error: '%s'."), $address, $this->mail->ErrorInfo));

web/lib/admin/view/SendSmsBox.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
      * {@inheritDoc}
      * @see \web\lib\admin\view\AbstractDialogBox::renderControls()
      */
-    protected function renderControls(){
+    protected function renderControls() {
         ?>
         <div style="position: relative; padding-bottom: 10px; width: 450px;">
              <label>Enter user phone: +</label>

web/lib/admin/http/SilverbulletController.php

Braces +17 added lines, -17 removed lines

@@ -37,15 +37,15 @@ 
             if($commandToken == SaveUsersCommand::COMMAND){
                 if(isset($_POST[DeleteUserCommand::COMMAND])){
                     $commandToken = DeleteUserCommand::COMMAND;
-                }elseif(isset($_POST[AddInvitationCommand::COMMAND])){
+                } elseif(isset($_POST[AddInvitationCommand::COMMAND])){
                     $commandToken = AddInvitationCommand::COMMAND;
-                }elseif(isset($_POST[UpdateUserCommand::COMMAND])){
+                } elseif(isset($_POST[UpdateUserCommand::COMMAND])){
                     $commandToken = UpdateUserCommand::COMMAND;
-                }elseif (isset($_POST[RevokeCertificateCommand::COMMAND])){
+                } elseif (isset($_POST[RevokeCertificateCommand::COMMAND])){
                     $commandToken = RevokeCertificateCommand::COMMAND;
-                }elseif (isset($_POST[RevokeInvitationCommand::COMMAND])){
+                } elseif (isset($_POST[RevokeInvitationCommand::COMMAND])){
                     $commandToken = RevokeInvitationCommand::COMMAND;
-                }elseif (isset($_POST[SaveUsersCommand::COMMAND])){
+                } elseif (isset($_POST[SaveUsersCommand::COMMAND])){
                     $commandToken = SaveUsersCommand::COMMAND;
                 }
             }
@@ -63,31 +63,31 @@ 
         if($this->context->isAgreementSigned()){
             if($commandToken == AddUserCommand::COMMAND){
                 return new AddUserCommand($commandToken, $this->context);
-            }elseif ($commandToken == AddUsersCommand::COMMAND){
+            } elseif ($commandToken == AddUsersCommand::COMMAND){
                 return new AddUsersCommand($commandToken, $this->context);
-            }elseif ($commandToken == DeleteUserCommand::COMMAND){
+            } elseif ($commandToken == DeleteUserCommand::COMMAND){
                 return new DeleteUserCommand($commandToken, $this->context);
-            }elseif ($commandToken == AddInvitationCommand::COMMAND){
+            } elseif ($commandToken == AddInvitationCommand::COMMAND){
                 return new AddInvitationCommand($commandToken, $this->context);
-            }elseif ($commandToken == UpdateUserCommand::COMMAND){
+            } elseif ($commandToken == UpdateUserCommand::COMMAND){
                 return new UpdateUserCommand($commandToken, $this->context);
-            }elseif ($commandToken == RevokeCertificateCommand::COMMAND){
+            } elseif ($commandToken == RevokeCertificateCommand::COMMAND){
                 return new RevokeCertificateCommand($commandToken, $this->context);
-            }elseif ($commandToken == RevokeInvitationCommand::COMMAND){
+            } elseif ($commandToken == RevokeInvitationCommand::COMMAND){
                 return new RevokeInvitationCommand($commandToken, $this->context);
-            }elseif ($commandToken == SaveUsersCommand::COMMAND){
+            } elseif ($commandToken == SaveUsersCommand::COMMAND){
                 return new SaveUsersCommand($commandToken, $this->context);
-            }elseif ($commandToken == SendTokenByEmail::COMMAND){
+            } elseif ($commandToken == SendTokenByEmail::COMMAND){
                 return new SendTokenByEmail($commandToken, $this->context);
-            }elseif ($commandToken == SendTokenBySms::COMMAND){
+            } elseif ($commandToken == SendTokenBySms::COMMAND){
                 return new SendTokenBySms($commandToken, $this->context);
-            }else{
+            } else{
                 return new DefaultCommand($commandToken);
             }
-        }else{
+        } else{
             if($commandToken == TermsOfUseCommand::COMMAND){
                 return new TermsOfUseCommand($commandToken, $this->context);
-            }else{
+            } else{
                 return new DefaultCommand($commandToken);
             }
         }

Spacing +27 added lines, -27 removed lines

@@ -6,7 +6,7 @@ 
  * @author Zilvinas Vaira
  *
  */
-class SilverbulletController extends AbstractController{
+class SilverbulletController extends AbstractController {
     
     /**
      * 
@@ -19,7 +19,7 @@ 
      * 
      * @param SilverbulletContext $context Requires silverbullet page context object
      */
-    public function __construct($context){
+    public function __construct($context) {
         $context->setController($this);
         $this->context = $context;
     }
@@ -29,22 +29,22 @@ 
      * {@inheritDoc}
      * @see \web\lib\admin\http\AbstractController::parseRequest()
      */
-    public function parseRequest(){
+    public function parseRequest() {
         $commandToken = '';
-        if(isset($_POST[SilverbulletController::COMMAND])){
+        if (isset($_POST[SilverbulletController::COMMAND])) {
             $commandToken = $_POST[SilverbulletController::COMMAND];
-            if($commandToken == SaveUsersCommand::COMMAND){
-                if(isset($_POST[DeleteUserCommand::COMMAND])){
+            if ($commandToken == SaveUsersCommand::COMMAND) {
+                if (isset($_POST[DeleteUserCommand::COMMAND])) {
                     $commandToken = DeleteUserCommand::COMMAND;
-                }elseif(isset($_POST[AddInvitationCommand::COMMAND])){
+                }elseif (isset($_POST[AddInvitationCommand::COMMAND])) {
                     $commandToken = AddInvitationCommand::COMMAND;
-                }elseif(isset($_POST[UpdateUserCommand::COMMAND])){
+                }elseif (isset($_POST[UpdateUserCommand::COMMAND])) {
                     $commandToken = UpdateUserCommand::COMMAND;
-                }elseif (isset($_POST[RevokeCertificateCommand::COMMAND])){
+                }elseif (isset($_POST[RevokeCertificateCommand::COMMAND])) {
                     $commandToken = RevokeCertificateCommand::COMMAND;
-                }elseif (isset($_POST[RevokeInvitationCommand::COMMAND])){
+                }elseif (isset($_POST[RevokeInvitationCommand::COMMAND])) {
                     $commandToken = RevokeInvitationCommand::COMMAND;
-                }elseif (isset($_POST[SaveUsersCommand::COMMAND])){
+                }elseif (isset($_POST[SaveUsersCommand::COMMAND])) {
                     $commandToken = SaveUsersCommand::COMMAND;
                 }
             }
@@ -58,35 +58,35 @@ 
      * {@inheritDoc}
      * @see \web\lib\admin\http\AbstractController::doCreateCommand()
      */
-    protected function doCreateCommand($commandToken){
-        if($this->context->isAgreementSigned()){
-            if($commandToken == AddUserCommand::COMMAND){
+    protected function doCreateCommand($commandToken) {
+        if ($this->context->isAgreementSigned()) {
+            if ($commandToken == AddUserCommand::COMMAND) {
                 return new AddUserCommand($commandToken, $this->context);
-            }elseif ($commandToken == AddUsersCommand::COMMAND){
+            }elseif ($commandToken == AddUsersCommand::COMMAND) {
                 return new AddUsersCommand($commandToken, $this->context);
-            }elseif ($commandToken == DeleteUserCommand::COMMAND){
+            }elseif ($commandToken == DeleteUserCommand::COMMAND) {
                 return new DeleteUserCommand($commandToken, $this->context);
-            }elseif ($commandToken == AddInvitationCommand::COMMAND){
+            }elseif ($commandToken == AddInvitationCommand::COMMAND) {
                 return new AddInvitationCommand($commandToken, $this->context);
-            }elseif ($commandToken == UpdateUserCommand::COMMAND){
+            }elseif ($commandToken == UpdateUserCommand::COMMAND) {
                 return new UpdateUserCommand($commandToken, $this->context);
-            }elseif ($commandToken == RevokeCertificateCommand::COMMAND){
+            }elseif ($commandToken == RevokeCertificateCommand::COMMAND) {
                 return new RevokeCertificateCommand($commandToken, $this->context);
-            }elseif ($commandToken == RevokeInvitationCommand::COMMAND){
+            }elseif ($commandToken == RevokeInvitationCommand::COMMAND) {
                 return new RevokeInvitationCommand($commandToken, $this->context);
-            }elseif ($commandToken == SaveUsersCommand::COMMAND){
+            }elseif ($commandToken == SaveUsersCommand::COMMAND) {
                 return new SaveUsersCommand($commandToken, $this->context);
-            }elseif ($commandToken == SendTokenByEmail::COMMAND){
+            }elseif ($commandToken == SendTokenByEmail::COMMAND) {
                 return new SendTokenByEmail($commandToken, $this->context);
-            }elseif ($commandToken == SendTokenBySms::COMMAND){
+            }elseif ($commandToken == SendTokenBySms::COMMAND) {
                 return new SendTokenBySms($commandToken, $this->context);
-            }else{
+            } else {
                 return new DefaultCommand($commandToken);
             }
-        }else{
-            if($commandToken == TermsOfUseCommand::COMMAND){
+        } else {
+            if ($commandToken == TermsOfUseCommand::COMMAND) {
                 return new TermsOfUseCommand($commandToken, $this->context);
-            }else{
+            } else {
                 return new DefaultCommand($commandToken);
             }
         }

web/lib/admin/http/RevokeInvitationCommand.php

Spacing +3 added lines, -3 removed lines

@@ -8,7 +8,7 @@ 
  * @author Zilvinas Vaira
  *
  */
-class RevokeInvitationCommand extends AbstractInvokerCommand{
+class RevokeInvitationCommand extends AbstractInvokerCommand {
 
     const COMMAND = 'revokeinvitation';
 
@@ -23,7 +23,7 @@ 
      * @param string $commandToken
      * @param SilverbulletContext $context
      */
-    public function __construct($commandToken, $context){
+    public function __construct($commandToken, $context) {
         parent::__construct($commandToken, $context);
         $this->context = $context;
     }
@@ -33,7 +33,7 @@ 
      * {@inheritDoc}
      * @see \web\lib\admin\http\AbstractCommand::execute()
      */
-    public function execute(){
+    public function execute() {
         $invitationId = $this->parseInt($_POST[self::COMMAND]);
         
         $invitation = SilverbulletInvitation::prepare($invitationId);