Upload files to 'classes'

classes/comments.class.php

@@ -1,21 +1,23 @@
-<?
-class Comments {
-  /*
-   * For all functions:
-   * $Page = 'artist', 'collages', 'requests' or 'torrents'
-   * $PageID = ArtistID, CollageID, RequestID or GroupID, respectively
-   */
-
-  /**
-   * Post a comment on an artist, request or torrent page.
-   * @param string $Page
-   * @param int $PageID
-   * @param string $Body
-   * @return int ID of the new comment
-   */
-  public static function post($Page, $PageID, $Body) {
-    $QueryID = G::$DB->get_query_id();
-    G::$DB->query("
+<?php
+class Comments
+{
+    /*
+     * For all functions:
+     * $Page = 'artist', 'collages', 'requests' or 'torrents'
+     * $PageID = ArtistID, CollageID, RequestID or GroupID, respectively
+     */
+
+    /**
+     * Post a comment on an artist, request or torrent page.
+     * @param string $Page
+     * @param int $PageID
+     * @param string $Body
+     * @return int ID of the new comment
+     */
+    public static function post($Page, $PageID, $Body)
+    {
+        $QueryID = G::$DB->get_query_id();
+        G::$DB->query("
       SELECT
         CEIL(
           (
@@ -25,36 +27,37 @@ class Comments {
               AND PageID = $PageID
           ) / " . TORRENT_COMMENTS_PER_PAGE . "
         ) AS Pages");
-    list($Pages) = G::$DB->next_record();
+        list($Pages) = G::$DB->next_record();
 
-    G::$DB->query("
+        G::$DB->query("
       INSERT INTO comments (Page, PageID, AuthorID, AddedTime, Body)
       VALUES ('$Page', $PageID, " . G::$LoggedUser['ID'] . ", NOW(), '" . db_string($Body) . "')");
-    $PostID = G::$DB->inserted_id();
+        $PostID = G::$DB->inserted_id();
 
-    $CatalogueID = floor((TORRENT_COMMENTS_PER_PAGE * $Pages - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
-    G::$Cache->delete_value($Page.'_comments_'.$PageID.'_catalogue_'.$CatalogueID);
-    G::$Cache->delete_value($Page.'_comments_'.$PageID);
+        $CatalogueID = floor((TORRENT_COMMENTS_PER_PAGE * $Pages - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
+        G::$Cache->delete_value($Page.'_comments_'.$PageID.'_catalogue_'.$CatalogueID);
+        G::$Cache->delete_value($Page.'_comments_'.$PageID);
 
-    Subscriptions::flush_subscriptions($Page, $PageID);
-    Subscriptions::quote_notify($Body, $PostID, $Page, $PageID);
+        Subscriptions::flush_subscriptions($Page, $PageID);
+        Subscriptions::quote_notify($Body, $PostID, $Page, $PageID);
 
-    G::$DB->set_query_id($QueryID);
+        G::$DB->set_query_id($QueryID);
 
-    return $PostID;
-  }
+        return $PostID;
+    }
 
-  /**
-   * Edit a comment
-   * @param int $PostID
-   * @param string $NewBody
-   * @param bool $SendPM If true, send a PM to the author of the comment informing him about the edit
-   * @todo move permission check out of here/remove hardcoded error(404)
-   */
-  public static function edit($PostID, $NewBody, $SendPM = false) {
-    $QueryID = G::$DB->get_query_id();
+    /**
+     * Edit a comment
+     * @param int $PostID
+     * @param string $NewBody
+     * @param bool $SendPM If true, send a PM to the author of the comment informing him about the edit
+     * @todo move permission check out of here/remove hardcoded error(404)
+     */
+    public static function edit($PostID, $NewBody, $SendPM = false)
+    {
+        $QueryID = G::$DB->get_query_id();
 
-    G::$DB->query("
+        G::$DB->query("
       SELECT
         Body,
         AuthorID,
@@ -63,25 +66,25 @@ class Comments {
         AddedTime
       FROM comments
       WHERE ID = $PostID");
-    if (!G::$DB->has_results()) {
-      return false;
-    }
-    list($OldBody, $AuthorID, $Page, $PageID, $AddedTime) = G::$DB->next_record();
+        if (!G::$DB->has_results()) {
+            return false;
+        }
+        list($OldBody, $AuthorID, $Page, $PageID, $AddedTime) = G::$DB->next_record();
 
-    if (G::$LoggedUser['ID'] != $AuthorID && !check_perms('site_moderate_forums')) {
-      return false;
-    }
+        if (G::$LoggedUser['ID'] != $AuthorID && !check_perms('site_moderate_forums')) {
+            return false;
+        }
 
-    G::$DB->query("
+        G::$DB->query("
       SELECT CEIL(COUNT(ID) / " . TORRENT_COMMENTS_PER_PAGE . ") AS Page
       FROM comments
       WHERE Page = '$Page'
         AND PageID = $PageID
         AND ID <= $PostID");
-    list($CommPage) = G::$DB->next_record();
+        list($CommPage) = G::$DB->next_record();
 
-    // Perform the update
-    G::$DB->query("
+        // Perform the update
+        G::$DB->query("
       UPDATE comments
       SET
         Body = '" . db_string($NewBody) . "',
@@ -89,49 +92,50 @@ class Comments {
         EditedTime = NOW()
       WHERE ID = $PostID");
 
-    // Update the cache
-    $CatalogueID = floor((TORRENT_COMMENTS_PER_PAGE * $CommPage - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
-    G::$Cache->delete_value($Page . '_comments_' . $PageID . '_catalogue_' . $CatalogueID);
+        // Update the cache
+        $CatalogueID = floor((TORRENT_COMMENTS_PER_PAGE * $CommPage - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
+        G::$Cache->delete_value($Page . '_comments_' . $PageID . '_catalogue_' . $CatalogueID);
 
-    if ($Page == 'collages') {
-      // On collages, we also need to clear the collage key (collage_$CollageID), because it has the comments in it... (why??)
-      G::$Cache->delete_value('collage_' . $PageID);
-    }
+        if ($Page == 'collages') {
+            // On collages, we also need to clear the collage key (collage_$CollageID), because it has the comments in it... (why??)
+            G::$Cache->delete_value('collage_' . $PageID);
+        }
 
-    G::$DB->query("
+        G::$DB->query("
       INSERT INTO comments_edits (Page, PostID, EditUser, EditTime, Body)
       VALUES ('$Page', $PostID, " . G::$LoggedUser['ID'] . ", NOW(), '" . db_string($OldBody) . "')");
 
-    G::$DB->set_query_id($QueryID);
+        G::$DB->set_query_id($QueryID);
 
-    if ($SendPM && G::$LoggedUser['ID'] != $AuthorID) {
-      // Send a PM to the user to notify them of the edit
-      $PMSubject = "Your comment #$PostID has been edited";
-      $PMurl = site_url()."comments.php?action=jump&postid=$PostID";
-      $ProfLink = '[url='.site_url().'user.php?id='.G::$LoggedUser['ID'].']'.G::$LoggedUser['Username'].'[/url]';
-      $PMBody = "One of your comments has been edited by $ProfLink: [url]{$PMurl}[/url]";
-      Misc::send_pm($AuthorID, 0, $PMSubject, $PMBody);
-    }
+        if ($SendPM && G::$LoggedUser['ID'] != $AuthorID) {
+            // Send a PM to the user to notify them of the edit
+            $PMSubject = "Your comment #$PostID has been edited";
+            $PMurl = site_url()."comments.php?action=jump&postid=$PostID";
+            $ProfLink = '[url='.site_url().'user.php?id='.G::$LoggedUser['ID'].']'.G::$LoggedUser['Username'].'[/url]';
+            $PMBody = "One of your comments has been edited by $ProfLink: [url]{$PMurl}[/url]";
+            Misc::send_pm($AuthorID, 0, $PMSubject, $PMBody);
+        }
 
-    return true; // TODO: this should reflect whether or not the update was actually successful, e.g. by checking G::$DB->affected_rows after the UPDATE query
-  }
-
-  /**
-   * Delete a comment
-   * @param int $PostID
-   */
-  public static function delete($PostID) {
-    $QueryID = G::$DB->get_query_id();
-    // Get page, pageid
-    G::$DB->query("SELECT Page, PageID FROM comments WHERE ID = $PostID");
-    if (!G::$DB->has_results()) {
-      // no such comment?
-      G::$DB->set_query_id($QueryID);
-      return false;
+        return true; // TODO: this should reflect whether or not the update was actually successful, e.g. by checking G::$DB->affected_rows after the UPDATE query
     }
-    list ($Page, $PageID) = G::$DB->next_record();
-    // get number of pages
-    G::$DB->query("
+
+    /**
+     * Delete a comment
+     * @param int $PostID
+     */
+    public static function delete($PostID)
+    {
+        $QueryID = G::$DB->get_query_id();
+        // Get page, pageid
+        G::$DB->query("SELECT Page, PageID FROM comments WHERE ID = $PostID");
+        if (!G::$DB->has_results()) {
+            // no such comment?
+            G::$DB->set_query_id($QueryID);
+            return false;
+        }
+        list($Page, $PageID) = G::$DB->next_record();
+        // get number of pages
+        G::$DB->query("
       SELECT
         CEIL(COUNT(ID) / " . TORRENT_COMMENTS_PER_PAGE . ") AS Pages,
         CEIL(SUM(IF(ID <= $PostID, 1, 0)) / " . TORRENT_COMMENTS_PER_PAGE . ") AS Page
@@ -139,148 +143,151 @@ class Comments {
       WHERE Page = '$Page'
         AND PageID = $PageID
       GROUP BY PageID");
-    if (!G::$DB->has_results()) {
-      // the comment $PostID was probably not posted on $Page
-      G::$DB->set_query_id($QueryID);
-      return false;
-    }
-    list($CommPages, $CommPage) = G::$DB->next_record();
+        if (!G::$DB->has_results()) {
+            // the comment $PostID was probably not posted on $Page
+            G::$DB->set_query_id($QueryID);
+            return false;
+        }
+        list($CommPages, $CommPage) = G::$DB->next_record();
 
-    // $CommPages = number of pages in the thread
-    // $CommPage = which page the post is on
-    // These are set for cache clearing.
+        // $CommPages = number of pages in the thread
+        // $CommPage = which page the post is on
+        // These are set for cache clearing.
 
-    G::$DB->query("
+        G::$DB->query("
       DELETE FROM comments
       WHERE ID = $PostID");
-    G::$DB->query("
+        G::$DB->query("
       DELETE FROM comments_edits
       WHERE Page = '$Page'
         AND PostID = $PostID");
 
-    G::$DB->query("
+        G::$DB->query("
       DELETE FROM users_notify_quoted
       WHERE Page = '$Page'
         AND PostID = $PostID");
 
-    Subscriptions::flush_subscriptions($Page, $PageID);
-    Subscriptions::flush_quote_notifications($Page, $PageID);
+        Subscriptions::flush_subscriptions($Page, $PageID);
+        Subscriptions::flush_quote_notifications($Page, $PageID);
 
-    //We need to clear all subsequential catalogues as they've all been bumped with the absence of this post
-    $ThisCatalogue = floor((TORRENT_COMMENTS_PER_PAGE * $CommPage - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
-    $LastCatalogue = floor((TORRENT_COMMENTS_PER_PAGE * $CommPages - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
-    for ($i = $ThisCatalogue; $i <= $LastCatalogue; ++$i) {
-      G::$Cache->delete_value($Page . '_comments_' . $PageID . '_catalogue_' . $i);
-    }
+        //We need to clear all subsequential catalogues as they've all been bumped with the absence of this post
+        $ThisCatalogue = floor((TORRENT_COMMENTS_PER_PAGE * $CommPage - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
+        $LastCatalogue = floor((TORRENT_COMMENTS_PER_PAGE * $CommPages - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
+        for ($i = $ThisCatalogue; $i <= $LastCatalogue; ++$i) {
+            G::$Cache->delete_value($Page . '_comments_' . $PageID . '_catalogue_' . $i);
+        }
 
-    G::$Cache->delete_value($Page . '_comments_' . $PageID);
+        G::$Cache->delete_value($Page . '_comments_' . $PageID);
 
-    if ($Page == 'collages') {
-      // On collages, we also need to clear the collage key (collage_$CollageID), because it has the comments in it... (why??)
-      G::$Cache->delete_value("collage_$PageID");
+        if ($Page == 'collages') {
+            // On collages, we also need to clear the collage key (collage_$CollageID), because it has the comments in it... (why??)
+            G::$Cache->delete_value("collage_$PageID");
+        }
+
+        G::$DB->set_query_id($QueryID);
+
+        return true;
     }
 
-    G::$DB->set_query_id($QueryID);
-
-    return true;
-  }
-
-  /**
-   * Get the URL to a comment, already knowing the Page and PostID
-   * @param string $Page
-   * @param int $PageID
-   * @param int $PostID
-   * @return string|bool The URL to the comment or false on error
-   */
-  public static function get_url($Page, $PageID, $PostID = null) {
-    $Post = (!empty($PostID) ? "&postid=$PostID#post$PostID" : '');
-    switch ($Page) {
-      case 'artist':
-        return "artist.php?id=$PageID$Post";
-      case 'collages':
-        return "collages.php?action=comments&collageid=$PageID$Post";
-      case 'requests':
-        return "requests.php?action=view&id=$PageID$Post";
-      case 'torrents':
-        return "torrents.php?id=$PageID$Post";
-      default:
-        return false;
+    /**
+     * Get the URL to a comment, already knowing the Page and PostID
+     * @param string $Page
+     * @param int $PageID
+     * @param int $PostID
+     * @return string|bool The URL to the comment or false on error
+     */
+    public static function get_url($Page, $PageID, $PostID = null)
+    {
+        $Post = (!empty($PostID) ? "&postid=$PostID#post$PostID" : '');
+        switch ($Page) {
+            case 'artist':
+                return "artist.php?id=$PageID$Post";
+            case 'collages':
+                return "collages.php?action=comments&collageid=$PageID$Post";
+            case 'requests':
+                return "requests.php?action=view&id=$PageID$Post";
+            case 'torrents':
+                return "torrents.php?id=$PageID$Post";
+            default:
+                return false;
+        }
     }
-  }
 
-  /**
-   * Get the URL to a comment
-   * @param int $PostID
-   * @return string|bool The URL to the comment or false on error
-   */
-  public static function get_url_query($PostID) {
-    $QueryID = G::$DB->get_query_id();
+    /**
+     * Get the URL to a comment
+     * @param int $PostID
+     * @return string|bool The URL to the comment or false on error
+     */
+    public static function get_url_query($PostID)
+    {
+        $QueryID = G::$DB->get_query_id();
 
-    G::$DB->query("
+        G::$DB->query("
       SELECT Page, PageID
       FROM comments
       WHERE ID = $PostID");
-    if (!G::$DB->has_results()) {
-      error(404);
+        if (!G::$DB->has_results()) {
+            error(404);
+        }
+        list($Page, $PageID) = G::$DB->next_record();
+
+        G::$DB->set_query_id($QueryID);
+
+        return self::get_url($Page, $PageID, $PostID);
     }
-    list($Page, $PageID) = G::$DB->next_record();
-
-    G::$DB->set_query_id($QueryID);
-
-    return self::get_url($Page, $PageID, $PostID);
-  }
-
-  /**
-   * Load a page's comments. This takes care of `postid` and (indirectly) `page` parameters passed in $_GET.
-   * Quote notifications and last read are also handled here, unless $HandleSubscriptions = false is passed.
-   * @param string $Page
-   * @param int $PageID
-   * @param bool $HandleSubscriptions Whether or not to handle subscriptions (last read & quote notifications)
-   * @return array ($NumComments, $Page, $Thread, $LastRead)
-   *     $NumComments: the total number of comments on this artist/request/torrent group
-   *     $Page: the page we're currently on
-   *     $Thread: an array of all posts on this page
-   *     $LastRead: ID of the last comment read by the current user in this thread;
-   *                will be false if $HandleSubscriptions == false or if there are no comments on this page
-   */
-  public static function load($Page, $PageID, $HandleSubscriptions = true) {
-    $QueryID = G::$DB->get_query_id();
-
-    // Get the total number of comments
-    $NumComments = G::$Cache->get_value($Page."_comments_$PageID");
-    if ($NumComments === false) {
-      G::$DB->query("
+
+    /**
+     * Load a page's comments. This takes care of `postid` and (indirectly) `page` parameters passed in $_GET.
+     * Quote notifications and last read are also handled here, unless $HandleSubscriptions = false is passed.
+     * @param string $Page
+     * @param int $PageID
+     * @param bool $HandleSubscriptions Whether or not to handle subscriptions (last read & quote notifications)
+     * @return array ($NumComments, $Page, $Thread, $LastRead)
+     *     $NumComments: the total number of comments on this artist/request/torrent group
+     *     $Page: the page we're currently on
+     *     $Thread: an array of all posts on this page
+     *     $LastRead: ID of the last comment read by the current user in this thread;
+     *                will be false if $HandleSubscriptions == false or if there are no comments on this page
+     */
+    public static function load($Page, $PageID, $HandleSubscriptions = true)
+    {
+        $QueryID = G::$DB->get_query_id();
+
+        // Get the total number of comments
+        $NumComments = G::$Cache->get_value($Page."_comments_$PageID");
+        if ($NumComments === false) {
+            G::$DB->query("
         SELECT COUNT(ID)
         FROM comments
         WHERE Page = '$Page'
           AND PageID = $PageID");
-      list($NumComments) = G::$DB->next_record();
-      G::$Cache->cache_value($Page."_comments_$PageID", $NumComments, 0);
-    }
-
-    // If a postid was passed, we need to determine which page that comment is on.
-    // Format::page_limit handles a potential $_GET['page']
-    if (isset($_GET['postid']) && is_number($_GET['postid']) && $NumComments > TORRENT_COMMENTS_PER_PAGE) {
-      G::$DB->query("
+            list($NumComments) = G::$DB->next_record();
+            G::$Cache->cache_value($Page."_comments_$PageID", $NumComments, 0);
+        }
+
+        // If a postid was passed, we need to determine which page that comment is on.
+        // Format::page_limit handles a potential $_GET['page']
+        if (isset($_GET['postid']) && is_number($_GET['postid']) && $NumComments > TORRENT_COMMENTS_PER_PAGE) {
+            G::$DB->query("
         SELECT COUNT(ID)
         FROM comments
         WHERE Page = '$Page'
           AND PageID = $PageID
           AND ID <= $_GET[postid]");
-      list($PostNum) = G::$DB->next_record();
-      list($CommPage, $Limit) = Format::page_limit(TORRENT_COMMENTS_PER_PAGE, $PostNum);
-    } else {
-      list($CommPage, $Limit) = Format::page_limit(TORRENT_COMMENTS_PER_PAGE, $NumComments);
-    }
-
-    // Get the cache catalogue
-    $CatalogueID = floor((TORRENT_COMMENTS_PER_PAGE * $CommPage - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
-
-    // Cache catalogue from which the page is selected, allows block caches and future ability to specify posts per page
-    $Catalogue = G::$Cache->get_value($Page.'_comments_'.$PageID.'_catalogue_'.$CatalogueID);
-    if ($Catalogue === false) {
-      $CatalogueLimit = $CatalogueID * THREAD_CATALOGUE . ', ' . THREAD_CATALOGUE;
-      G::$DB->query("
+            list($PostNum) = G::$DB->next_record();
+            list($CommPage, $Limit) = Format::page_limit(TORRENT_COMMENTS_PER_PAGE, $PostNum);
+        } else {
+            list($CommPage, $Limit) = Format::page_limit(TORRENT_COMMENTS_PER_PAGE, $NumComments);
+        }
+
+        // Get the cache catalogue
+        $CatalogueID = floor((TORRENT_COMMENTS_PER_PAGE * $CommPage - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
+
+        // Cache catalogue from which the page is selected, allows block caches and future ability to specify posts per page
+        $Catalogue = G::$Cache->get_value($Page.'_comments_'.$PageID.'_catalogue_'.$CatalogueID);
+        if ($Catalogue === false) {
+            $CatalogueLimit = $CatalogueID * THREAD_CATALOGUE . ', ' . THREAD_CATALOGUE;
+            G::$DB->query("
         SELECT
           c.ID,
           c.AuthorID,
@@ -295,20 +302,20 @@ class Comments {
           AND c.PageID = $PageID
         ORDER BY c.ID
         LIMIT $CatalogueLimit");
-      $Catalogue = G::$DB->to_array(false, MYSQLI_ASSOC);
-      G::$Cache->cache_value($Page.'_comments_'.$PageID.'_catalogue_'.$CatalogueID, $Catalogue, 0);
-    }
-
-    //This is a hybrid to reduce the catalogue down to the page elements: We use the page limit % catalogue
-    $Thread = array_slice($Catalogue, ((TORRENT_COMMENTS_PER_PAGE * $CommPage - TORRENT_COMMENTS_PER_PAGE) % THREAD_CATALOGUE), TORRENT_COMMENTS_PER_PAGE, true);
-
-    if ($HandleSubscriptions && count($Thread) > 0) {
-      // quote notifications
-      $LastPost = end($Thread);
-      $LastPost = $LastPost['ID'];
-      $FirstPost = reset($Thread);
-      $FirstPost = $FirstPost['ID'];
-      G::$DB->query("
+            $Catalogue = G::$DB->to_array(false, MYSQLI_ASSOC);
+            G::$Cache->cache_value($Page.'_comments_'.$PageID.'_catalogue_'.$CatalogueID, $Catalogue, 0);
+        }
+
+        //This is a hybrid to reduce the catalogue down to the page elements: We use the page limit % catalogue
+        $Thread = array_slice($Catalogue, ((TORRENT_COMMENTS_PER_PAGE * $CommPage - TORRENT_COMMENTS_PER_PAGE) % THREAD_CATALOGUE), TORRENT_COMMENTS_PER_PAGE, true);
+
+        if ($HandleSubscriptions && count($Thread) > 0) {
+            // quote notifications
+            $LastPost = end($Thread);
+            $LastPost = $LastPost['ID'];
+            $FirstPost = reset($Thread);
+            $FirstPost = $FirstPost['ID'];
+            G::$DB->query("
         UPDATE users_notify_quoted
         SET UnRead = false
         WHERE UserID = " . G::$LoggedUser['ID'] . "
@@ -316,126 +323,128 @@ class Comments {
           AND PageID = $PageID
           AND PostID >= $FirstPost
           AND PostID <= $LastPost");
-      if (G::$DB->affected_rows()) {
-        G::$Cache->delete_value('notify_quoted_' . G::$LoggedUser['ID']);
-      }
+            if (G::$DB->affected_rows()) {
+                G::$Cache->delete_value('notify_quoted_' . G::$LoggedUser['ID']);
+            }
 
-      // last read
-      G::$DB->query("
+            // last read
+            G::$DB->query("
         SELECT PostID
         FROM users_comments_last_read
         WHERE UserID = " . G::$LoggedUser['ID'] . "
           AND Page = '$Page'
           AND PageID = $PageID");
-      list($LastRead) = G::$DB->next_record();
-      if ($LastRead < $LastPost) {
-        G::$DB->query("
+            list($LastRead) = G::$DB->next_record();
+            if ($LastRead < $LastPost) {
+                G::$DB->query("
           INSERT INTO users_comments_last_read
             (UserID, Page, PageID, PostID)
           VALUES
             (" . G::$LoggedUser['ID'] . ", '$Page', $PageID, $LastPost)
           ON DUPLICATE KEY UPDATE
             PostID = $LastPost");
-        G::$Cache->delete_value('subscriptions_user_new_' . G::$LoggedUser['ID']);
-      }
-    } else {
-      $LastRead = false;
-    }
+                G::$Cache->delete_value('subscriptions_user_new_' . G::$LoggedUser['ID']);
+            }
+        } else {
+            $LastRead = false;
+        }
 
-    G::$DB->set_query_id($QueryID);
+        G::$DB->set_query_id($QueryID);
 
-    return array($NumComments, $CommPage, $Thread, $LastRead);
-  }
+        return array($NumComments, $CommPage, $Thread, $LastRead);
+    }
 
-  /**
-   * Merges all comments from $Page/$PageID into $Page/$TargetPageID. This also takes care of quote notifications, subscriptions and cache.
-   * @param type $Page
-   * @param type $PageID
-   * @param type $TargetPageID
-   */
-  public static function merge($Page, $PageID, $TargetPageID) {
-    $QueryID = G::$DB->get_query_id();
+    /**
+     * Merges all comments from $Page/$PageID into $Page/$TargetPageID. This also takes care of quote notifications, subscriptions and cache.
+     * @param type $Page
+     * @param type $PageID
+     * @param type $TargetPageID
+     */
+    public static function merge($Page, $PageID, $TargetPageID)
+    {
+        $QueryID = G::$DB->get_query_id();
 
-    G::$DB->query("
+        G::$DB->query("
       UPDATE comments
       SET PageID = $TargetPageID
       WHERE Page = '$Page'
         AND PageID = $PageID");
 
-    // quote notifications
-    G::$DB->query("
+        // quote notifications
+        G::$DB->query("
       UPDATE users_notify_quoted
       SET PageID = $TargetPageID
       WHERE Page = '$Page'
         AND PageID = $PageID");
 
-    // comment subscriptions
-    Subscriptions::move_subscriptions($Page, $PageID, $TargetPageID);
+        // comment subscriptions
+        Subscriptions::move_subscriptions($Page, $PageID, $TargetPageID);
 
-    // cache (we need to clear all comment catalogues)
-    G::$DB->query("
+        // cache (we need to clear all comment catalogues)
+        G::$DB->query("
       SELECT
         CEIL(COUNT(ID) / " . TORRENT_COMMENTS_PER_PAGE . ") AS Pages
       FROM comments
       WHERE Page = '$Page'
         AND PageID = $TargetPageID
       GROUP BY PageID");
-    list($CommPages) = G::$DB->next_record();
-    $LastCatalogue = floor((TORRENT_COMMENTS_PER_PAGE * $CommPages - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
-    for ($i = 0; $i <= $LastCatalogue; ++$i) {
-      G::$Cache->delete_value($Page . "_comments_$TargetPageID" . "_catalogue_$i");
+        list($CommPages) = G::$DB->next_record();
+        $LastCatalogue = floor((TORRENT_COMMENTS_PER_PAGE * $CommPages - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
+        for ($i = 0; $i <= $LastCatalogue; ++$i) {
+            G::$Cache->delete_value($Page . "_comments_$TargetPageID" . "_catalogue_$i");
+        }
+        G::$Cache->delete_value($Page."_comments_$TargetPageID");
+        G::$DB->set_query_id($QueryID);
     }
-    G::$Cache->delete_value($Page."_comments_$TargetPageID");
-    G::$DB->set_query_id($QueryID);
-  }
-
-  /**
-   * Delete all comments on $Page/$PageID (deals with quote notifications and subscriptions as well)
-   * @param string $Page
-   * @param int $PageID
-   * @return boolean
-   */
-  public static function delete_page($Page, $PageID) {
-    $QueryID = G::$DB->get_query_id();
-
-    // get number of pages
-    G::$DB->query("
+
+    /**
+     * Delete all comments on $Page/$PageID (deals with quote notifications and subscriptions as well)
+     * @param string $Page
+     * @param int $PageID
+     * @return boolean
+     */
+    public static function delete_page($Page, $PageID)
+    {
+        $QueryID = G::$DB->get_query_id();
+
+        // get number of pages
+        G::$DB->query("
       SELECT
         CEIL(COUNT(ID) / " . TORRENT_COMMENTS_PER_PAGE . ") AS Pages
       FROM comments
       WHERE Page = '$Page'
         AND PageID = $PageID
       GROUP BY PageID");
-    if (!G::$DB->has_results()) {
-      return false;
-    }
-    list($CommPages) = G::$DB->next_record();
+        if (!G::$DB->has_results()) {
+            return false;
+        }
+        list($CommPages) = G::$DB->next_record();
 
-    // Delete comments
-    G::$DB->query("
+        // Delete comments
+        G::$DB->query("
       DELETE FROM comments
       WHERE Page = '$Page'
         AND PageID = $PageID");
 
-    // Delete quote notifications
-    Subscriptions::flush_quote_notifications($Page, $PageID);
-    G::$DB->query("
+        // Delete quote notifications
+        Subscriptions::flush_quote_notifications($Page, $PageID);
+        G::$DB->query("
       DELETE FROM users_notify_quoted
       WHERE Page = '$Page'
         AND PageID = $PageID");
 
-    // Deal with subscriptions
-    Subscriptions::move_subscriptions($Page, $PageID, null);
+        // Deal with subscriptions
+        Subscriptions::move_subscriptions($Page, $PageID, null);
 
-    // Clear cache
-    $LastCatalogue = floor((TORRENT_COMMENTS_PER_PAGE * $CommPages - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
-    for ($i = 0; $i <= $LastCatalogue; ++$i) {
-      G::$Cache->delete_value($Page . '_comments_' . $PageID . '_catalogue_' . $i);
-    }
-    G::$Cache->delete_value($Page.'_comments_'.$PageID);
+        // Clear cache
+        $LastCatalogue = floor((TORRENT_COMMENTS_PER_PAGE * $CommPages - TORRENT_COMMENTS_PER_PAGE) / THREAD_CATALOGUE);
+        for ($i = 0; $i <= $LastCatalogue; ++$i) {
+            G::$Cache->delete_value($Page . '_comments_' . $PageID . '_catalogue_' . $i);
+        }
+        G::$Cache->delete_value($Page.'_comments_'.$PageID);
 
-    G::$DB->set_query_id($QueryID);
+        G::$DB->set_query_id($QueryID);
 
-    return true;
-  }
+        return true;
+    }
 }

classes/commentsview.class.php

@@ -1,95 +1,99 @@
-<?
-class CommentsView {
-  /**
-   * Render a thread of comments
-   * @param array $Thread An array as returned by Comments::load
-   * @param int $LastRead PostID of the last read post
-   * @param string $Baselink Link to the site these comments are on
-   */
-  public static function render_comments($Thread, $LastRead, $Baselink) {
-    foreach ($Thread as $Post) {
-      list($PostID, $AuthorID, $AddedTime, $CommentBody, $EditedUserID, $EditedTime, $EditedUsername) = array_values($Post);
-      self::render_comment($AuthorID, $PostID, $CommentBody, $AddedTime, $EditedUserID, $EditedTime, $Baselink . "&amp;postid=$PostID#post$PostID", ($PostID > $LastRead));
+<?php
+class CommentsView
+{
+    /**
+     * Render a thread of comments
+     * @param array $Thread An array as returned by Comments::load
+     * @param int $LastRead PostID of the last read post
+     * @param string $Baselink Link to the site these comments are on
+     */
+    public static function render_comments($Thread, $LastRead, $Baselink)
+    {
+        foreach ($Thread as $Post) {
+            list($PostID, $AuthorID, $AddedTime, $CommentBody, $EditedUserID, $EditedTime, $EditedUsername) = array_values($Post);
+            self::render_comment($AuthorID, $PostID, $CommentBody, $AddedTime, $EditedUserID, $EditedTime, $Baselink . "&amp;postid=$PostID#post$PostID", ($PostID > $LastRead));
+        }
     }
-  }
 
-  /**
-   * Render one comment
-   * @param int $AuthorID
-   * @param int $PostID
-   * @param string $Body
-   * @param string $AddedTime
-   * @param int $EditedUserID
-   * @param string $EditedTime
-   * @param string $Link The link to the post elsewhere on the site
-   * @param string $Header The header used in the post
-   * @param bool $Tools Whether or not to show [Edit], [Report] etc.
-   * @todo Find a better way to pass the page (artist, collages, requests, torrents) to this function than extracting it from $Link
-   */
-  static function render_comment($AuthorID, $PostID, $Body, $AddedTime, $EditedUserID, $EditedTime, $Link, $Unread = false, $Header = '', $Tools = true) {
-    $UserInfo = Users::user_info($AuthorID);
-    $Header = Users::format_username($AuthorID, true, true, true, true, true) . time_diff($AddedTime) . $Header;
-?>
+    /**
+     * Render one comment
+     * @param int $AuthorID
+     * @param int $PostID
+     * @param string $Body
+     * @param string $AddedTime
+     * @param int $EditedUserID
+     * @param string $EditedTime
+     * @param string $Link The link to the post elsewhere on the site
+     * @param string $Header The header used in the post
+     * @param bool $Tools Whether or not to show [Edit], [Report] etc.
+     * @todo Find a better way to pass the page (artist, collages, requests, torrents) to this function than extracting it from $Link
+     */
+    public static function render_comment($AuthorID, $PostID, $Body, $AddedTime, $EditedUserID, $EditedTime, $Link, $Unread = false, $Header = '', $Tools = true)
+    {
+        $UserInfo = Users::user_info($AuthorID);
+        $Header = Users::format_username($AuthorID, true, true, true, true, true) . time_diff($AddedTime) . $Header; ?>
     <table class="forum_post box vertical_margin<?=(!Users::has_avatars_enabled() ? ' noavatar' : '') . ($Unread ? ' forum_unread' : '')?>" id="post<?=$PostID?>">
       <colgroup>
-<?    if (Users::has_avatars_enabled()) { ?>
+        <?php    if (Users::has_avatars_enabled()) { ?>
         <col class="col_avatar" />
-<?    } ?>
+        <?php    } ?>
         <col class="col_post_body" />
       </colgroup>
       <tr class="colhead_dark">
         <td colspan="<?=(Users::has_avatars_enabled() ? 2 : 1)?>">
           <div class="float_left"><a class="post_id" href="<?=$Link?>">#<?=$PostID?></a>
             <?=$Header?>
-<?    if ($Tools) { ?>
+        <?php    if ($Tools) { ?>
             - <a href="#quickpost" onclick="Quote('<?=$PostID?>','<?=$UserInfo['Username']?>', true);" class="brackets">Quote</a>
-<?      if ($AuthorID == G::$LoggedUser['ID'] || check_perms('site_moderate_forums')) { ?>
+            <?php      if ($AuthorID == G::$LoggedUser['ID'] || check_perms('site_moderate_forums')) { ?>
             - <a href="#post<?=$PostID?>" onclick="Edit_Form('<?=$PostID?>','');" class="brackets">Edit</a>
-<?      }
-      if (check_perms('site_moderate_forums')) { ?>
+            <?php      }
+            if (check_perms('site_moderate_forums')) { ?>
             - <a href="#post<?=$PostID?>" onclick="Delete('<?=$PostID?>');" class="brackets">Delete</a>
-<?      } ?>
+            <?php      } ?>
           </div>
           <div id="bar<?=$PostID?>" class="float_right">
             <a href="reports.php?action=report&amp;type=comment&amp;id=<?=$PostID?>" class="brackets">Report</a>
-<?
-      if (check_perms('users_warn') && $AuthorID != G::$LoggedUser['ID'] && G::$LoggedUser['Class'] >= $UserInfo['Class']) {
-?>
+            <?php
+            if (check_perms('users_warn') && $AuthorID != G::$LoggedUser['ID'] && G::$LoggedUser['Class'] >= $UserInfo['Class']) {
+                ?>
             <form class="manage_form hidden" name="user" id="warn<?=$PostID?>" action="comments.php" method="post">
               <input type="hidden" name="action" value="warn" />
               <input type="hidden" name="postid" value="<?=$PostID?>" />
             </form>
             - <a href="#" onclick="$('#warn<?=$PostID?>').raw().submit(); return false;" class="brackets">Warn</a>
-<?      } ?>
+                <?php
+            } ?>
             &nbsp;
             <a href="#">&uarr;</a>
-<?    } ?>
+        <?php    } ?>
           </div>
         </td>
       </tr>
       <tr>
-<?    if (Users::has_avatars_enabled()) { ?>
+        <?php    if (Users::has_avatars_enabled()) { ?>
         <td class="avatar" valign="top">
-        <?=Users::show_avatar($UserInfo['Avatar'], $AuthorID, $UserInfo['Username'], G::$LoggedUser['DisableAvatars'])?>
+            <?=Users::show_avatar($UserInfo['Avatar'], $AuthorID, $UserInfo['Username'], G::$LoggedUser['DisableAvatars'])?>
         </td>
-<?    } ?>
+        <?php    } ?>
         <td class="body" valign="top">
           <div id="content<?=$PostID?>">
             <?=Text::full_format($Body)?>
-<?    if ($EditedUserID) { ?>
+        <?php    if ($EditedUserID) { ?>
             <br />
             <br />
             <div class="last_edited">
-<?      if (check_perms('site_admin_forums')) { ?>
+            <?php      if (check_perms('site_admin_forums')) { ?>
               <a href="#content<?=$PostID?>" onclick="LoadEdit('<?=substr($Link, 0, strcspn($Link, '.'))?>', <?=$PostID?>, 1); return false;">&laquo;</a>
-<?      } ?>
+            <?php      } ?>
               Last edited by
               <?=Users::format_username($EditedUserID, false, false, false) ?> <?=time_diff($EditedTime, 2, true, true)?>
-<?    } ?>
+        <?php    } ?>
             </div>
           </div>
         </td>
       </tr>
     </table>
-<?  }
+        <?php
+    }
 }

classes/cookie.class.php

@@ -1,4 +1,4 @@
-<?
+<?php
 /*************************************************************************|
 |--------------- Cookie class --------------------------------------------|
 |*************************************************************************|
@@ -19,30 +19,35 @@ interface COOKIE_INTERFACE {
 }
 */
 
-class COOKIE /*implements COOKIE_INTERFACE*/ {
-  const LIMIT_ACCESS = true; //If true, blocks JS cookie API access by default (can be overridden case by case)
-  const PREFIX = ''; //In some cases you may desire to prefix your cookies
+class COOKIE /*implements COOKIE_INTERFACE*/
+{
+    const LIMIT_ACCESS = true; //If true, blocks JS cookie API access by default (can be overridden case by case)
+    const PREFIX = ''; //In some cases you may desire to prefix your cookies
+
+    public function get($Key)
+    {
+        if (!isset($_COOKIE[self::PREFIX.$Key])) {
+            return false;
+        }
+        return $_COOKIE[self::PREFIX.$Key];
+    }
+
+    //Pass the 4th optional param as false to allow JS access to the cookie
+    public function set($Key, $Value, $Seconds = 86400, $LimitAccess = self::LIMIT_ACCESS)
+    {
+        setcookie(self::PREFIX.$Key, $Value, time() + $Seconds, '/', SITE_DOMAIN, $_SERVER['SERVER_PORT'] === '443', $LimitAccess, false);
+    }
 
-  public function get($Key) {
-    if (!isset($_COOKIE[SELF::PREFIX.$Key])) {
-      return false;
+    public function del($Key)
+    {
+        setcookie(self::PREFIX.$Key, '', time() - 24 * 3600); //3600 vs 1 second to account for potential clock desyncs
     }
-    return $_COOKIE[SELF::PREFIX.$Key];
-  }
-
-  //Pass the 4th optional param as false to allow JS access to the cookie
-  public function set($Key, $Value, $Seconds = 86400, $LimitAccess = SELF::LIMIT_ACCESS) {
-    setcookie(SELF::PREFIX.$Key, $Value, time() + $Seconds, '/', SITE_DOMAIN, $_SERVER['SERVER_PORT'] === '443', $LimitAccess, false);
-  }
-
-  public function del($Key) {
-    setcookie(SELF::PREFIX.$Key, '', time() - 24 * 3600); //3600 vs 1 second to account for potential clock desyncs
-  }
-
-  public function flush() {
-    $Cookies = array_keys($_COOKIE);
-    foreach ($Cookies as $Cookie) {
-      $this->del($Cookie);
+
+    public function flush()
+    {
+        $Cookies = array_keys($_COOKIE);
+        foreach ($Cookies as $Cookie) {
+            $this->del($Cookie);
+        }
     }
-  }
 }

classes/crypto.class.php

@@ -1,37 +1,39 @@
-<?
-class Crypto {
-  /**
-   * Encrypts input text for use in database
-   *
-   * @param string $plaintext
-   * @return encrypted string or false if DB key not accessible
-   */
-  public static function encrypt($plaintext) {
-    if (apcu_exists('DBKEY')) {
-      $iv_size = openssl_cipher_iv_length('AES-128-CBC');
-      $iv = openssl_random_pseudo_bytes($iv_size);
-      $ret = base64_encode($iv.openssl_encrypt($plaintext, 'AES-128-CBC', apcu_fetch('DBKEY'), OPENSSL_RAW_DATA, $iv));
-      return $ret;
-    } else {
-      return false;
+<?php
+class Crypto
+{
+    /**
+     * Encrypts input text for use in database
+     *
+     * @param string $plaintext
+     * @return encrypted string or false if DB key not accessible
+     */
+    public static function encrypt($plaintext)
+    {
+        if (apcu_exists('DBKEY')) {
+            $iv_size = openssl_cipher_iv_length('AES-128-CBC');
+            $iv = openssl_random_pseudo_bytes($iv_size);
+            $ret = base64_encode($iv.openssl_encrypt($plaintext, 'AES-128-CBC', apcu_fetch('DBKEY'), OPENSSL_RAW_DATA, $iv));
+            return $ret;
+        } else {
+            return false;
+        }
     }
-  }
 
-  /**
-   * Decrypts input text from database
-   *
-   * @param string $ciphertext
-   * @return decrypted string string or false if DB key not accessible
-   */
-  public static function decrypt($ciphertext) {
-    if (apcu_exists('DBKEY')) {
-      $iv_size = openssl_cipher_iv_length('AES-128-CBC');
-      $iv = substr(base64_decode($ciphertext), 0, $iv_size);
-      $ciphertext = substr(base64_decode($ciphertext), $iv_size);
-      return openssl_decrypt($ciphertext, 'AES-128-CBC', apcu_fetch('DBKEY'), OPENSSL_RAW_DATA, $iv);
-    } else {
-      return false;
+    /**
+     * Decrypts input text from database
+     *
+     * @param string $ciphertext
+     * @return decrypted string string or false if DB key not accessible
+     */
+    public static function decrypt($ciphertext)
+    {
+        if (apcu_exists('DBKEY')) {
+            $iv_size = openssl_cipher_iv_length('AES-128-CBC');
+            $iv = substr(base64_decode($ciphertext), 0, $iv_size);
+            $ciphertext = substr(base64_decode($ciphertext), $iv_size);
+            return openssl_decrypt($ciphertext, 'AES-128-CBC', apcu_fetch('DBKEY'), OPENSSL_RAW_DATA, $iv);
+        } else {
+            return false;
+        }
     }
-  }
 }
-?>