Remove site history stuff

classes/mysql.class.php

@@ -128,6 +128,7 @@ function db_string($String, $DisableWildcards = false)
     # Connect and mysqli_real_escape_string()
     # Previously called $DB->escape_str, now below
     # todo: Fix the bad escapes everywhere; see below
+
     #if (!is_string($String)) { # This is the correct way,
     if (is_array($String)) { # but this prevents errors
         error('Attempted to escape non-string.', $NoHTML = true);
@@ -163,7 +164,6 @@ function db_array($Array, $DontEscape = [], $Quote = false)
 }
 
 
-// todo: Revisit access levels once Drone is replaced by ZeRobot
 class DB_MYSQL
 {
     public $LinkID = false;
@@ -256,7 +256,7 @@ class DB_MYSQL
                 $this->LinkID,
                 $this->Key,
                 $this->Cert,
-                $this->Ca,
+                $this->CA,
                 null,
                 null
             );
@@ -453,7 +453,7 @@ class DB_MYSQL
     }
 
 
-    /*
+    /**
      * Returns an integer with the number of rows found
      * Returns a string if the number of rows found exceeds MAXINT
      */
@@ -465,7 +465,7 @@ class DB_MYSQL
     }
 
 
-    /*
+    /**
      * Returns true if the query exists and there were records found
      * Returns false if the query does not exist or if there were 0 records returned
      */

classes/security.class.php

