GEANT / CAT

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);

web/lib/admin/view/UserCredentialsForm.php

Braces +3 added lines, -3 removed lines

@@ -158,7 +158,7 @@ 
         $acknowledgeLevel = $user->getAcknowledgeLevel();
         if($acknowledgeLevel == SilverbulletUser::LEVEL_YELLOW){
             $row->addAttribute('style', 'background-color:#F0EAC0;');
-        }elseif ($acknowledgeLevel == SilverbulletUser::LEVEL_RED){
+        } elseif ($acknowledgeLevel == SilverbulletUser::LEVEL_RED){
             $row->addAttribute('style', 'background-color:#F0C0C0;');
         }
         
@@ -215,11 +215,11 @@ 
                 $certificateBox->addAttribute('style', 'background-color:#F0C0C0;');
                 $buttonContainer->addAttribute('style', 'height:22px; margin-top:7px; text-align:center;');
                 $buttonContainer->addText(_("REVOKED"));
-            }elseif ($certificate->isExpired()){
+            } elseif ($certificate->isExpired()){
                 $certificateBox->addAttribute('style', 'background-color:lightgrey;');
                 $buttonContainer->addAttribute('style', 'height:22px; margin-top:7px; text-align:center;');
                 $buttonContainer->addText(_("EXPIRED"));
-            }else{
+            } else{
                 $buttonContainer->addAttribute('style', 'text-align:right;padding-top: 5px;');
                 $buttonContainer->addText(new Button(_('Revoke'), 'submit', RevokeCertificateCommand::COMMAND, $certificate->getIdentifier(), 'delete'));
             }

Spacing +23 added lines, -23 removed lines

@@ -26,7 +26,7 @@ 
  * @author Zilvinas Vaira
  *
  */
