Release v2.8.5

wilsonodk · Aug 18, 2014 · 061e779 · 061e779
2 parents b1786b6 + 012c31d
commit 061e779
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/VERSION.txt b/VERSION.txt
@@ -1 +1 @@
-v2.8.4
+v2.8.5
diff --git a/controllers/AdminPlayerController.php b/controllers/AdminPlayerController.php
@@ -52,8 +52,8 @@ static function player_do_add() {
         if (self::checkPerms()) {
             $db = option('db');
             $log = option('log');
-            // uid, username, email, password, wins, loses, permissions
-            $query = 'INSERT INTO {{users}} VALUES (NULL, "%s", "%s", "%s", 0, 0, %s, 0)';
+            // uid, username, email, password, wins, loses, permissions, submission, reminder
+            $query = 'INSERT INTO {{users}} VALUES (NULL, "%s", "%s", "%s", 0, 0, %s, 0, 1)';
 
             if ($db->qry(
                     $query,

diff --git a/controllers/MainController.php b/controllers/MainController.php
@@ -443,6 +443,12 @@ static function save_week() {
     /* HELPERS                                                                */
     /**************************************************************************/
 
+    static function get_percentage($wins, $loses) {
+        $total = $wins + $loses;
+
+        return $total > 0 ? round(($wins / $total) * 100, 2) : 0;
+    }
+
     static function get_standings() {
         $db    = option('db');
         $i     = 1;
@@ -461,7 +467,7 @@ static function get_standings() {
                 $temp->place = '';
                 $temp->path  = ($obj->submission ? "/week/$week/" : '/picks/' ) . strtolower($obj->username);
                 $temp->alt   = $obj->submission ? 'Picks for this week' : 'All picks';
-                $temp->per   = round(($temp->wins / ($temp->wins + $temp->loses)) * 100, 2);
+                $temp->per   = self::get_percentage($temp->wins, $temp->loses);
 
                 // Set proper place
                 if ($temp->wins !== $score) {