@@ -18,6 +18,9 @@ class Security
      */
     public function CheckID($ID)
     {
+        # Temporary failsafe
+        # (int) 'dingus' = 0
+        # (int) 3.14 = 3
         $ID = (int) $ID;
 
         if (!is_int($ID) || $ID < 1) {

classes/sitehistory.class.php

@@ -1,305 +0,0 @@
-<?php
-#declare(strict_types=1);
-
-class SiteHistory
-{
-    private static $Categories = array(1 => "Code", "Event", "Milestone", "Policy", "Release", "Staff Change");
-    private static $SubCategories = array(1 => "Announcement", "Blog Post", "Forum Post", "Wiki", "Other", "External Source");
-    private static $Tags = array(
-        "api",
-        "celebration",
-        "class.primary",
-        "class.secondary",
-        "collage",
-        "community",
-        "conclusion",
-        "contest",
-        "design",
-        "donate",
-        "editing",
-        "editorial",
-        "feature",
-        "featured.article",
-        "featured.album",
-        "featured.product",
-        "finances",
-        "format",
-        "forum",
-        "freeleech",
-        "freeleech.tokens",
-        "gazelle",
-        "hierarchy",
-        "inbox",
-        "infrastructure",
-        "interview",
-        "irc",
-        "log",
-        "neutral.leech",
-        "notifications",
-        "ocelot",
-        "paranoia",
-        "picks.guest",
-        "picks.staff",
-        "promotion",
-        "ratio",
-        "record",
-        "report",
-        "request",
-        "requirement",
-        "retirement",
-        "rippy",
-        "search",
-        "settings",
-        "start",
-        "stats",
-        "store",
-        "stylesheet",
-        "tagging",
-        "transcode",
-        "toolbox",
-        "top.10",
-        "torrent",
-        "torrent.group",
-        "upload",
-        "vanity.house",
-        "voting",
-        "whitelist",
-        "wiki"
-    );
-
-    public static function get_months()
-    {
-        $Results = G::$Cache->get_value("site_history_months");
-        if (!$Results) {
-            $QueryID = G::$DB->get_query_id();
-            G::$DB->query("
-            SELECT DISTINCT
-              YEAR(DATE) AS Year, MONTH(Date) AS Month, MONTHNAME(Date) AS MonthName
-            FROM site_history
-              ORDER BY Date DESC");
-
-            $Results = G::$DB->to_array();
-            G::$DB->set_query_id($QueryID);
-            G::$Cache->cache_value("site_history_months", $Results, 0);
-        }
-        return $Results;
-    }
-
-    public static function get_event($ID)
-    {
-        if (!empty($ID)) {
-            $QueryID = G::$DB->get_query_id();
-            G::$DB->query("
-            SELECT
-              ID, Title, Url, Category, SubCategory, Tags, Body, AddedBy, Date
-            FROM site_history
-              WHERE ID = '$ID'
-              ORDER BY Date DESC");
-
-            $Event = G::$DB->next_record();
-            G::$DB->set_query_id($QueryID);
-            return $Event;
-        }
-    }
-
-    public static function get_latest_events($Limit)
-    {
-        self::get_events(null, null, null, null, null, null, $Limit);
-    }
-
-    public static function get_events($Month, $Year, $Title, $Category, $SubCategory, $Tags, $Limit)
-    {
-        $Month = (int)$Month;
-        $Year = (int)$Year;
-        $Title = db_string($Title);
-        $Category = (int)$Category;
-        $SubCategory = (int)$SubCategory;
-        $Tags = db_string($Tags);
-        $Limit = (int)$Limit;
-        $Where = [];
-
-        if (!empty($Month)) {
-            $Where[] = " MONTH(Date) = '$Month' ";
-        }
-
-        if (!empty($Year)) {
-            $Where[] = " YEAR(Date) = '$Year' ";
-        }
-
-        if (!empty($Title)) {
-            $Where[] = " Title LIKE '%$Title%' ";
-        }
-
-        if (!empty($Category)) {
-            $Where[] = " Category = '$Category '";
-        }
-
-        if (!empty($SubCategory)) {
-            $Where[] = " SubCategory = '$SubCategory '";
-        }
-
-        if (!empty($Tags)) {
-            $Tags = explode(',', $Tags);
-            $Or = '(';
-
-            foreach ($Tags as $Tag) {
-                $Tag = trim($Tag);
-                $Or .= " Tags LIKE '%$Tag%' OR ";
-            }
-
-            if (strlen($Or) > 1) {
-                $Or = rtrim($Or, 'OR ');
-                $Or .= ')';
-                $Where[] = $Or;
-            }
-        }
-
-        if (!empty($Limit)) {
-            $Limit = " LIMIT $Limit";
-        } else {
-            $Limit = '';
-        }
-
-        if (count($Where) > 0) {
-            $Query = ' WHERE ' . implode('AND', $Where);
-        } else {
-            $Query = '';
-        }
-
-        $QueryID = G::$DB->get_query_id();
-        G::$DB->query("
-        SELECT
-          ID, Title, Url, Category, SubCategory, Tags, Body, AddedBy, Date
-        FROM site_history
-          $Query
-          ORDER BY Date DESC
-          $Limit");
-
-        $Events = G::$DB->to_array();
-        G::$DB->set_query_id($QueryID);
-        return $Events;
-    }
-
-    public static function add_event($Date, $Title, $Link, $Category, $SubCategory, $Tags, $Body, $UserID)
-    {
-        if (empty($Date)) {
-            $Date = sqltime();
-        } else {
-            list($Y, $M, $D) = explode('-', $Date);
-            if (!checkdate($M, $D, $Y)) {
-                error("Error");
-            }
-        }
-
-        $Title = db_string($Title);
-        $Link = db_string($Link);
-        $Category = (int)$Category;
-        $SubCategory = (int)$SubCategory;
-        $Tags = db_string(strtolower((preg_replace('/\s+/', '', $Tags))));
-        $ExplodedTags = explode(',', $Tags);
-
-        foreach ($ExplodedTags as $Tag) {
-            if (!in_array($Tag, self::get_tags())) {
-                error("Invalid tag");
-            }
-        }
-
-        $Body = db_string($Body);
-        $UserID = (int)$UserID;
-
-        if (empty($Title) || empty($Category) || empty($SubCategory)) {
-            error("Error");
-        }
-
-        $QueryID = G::$DB->get_query_id();
-        G::$DB->query("
-        INSERT INTO site_history
-          (Title, Url, Category, SubCategory, Tags, Body, AddedBy, Date)
-        VALUES
-          ('$Title', '$Link', '$Category', '$SubCategory', '$Tags', '$Body', '$UserID', '$Date')");
-
-        G::$DB->set_query_id($QueryID);
-        G::$Cache->delete_value("site_history_months");
-    }
-
-    public static function update_event($ID, $Date, $Title, $Link, $Category, $SubCategory, $Tags, $Body, $UserID)
-    {
-        if (empty($Date)) {
-            $Date = sqltime();
-        } else {
-            $Date = db_string($Date);
-            list($Y, $M, $D) = explode('-', $Date);
-            if (!checkdate($M, $D, $Y)) {
-                error("Error");
-            }
-        }
-
-        $ID = (int)$ID;
-        $Title = db_string($Title);
-        $Link = db_string($Link);
-        $Category = (int)$Category;
-        $SubCategory = (int)$SubCategory;
-        $Tags = db_string(strtolower((preg_replace('/\s+/', '', $Tags))));
-        $ExplodedTags = explode(",", $Tags);
-
-        foreach ($ExplodedTags as $Tag) {
-            if (!in_array($Tag, self::get_tags())) {
-                error("Invalid tag");
-            }
-        }
-
-        $Body = db_string($Body);
-        $UserID = (int)$UserID;
-
-        if (empty($ID) || empty($Title) || empty($Category) || empty($SubCategory)) {
-            error("Error");
-        }
-
-        $QueryID = G::$DB->get_query_id();
-        G::$DB->query("
-        UPDATE site_history
-        SET
-          Title = '$Title',
-          Url = '$Link',
-          Category = '$Category',
-          SubCategory = '$SubCategory',
-          Tags = '$Tags',
-          Body = '$Body',
-          AddedBy = '$UserID',
-          Date = '$Date'
-        WHERE ID = '$ID'");
-
-        G::$DB->set_query_id($QueryID);
-        G::$Cache->delete_value("site_history_months");
-    }
-
-    public static function delete_event($ID)
-    {
-        if (!is_numeric($ID)) {
-            error(404);
-        }
-
-        $QueryID = G::$DB->get_query_id();
-        G::$DB->query("
-        DELETE FROM site_history
-          WHERE ID = '$ID'");
-          
-        G::$DB->set_query_id($QueryID);
-        G::$Cache->delete_value("site_history_months");
-    }
-
-    public static function get_categories()
-    {
-        return self::$Categories;
-    }
-
-    public static function get_sub_categories()
-    {
-        return self::$SubCategories;
-    }
-
-    public static function get_tags()
-    {
-        return self::$Tags;
-    }
-}

classes/sitehistoryview.class.php

@@ -1,237 +0,0 @@
-<?php
-#declare(strict_types=1);
-
-class SiteHistoryView
-{
-    public static function render_linkbox()
-    {
-        if (check_perms('users_mod')
-      ) {
-            ?>
-<div class="linkbox">
-  <a href="sitehistory.php?action=edit" class="brackets">Create new event</a>
-</div>
-<?php
-        }
-    }
-
-    public static function render_events($Events)
-    {
-        $Categories = SiteHistory::get_categories();
-        $SubCategories = SiteHistory::get_sub_categories();
-        $CanEdit = check_perms('users_mod') ;
-        foreach ($Events as $Event) {
-            ?>
-<div class="box">
-  <div class="head colhead_dark">
-    <div class="title">
-      <?php if ($CanEdit) { ?>
-      <a class="brackets"
-        href="sitehistory.php?action=edit&amp;id=<?=$Event['ID']?>">Edit</a>
-      <?php } ?>
-
-      <?=date('F d, Y', strtotime($Event['Date'])); ?>
-      -
-      <a href="sitehistory.php?action=search&amp;category=<?=$Event['Category']?>"
-        class="brackets"><?=$Categories[$Event['Category']]?></a>
-      <a href="sitehistory.php?action=search&amp;subcategory=<?=$Event['SubCategory']?>"
-        class="brackets"><?=$SubCategories[$Event['SubCategory']]?></a>
-
-      <?php if (!empty($Event['Url'])) { ?>
-      <a href="<?=$Event['Url']?>"><?=$Event['Title']?></a>
-      <?php } else { ?>
-      <?=$Event['Title']?>
-      <?php } ?>
-    </div>
-    <div class="tags">
-      <?php self::render_tags($Event['Tags'])?>
-    </div>
-  </div>
-  <?php if (!empty($Event['Body'])) { ?>
-  <div class="body">
-    <?=Text::full_format($Event['Body'])?>
-  </div>
-  <?php } ?>
-</div>
-<?php
-        }
-    }
-
-    private static function render_tags($Tags)
-    {
-        $Tags = explode(',', $Tags);
-        natcasesort($Tags);
-        $FormattedTags = '';
-        foreach ($Tags as $Tag) {
-            $FormattedTags .= "<a href=\"sitehistory.php?action=search&amp;tags=$Tag\">$Tag" . "</a>, ";
-        }
-        echo rtrim($FormattedTags, ', ');
-    }
-
-    public static function render_search() { ?>
-<div class="box">
-  <div class="head">Search</div>
-  <div class="pad">
-    <form class="search_form" action="sitehistory.php" method="get">
-      <input type="hidden" name="action" value="search" />
-      <input type="text" id="title" name="title" size="20" placeholder="Title" />
-      <br /><br />
-      <input type="text" id="tags" name="tags" size="20" placeholder="Comma-separated tags" />
-      <br /><br />
-      <select name="category" id="category">
-        <option value="0">Choose a category</option>
-        <?php
-      $Categories = SiteHistory::get_categories();
-      foreach ($Categories as $Key => $Value) {
-          ?>
-        <option<?=$Key == $Event['Category'] ? ' selected="selected"' : ''?>
-          value="<?=$Key?>"><?=$Value?>
-          </option>
-          <?php
-      } ?>
-      </select>
-      <br /><br />
-      <select name="subcategory">
-        <option value="0">Choose a subcategory</option>
-        <?php
-      $SubCategories = SiteHistory::get_sub_categories();
-      foreach ($SubCategories as $Key => $Value) {
-          ?>
-        <option<?=$Key == $Event['SubCategory'] ? ' selected="selected"' : ''?>
-          value="<?=$Key?>"><?=$Value?>
-          </option>
-          <?php
-      } ?>
-      </select>
-      <br /><br />
-      <input value="Search" type="submit" />
-    </form>
-  </div>
-</div>
-<?php }
-
-    public static function render_edit_form($Event) { ?>
-<form id="event_form" method="post" action="">
-  <?php if ($Event) { ?>
-  <input type="hidden" name="action" value="take_edit" />
-  <input type="hidden" name="id"
-    value="<?=$Event['ID']?>" />
-  <?php } else { ?>
-  <input type="hidden" name="action" value="take_create" />
-  <?php } ?>
-  <input type="hidden" name="auth"
-    value="<?=G::$LoggedUser['AuthKey']?>" />
-  <table cellpadding="6" cellspacing="1" border="0" class="layout border" width="100%">
-    <tr>
-      <td class="label">Title:</td>
-      <td>
-        <input type="text" id="title" name="title" size="50" class="required"
-          value="<?=$Event['Title']?>" />
-      </td>
-    </tr>
-    <tr>
-      <td class="label">Link:</td>
-      <td>
-        <input type="text" id="url" name="url" size="50"
-          value="<?=$Event['Url']?>" />
-      </td>
-    </tr>
-    <tr>
-      <td class="label">Date:</td>
-      <td>
-        <input type="date" id="date" name="date" class="required" <?=$Event ? ' value="' . date('Y-m-d', strtotime($Event['Date'])) . '"' : ''?>
-        />
-      </td>
-    </tr>
-    <tr>
-      <td class="label">Category:</td>
-      <td>
-        <select id="category" name="category" class="required">
-          <option value="0">Choose a category</option>
-          <?php
-    $Categories = SiteHistory::get_categories();
-    foreach ($Categories as $Key => $Value) {
-        ?>
-          <option<?=$Key == $Event['Category'] ? ' selected="selected"' : ''?>
-            value="<?=$Key?>"><?=$Value?>
-            </option>
-            <?php
-    } ?>
-        </select>
-      </td>
-    </tr>
-    <tr>
-      <td class="label">Subcategory:</td>
-      <td>
-        <select id="category" name="sub_category" class="required">
-          <option value="0">Choose a subcategory</option>
-          <?php $SubCategories = SiteHistory::get_sub_categories();
-    foreach ($SubCategories as $Key => $Value) { ?>
-          <option<?=$Key == $Event['SubCategory'] ? ' selected="selected"' : ''?>
-            value="<?=$Key?>"><?=$Value?>
-            </option>
-            <?php } ?>
-        </select>
-      </td>
-    </tr>
-    <tr>
-      <td class="label">Tags:</td>
-      <td>
-        <input type="text" id="tags" name="tags" placeholder="Comma-separated tags; use periods/dots for spaces"
-          size="50"
-          value="<?=$Event['Tags']?>" />
-        <select id="tag_list">
-          <option>Choose tags</option>
-          <?php
-    $Tags = SiteHistory::get_tags();
-    foreach ($Tags as $Tag) {
-        ?>
-          <option><?=$Tag?>
-          </option>
-          <?php
-    } ?>
-        </select>
-      </td>
-    </tr>
-    <tr>
-      <td class="label">Body:</td>
-      <td>
-        <textarea id="body" name="body" cols="90" rows="8" tabindex="1"
-          onkeyup="resize('body');"><?=$Event['Body']?></textarea>
-      </td>
-    </tr>
-  </table>
-  <input type="submit" name="submit" value="Submit" />
-  <?php if ($Event) { ?>
-  <input type="submit" name="delete" value="Delete" />
-  <?php } ?>
-</form>
-<?php
-  }
-
-    public static function render_recent_sidebar($Events) { ?>
-<div class="box">
-  <div class="head colhead_dark">
-    <strong><a href="sitehistory.php">Latest site history</a></strong>
-  </div>
-  <ul class="stats nobullet">
-    <?php
-    $Categories = SiteHistory::get_categories();
-    foreach ($Events as $Event) {
-        ?>
-    <li>
-      <a href="sitehistory.php?action=search&amp;category=<?=$Event['Category']?>"
-        class="brackets"><?=$Categories[$Event['Category']]?></a>
-      <?php if (!empty($Event['Url'])) { ?>
-      <a href="<?=$Event['Url']?>"><?=Format::cut_string($Event['Title'], 20)?></a>
-      <?php } else { ?>
-      <?=Format::cut_string($Event['Title'], 20)?>
-      <?php } ?>
-    </li>
-    <?php
-    } ?>
-  </ul>
-</div>
-<?php
-  }
-}

sections/sitehistory/edit.php

@@ -1,34 +0,0 @@
-<?php
-#declare(strict_types = 1);
-
-if (!check_perms('users_mod')) {
-    error(403);
-}
-if (is_number($_GET['id'])) {
-    $ID = $_GET['id'];
-    $Event = SiteHistory::get_event($ID);
-}
-
-if ($ID) {
-    $Title = "Edit";
-} else {
-    $Title = "Create";
-}
-View::show_header(
-    $Title,
-    "vendor/jquery.validate.min,form_validate,site_history"
-);
-
-?>
-
-<div class="header">
-  <?php if ($ID) { ?>
-  <h2>Edit event</h2>
-  <?php } else { ?>
-  <h2>Create new event</h2>
-  <?php } ?>
-</div>
-
-<?php
-SiteHistoryView::render_edit_form($Event);
-View::show_footer();

sections/sitehistory/history.php

@@ -1,51 +0,0 @@
-<?php
-define('DEFAULT_LIMIT', 10);
-
-$Limit = DEFAULT_LIMIT;
-if (is_number($_GET['month'])) {
-    $Month = $_GET['month'];
-    $Limit = null;
-}
-if (is_number($_GET['year'])) {
-    $Year = $_GET['year'];
-    $Limit = null;
-}
-if (!empty($_GET['title'])) {
-    $Title = $_GET['title'];
-    $Limit = null;
-}
-if (!empty($_GET['category'])) {
-    $Category = $_GET['category'];
-    $Limit = null;
-}
-if (!empty($_GET['subcategory'])) {
-    $SubCategory = $_GET['subcategory'];
-    $Limit = null;
-}
-if (!empty($_GET['tags'])) {
-    $Tags = $_GET['tags'];
-    $Limit = null;
-}
-$Events = SiteHistory::get_events($Month, $Year, $Title, $Category, $SubCategory, $Tags, $Limit);
-$Months = SiteHistory::get_months();
-View::show_header("Site History");
-?>
-<div class="header">
-  <h2><a href="sitehistory.php">Site History</a> <?=$Month && $Year ? date("- F, Y", mktime(0, 0, 0, $Month, 1, $Year)) : '' ?>
-  </h2>
-  <?php
-  SiteHistoryView::render_linkbox();
-?>
-</div>
-<div class="sidebar">
-  <?php
-  SiteHistoryView::render_search();
-?>
-</div>
-<div class="main_column">
-  <?php
-  SiteHistoryView::render_events($Events);
-?>
-</div>
-<?php
-View::show_footer();

sections/sitehistory/index.php

@@ -1,33 +0,0 @@
-<?
-enforce_login();
-
-if (!check_perms('users_mod')) {
-  error(403);
-}
-if (!empty($_POST['action'])) {
-  switch ($_POST['action']) {
-    case 'take_create':
-      include(SERVER_ROOT . '/sections/sitehistory/take_create.php');
-      break;
-    case 'take_edit':
-      include(SERVER_ROOT . '/sections/sitehistory/take_edit.php');
-      break;
-    default:
-      error(404);
-      break;
-  }
-} elseif (!empty($_GET['action'])) {
-  switch ($_GET['action']) {
-    case 'search':
-      include(SERVER_ROOT . '/sections/sitehistory/history.php');
-      break;
-    case 'edit':
-      include(SERVER_ROOT . '/sections/sitehistory/edit.php');
-      break;
-    default:
-      error(404);
-      break;
-  }
-} else {
-  include(SERVER_ROOT . '/sections/sitehistory/history.php');
-}

sections/sitehistory/take_create.php

@@ -1,9 +0,0 @@
-<?
-authorize();
-if (!check_perms('users_mod') ) {
-  error(403);
-}
-
-SiteHistory::add_event($_POST['date'], $_POST['title'], $_POST['url'], $_POST['category'], $_POST['sub_category'], $_POST['tags'], $_POST['body'], $LoggedUser['ID']);
-
-header("Location: sitehistory.php");

sections/sitehistory/take_edit.php

@@ -1,12 +0,0 @@
-<?
-authorize();
-if (!check_perms('users_mod') ) {
-  error(403);
-}
-
-if ($_POST['submit']) {
-  SiteHistory::update_event($_POST['id'], $_POST['date'], $_POST['title'], $_POST['url'], $_POST['category'], $_POST['sub_category'], $_POST['tags'], $_POST['body'], $LoggedUser['ID']);
-} elseif ($_POST['delete']) {
-  SiteHistory::delete_event($_POST['id']);
-}
-header("Location: sitehistory.php");

sections/torrents/editgroup.php

@@ -8,10 +8,11 @@ declare(strict_types = 1);
  * and clears the cache for the torrent group page.
  */
 
-Security::CheckID($_GET['groupid']);
+$GroupID = $_GET['groupid'];
+Security::CheckID($GroupID);
 
 // Get the torrent group name and the body of the last revision
-$DB->query("
+$DB->prepare_query("
 SELECT
   tg.`Name`,
   tg.`Title2`,
@@ -31,15 +32,16 @@ LEFT JOIN `wiki_torrents` AS wt
 ON
   wt.`RevisionID` = tg.`RevisionID`
 WHERE
-  tg.`ID` = '".db_string($GroupID)."'
+  tg.`ID` = '$GroupID'
 ");
+$DB->exec_prepared_query();
 
 if (!$DB->has_results()) {
     error(404);
 }
 list($Name, $Title2, $NameJP, $Image, $Body, $WikiImage, $WikiBody, $Year, $Studio, $Series, $CatalogueNumber, $CategoryID) = $DB->next_record();
 
-$DB->query("
+$DB->prepare_query("
 SELECT
   `ID`,
   `UserID`,
@@ -50,6 +52,7 @@ FROM
 WHERE
   `TorrentID` = '$GroupID'
 ");
+$DB->exec_prepared_query();
 
 if ($DB->has_results()) {
     $Screenshots = [];

sitehistory.php

@@ -1,4 +0,0 @@
-<?php
-declare(strict_types=1);
-
-require_once 'classes/script_start.php';

static/functions/form_validate.js

@@ -6,6 +6,5 @@ $(document).ready(function() {
   if (path == "inbox" && (action == "viewconv" || action == "compose")) $("#messageform").validate();
   if (path == "user" && action == "notify") $("#filter_form").validate();
   if (path == "requests" && action == "new") $("#request_form").preventDoubleSubmission();
-  if (path == "sitehistory" && action == "edit") $("#event_form").validate();
   if (path == "tools" && action == "mass_pm") $("#messageform").validate();
 });

static/functions/site_history.js

@@ -1,18 +0,0 @@
-$(document).ready(function() {
-  var trimmed = false;
-  var tags = $("#tags");
-  $("#tag_list").change(function() {
-    if (tags.val().length == 0) {
-      trimmed = false;
-    } else {
-      trimmed = true;
-    }
-    if ($(this).prop("selectedIndex")) {
-      tags.val(tags.val() + "," + $(this).val());
-      if (!trimmed) {
-        tags.val(tags.val().substr(1, tags.val().length));
-        trimmed = true;
-      }
-    }
-  });
-});