CorWatts / fasterscale

console/migrations/m140609_010314_populate_category_and_options_table.php

Indentation +1 added lines, -1 removed lines

@@ -152,7 +152,7 @@
             ["condemnation", 7],
             ["guilt", 7],
             ["aloneness", 7]
-       ]);
+        ]);
     }
 
     public function safeDown()

console/migrations/m160403_060352_delete_incorrect_behaviors.php

Indentation +1 added lines, -1 removed lines

@@ -15,6 +15,6 @@
         $this->batchInsert("{{%option}}", ["name", "category_id"], [
             ["masturbation", 3]
             , ["pornography", 3]
-       ]);
+        ]);
     }
 }

site/models/CheckinForm.php

Spacing +5 added lines, -5 removed lines

@@ -110,7 +110,7 @@ 
         }
 
         $self = $this;
-        array_walk($clean_behaviors, function ($bhvr) use (&$self) {
+        array_walk($clean_behaviors, function($bhvr) use (&$self) {
             if (preg_match('/.*-custom/', $bhvr)) {
                 $self->custom_behaviors[] = $bhvr;
             } else {
@@ -141,8 +141,8 @@ 
         );
 
         // delete cached behaviors
-        array_map(function ($period) use ($time) {
-            $key = "checkins_".Yii::$app->user->id."_{$period}_".$time->getLocalDate();
+        array_map(function($period) use ($time) {
+            $key = "checkins_" . Yii::$app->user->id . "_{$period}_" . $time->getLocalDate();
             Yii::$app->cache->delete($key);
         }, [30, 90, 180]);
     }
@@ -156,7 +156,7 @@ 
     public function mergeWithDefault(array $user_behaviors)
     {
         $behaviors = AH::index(Yii::$container->get(BehaviorInterface::class)::$behaviors, 'name', "category_id");
-        array_walk($behaviors, function (&$bhvrs, $cat_id) use ($user_behaviors) {
+        array_walk($behaviors, function(&$bhvrs, $cat_id) use ($user_behaviors) {
             if (array_key_exists($cat_id, $user_behaviors)) {
                 $bhvrs = AH::merge($bhvrs, $user_behaviors[$cat_id]);
             }
@@ -175,7 +175,7 @@ 
      */
     public function getCustomBehaviors()
     {
-        return array_map(function ($cs) {
+        return array_map(function($cs) {
             return (int)explode('-', $cs)[0];
         }, $this->custom_behaviors);
     }

site/models/EditProfileForm.php

Indentation +3 added lines, -3 removed lines

@@ -64,8 +64,8 @@ 
     }
 
     /**
-       * @codeCoverageIgnore
-       */
+     * @codeCoverageIgnore
+     */
     public function attributeLabels()
     {
         return [
@@ -99,7 +99,7 @@ 
 
                 // generate behaviors graph image
                 $checkins_last_month = (Yii::$container->get(\common\interfaces\UserBehaviorInterface::class))
-                                               ->getCheckInBreakdown();
+                                                ->getCheckInBreakdown();
 
                 // if they haven't done a check-in in the last month this will explode
                 // because $checkins_last_month is an empty array

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
       ['email_category', 'in', 'range'=>array_keys($this->categories)],
       [['partner_email1', 'partner_email2', 'partner_email3'], 'email'],
       [['partner_email1', 'email_category'], 'required',
-        'when' => function ($model) {
+        'when' => function($model) {
             return $model->send_email;
         },
         'message' => "If you've elected to send email reports, at least one partner email must be set.",

site/models/SignupForm.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
       [
         ['partner_email1', 'email_category'],
         'required',
-        'when' => function ($model) {
+        'when' => function($model) {
             return $model->send_email;
         },
         'message' => "If you've elected to send email reports, at least one partner email must be set.",

site/models/QuestionForm.php

Indentation +6 added lines, -6 removed lines

@@ -163,12 +163,12 @@
             foreach ($this->behaviorToAnswers($behavior_id) as $answer_letter => $answer) {
                 $question_id = \common\models\Question::$TYPES[$answer_letter];
                 array_push($answers, [
-                               'behavior_id'    => $user_bhvr->behavior_id,
-                               'category_id'    => $user_bhvr->category_id,
-                               'user_bhvr_id' => $user_bhvr->id,
-                               'question_id'  => $question_id,
-                               'answer'       => $answer,
-                             ]);
+                                'behavior_id'    => $user_bhvr->behavior_id,
+                                'category_id'    => $user_bhvr->category_id,
+                                'user_bhvr_id' => $user_bhvr->id,
+                                'question_id'  => $question_id,
+                                'answer'       => $answer,
+                              ]);
             }
         }
         return $answers;

Spacing +7 added lines, -7 removed lines

@@ -55,7 +55,7 @@ 
         $config = []
     )
     {
-        $this->question    = $question;
+        $this->question = $question;
         parent::__construct($config);
     }
 
@@ -73,7 +73,7 @@ 
         'whenClient' => "function(attribute, value) { return false; }", // lame, but acceptable
         "message" => "You must select a behavior your responses apply to."],
 
-      [['answer_1a','answer_1b','answer_1c','answer_2a','answer_2b','answer_2c','answer_3a','answer_3b','answer_3c','answer_4a','answer_4b','answer_4c','answer_5a', 'answer_5b','answer_5c','answer_6a', 'answer_6b','answer_6c','answer_7a','answer_7b','answer_7c'], 'safe']
+      [['answer_1a', 'answer_1b', 'answer_1c', 'answer_2a', 'answer_2b', 'answer_2c', 'answer_3a', 'answer_3b', 'answer_3c', 'answer_4a', 'answer_4b', 'answer_4c', 'answer_5a', 'answer_5b', 'answer_5c', 'answer_6a', 'answer_6b', 'answer_6c', 'answer_7a', 'answer_7b', 'answer_7c'], 'safe']
     ];
     }
 
@@ -92,8 +92,8 @@ 
 
     public function getBhvrValidator()
     {
-        return function ($model, $attr) {
-            $attrNum = $attr[strlen($attr)-1];
+        return function($model, $attr) {
+            $attrNum = $attr[strlen($attr) - 1];
             foreach (['a', 'b', 'c'] as $l) {
                 $attr = "answer_{$attrNum}{$l}";
                 if ($model->$attr) {
@@ -124,10 +124,10 @@ 
     public function behaviorToAnswers($bhvr_number)
     {
         $prop_names   = array_keys($this->getPrefixProps("answer_$bhvr_number"));
-        $prop_letters = array_map(function ($n) {
+        $prop_letters = array_map(function($n) {
             return substr($n, -1, 1);
         }, $prop_names);
-        $answers = array_map(function ($n) {
+        $answers = array_map(function($n) {
             return $this->$n;
         }, $prop_names);
  
@@ -136,7 +136,7 @@ 
 
     public function getPrefixProps($prefix)
     {
-        return array_filter(get_object_vars($this), function ($v, $k) use ($prefix) {
+        return array_filter(get_object_vars($this), function($v, $k) use ($prefix) {
             if (strpos($k, $prefix) === 0 && strlen($v)) {
                 return true;
             }

site/models/ContactForm.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
       ['email', 'filter', 'filter' => 'strtolower'],
       ['email', 'email'],
       // verifyCode needs to be entered correctly
-      ['verifyCode', 'captcha', 'when' => function () {
+      ['verifyCode', 'captcha', 'when' => function() {
           return Yii::$app->user->isGuest;
       }, 'skipOnEmpty' => !!YII_ENV_TEST],
     ];
@@ -57,7 +57,7 @@ 
         return Yii::$app->mailer->compose()
       ->setTo($email)
       ->setFrom([$this->email => $this->name])
-      ->setSubject("[FSA Contact] ".$this->subject)
+      ->setSubject("[FSA Contact] " . $this->subject)
       ->setTextBody($this->body)
       ->send();
     }

site/tests/unit/models/QuestionFormTest.php

Indentation +4 added lines, -4 removed lines

@@ -101,10 +101,10 @@
             $model->answer_2c = 'answering question c';
 
             expect('behaviorToAnswers should return the answer properties related to the behavior number supplied', $this->assertEquals($model->behaviorToAnswers(1), [
-                                                               'a' => 'answering question a',
-                                                               'b' => 'answering question b',
-                                                               'c' => 'answering question c'
-                                                             ]));
+                                                                'a' => 'answering question a',
+                                                                'b' => 'answering question b',
+                                                                'c' => 'answering question c'
+                                                              ]));
 
             expect('behaviorToAnswers should return the the empty set when there are no answers associated with the supplied behavior number', $this->assertEmpty($model->behaviorToAnswers(7)));
         });

Spacing +7 added lines, -7 removed lines

@@ -18,14 +18,14 @@ 
         $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\QuestionInterface', '\site\tests\_support\MockQuestion');
-        $this->container->set('common\interfaces\TimeInterface', function () {
+        $this->container->set('common\interfaces\TimeInterface', function() {
             return new \common\components\Time('America/Los_Angeles');
         });
     }
 
     public function testAttributeLabels()
     {
-        $this->specify('setBehaviors should function properly', function () {
+        $this->specify('setBehaviors should function properly', function() {
             $model = $this->container->get('\site\models\QuestionForm');
             expect('attributeLabels should be correct', $this->assertEquals($model->attributeLabels(), [
         'user_behavior_id1' => 'Restoration',
@@ -44,7 +44,7 @@ 
         $model = $this->container->get('\site\models\QuestionForm');
         $validator = $model->getBhvrValidator();
 
-        $this->specify('getBhvrValidator should function properly', function () use ($model, $validator) {
+        $this->specify('getBhvrValidator should function properly', function() use ($model, $validator) {
             expect('getBhvrValidator should return false when nothing is set on the form', $this->assertFalse($validator($model, "user_behavior_id1")));
             expect('getBhvrValidator should return false when nothing is set on the form', $this->assertFalse($validator($model, "user_behavior_id7")));
 
@@ -65,7 +65,7 @@ 
     {
         $model = $this->container->get('\site\models\QuestionForm');
 
-        $this->specify('getPrefixProps should function properly', function () use ($model) {
+        $this->specify('getPrefixProps should function properly', function() use ($model) {
             expect('getPrefixProps should strip out all the falsy propeties it finds', $this->assertEmpty($model->getPrefixProps('answer')));
 
             $model->answer_1a = 'processing emotions';
@@ -90,7 +90,7 @@ 
 
     public function testBehaviorToAnswers()
     {
-        $this->specify('behaviorToAnswers should function properly', function () {
+        $this->specify('behaviorToAnswers should function properly', function() {
             $model = $this->container->get('\site\models\QuestionForm');
             $model->answer_1a = 'answering question a';
             $model->answer_1b = 'answering question b';
@@ -114,7 +114,7 @@ 
     {
         $model = $this->container->get('\site\models\QuestionForm');
 
-        $this->specify('getAnswers should function properly', function () use ($model) {
+        $this->specify('getAnswers should function properly', function() use ($model) {
             $model->user_behavior_id1 = 'dummy';
             $model->user_behavior_id2 = 'dummy';
             $model->user_behavior_id3 = 'dummy';
@@ -132,7 +132,7 @@ 
         $this->fakeModel(7, 280, 8),
         $this->fakeModel(13, 281, 8),
         $this->fakeModel(28, 284, 8)
-      ]), [ [
+      ]), [[
                                         'behavior_id' => 280,
                     'category_id' => 8,
                                         'user_bhvr_id' => 7,

site/tests/unit/models/ChangeEmailFormTest.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
 
     public function testChangeEmail()
     {
-        $this->specify('changeEmail() should return true and do nothing when the email is already taken', function () {
+        $this->specify('changeEmail() should return true and do nothing when the email is already taken', function() {
             $user = $this->getUser();
             $user
         ->expects($this->never())
@@ -41,7 +41,7 @@ 
             $this->assertNotEquals($user->desired_email, $desired_email);
         });
 
-        $this->specify('changeEmail() should return true, set values, and send an email when the email is available', function () {
+        $this->specify('changeEmail() should return true, set values, and send an email when the email is available', function() {
             $user = $this->getUser();
             $desired_email = 'new_email@email.com';
             $user