Series API

gatherling/admin/clean_decks.php

@@ -1,5 +1,9 @@
 <?php
 
+use Gatherling\Database;
+use Gatherling\Deck;
+use Gatherling\Player;
+
 require '../lib.php';
 
 session_start();

gatherling/admin/db-upgrade.php

@@ -813,6 +813,12 @@ function redirect_deck_update($latest_id = 0)
     do_query('ALTER TABLE `players`
             CHANGE COLUMN `discord_id` `discord_id` VARCHAR(20) NULL DEFAULT NULL AFTER `theme`;');
 });
+upgrade_db(50, 'Fix series discord fields', function () {
+    do_query('ALTER TABLE `series`
+	        CHANGE COLUMN `discord_guild_id` `discord_guild_id` VARCHAR(20) NULL DEFAULT NULL AFTER `mtgo_room`,
+	        ADD COLUMN `discord_channel_id` VARCHAR(20) NULL DEFAULT NULL AFTER `discord_guild_id`,
+	        CHANGE COLUMN `discord_require_membership` `discord_require_membership` INT NULL DEFAULT NULL AFTER `discord_guild_invite`;');
+});
 $db->autocommit(true);
 
 info('DB is up to date!');

gatherling/api.php

@@ -38,9 +38,9 @@
         try {
             $series = new Series($seriesname);
             $result = repr_json_series($series);
-            $result['sucess'] = true;
+            $result['success'] = true;
         } catch (Exception $e) {
-            $result['sucess'] = false;
+            $result['success'] = false;
             $result['error'] = $e->getMessage();
         }
         break;

gatherling/api_lib.php

@@ -1,5 +1,7 @@
 <?php
 
+require_once 'lib.php';
+
 //## Helper Functions
 
 use Gatherling\Event;
@@ -16,6 +18,7 @@ function populate($array, $src, $keys)
     return $array;
 }
 
+/** @return bool  */
 function is_admin()
 {
     if (!isset($_SESSION['infobot'])) {
@@ -62,6 +65,12 @@ function auth()
     return Player::isLoggedIn();
 }
 
+/**
+ * @param string $msg
+ * @param mixed  $extra
+ *
+ * @return never
+ */
 function error($msg, $extra = null)
 {
     $result = [];
@@ -74,6 +83,12 @@ function error($msg, $extra = null)
     exit(json_encode($result));
 }
 
+/**
+ * @param string $key
+ * @param mixed  $default
+ *
+ * @return mixed
+ */
 function arg($key, $default = null)
 {
     if (!isset($_REQUEST[$key])) {
@@ -181,9 +196,14 @@ function repr_json_deck($deck)
     return $json;
 }
 
+/**
+ * @param Gatherling\Series $series
+ *
+ * @return mixed
+ */
 function repr_json_series($series)
 {
-    $json = populate([], $series, ['name', 'active', 'start_day', 'start_time', 'organizers', 'mtgo_room', 'this_season_format', 'this_season_master_link', 'this_season_season']);
+    $json = populate([], $series, ['name', 'active', 'start_day', 'start_time', 'organizers', 'mtgo_room', 'this_season_format', 'this_season_master_link', 'this_season_season', 'discord_guild_id', 'discord_channel_id', 'discord_channel_name', 'discord_guild_name']);
 
     return $json;
 }
@@ -272,6 +292,13 @@ function drop_player_from_event($event, $name)
     return $result;
 }
 
+/**
+ * @param string $newseries
+ * @param bool   $active
+ * @param string $day
+ *
+ * @return array
+ */
 function create_series($newseries, $active, $day)
 {
     $result = [];

gatherling/models/Decksearch.php

@@ -151,7 +151,7 @@ public function searchByMedals($medal)
      *
      *  @param [$color_str_input] Array of the input color
      *
-     *  @return mixed true if success/false otherwise
+     * @return mixed true if success/false otherwise
      */
     public function searchByColor($color_str_input)
     {

gatherling/models/Event.php

@@ -499,6 +499,11 @@ public function getTrophyImageLink()
            .self::trophy_image_tag($this->name)."\n</a>\n";
     }
 
+    /**
+     * @param string $name
+     *
+     * @return bool
+     */
     public function isHost($name)
     {
         $ishost = !is_null($this->host) && strcasecmp($name, $this->host) == 0;
@@ -507,11 +512,17 @@ public function isHost($name)
         return $ishost || $iscohost;
     }
 
+    /** @return bool  */
     public function isFinalized()
     {
         return $this->finalized != 0;
     }
 
+    /**
+     * @param string $name
+     *
+     * @return bool
+     */
     public function isOrganizer($name)
     {
         $isOrganizer = false;
@@ -528,6 +539,11 @@ public function isOrganizer($name)
         return $isOrganizer;
     }
 
+    /**
+     * @param string $playername
+     *
+     * @return bool
+     */
     public function authCheck($playername)
     {
         $player = new Player($playername);
@@ -541,6 +557,11 @@ public function authCheck($playername)
         return false;
     }
 
+    /**
+     * @throws Exception
+     *
+     * @return int
+     */
     public function getPlayerCount()
     {
         return Database::single_result_single_param('SELECT count(*) FROM entries WHERE event_id = ?', 'd', $this->id);
@@ -1707,6 +1728,8 @@ public function AssignMedalsbyStandings()
 
         $t8 = [];
         $t4 = [];
+        $sec = null;
+        $win = null;
 
         switch ($medalCount) {
             case 8: