CorWatts / fasterscale

common/models/Behavior.php

Spacing +98 added lines, -98 removed lines

@@ -14,103 +14,103 @@ 
 class Behavior extends \yii\base\BaseObject implements BehaviorInterface
 {
   public static $behaviors = [
-    ['id' => 1,   'name' => 'no current secrets', 'category_id' => 1],
-    ['id' => 2,   'name' => 'resolving problems', 'category_id' => 1],
-    ['id' => 3,   'name' => 'identifying fears and feelings', 'category_id' => 1],
-    ['id' => 4,   'name' => 'keeping commitments to meetings, prayer, family, church, people, goals, and self', 'category_id' => 1],
-    ['id' => 5,   'name' => 'being open', 'category_id' => 1],
-    ['id' => 6,   'name' => 'being honest', 'category_id' => 1],
-    ['id' => 7,   'name' => 'making eye contact', 'category_id' => 1],
-    ['id' => 8,   'name' => 'reaching out to others', 'category_id' => 1],
-    ['id' => 9,   'name' => 'increasing in relationships with God and others', 'category_id' => 1],
-    ['id' => 10,  'name' => 'accountability', 'category_id' => 1],
-    ['id' => 11,  'name' => 'secrets', 'category_id' => 2],
-    ['id' => 12,  'name' => 'bored', 'category_id' => 2],
-    ['id' => 13,  'name' => 'less time/energy for God, meetings, and church', 'category_id' => 2],
-    ['id' => 14,  'name' => 'avoiding support and accountability towards people', 'category_id' => 2],
-    ['id' => 15,  'name' => 'superficial conversations', 'category_id' => 2],
-    ['id' => 16,  'name' => 'sarcasm', 'category_id' => 2],
-    ['id' => 17,  'name' => 'isolating yourself', 'category_id' => 2],
-    ['id' => 18,  'name' => 'changes in goals', 'category_id' => 2],
-    ['id' => 19,  'name' => 'flirting', 'category_id' => 2],
-    ['id' => 20,  'name' => 'obsessed with relationships', 'category_id' => 2],
-    ['id' => 21,  'name' => 'breaking promises/commitments', 'category_id' => 2],
-    ['id' => 22,  'name' => 'neglecting family', 'category_id' => 2],
-    ['id' => 23,  'name' => 'preoccupation with material things, television, or entertainment', 'category_id' => 2],
-    ['id' => 24,  'name' => 'procrastination', 'category_id' => 2],
-    ['id' => 25,  'name' => 'lying', 'category_id' => 2],
-    ['id' => 26,  'name' => 'over-confidence', 'category_id' => 2],
-    ['id' => 27,  'name' => 'hiding money', 'category_id' => 2],
-    ['id' => 28,  'name' => 'worry', 'category_id' => 3],
-    ['id' => 29,  'name' => 'using profanity', 'category_id' => 3],
-    ['id' => 30,  'name' => 'being fearful', 'category_id' => 3],
-    ['id' => 31,  'name' => 'being resentful', 'category_id' => 3],
-    ['id' => 32,  'name' => 'replaying old, negative thoughts', 'category_id' => 3],
-    ['id' => 33,  'name' => 'perfectionism', 'category_id' => 3],
-    ['id' => 34,  'name' => 'judging others\' motives', 'category_id' => 3],
-    ['id' => 35,  'name' => 'making goals and lists you can\'t complete', 'category_id' => 3],
-    ['id' => 36,  'name' => 'poor planning', 'category_id' => 3],
-    ['id' => 37,  'name' => 'mind reading', 'category_id' => 3],
-    ['id' => 38,  'name' => 'fantasy', 'category_id' => 3],
-    ['id' => 41,  'name' => 'co-dependent rescuing', 'category_id' => 3],
-    ['id' => 42,  'name' => 'sleep problems', 'category_id' => 3],
-    ['id' => 43,  'name' => 'trouble concentrating', 'category_id' => 3],
-    ['id' => 44,  'name' => 'seeking/creating drama', 'category_id' => 3],
-    ['id' => 45,  'name' => 'gossip', 'category_id' => 3],
-    ['id' => 46,  'name' => 'using over-the-counter medication for pain, sleep, and weight control', 'category_id' => 3],
-    ['id' => 47,  'name' => 'super busy', 'category_id' => 4],
-    ['id' => 48,  'name' => 'workaholic', 'category_id' => 4],
-    ['id' => 49,  'name' => 'can\'t relax', 'category_id' => 4],
-    ['id' => 50,  'name' => 'driving too fast', 'category_id' => 4],
-    ['id' => 51,  'name' => 'avoiding slowing down', 'category_id' => 4],
-    ['id' => 52,  'name' => 'feeling driven', 'category_id' => 4],
-    ['id' => 53,  'name' => 'in a hurry', 'category_id' => 4],
-    ['id' => 54,  'name' => 'can\'t turn off thoughts', 'category_id' => 4],
-    ['id' => 55,  'name' => 'skipping meals', 'category_id' => 4],
-    ['id' => 56,  'name' => 'binge eating (usually at night)', 'category_id' => 4],
-    ['id' => 57,  'name' => 'overspending', 'category_id' => 4],
-    ['id' => 58,  'name' => 'can\'t identify own feelings/needs', 'category_id' => 4],
-    ['id' => 59,  'name' => 'repetitive, negative thoughts', 'category_id' => 4],
-    ['id' => 60,  'name' => 'irritable', 'category_id' => 4],
-    ['id' => 61,  'name' => 'making excuses for "having to do it all"', 'category_id' => 4],
-    ['id' => 62,  'name' => 'dramatic mood swings', 'category_id' => 4],
-    ['id' => 63,  'name' => 'lust', 'category_id' => 4],
-    ['id' => 64,  'name' => 'too much caffeine', 'category_id' => 4],
-    ['id' => 65,  'name' => 'over exercising', 'category_id' => 4],
-    ['id' => 66,  'name' => 'nervousness', 'category_id' => 4],
-    ['id' => 67,  'name' => 'difficulty being alone or with people', 'category_id' => 4],
-    ['id' => 68,  'name' => 'difficulty listening to others', 'category_id' => 4],
-    ['id' => 69,  'name' => 'avoiding support', 'category_id' => 4],
-    ['id' => 70,  'name' => 'procrastination causing crises in money, work, or relationships', 'category_id' => 5],
-    ['id' => 71,  'name' => 'sarcasm', 'category_id' => 5],
-    ['id' => 72,  'name' => 'black and white, all or nothing thinking', 'category_id' => 5],
-    ['id' => 73,  'name' => 'feeling that no one understands', 'category_id' => 5],
-    ['id' => 74,  'name' => 'overreacting', 'category_id' => 5],
-    ['id' => 75,  'name' => 'road rage', 'category_id' => 5],
-    ['id' => 76,  'name' => 'constant resentments', 'category_id' => 5],
-    ['id' => 77,  'name' => 'pushing others away', 'category_id' => 5],
-    ['id' => 78,  'name' => 'increased isolation', 'category_id' => 5],
-    ['id' => 79,  'name' => 'blaming', 'category_id' => 5],
-    ['id' => 80,  'name' => 'self pity', 'category_id' => 5],
-    ['id' => 81,  'name' => 'arguing', 'category_id' => 5],
-    ['id' => 82,  'name' => 'irrationality', 'category_id' => 5],
-    ['id' => 83,  'name' => 'can\'t handle criticism', 'category_id' => 5],
-    ['id' => 84,  'name' => 'defensive', 'category_id' => 5],
-    ['id' => 85,  'name' => 'people are avoiding you', 'category_id' => 5],
-    ['id' => 86,  'name' => 'having to be right', 'category_id' => 5],
-    ['id' => 87,  'name' => 'digestive problems', 'category_id' => 5],
-    ['id' => 88,  'name' => 'headaches', 'category_id' => 5],
-    ['id' => 89,  'name' => 'obsessive (stuck) thoughts', 'category_id' => 5],
-    ['id' => 90,  'name' => 'can\'t forgive', 'category_id' => 5],
-    ['id' => 91,  'name' => 'feeling grandiose (superior)', 'category_id' => 5],
-    ['id' => 92,  'name' => 'intimidation', 'category_id' => 5],
-    ['id' => 93,  'name' => 'feeling aggressive', 'category_id' => 5],
-    ['id' => 94,  'name' => 'depressed', 'category_id' => 6],
-    ['id' => 95,  'name' => 'panicked', 'category_id' => 6],
-    ['id' => 96,  'name' => 'confused', 'category_id' => 6],
-    ['id' => 97,  'name' => 'hopeless', 'category_id' => 6],
-    ['id' => 98,  'name' => 'sleeping too much or too little', 'category_id' => 6],
-    ['id' => 99,  'name' => 'can\'t cope', 'category_id' => 6],
+    ['id' => 1, 'name' => 'no current secrets', 'category_id' => 1],
+    ['id' => 2, 'name' => 'resolving problems', 'category_id' => 1],
+    ['id' => 3, 'name' => 'identifying fears and feelings', 'category_id' => 1],
+    ['id' => 4, 'name' => 'keeping commitments to meetings, prayer, family, church, people, goals, and self', 'category_id' => 1],
+    ['id' => 5, 'name' => 'being open', 'category_id' => 1],
+    ['id' => 6, 'name' => 'being honest', 'category_id' => 1],
+    ['id' => 7, 'name' => 'making eye contact', 'category_id' => 1],
+    ['id' => 8, 'name' => 'reaching out to others', 'category_id' => 1],
+    ['id' => 9, 'name' => 'increasing in relationships with God and others', 'category_id' => 1],
+    ['id' => 10, 'name' => 'accountability', 'category_id' => 1],
+    ['id' => 11, 'name' => 'secrets', 'category_id' => 2],
+    ['id' => 12, 'name' => 'bored', 'category_id' => 2],
+    ['id' => 13, 'name' => 'less time/energy for God, meetings, and church', 'category_id' => 2],
+    ['id' => 14, 'name' => 'avoiding support and accountability towards people', 'category_id' => 2],
+    ['id' => 15, 'name' => 'superficial conversations', 'category_id' => 2],
+    ['id' => 16, 'name' => 'sarcasm', 'category_id' => 2],
+    ['id' => 17, 'name' => 'isolating yourself', 'category_id' => 2],
+    ['id' => 18, 'name' => 'changes in goals', 'category_id' => 2],
+    ['id' => 19, 'name' => 'flirting', 'category_id' => 2],
+    ['id' => 20, 'name' => 'obsessed with relationships', 'category_id' => 2],
+    ['id' => 21, 'name' => 'breaking promises/commitments', 'category_id' => 2],
+    ['id' => 22, 'name' => 'neglecting family', 'category_id' => 2],
+    ['id' => 23, 'name' => 'preoccupation with material things, television, or entertainment', 'category_id' => 2],
+    ['id' => 24, 'name' => 'procrastination', 'category_id' => 2],
+    ['id' => 25, 'name' => 'lying', 'category_id' => 2],
+    ['id' => 26, 'name' => 'over-confidence', 'category_id' => 2],
+    ['id' => 27, 'name' => 'hiding money', 'category_id' => 2],
+    ['id' => 28, 'name' => 'worry', 'category_id' => 3],
+    ['id' => 29, 'name' => 'using profanity', 'category_id' => 3],
+    ['id' => 30, 'name' => 'being fearful', 'category_id' => 3],
+    ['id' => 31, 'name' => 'being resentful', 'category_id' => 3],
+    ['id' => 32, 'name' => 'replaying old, negative thoughts', 'category_id' => 3],
+    ['id' => 33, 'name' => 'perfectionism', 'category_id' => 3],
+    ['id' => 34, 'name' => 'judging others\' motives', 'category_id' => 3],
+    ['id' => 35, 'name' => 'making goals and lists you can\'t complete', 'category_id' => 3],
+    ['id' => 36, 'name' => 'poor planning', 'category_id' => 3],
+    ['id' => 37, 'name' => 'mind reading', 'category_id' => 3],
+    ['id' => 38, 'name' => 'fantasy', 'category_id' => 3],
+    ['id' => 41, 'name' => 'co-dependent rescuing', 'category_id' => 3],
+    ['id' => 42, 'name' => 'sleep problems', 'category_id' => 3],
+    ['id' => 43, 'name' => 'trouble concentrating', 'category_id' => 3],
+    ['id' => 44, 'name' => 'seeking/creating drama', 'category_id' => 3],
+    ['id' => 45, 'name' => 'gossip', 'category_id' => 3],
+    ['id' => 46, 'name' => 'using over-the-counter medication for pain, sleep, and weight control', 'category_id' => 3],
+    ['id' => 47, 'name' => 'super busy', 'category_id' => 4],
+    ['id' => 48, 'name' => 'workaholic', 'category_id' => 4],
+    ['id' => 49, 'name' => 'can\'t relax', 'category_id' => 4],
+    ['id' => 50, 'name' => 'driving too fast', 'category_id' => 4],
+    ['id' => 51, 'name' => 'avoiding slowing down', 'category_id' => 4],
+    ['id' => 52, 'name' => 'feeling driven', 'category_id' => 4],
+    ['id' => 53, 'name' => 'in a hurry', 'category_id' => 4],
+    ['id' => 54, 'name' => 'can\'t turn off thoughts', 'category_id' => 4],
+    ['id' => 55, 'name' => 'skipping meals', 'category_id' => 4],
+    ['id' => 56, 'name' => 'binge eating (usually at night)', 'category_id' => 4],
+    ['id' => 57, 'name' => 'overspending', 'category_id' => 4],
+    ['id' => 58, 'name' => 'can\'t identify own feelings/needs', 'category_id' => 4],
+    ['id' => 59, 'name' => 'repetitive, negative thoughts', 'category_id' => 4],
+    ['id' => 60, 'name' => 'irritable', 'category_id' => 4],
+    ['id' => 61, 'name' => 'making excuses for "having to do it all"', 'category_id' => 4],
+    ['id' => 62, 'name' => 'dramatic mood swings', 'category_id' => 4],
+    ['id' => 63, 'name' => 'lust', 'category_id' => 4],
+    ['id' => 64, 'name' => 'too much caffeine', 'category_id' => 4],
+    ['id' => 65, 'name' => 'over exercising', 'category_id' => 4],
+    ['id' => 66, 'name' => 'nervousness', 'category_id' => 4],
+    ['id' => 67, 'name' => 'difficulty being alone or with people', 'category_id' => 4],
+    ['id' => 68, 'name' => 'difficulty listening to others', 'category_id' => 4],
+    ['id' => 69, 'name' => 'avoiding support', 'category_id' => 4],
+    ['id' => 70, 'name' => 'procrastination causing crises in money, work, or relationships', 'category_id' => 5],
+    ['id' => 71, 'name' => 'sarcasm', 'category_id' => 5],
+    ['id' => 72, 'name' => 'black and white, all or nothing thinking', 'category_id' => 5],
+    ['id' => 73, 'name' => 'feeling that no one understands', 'category_id' => 5],
+    ['id' => 74, 'name' => 'overreacting', 'category_id' => 5],
+    ['id' => 75, 'name' => 'road rage', 'category_id' => 5],
+    ['id' => 76, 'name' => 'constant resentments', 'category_id' => 5],
+    ['id' => 77, 'name' => 'pushing others away', 'category_id' => 5],
+    ['id' => 78, 'name' => 'increased isolation', 'category_id' => 5],
+    ['id' => 79, 'name' => 'blaming', 'category_id' => 5],
+    ['id' => 80, 'name' => 'self pity', 'category_id' => 5],
+    ['id' => 81, 'name' => 'arguing', 'category_id' => 5],
+    ['id' => 82, 'name' => 'irrationality', 'category_id' => 5],
+    ['id' => 83, 'name' => 'can\'t handle criticism', 'category_id' => 5],
+    ['id' => 84, 'name' => 'defensive', 'category_id' => 5],
+    ['id' => 85, 'name' => 'people are avoiding you', 'category_id' => 5],
+    ['id' => 86, 'name' => 'having to be right', 'category_id' => 5],
+    ['id' => 87, 'name' => 'digestive problems', 'category_id' => 5],
+    ['id' => 88, 'name' => 'headaches', 'category_id' => 5],
+    ['id' => 89, 'name' => 'obsessive (stuck) thoughts', 'category_id' => 5],
+    ['id' => 90, 'name' => 'can\'t forgive', 'category_id' => 5],
+    ['id' => 91, 'name' => 'feeling grandiose (superior)', 'category_id' => 5],
+    ['id' => 92, 'name' => 'intimidation', 'category_id' => 5],
+    ['id' => 93, 'name' => 'feeling aggressive', 'category_id' => 5],
+    ['id' => 94, 'name' => 'depressed', 'category_id' => 6],
+    ['id' => 95, 'name' => 'panicked', 'category_id' => 6],
+    ['id' => 96, 'name' => 'confused', 'category_id' => 6],
+    ['id' => 97, 'name' => 'hopeless', 'category_id' => 6],
+    ['id' => 98, 'name' => 'sleeping too much or too little', 'category_id' => 6],
+    ['id' => 99, 'name' => 'can\'t cope', 'category_id' => 6],
     ['id' => 100, 'name' => 'overwhelmed', 'category_id' => 6],
     ['id' => 101, 'name' => 'crying for "no reason"', 'category_id' => 6],
     ['id' => 102, 'name' => 'can\'t think', 'category_id' => 6],
@@ -180,7 +180,7 @@ 
   public function getCategories() {
     $bhvrs_by_cat = AH::index(self::$behaviors, null, 'category_id');
     $cats = AH::index(\common\models\Category::$categories, "id");
-    foreach($cats as $id => &$cat) {
+    foreach ($cats as $id => &$cat) {
       $cat['behavior_count'] = count($bhvrs_by_cat[$id]); // add count of behaviors
       $cat['category_id'] = $cat['id']; // rename id to category_id 
       unset($cat['id']);

common/models/User.php

Spacing +26 added lines, -26 removed lines

@@ -143,7 +143,7 @@ 
    */
   public function findByPasswordResetToken($token)
   {
-    if(!$this->isTokenCurrent($token)) {
+    if (!$this->isTokenCurrent($token)) {
       return null;
     }
 
@@ -161,15 +161,15 @@ 
    */
   public function findByVerifyEmailToken($token)
   {
-    if($this->isTokenConfirmed($token)) return null;
+    if ($this->isTokenConfirmed($token)) return null;
 
     $user = $this->findOne([
       'verify_email_token' => [$token, $token . self::CONFIRMED_STRING],
       'status' => self::STATUS_ACTIVE,
     ]);
 
-    if($user) {
-      if(!$this->isTokenConfirmed($token) &&
+    if ($user) {
+      if (!$this->isTokenConfirmed($token) &&
          !$this->isTokenCurrent($token, 'user.verifyAccountTokenExpire')) {
         return null;
       }
@@ -191,8 +191,8 @@ 
       'status' => self::STATUS_ACTIVE,
     ]);
 
-    if($user) {
-      if(!$user->isTokenCurrent($token, 'user.verifyAccountTokenExpire')) {
+    if ($user) {
+      if (!$user->isTokenCurrent($token, 'user.verifyAccountTokenExpire')) {
         return null;
       }
     }
@@ -210,7 +210,7 @@ 
   public function isTokenCurrent($token, String $paramPath = 'user.passwordResetTokenExpire') {
     $expire = \Yii::$app->params[$paramPath];
     $parts = explode('_', $token);
-    $timestamp = (int) end($parts);
+    $timestamp = (int)end($parts);
     if ($timestamp + $expire < time()) {
       // token expired
       return false;
@@ -225,7 +225,7 @@ 
    * @param string    $match the needle to search for
    */
   public function isTokenConfirmed($token = null, String $match = self::CONFIRMED_STRING) {
-    if(is_null($token)) $token = $this->verify_email_token;
+    if (is_null($token)) $token = $this->verify_email_token;
     return substr($token, -strlen($match)) === $match;
   }
 
@@ -250,7 +250,7 @@ 
   }
 
   public function isVerified() {
-    if(is_null($this->verify_email_token)) {
+    if (is_null($this->verify_email_token)) {
       // for old users who verified their accounts before the addition of
       // '_confirmed' to the token
       return true;
@@ -358,14 +358,14 @@ 
   }
 
   public function sendEmailReport($date) {
-    if(!$this->send_email) return false; // no partner emails set
+    if (!$this->send_email) return false; // no partner emails set
     list($start, $end) = $this->time->getUTCBookends($date);
-    $user_behaviors   = $this->getUserBehaviors($date);
+    $user_behaviors = $this->getUserBehaviors($date);
     $user_questions = $this->getUserQuestions($date);
 
     $messages = [];
-    foreach($this->getPartnerEmails() as $email) {
-      if($email) {
+    foreach ($this->getPartnerEmails() as $email) {
+      if ($email) {
         $messages[] = Yii::$app->mailer->compose('checkinReport', [
           'user'          => $this,
           'email'         => $email,
@@ -377,7 +377,7 @@ 
           'behaviors_list'  => \common\models\Behavior::$behaviors,
         ])->setFrom([Yii::$app->params['supportEmail'] => Yii::$app->name])
         ->setReplyTo($this->email)
-        ->setSubject($this->email." has completed a Faster Scale check-in")
+        ->setSubject($this->email . " has completed a Faster Scale check-in")
         ->setTo($email);
       }
     }
@@ -450,7 +450,7 @@ 
     return \Yii::$app->mailer->compose('signupNotification')
       ->setFrom([\Yii::$app->params['supportEmail'] => \Yii::$app->name])
       ->setTo(\Yii::$app->params['adminEmail'])
-      ->setSubject('A new user has signed up for '.\Yii::$app->name)
+      ->setSubject('A new user has signed up for ' . \Yii::$app->name)
       ->send();
   }
 
@@ -458,20 +458,20 @@ 
     return \Yii::$app->mailer->compose('verifyEmail', ['user' => $this])
       ->setFrom([\Yii::$app->params['supportEmail'] => \Yii::$app->name])
       ->setTo($this->email)
-      ->setSubject('Please verify your '.\Yii::$app->name .' account')
+      ->setSubject('Please verify your ' . \Yii::$app->name . ' account')
       ->send();
   }
 
   public function sendDeleteNotificationEmail() {
     $messages = [];
-    foreach(array_merge([$this->email], $this->getPartnerEmails()) as $email) {
-      if($email) {
+    foreach (array_merge([$this->email], $this->getPartnerEmails()) as $email) {
+      if ($email) {
         $messages[] = Yii::$app->mailer->compose('deleteNotification', [
           'user' => $this,
           'email' => $email
         ])->setFrom([Yii::$app->params['supportEmail'] => Yii::$app->name])
         ->setReplyTo($this->email)
-        ->setSubject($this->email." has deleted their The Faster Scale App account")
+        ->setSubject($this->email . " has deleted their The Faster Scale App account")
         ->setTo($email);
       }
     }
@@ -480,13 +480,13 @@ 
   }
 
   public function getUserQuestions($local_date = null) {
-    if(is_null($local_date)) $local_date = $this->time->getLocalDate();
+    if (is_null($local_date)) $local_date = $this->time->getLocalDate();
     $questions = $this->getQuestionData($local_date);
     return $this->parseQuestionData($questions);
   }
 
   public function getUserBehaviors($local_date = null) {
-    if(is_null($local_date)) $local_date = $this->time->getLocalDate();
+    if (is_null($local_date)) $local_date = $this->time->getLocalDate();
 
     $behaviors = $this->getBehaviorData($local_date);
     $behaviors = $this->user_behavior::decorateWithCategory($behaviors);
@@ -494,10 +494,10 @@ 
   }
 
   public function parseQuestionData($questions) {
-    if(!$questions) return [];
+    if (!$questions) return [];
 
     $question_answers = [];
-    foreach($questions as $question) {
+    foreach ($questions as $question) {
       $behavior = $question['behavior'];
 
       $question_answers[$behavior['id']]['question'] = [
@@ -515,10 +515,10 @@ 
   }
  
   public function parseBehaviorData($behaviors) {
-    if(!$behaviors) return [];
+    if (!$behaviors) return [];
 
     $opts_by_cat = [];
-    foreach($behaviors as $behavior) {
+    foreach ($behaviors as $behavior) {
       $indx = $behavior['behavior']['category_id'];
 
       $opts_by_cat[$indx]['category_name'] = $behavior['behavior']['category']['name'];
@@ -610,7 +610,7 @@ 
     return rtrim(
       strtr(
         base64_encode(
-          hash('sha256', $this->id."::".$this->created_at, true)
+          hash('sha256', $this->id . "::" . $this->created_at, true)
         ),
       '+/', '-_'),
     '=');

Braces +22 added lines, -7 removed lines

@@ -161,7 +161,9 @@ 
    */
   public function findByVerifyEmailToken($token)
   {
-    if($this->isTokenConfirmed($token)) return null;
+    if($this->isTokenConfirmed($token)) {
+      return null;
+    }
 
     $user = $this->findOne([
       'verify_email_token' => [$token, $token . self::CONFIRMED_STRING],
@@ -225,7 +227,9 @@ 
    * @param string    $match the needle to search for
    */
   public function isTokenConfirmed($token = null, String $match = self::CONFIRMED_STRING) {
-    if(is_null($token)) $token = $this->verify_email_token;
+    if(is_null($token)) {
+      $token = $this->verify_email_token;
+    }
     return substr($token, -strlen($match)) === $match;
   }
 
@@ -358,7 +362,10 @@ 
   }
 
   public function sendEmailReport($date) {
-    if(!$this->send_email) return false; // no partner emails set
+    if(!$this->send_email) {
+      return false;
+    }
+    // no partner emails set
     list($start, $end) = $this->time->getUTCBookends($date);
     $user_behaviors   = $this->getUserBehaviors($date);
     $user_questions = $this->getUserQuestions($date);
@@ -480,13 +487,17 @@ 
   }
 
   public function getUserQuestions($local_date = null) {
-    if(is_null($local_date)) $local_date = $this->time->getLocalDate();
+    if(is_null($local_date)) {
+      $local_date = $this->time->getLocalDate();
+    }
     $questions = $this->getQuestionData($local_date);
     return $this->parseQuestionData($questions);
   }
 
   public function getUserBehaviors($local_date = null) {
-    if(is_null($local_date)) $local_date = $this->time->getLocalDate();
+    if(is_null($local_date)) {
+      $local_date = $this->time->getLocalDate();
+    }
 
     $behaviors = $this->getBehaviorData($local_date);
     $behaviors = $this->user_behavior::decorateWithCategory($behaviors);
@@ -494,7 +505,9 @@ 
   }
 
   public function parseQuestionData($questions) {
-    if(!$questions) return [];
+    if(!$questions) {
+      return [];
+    }
 
     $question_answers = [];
     foreach($questions as $question) {
@@ -515,7 +528,9 @@ 
   }
  
   public function parseBehaviorData($behaviors) {
-    if(!$behaviors) return [];
+    if(!$behaviors) {
+      return [];
+    }
 
     $opts_by_cat = [];
     foreach($behaviors as $behavior) {

site/models/EditProfileForm.php

Spacing +4 added lines, -4 removed lines

@@ -69,16 +69,16 @@
   public function saveProfile()
   {
     if ($this->validate()) {
-      $user  = $this->user;
+      $user = $this->user;
 
-      if($this->timezone)
+      if ($this->timezone)
         $user->timezone = $this->timezone;
-      if($this->expose_graph) {
+      if ($this->expose_graph) {
         $user->expose_graph = true;
       } else {
         $user->expose_graph = false;
       }
-      if($this->send_email) {
+      if ($this->send_email) {
         $user->partner_email1  = $this->partner_email1;
         $user->partner_email2  = $this->partner_email2;
         $user->partner_email3  = $this->partner_email3;

Braces +3 added lines, -2 removed lines

@@ -71,8 +71,9 @@
     if ($this->validate()) {
       $user  = $this->user;
 
-      if($this->timezone)
-        $user->timezone = $this->timezone;
+      if($this->timezone) {
+              $user->timezone = $this->timezone;
+      }
       if($this->expose_graph) {
         $user->expose_graph = true;
       } else {

site/views/profile/index.php

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
   ]); ?>
           <?= $form->field($profile, 'timezone')->dropDownList(array_combine($timezones, $timezones)); ?>
           <?= $form->field($profile, 'send_email')->checkbox() ?>
-          <div id='send_email_fields' <?php if(!$profile->send_email) { ?>style="display: none;"<?php } ?>>
+          <div id='send_email_fields' <?php if (!$profile->send_email) { ?>style="display: none;"<?php } ?>>
             <?= $form->field($profile, 'partner_email1')->input('email'); ?>
             <?= $form->field($profile, 'partner_email2')->input('email'); ?>
             <?= $form->field($profile, 'partner_email3')->input('email'); ?>
@@ -94,11 +94,11 @@ 
 ]); ?>
           <?= $form->field($profile, 'timezone')->dropDownList(array_combine($timezones, $timezones)); ?>
           <?= $form->field($profile, 'expose_graph')->checkbox() ?>
-          <?php if($profile->expose_graph): ?>
+          <?php if ($profile->expose_graph): ?>
           <div class='alert alert-success score-graph-info'>Your score graph can be found at:<br /> <a id="score-graph-link" target="_blank" href="<?=$graph_url?>"><?=$graph_url?></a></div>
           <?php endif; ?>
         <?= $form->field($profile, 'send_email')->checkbox() ?>
-        <div id='email_threshold_fields' <?php if(!$profile->send_email) { ?>style="display: none;"<?php } ?>>
+        <div id='email_threshold_fields' <?php if (!$profile->send_email) { ?>style="display: none;"<?php } ?>>
           <?= $form->field($profile, 'email_threshold')->textInput(['class'=>'form-control', 'style'=>'width: 50px;'])->input('number', ['min' => 0, 'max' => 1000]) ?>
           <?= $form->field($profile, 'partner_email1')->input('email'); ?>
           <?= $form->field($profile, 'partner_email2')->input('email'); ?>