-class UserCredentialsForm implements PageElementInterface{
+class UserCredentialsForm implements PageElementInterface {
     
     const EDITABLEBLOCK_CLASS = 'sb-editable-block';
     const TITLEROW_CLASS = 'sb-title-row';
@@ -118,10 +118,10 @@ 
         
         $this->acknowledgeDays = CONFIG_CONFASSISTANT['SILVERBULLET']['gracetime'] ?? SilverbulletUser::MAX_ACKNOWLEDGE;
         $this->acknowledgeText = $acknowledgeText;
-        if($isAcknowledgeEnabled){
+        if ($isAcknowledgeEnabled) {
             $div = new CompositeTag('div');
             $div->addAttribute('style', 'padding-bottom: 20px;');
-            $this->acknowledgeNotice = new Tag ('p');
+            $this->acknowledgeNotice = new Tag('p');
             $div->addTag($this->acknowledgeNotice);
             $checkbox = new UnaryTag('input');
             $checkbox->addAttribute('type', 'checkbox');
@@ -132,7 +132,7 @@ 
             $label->addText('I have verified that all configured users are still eligible for eduroam');
             $div->addTag($label);
             $this->decorator->addHtmlElement($div);
-            $this->decorator->addHtmlElement(new Button(_('Save'),'submit', SaveUsersCommand::COMMAND, SaveUsersCommand::COMMAND));
+            $this->decorator->addHtmlElement(new Button(_('Save'), 'submit', SaveUsersCommand::COMMAND, SaveUsersCommand::COMMAND));
         }
         
         $this->addTitleRow();
@@ -141,7 +141,7 @@ 
     /**
      * 
      */
-    private function addTitleRow(){
+    private function addTitleRow() {
         $row = new Row(array(self::USER_COLUMN => _('User'), self::TOKEN_COLUMN => _('Token/Certificate details'), self::EXPIRY_COLUMN => _('User/Token Expiry'), self::ACTION_COLUMN => _('Actions')));
         $row->addAttribute('class', self::TITLEROW_CLASS);
         $this->table->addRow($row);
@@ -151,19 +151,19 @@ 
      * 
      * @param SilverbulletUser $user
      */
-    public function addUserRow($user){
+    public function addUserRow($user) {
         $row = new Row(array(self::USER_COLUMN => $user->getUsername(), self::EXPIRY_COLUMN => new DatePicker(SaveUsersCommand::PARAM_EXPIRY_MULTIPLE, $user->getExpiry())));
         $row->addAttribute('class', self::USERROW_CLASS);
         
         $acknowledgeLevel = $user->getAcknowledgeLevel();
-        if($acknowledgeLevel == SilverbulletUser::LEVEL_YELLOW){
+        if ($acknowledgeLevel == SilverbulletUser::LEVEL_YELLOW) {
             $row->addAttribute('style', 'background-color:#F0EAC0;');
-        }elseif ($acknowledgeLevel == SilverbulletUser::LEVEL_RED){
+        }elseif ($acknowledgeLevel == SilverbulletUser::LEVEL_RED) {
             $row->addAttribute('style', 'background-color:#F0C0C0;');
         }
         
         $acknowledgeDays = $user->getAcknowledgeDays();
-        if($acknowledgeDays < $this->acknowledgeDays){
+        if ($acknowledgeDays < $this->acknowledgeDays) {
             $this->acknowledgeDays = $acknowledgeDays;
         }
         
@@ -183,7 +183,7 @@ 
         $action = new CompositeTag('div');
         $action->addAttribute('class', 'sb-user-buttons');
             $deactivationButton = new Button(_('Deactivate User'), 'submit', DeleteUserCommand::COMMAND, $user->getIdentifier(), 'delete');
-            if($user->isDeactivated()){
+            if ($user->isDeactivated()) {
                 $deactivationButton->addAttribute('disabled', 'disabled'); 
             }
         $action->addTag($deactivationButton);
@@ -206,8 +206,8 @@ 
      * 
      * @param SilverbulletCertificate $certificate
      */
-    public function addCertificateRow($certificate){
-        if($certificate->isGenerated()){
+    public function addCertificateRow($certificate) {
+        if ($certificate->isGenerated()) {
 
             //Create certificate box
             $certificateBox = new CompositeTag('div');
@@ -221,15 +221,15 @@ 
 
             //Create button container div
             $buttonContainer = new Tag('div');
-            if($certificate->isRevoked()){
+            if ($certificate->isRevoked()) {
                 $certificateBox->addAttribute('style', 'background-color:#F0C0C0;');
                 $buttonContainer->addAttribute('style', 'height:22px; margin-top:7px; text-align:center;');
                 $buttonContainer->addText(_("REVOKED"));
-            }elseif ($certificate->isExpired()){
+            }elseif ($certificate->isExpired()) {
                 $certificateBox->addAttribute('style', 'background-color:lightgrey;');
                 $buttonContainer->addAttribute('style', 'height:22px; margin-top:7px; text-align:center;');
                 $buttonContainer->addText(_("EXPIRED"));
-            }else{
+            } else {
                 $buttonContainer->addAttribute('style', 'text-align:right;padding-top: 5px;');
                 $buttonContainer->addText(new Button(_('Revoke'), 'submit', RevokeCertificateCommand::COMMAND, $certificate->getIdentifier(), 'delete'));
             }
@@ -243,7 +243,7 @@ 
      *
      * @param SilverbulletInvitation $invitation
      */
-    public function addInvitationRow($invitation){
+    public function addInvitationRow($invitation) {
         $expiryColumn = _("Expiry Date: ") . $invitation->getExpiry();
         $expiryColumn .= "<br>";
         $expiryColumn .= _("Activations: ") . $invitation->getRemainingActivations();
@@ -251,13 +251,13 @@ 
         $row->addAttribute('class', self::CERTIFICATEROW_CLASS);
         $index = $this->table->size();
         $this->table->addRow($row);
-        if(!$invitation->isExpired() && !$invitation->isRevoked() && !$invitation->isAbsent()){
+        if (!$invitation->isExpired() && !$invitation->isRevoked() && !$invitation->isAbsent()) {
             $this->table->addToCell($index, self::TOKEN_COLUMN, new Button(_('Copy to Clipboard'), 'button', '', '', self::INVITATION_TOKEN_CLASS . '-copy'));
-            $this->table->addToCell($index, self::TOKEN_COLUMN, new Button(_('Compose mail...'), 'button', '', '', self::INVITATION_TOKEN_CLASS. '-compose'));
-            $this->table->addToCell($index, self::TOKEN_COLUMN, new Button(_('Send in SMS...'), 'button', '', '', self::INVITATION_TOKEN_CLASS. '-sms'));
-            $this->table->addToCell($index, self::TOKEN_COLUMN, new Button(_('Generate QR code...'), 'button', '', '', self::INVITATION_TOKEN_CLASS. '-qrcode'));
+            $this->table->addToCell($index, self::TOKEN_COLUMN, new Button(_('Compose mail...'), 'button', '', '', self::INVITATION_TOKEN_CLASS . '-compose'));
+            $this->table->addToCell($index, self::TOKEN_COLUMN, new Button(_('Send in SMS...'), 'button', '', '', self::INVITATION_TOKEN_CLASS . '-sms'));
+            $this->table->addToCell($index, self::TOKEN_COLUMN, new Button(_('Generate QR code...'), 'button', '', '', self::INVITATION_TOKEN_CLASS . '-qrcode'));
         }
-        if(!$invitation->isRevoked()){
+        if (!$invitation->isRevoked()) {
             $this->table->addToCell($index, self::ACTION_COLUMN, new Button(_('Revoke'), 'submit', RevokeInvitationCommand::COMMAND, $invitation->getIdentifier(), 'delete'));
         }
     }
@@ -268,11 +268,11 @@ 
      * @see \web\lib\admin\view\PageElementInterface::render()
      */
     public function render() {
-        if($this->acknowledgeNotice!=null){
+        if ($this->acknowledgeNotice != null) {
             $this->acknowledgeNotice->addText(sprintf($this->acknowledgeText, $this->acknowledgeDays));
         }
         ?>
-        <div class="<?php echo self::EDITABLEBLOCK_CLASS;?>">
+        <div class="<?php echo self::EDITABLEBLOCK_CLASS; ?>">
             <?php $this->decorator->render(); ?>
         </div>
         <?php

web/lib/admin/domain/SilverbulletInvitation.php

Doc Comments +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
     
     /**
      * 
-     * @param $silverbulletUser SilverbulletUser
+     * @param SilverbulletUser|null $silverbulletUser SilverbulletUser
      */
     public function __construct($silverbulletUser) {
         parent::__construct(self::TABLE, self::TYPE_INST);

Braces +2 added lines, -2 removed lines

@@ -160,9 +160,9 @@
         
         if ($this->isRevoked()){
             $link = _('This token has been revoked!');
-        }else if($this->isAbsent()){
+        } else if($this->isAbsent()){
             $link = _('All available activations were used!');
-        }else if ($this->isExpired()) {
+        } else if ($this->isExpired()) {
             $link = _('User did not consume the token and it expired!');
         } else {
             $link .= '/accountstatus/accountstatus.php?token=' . $this->get(self::TOKEN);

Spacing +12 added lines, -12 removed lines

@@ -82,7 +82,7 @@ 
      * {@inheritDoc}
      * @see \web\lib\admin\domain\PersistentInterface::validate()
      */
-    public function validate(){
+    public function validate() {
         return (
             !empty($this->get(self::PROFILEID)) &&
             !empty($this->get(self::SILVERBULLETUSERID))
@@ -109,7 +109,7 @@ 
      * 
      * @param int $quantity
      */
-    public function setQuantity($quantity){
+    public function setQuantity($quantity) {
         $this->set(self::QUANTITY, $quantity);
     }
 
@@ -117,7 +117,7 @@ 
      *
      * @return string
      */
-    public function getRemainingActivations(){
+    public function getRemainingActivations() {
         return sprintf(_("%s of %s"), count($this->certificates), $this->get(self::QUANTITY));
     }
     
@@ -125,12 +125,12 @@ 
      * 
      * @return boolean
      */
-    public function isRevoked(){
+    public function isRevoked() {
         $quantity = $this->get(self::QUANTITY);
         return $quantity <= 0;
     }
     
-    public function isAbsent(){
+    public function isAbsent() {
         $quantity = $this->get(self::QUANTITY);
         return count($this->certificates) >= $quantity;
     }
@@ -162,7 +162,7 @@ 
             }
             $link .= $_SERVER['SERVER_NAME'];
             $relPath = dirname(dirname($_SERVER['SCRIPT_NAME']));
-            if ($relPath[strlen($relPath) -1] == '/') {
+            if ($relPath[strlen($relPath) - 1] == '/') {
                 $relPath = substr($relPath, 0, strlen($relPath) - 1);
             }
             $link = $link . $relPath;
@@ -170,11 +170,11 @@ 
             $link = $host;
         }
         
-        if ($this->isRevoked()){
+        if ($this->isRevoked()) {
             $link = _('This token has been revoked!');
-        }else if($this->isAbsent()){
+        } else if ($this->isAbsent()) {
             $link = _('All available activations were used!');
-        }else if ($this->isExpired()) {
+        } else if ($this->isExpired()) {
             $link = _('User did not consume the token and it expired!');
         } else {
             $link .= '/accountstatus/accountstatus.php?token=' . $this->get(self::TOKEN);
@@ -234,7 +234,7 @@ 
      * 
      * @return \web\lib\admin\domain\SilverbulletCertificate[]
      */
-    public function getCertificates(){
+    public function getCertificates() {
         return $this->certificates;
     }
     
@@ -243,7 +243,7 @@ 
      * {@inheritDoc}
      * @see \web\lib\admin\domain\PersistentEntity::load()
      */
-    public function load($searchAttribute = null){
+    public function load($searchAttribute = null) {
         $state = parent::load($searchAttribute);
         $this->certificates = SilverbulletCertificate::getList(null, new Attribute(SilverbulletCertificate::SILVERBULLETINVITATIONID, $this->getIdentifier(), Attribute::TYPE_INTEGER));
         return $state;
@@ -276,7 +276,7 @@ 
             $attrValue = $searchAttribute->value;
             $query = sprintf("SELECT * FROM `%s` WHERE `%s`=? AND `%s`=? ORDER BY `%s` DESC", self::TABLE, self::SILVERBULLETUSERID, $searchAttribute->key, self::EXPIRY);
             $result = $databaseHandle->exec($query, $userType . $attrType, $userValue, $attrValue);
-        } else if($silverbulletUser != null) {
+        } else if ($silverbulletUser != null) {
             $userId = $silverbulletUser->getAttribute(self::ID);
             $userType = $userId->getType();
             $userValue = $userId->value;

tests/unit/web/lib/admin/domain/SilverBulletUserTest.php

Spacing +8 added lines, -8 removed lines

@@ -5,7 +5,7 @@ 
 use web\lib\admin\domain\Attribute;
 use web\lib\admin\domain\SilverbulletInvitation;
 
-class SilverbulletUserTest extends PHPUnit_Framework_TestCase{
+class SilverbulletUserTest extends PHPUnit_Framework_TestCase {
     
     private $username = 'testusername';
     
@@ -26,7 +26,7 @@ 
         $this->newUser->setExpiry('now');
     }
     
-    public function testNewUser(){
+    public function testNewUser() {
         $this->newUser->save();
         $this->assertNotEmpty($this->newUser->getIdentifier());
         
@@ -50,7 +50,7 @@ 
         $list = SilverbulletUser::getList($this->profileId);
         $found = false;
         foreach ($list as $user) {
-            if($user->getIdentifier() == $this->newUser->getIdentifier()){
+            if ($user->getIdentifier() == $this->newUser->getIdentifier()) {
                 $found = true;
                 break;
             }
@@ -61,12 +61,12 @@ 
         $this->assertTrue($result);
     }
     
-    public function testInactiveUser(){
+    public function testInactiveUser() {
         $this->newUser->save();
         $this->assertFalse($this->newUser->hasCertificates());
     }
     
-    public function testActiveUser(){
+    public function testActiveUser() {
         $this->newUser->save();
         $invitation = new SilverbulletInvitation($this->newUser);
         $invitation->setQuantity(2);
@@ -83,10 +83,10 @@ 
         $this->assertEquals(2, count($certificates));
     }
     
-    public function testSetDeactivated(){
+    public function testSetDeactivated() {
         $serial = '29837498273948';
         $cn = 'testCommonName';
-        $expiry = date('Y-m-d',strtotime("tomorrow"));
+        $expiry = date('Y-m-d', strtotime("tomorrow"));
         
         //Testing new user deactivation
         $this->newUser->save();
@@ -130,7 +130,7 @@ 
         $this->assertEquals(SilverbulletUser::ACTIVE, $existingUser->get(SilverbulletUser::DEACTIVATION_STATUS));
     }
     
-    protected function tearDown(){
+    protected function tearDown() {
         $this->newUser->delete();
         $this->profile->delete();
     }

tests/unit/web/lib/admin/domain/SilverbulletControllerTest.php

Braces +3 added lines, -1 removed lines

@@ -15,7 +15,9 @@
 use web\lib\admin\http\SilverbulletContext;
 use web\lib\admin\domain\SilverbulletInvitation;
 
-if ( !isset( $_SESSION ) ) $_SESSION = array();
+if ( !isset( $_SESSION ) ) {
+    $_SESSION = array();
+}
 
 class MockInstitutionPageBuilder extends InstitutionPageBuilder{
     

Spacing +12 added lines, -12 removed lines

@@ -15,24 +15,24 @@ 
 use web\lib\admin\http\SilverbulletContext;
 use web\lib\admin\domain\SilverbulletInvitation;
 
-if ( !isset( $_SESSION ) ) $_SESSION = array();
+if (!isset($_SESSION)) $_SESSION = array();
 
-class MockInstitutionPageBuilder extends InstitutionPageBuilder{
+class MockInstitutionPageBuilder extends InstitutionPageBuilder {
     
     private $profile;
     
-    public function __construct($profile){
+    public function __construct($profile) {
         parent::__construct(new DefaultHtmlPage("Test Page"));
         $this->profile = $profile; 
     }
     
-    public function getProfile(){
+    public function getProfile() {
         return $this->profile;
     }
     
 }
 
-class SilverbulletControllerTest extends PHPUnit_Framework_TestCase{
+class SilverbulletControllerTest extends PHPUnit_Framework_TestCase {
     
     private $username = 'testusername';
     
@@ -44,7 +44,7 @@ 
     
     private $factory;
     
-    protected function setUp(){
+    protected function setUp() {
         $this->databaseHandle = \core\DBConnection::handle('INST');
         
         $this->profile = new MockProfileSilverbullet($this->databaseHandle);
@@ -64,7 +64,7 @@ 
         $usersAfter = count(SilverbulletUser::getList($this->profile->identifier));
         $this->assertFalse($usersAfter > $usersBefore);
         
-        $_POST[AddUserCommand::PARAM_EXPIRY] = date('Y-m-d',strtotime("tomorrow"));
+        $_POST[AddUserCommand::PARAM_EXPIRY] = date('Y-m-d', strtotime("tomorrow"));
         $this->factory->parseRequest();
 
         // Is not going to work since the $_POST variables can't be modified at runtime for filter_input function
@@ -73,10 +73,10 @@ 
         
     }
 
-    private function countActiveUsers($users){
+    private function countActiveUsers($users) {
         $count = 0;
         foreach ($users as $user) {
-            if(!$user->isDeactivated()){
+            if (!$user->isDeactivated()) {
                 $count++;
             }
         }
@@ -154,7 +154,7 @@ 
     public function testRevokeGeneratedCertificate() {
         $serial = '29837498273948';
         $cn = 'testCommonName';
-        $expiry = date('Y-m-d',strtotime("tomorrow"));
+        $expiry = date('Y-m-d', strtotime("tomorrow"));
         
         $this->user->save();
 
@@ -181,9 +181,9 @@ 
         
     }
     
-    protected function tearDown(){
+    protected function tearDown() {
         $this->user->delete();
-        $this->databaseHandle->exec("DELETE FROM `".SilverbulletUser::TABLE."` WHERE `".SilverbulletUser::USERNAME."`='".$this->username."'");
+        $this->databaseHandle->exec("DELETE FROM `" . SilverbulletUser::TABLE . "` WHERE `" . SilverbulletUser::USERNAME . "`='" . $this->username . "'");
         $this->profile->delete();
     }
 }

web/lib/admin/http/AddInvitationCommand.php

Spacing +7 added lines, -7 removed lines

@@ -8,7 +8,7 @@ 
  * @author Zilvinas Vaira
  *
  */
-class AddInvitationCommand extends AbstractInvokerCommand{
+class AddInvitationCommand extends AbstractInvokerCommand {
 
     const COMMAND = 'newinvitation';
 
@@ -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() {
         $userIndex = $this->parseInt($_POST[self::COMMAND]);
         $userIds = $this->parseArray($_POST[SaveUsersCommand::PARAM_ID]);
         $invitationsQuantities = $this->parseArray($_POST[SaveUsersCommand::PARAM_QUANTITY]);
@@ -43,14 +43,14 @@ 
         $user = SilverbulletUser::prepare($userId);
         $user->load();
         
-        if($user->isExpired()){
+        if ($user->isExpired()) {
             $this->storeErrorMessage(sprintf(_("User '%s' has expired. In order to generate credentials please extend the expiry date!"), $user->getUsername()));
-        }else{
+        } else {
             $this->context->createInvitation($user, $this, (int) $invitationsQuantity);
-            if(!is_numeric($invitationsQuantity)){
+            if (!is_numeric($invitationsQuantity)) {
                 $this->storeErrorMessage(sprintf(_("Invitations quantity '%' provided for user '%s' was not numeric. Assumed quantity as '1' !"), $invitationsQuantity, $user->getUsername()));
             }
-            if($user->isDeactivated()){
+            if ($user->isDeactivated()) {
                 $user->setDeactivated(false, $this->context->getProfile());
                 $user->save();
             }

Braces +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         
         if($user->isExpired()){
             $this->storeErrorMessage(sprintf(_("User '%s' has expired. In order to generate credentials please extend the expiry date!"), $user->getUsername()));
-        }else{
+        } else{
             $this->context->createInvitation($user, $this, (int) $invitationsQuantity);
             if(!is_numeric($invitationsQuantity)){
                 $this->storeErrorMessage(sprintf(_("Invitations quantity '%' provided for user '%s' was not numeric. Assumed quantity as '1' !"), $invitationsQuantity, $user->getUsername()));

web/skins/classic/basic.php

Spacing +3 added lines, -3 removed lines

@@ -142,7 +142,7 @@ 
     public function listIdPs() {
         $instList = $this->orderIdentityProviders($this->country->identifier);
         $out = '';
-        $out .= sprintf(_("Select your %s"), $this->nomenclature_inst );
+        $out .= sprintf(_("Select your %s"), $this->nomenclature_inst);
         $out .= '<select name="idp" onchange="submit_form(this)">';
         if (!empty($instList)) {
             if (!isset($this->idp)) {
@@ -236,7 +236,7 @@ 
                 }
             }
             $out .= '>' . $disp . '</option>';
-            $deviceRedirects .= 'redirects[' . $iterator . '] = ' . ( $oneDevice['redirect'] ? 1 : 0 ) . ';';
+            $deviceRedirects .= 'redirects[' . $iterator . '] = ' . ($oneDevice['redirect'] ? 1 : 0) . ';';
             $iterator++;
         }
         $out .= '</select>';
@@ -430,7 +430,7 @@ 
         </script>
     </head>
     <body style="">
-        <?php print '<div id="motd">' . ( isset(CONFIG['APPEARANCE']['MOTD']) ? CONFIG['APPEARANCE']['MOTD'] : '&nbsp' ) . '</div>'; ?>
+        <?php print '<div id="motd">' . (isset(CONFIG['APPEARANCE']['MOTD']) ? CONFIG['APPEARANCE']['MOTD'] : '&nbsp') . '</div>'; ?>
         <form name="my_form" method="POST" action="<?php echo $_SERVER['SCRIPT_NAME'] ?>" accept-charset='UTF-8'>
             <img src="<?php echo $skinObject->findResourceUrl("IMAGES", "consortium_logo.png"); ?>" style="width: 20%; padding-right:20px; padding-top:0px; float:right" alt="logo" />
             <?php

web/skins/classic/user/js/cat_js.php

Indentation +9 added lines, -9 removed lines

@@ -11,7 +11,7 @@ 
 <?php 
 $cat = new core\CAT();
 function escaped_echo($s) {
-   echo preg_replace('/"/','&quot;',$s);
+    echo preg_replace('/"/','&quot;',$s);
 }
 
 $langObject = new \core\common\Language();
@@ -24,7 +24,7 @@ 
 if (! is_numeric($profileId)) {
     exit;
 }
- ?>
+    ?>
 var n;
 var profile;
 var generateTimer;
@@ -359,7 +359,7 @@ 
 }
 
 <?php if($idpId) { 
-      print "front_page = 0;\n";
+        print "front_page = 0;\n";
 } ?>
 
 function showTOU(){
@@ -493,7 +493,7 @@ 
 if ($profileId) {
     print "listProfiles($idpId,$profileId);";
 }
- ?>
+    ?>
 
 $(".signin").click(function(event){
      pressedButton = $(this);
@@ -554,12 +554,12 @@ 
         });
 DiscoJuice.Constants.Countries = {
 <?php 
-  $C = $Gui->printCountryList(1);
-     $ret = '';
-   foreach ($C as $key => $val) {
+    $C = $Gui->printCountryList(1);
+        $ret = '';
+    foreach ($C as $key => $val) {
     $ret .= "'$key': \"$val\",";
-   }
-   echo substr($ret, 0, -1);
+    }
+    echo substr($ret, 0, -1);
 ?>
         };
 

Spacing +27 added lines, -27 removed lines

@@ -11,17 +11,17 @@ 
 <?php 
 $cat = new core\CAT();
 function escaped_echo($s) {
-   echo preg_replace('/"/','&quot;',$s);
+   echo preg_replace('/"/', '&quot;', $s);
 }
 
 $langObject = new \core\common\Language();
 $langObject->setTextDomain('web_user');
 $idpId = empty($_REQUEST['idp']) ? 0 : $_REQUEST['idp'];
-if (! is_numeric($idpId)) {
+if (!is_numeric($idpId)) {
     exit;
 }
 $profileId = empty($_REQUEST['profile']) ? 0 : $_REQUEST['profile'];
-if (! is_numeric($profileId)) {
+if (!is_numeric($profileId)) {
     exit;
 }
  ?>
@@ -76,7 +76,7 @@ 
     result = j.status;
     if(! result) {
       alert("<?php escaped_echo(_("no matching data found"))?>");
-      document.location.href='<?php echo rtrim(dirname($_SERVER['SCRIPT_NAME']),'/').'/'?>';
+      document.location.href='<?php echo rtrim(dirname($_SERVER['SCRIPT_NAME']), '/') . '/'?>';
     }
     j = j.data;
     n = j.length;
@@ -84,7 +84,7 @@ 
     inst_name = j[0].idp_name;
     logo = j[0].logo;
     $("#inst_name").val(inst_name);
-    $("#inst_name_span").html("<?php escaped_echo(sprintf(_("Selected %s:"),$cat->nomenclature_inst))?> <strong>"+inst_name+"</strong>");
+    $("#inst_name_span").html("<?php escaped_echo(sprintf(_("Selected %s:"), $cat->nomenclature_inst))?> <strong>"+inst_name+"</strong>");
     $(".inst_name").text(inst_name);
     $("#user_page").show();
     $("#institution_name").show();
@@ -197,8 +197,8 @@ 
        j1 = $.parseJSON(data);
        result = j1.status;
        if(! result) {
-            alert("<?php escaped_echo( _("no matching data found"))?>");
-            document.location.href='<?php echo rtrim(dirname($_SERVER['SCRIPT_NAME']),'/').'/'?>';
+            alert("<?php escaped_echo(_("no matching data found"))?>");
+            document.location.href='<?php echo rtrim(dirname($_SERVER['SCRIPT_NAME']), '/') . '/'?>';
        }
        j = j1.data;
        if(j.description !== undefined && j.description) {
@@ -209,15 +209,15 @@ 
          $("#profile_desc").text('');
        }
        if(j.local_url !== undefined && j.local_url) 
-         txt = txt+'<tr><td><?php escaped_echo(_("WWW:"));?></td><td><a href="'+j.local_url+'" target="_blank">'+j.local_url+'</a></td></tr>';
+         txt = txt+'<tr><td><?php escaped_echo(_("WWW:")); ?></td><td><a href="'+j.local_url+'" target="_blank">'+j.local_url+'</a></td></tr>';
        if(j.local_email !== undefined && j.local_email) 
-         txt = txt+'<tr><td><?php escaped_echo(_("email:"));?></td><td><a href=mailto:"'+j.local_email+'">'+j.local_email+'</a></td></tr>';
+         txt = txt+'<tr><td><?php escaped_echo(_("email:")); ?></td><td><a href=mailto:"'+j.local_email+'">'+j.local_email+'</a></td></tr>';
        if(j.local_phone !== undefined && j.local_phone) 
-         txt = txt+'<tr><td><?php escaped_echo(_("tel:"));?></td><td>'+j.local_phone+'</td></tr>';
+         txt = txt+'<tr><td><?php escaped_echo(_("tel:")); ?></td><td>'+j.local_phone+'</td></tr>';
        if(txt) 
-         txt = "<table><tr><th colspan='2'><?php escaped_echo(sprintf(_("If you encounter problems, then you can obtain direct assistance from your %s at:"),$cat->nomenclature_inst)); ?></th></tr>"+txt+'</table>';
+         txt = "<table><tr><th colspan='2'><?php escaped_echo(sprintf(_("If you encounter problems, then you can obtain direct assistance from your %s at:"), $cat->nomenclature_inst)); ?></th></tr>"+txt+'</table>';
         else 
-         txt = "<table><tr><th colspan='2'><?php escaped_echo(sprintf(_("If you encounter problems you should ask for help at your %s"),$cat->nomenclature_inst)); ?>.</th></tr></table>";
+         txt = "<table><tr><th colspan='2'><?php escaped_echo(sprintf(_("If you encounter problems you should ask for help at your %s"), $cat->nomenclature_inst)); ?>.</th></tr></table>";
       $("#user_info").html(txt);
       $("#user_info").show();
       if(j.silverbullet) {
@@ -235,7 +235,7 @@ 
           $("#g_"+v.id).addClass('alertButton');
           $("#cross_icon_"+v.id).show();
           $("#"+v.id).addClass('disabledDevice');
-          $("#download_button_header_"+v.id).html("<?php escaped_echo(sprintf(_("This device cannot be configured with settings provided by your %s"),$cat->nomenclature_inst))?>");
+          $("#download_button_header_"+v.id).html("<?php escaped_echo(sprintf(_("This device cannot be configured with settings provided by your %s"), $cat->nomenclature_inst))?>");
           $("#info_b_"+v.id+",#g_info_b_"+v.id).hide();
         } else  {
           if(v.status == -1)
@@ -247,7 +247,7 @@ 
           $("#"+v.id+",#g_"+v.id).addClass('additionalInfo');
           $("#"+v.id+",#g_"+v.id).click(function(event){
             i_div = $("#info_"+$(this).attr('id'));
-            t = "<?php escaped_echo(_("Your site administrator has specified that this device should be configured with resources located on a local page. When you click <b>Continue</b> this page will be opened in a new window/tab."))?>"+"<br><span class='redirect_link'><a href='"+v.redirect+"' target='_blank'><?php escaped_echo(_("Continue"));?></a></span>";
+            t = "<?php escaped_echo(_("Your site administrator has specified that this device should be configured with resources located on a local page. When you click <b>Continue</b> this page will be opened in a new window/tab."))?>"+"<br><span class='redirect_link'><a href='"+v.redirect+"' target='_blank'><?php escaped_echo(_("Continue")); ?></a></span>";
             i_div.html(t);
             $(".redirect_link").click(function(event) {
                i_div.hide();
@@ -255,13 +255,13 @@ 
                
           });
         } else if(v.device_customtext != '0' || v.eap_customtext != '0' || v.message != '0' || v.status > 0) {
-          var continue_text = "<?php escaped_echo(_("Continue"));?>";
+          var continue_text = "<?php escaped_echo(_("Continue")); ?>";
           $("#"+v.id+",#g_"+v.id).addClass('additionalInfo');
           $("#"+v.id+",#g_"+v.id).click(function(event){
             i_div = $("#info_"+$(this).attr('id'));
             if(v.status > 0) {
-              t = "<?php escaped_echo(sprintf(_("This device cannot be configured with settings provided by your %s"),$cat->nomenclature_inst))?>";
-              continue_text = "<?php escaped_echo(_("Close"));?>";
+              t = "<?php escaped_echo(sprintf(_("This device cannot be configured with settings provided by your %s"), $cat->nomenclature_inst))?>";
+              continue_text = "<?php escaped_echo(_("Close")); ?>";
             } else {
             t = i_div.html();
             if(v.message != '0') {
@@ -318,7 +318,7 @@ 
   }
 
   function infoCAT(k,title) {
-      $.post('<?php echo $Gui->skinObject->findResourceUrl("BASE", "user/cat_info.php");?>', {page: k, lang: lang}, function(data) {
+      $.post('<?php echo $Gui->skinObject->findResourceUrl("BASE", "user/cat_info.php"); ?>', {page: k, lang: lang}, function(data) {
     if(data.substring(0,8) == 'no_title') {
        data = data.substring(8,data.length);
     } else {
@@ -337,7 +337,7 @@ 
    $("#loading_ico").css('left',x+'px');
    $("#loading_ico").attr('src','resources/images/icons/loading9.gif');
    $("#loading_ico").show();
-   window.location.replace("<?php echo $Gui->skinObject->findResourceUrl("BASE","admin/overview_user.php");?>?lang="+lang);
+   window.location.replace("<?php echo $Gui->skinObject->findResourceUrl("BASE", "admin/overview_user.php"); ?>?lang="+lang);
 }
 
 /* Get horizontal center of the Browser Window */
@@ -358,7 +358,7 @@ 
     return(Math.round(windowWidth/2));
 }
 
-<?php if($idpId) { 
+<?php if ($idpId) { 
       print "front_page = 0;\n";
 } ?>
 
@@ -396,10 +396,10 @@ 
     $("#download_info a").attr('href',download_link);
     $('#download_info').show();
     if( generateTimer > 0 ) {
-       setTimeout("document.location.href='<?php echo rtrim(dirname($_SERVER['SCRIPT_NAME']),'/')?>'+'/'+download_link",generateTimer);
+       setTimeout("document.location.href='<?php echo rtrim(dirname($_SERVER['SCRIPT_NAME']), '/')?>'+'/'+download_link",generateTimer);
     }
     else {
-       document.location.href='<?php echo rtrim(dirname($_SERVER['SCRIPT_NAME']),'/')?>'+'/'+download_link;
+       document.location.href='<?php echo rtrim(dirname($_SERVER['SCRIPT_NAME']), '/')?>'+'/'+download_link;
     }
   }
 }
@@ -517,13 +517,13 @@ 
    "overlay":true,"cookie":true,"type":false,
    "country":true,"location":true,
    "title":"<?php escaped_echo($cat->nomenclature_inst) ?>",
-   "subtitle":"<?php escaped_echo(sprintf(_("Select your <strong>%s<\/strong>"),$cat->nomenclature_inst)) ?>",
-   "textHelp": "<?php escaped_echo(sprintf(_("Help, my %s is not on the list"),$cat->nomenclature_inst)) ?>",
-   "textHelpMore": "<?php escaped_echo(sprintf(_("This system relies on information supplied by local %s administrators. If your %s is not on the list, then nag them to add information to the %s database."),CONFIG_CONFASSISTANT['CONSORTIUM']['display_name'], $cat->nomenclature_inst, CONFIG['APPEARANCE']['productname'])); ?>",
+   "subtitle":"<?php escaped_echo(sprintf(_("Select your <strong>%s<\/strong>"), $cat->nomenclature_inst)) ?>",
+   "textHelp": "<?php escaped_echo(sprintf(_("Help, my %s is not on the list"), $cat->nomenclature_inst)) ?>",
+   "textHelpMore": "<?php escaped_echo(sprintf(_("This system relies on information supplied by local %s administrators. If your %s is not on the list, then nag them to add information to the %s database."), CONFIG_CONFASSISTANT['CONSORTIUM']['display_name'], $cat->nomenclature_inst, CONFIG['APPEARANCE']['productname'])); ?>",
    "textLocateMe": "<?php escaped_echo(_("Locate me more accurately using HTML5 Geo-Location")) ?>",
-   "textShowProviders": "<?php escaped_echo(sprintf(_("Show %ss in"),$cat->nomenclature_inst)) ?>",
+   "textShowProviders": "<?php escaped_echo(sprintf(_("Show %ss in"), $cat->nomenclature_inst)) ?>",
    "textAllCountries": "<?php escaped_echo(_("all countries")) ?>",
-   "textSearch" : "<?php escaped_echo(sprintf(_("or search for an %s, in example Univerity of Oslo"),$cat->nomenclature_inst)) ?>",
+   "textSearch" : "<?php escaped_echo(sprintf(_("or search for an %s, in example Univerity of Oslo"), $cat->nomenclature_inst)) ?>",
    "textShowAllCountries": "<?php escaped_echo(_("show all countries")) ?>",
    "textLimited1" : "<?php escaped_echo(_("Results limited to"))?>",
    "textLimited2" : "<?php escaped_echo(_("entries - show more"))?>",