CorWatts / fasterscale

site/tests/unit/models/DeleteAccountFormTest.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
 
     public function testDeleteAccount()
     {
-        $this->specify('deleteAccount() should return false if the form does not validate', function () {
+        $this->specify('deleteAccount() should return false if the form does not validate', function() {
             $user = $this->getUser();
             $user
         ->expects($this->never())
@@ -37,7 +37,7 @@ 
             $this->assertFalse($form->deleteAccount(), 'deleteAccount should not be successful');
         });
 
-        $this->specify('deleteAccount() should return false if the user\'s password is incorrect', function () {
+        $this->specify('deleteAccount() should return false if the user\'s password is incorrect', function() {
             $user = $this->getUser();
             $user
         ->expects($this->never())
@@ -48,7 +48,7 @@ 
             $this->assertFalse($form->deleteAccount(), 'deleteAccount should not be successful');
         });
 
-        $this->specify('deleteAccount() should return true, send a notification email, and delete the user when the form validates and the user\'s password is correct', function () {
+        $this->specify('deleteAccount() should return true, send a notification email, and delete the user when the form validates and the user\'s password is correct', function() {
             $password = 'password';
             $user = $this->getUser();
             $user->email = 'user@example.com';

site/tests/unit/models/CheckinFormTest.php

Spacing +19 added lines, -19 removed lines

@@ -118,7 +118,7 @@ 
     {
         $this->container = new \yii\di\Container;
         $this->container->set('common\interfaces\UserBehaviorInterface', '\site\tests\_support\MockUserBehavior');
-        $this->container->set('common\interfaces\TimeInterface', function () {
+        $this->container->set('common\interfaces\TimeInterface', function() {
             return new \common\components\Time('America/Los_Angeles');
         });
         parent::setUp();
@@ -126,7 +126,7 @@ 
 
     public function testAttributeLabels()
     {
-        $this->specify('attributeLabels should function properly', function () {
+        $this->specify('attributeLabels should function properly', function() {
             $model = $this->container->get('\site\models\CheckinForm');
             expect('attributeLabels should be correct', $this->assertEquals([
             'behaviors1' => 'Restoration',
@@ -142,32 +142,32 @@ 
 
     public function testSetBehaviors()
     {
-        $this->specify('setBehaviors should function properly', function () {
+        $this->specify('setBehaviors should function properly', function() {
             $model = $this->container->get('\site\models\CheckinForm');
 
             $model->setBehaviors($this->behaviors);
-            expect('behaviors1 should be correct', $this->assertEquals([ 0 => 7 ], $model->behaviors1));
-            expect('behaviors2 should be correct', $this->assertEquals([ 0 => 12, 1 => 13, 2 => 17, 3 => 18 ], $model->behaviors2));
-            expect('behaviors3 should be correct', $this->assertEquals([ 0 => 28, 1 => 38, 2 => 46 ], $model->behaviors3));
-            expect('behaviors4 should be correct', $this->assertEquals([ 0 => 47, 1 => 56, 2 => 62 ], $model->behaviors4));
-            expect('behaviors5 should be correct', $this->assertEquals([ 0 => 78, 1 => 79 ], $model->behaviors5));
-            expect('behaviors6 should be correct', $this->assertEquals([ 0 => 104 ], $model->behaviors6));
-            expect('behaviors7 should be correct', $this->assertEquals([ 0 => 128 ], $model->behaviors7));
+            expect('behaviors1 should be correct', $this->assertEquals([0 => 7], $model->behaviors1));
+            expect('behaviors2 should be correct', $this->assertEquals([0 => 12, 1 => 13, 2 => 17, 3 => 18], $model->behaviors2));
+            expect('behaviors3 should be correct', $this->assertEquals([0 => 28, 1 => 38, 2 => 46], $model->behaviors3));
+            expect('behaviors4 should be correct', $this->assertEquals([0 => 47, 1 => 56, 2 => 62], $model->behaviors4));
+            expect('behaviors5 should be correct', $this->assertEquals([0 => 78, 1 => 79], $model->behaviors5));
+            expect('behaviors6 should be correct', $this->assertEquals([0 => 104], $model->behaviors6));
+            expect('behaviors7 should be correct', $this->assertEquals([0 => 128], $model->behaviors7));
 
             $model->setBehaviors($this->behaviors);
-            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([ 0 => 7 ], $model->behaviors1));
-            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([ 0 => 12, 1 => 13, 2 => 17, 3 => 18 ], $model->behaviors2));
-            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([ 0 => 28, 1 => 38, 2 => 46 ], $model->behaviors3));
-            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([ 0 => 47, 1 => 56, 2 => 62 ], $model->behaviors4));
-            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([ 0 => 78, 1 => 79 ], $model->behaviors5));
-            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([ 0 => 104 ], $model->behaviors6));
-            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([ 0 => 128 ], $model->behaviors7));
+            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([0 => 7], $model->behaviors1));
+            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([0 => 12, 1 => 13, 2 => 17, 3 => 18], $model->behaviors2));
+            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([0 => 28, 1 => 38, 2 => 46], $model->behaviors3));
+            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([0 => 47, 1 => 56, 2 => 62], $model->behaviors4));
+            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([0 => 78, 1 => 79], $model->behaviors5));
+            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([0 => 104], $model->behaviors6));
+            expect('setBehaviors should not append behaviors to existing ones', $this->assertEquals([0 => 128], $model->behaviors7));
         });
     }
 
     public function testValidateBehaviors()
     {
-        $this->specify('validateBehaviors should function properly', function () {
+        $this->specify('validateBehaviors should function properly', function() {
             $model = $this->container->get('\site\models\CheckinForm');
             $model->setBehaviors($this->behaviors);
             expect('validation should be good', $this->assertTrue($model->validate()));
@@ -215,7 +215,7 @@ 
 
         $model = $this->container->get('\site\models\CheckinForm');
         $model->custom_behaviors = [];
-        $model->compiled_behaviors = [ 1, 2, 3];
+        $model->compiled_behaviors = [1, 2, 3];
         expect('to return an empty array if no custom behaviors are set', $this->assertEquals([], $model->getCustomBehaviors()));
     }
 

site/tests/unit/models/SignupFormTest.php

Spacing +9 added lines, -9 removed lines

@@ -33,13 +33,13 @@ 
         $this->container = new \yii\di\Container;
         $this->container->set('common\interfaces\UserInterface', '\site\tests\_support\MockUser');
         $this->container->set('common\interfaces\UserBehaviorInterface', '\site\tests\_support\MockUserBehavior');
-        $this->container->set('common\interfaces\TimeInterface', function () {
+        $this->container->set('common\interfaces\TimeInterface', function() {
             return new \common\components\Time('America/Los_Angeles');
         });
 
         // instantiate mock objects
         $user_behavior = $this->container->get(\common\interfaces\UserBehaviorInterface::class);
-        $time        = $this->container->get(\common\interfaces\TimeInterface::class);
+        $time = $this->container->get(\common\interfaces\TimeInterface::class);
 
         $question = $this->getMockBuilder(\common\models\Question::class)
       ->setMethods(['save', 'attributes'])
@@ -111,7 +111,7 @@ 
 
     // set up a side effect to occur when the user's save() function is called
         $saved = false;
-        $this->user->method('save')->will($this->returnCallback(function ($runValidation=true, $attributeNames=null) use (&$saved) {
+        $this->user->method('save')->will($this->returnCallback(function($runValidation = true, $attributeNames = null) use (&$saved) {
             $saved = true;
             return $this->user;
         }));
@@ -120,7 +120,7 @@ 
 
         /////////////////////////////////////////////////////////////////////
 
-        $this->specify('a brand-new user should be able to sign up', function () use ($form, &$saved) {
+        $this->specify('a brand-new user should be able to sign up', function() use ($form, &$saved) {
             expect('the form should validate correctly', $this->assertTrue($form->validate()));
             $user = $form->signup();
             expect(
@@ -146,7 +146,7 @@ 
     public function testSignupExistingConfirmedUser()
     {
         $saved = false;
-        $this->existing_user->method('save')->will($this->returnCallback(function ($runValidation=true, $attributeNames=null) use (&$saved) {
+        $this->existing_user->method('save')->will($this->returnCallback(function($runValidation = true, $attributeNames = null) use (&$saved) {
             $saved = true;
             return $this->existing_user;
         }));
@@ -155,7 +155,7 @@ 
         $this->existing_user->verify_email_token = $this->getValidToken() . "_confirmed";
         $form2 = $this->getForm();
 
-        $this->specify('an existing and confirmed user should not be signed up', function () use ($form2, &$saved) {
+        $this->specify('an existing and confirmed user should not be signed up', function() use ($form2, &$saved) {
             expect('the form should validate correctly', $this->assertTrue($form2->validate()));
             $user = $form2->signup();
             expect('when the user exists and is confirmed, signup() should return null', $this->assertNull($user));
@@ -166,7 +166,7 @@ 
     public function testSignupExistingUnconfirmedUser()
     {
         $saved = false;
-        $this->existing_user->method('save')->will($this->returnCallback(function ($runValidation=true, $attributeNames=null) use (&$saved) {
+        $this->existing_user->method('save')->will($this->returnCallback(function($runValidation = true, $attributeNames = null) use (&$saved) {
             $saved = true;
             return $this->existing_user;
         }));
@@ -179,7 +179,7 @@ 
         $this->user->method('findByEmail')->willReturn($this->existing_user);
         $form3 = $this->getForm();
 
-        $this->specify('an existing and unconfirmed user should be able to sign up again, but their information will be UPDATED', function () use ($form3, &$saved) {
+        $this->specify('an existing and unconfirmed user should be able to sign up again, but their information will be UPDATED', function() use ($form3, &$saved) {
             expect('the form should validate correctly', $this->assertTrue($form3->validate()));
             $user = $form3->signup();
             expect('This user should not be an isNewRecord -- they should already exist', $this->assertFalse($user->getIsNewRecord()));
@@ -199,7 +199,7 @@ 
         $form = $this->setAttrs($form, $this->vals);
 
         $result = $form->setFields($this->user);
-        $this->specify('populate should set the values of the user model from the form', function () use ($result) {
+        $this->specify('populate should set the values of the user model from the form', function() use ($result) {
             foreach ($this->whitelist as $attr) {
                 $this->assertEquals($this->vals[$attr], $result->$attr);
             }

site/views/site/signup.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
                 'id' => 'form-signup',
                 'enableClientValidation' => true,
         'enableAjaxValidation' => false,
-        'options' => [ 'validateOnSubmit' => true ]
+        'options' => ['validateOnSubmit' => true]
             ]); ?>
         <?= $form->field($model, 'email', ['inputTemplate' => '<div class="input-group"><span class="input-group-addon">@</span>{input}</div>'])->input('email') ?>
         <?= $form->field($model, 'password', ['inputTemplate' => '<div class="input-group">{input}<span class="input-group-btn"><button id="password-toggle" class="btn btn-default" type="button">Show</button></span></div>'])->passwordInput() ?>

site/views/layouts/main.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 site\assets\AppAsset::register($this);
 
 if ($hash = Utility::getRevHash()) {
-    $rev_link = '<a href="'.Utility::getGithubRevUrl().'">'.Utility::getRevHash().'</a>';
+    $rev_link = '<a href="' . Utility::getGithubRevUrl() . '">' . Utility::getRevHash() . '</a>';
 } else {
     $rev_link = 'DEVELOPMENT';
 }

site/views/checkin/view.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
         foreach ($behavior_questions['answers'] as $question) {
             print "<div class='col-md-4'>";
             print "<p><strong>{$question['title']}</strong></p>";
-            print "<p>".Html::encode($question['answer'])."</p>";
+            print "<p>" . Html::encode($question['answer']) . "</p>";
             print "</div>";
         }
         print "</div></div>";

site/views/checkin/index.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     $behaviors = AH::map($behaviorsList[$category['id']], 'id', 'name');
     $custom = [];
     if (array_key_exists($category['id'], $customList)) {
-        $custom = AH::map($customList[$category['id']], function ($cbhvr) {
+        $custom = AH::map($customList[$category['id']], function($cbhvr) {
             return $cbhvr['id'] . '-custom';
         }, 'name');
     }

site/views/checkin/report.php

Spacing +4 added lines, -4 removed lines

@@ -28,10 +28,10 @@
               </tr>
   <?php foreach ($top_behaviors as $key => $row) {
     $num = $key + 1;
-    print "<tr>".
-    "<td>{$row['count']}</td>".
-    "<td>{$row['behavior']['name']}</td>".
-    "<td>{$row['category']['name']}</td>".
+    print "<tr>" .
+    "<td>{$row['count']}</td>" .
+    "<td>{$row['behavior']['name']}</td>" .
+    "<td>{$row['category']['name']}</td>" .
     "</tr>";
 }
   ?>

site/views/checkin/questions.php

Indentation +3 added lines, -3 removed lines

@@ -14,9 +14,9 @@
         $name,
         $checked,
         ['value' => $value,
-     'label' => $label,
-     'container' => false,
-     'labelOptions' => [ 'class' => $checked
+      'label' => $label,
+      'container' => false,
+      'labelOptions' => [ 'class' => $checked
                                      ? 'btn btn-info active'
                                      : 'btn btn-info'],
     ]

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
         ['value' => $value,
      'label' => $label,
      'container' => false,
-     'labelOptions' => [ 'class' => $checked
+     'labelOptions' => ['class' => $checked
                                      ? 'btn btn-info active'
                                      : 'btn btn-info'],
     ]