Update to image_block based on bharat's feedback

1) move the rand_key column into core
2) don't do a max rand, just try to a get a random number less than
the current random number if that doesn't successd look the other way

3 files changed, 6 insertions, 43 deletions

diff --git a/modules/image_block/helpers/image_block_event.php b/modules/image_block/helpers/image_block_event.php
deleted file mode 100644
index 0783773b..00000000
--- a/modules/image_block/helpers/image_block_event.php
+++ /dev/null
@@ -1,25 +0,0 @@
-<?php defined("SYSPATH") or die("No direct script access.");
-/**
- * Gallery - a web based photo album viewer and editor
- * Copyright (C) 2000-2008 Bharat Mediratta
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or (at
- * your option) any later version.
- *
- * This program is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston, MA  02110-1301, USA.
- */
-class image_block_event_Core {
-  static function item_created($item) {
-    $db = Database::instance();
-    $db->query("UPDATE `items` SET `rand_key` = RAND() WHERE id = {$item->id}");
-  }
-}
diff --git a/modules/image_block/helpers/image_block_installer.php b/modules/image_block/helpers/image_block_installer.php
index 6b0ec20b..386ad2f5 100644
--- a/modules/image_block/helpers/image_block_installer.php
+++ b/modules/image_block/helpers/image_block_installer.php
@@ -24,18 +24,12 @@ class image_block_installer {
     if ($version == 0) {
       $db = Database::instance();
       
-      $db->query("ALTER TABLE `items` ADD `rand_key` FLOAT DEFAULT NULL");
-      $db->query("UPDATE `items` SET `rand_key` = RAND()");
-      $db->query("CREATE INDEX `random_index` ON `items` (rand_key DESC)");
-
       module::set_version("image_block", 1);
     }
   }
 
   static function uninstall() {
     $db = Database::instance();
-    // Dropping the column should drop the index as well.
-    $db->query("ALTER TABLE `items` DROP `rand_key`");
 
     module::delete("image_block");
   }
diff --git a/modules/image_block/helpers/image_block_theme.php b/modules/image_block/helpers/image_block_theme.php
index 0a103add..96ee58e2 100644
--- a/modules/image_block/helpers/image_block_theme.php
+++ b/modules/image_block/helpers/image_block_theme.php
@@ -19,11 +19,12 @@
  */
 class image_block_theme_Core {
   static function sidebar_blocks($theme) {
-    $result = ORM::factory("item")
+    // Leave as a count so we can filter based on viewable items.
+    $viewable_items = ORM::factory("item")
       ->viewable()
       ->select("COUNT(*) AS C")
-      ->find();
-    if (empty($result->C)) {
+      ->find()->C;
+    if (empty($viewable_items)) {
       return "";
     }
 
@@ -32,18 +33,12 @@ class image_block_theme_Core {
     $block->title = t("Random Image");
     $block->content = new View("image_block_block.html");
 
-    $result = ORM::factory("item")
-      ->viewable()
-      ->select("MAX(rand_key) AS max_random")
-      ->find();
-
-    $max_rand = $result->max_random;
     $random = ((float)mt_rand()) / (float)mt_getrandmax();
 
     $items = ORM::factory("item")
       ->viewable()
       ->where("type !=", "album")
-      ->where("rand_key < ", $max_rand * $random)
+      ->where("rand_key < ", $random)
       ->orderby(array("rand_key" => "DESC"))
       ->find_all(1);
 
@@ -51,10 +46,9 @@ class image_block_theme_Core {
       $items = ORM::factory("item")
         ->viewable()
         ->where("type !=", "album")
-        ->where("rand_key > ", $max_rand * $random)
+        ->where("rand_key >= ", $random)
         ->orderby(array("rand_key" => "DESC"))
         ->find_all(1);
-     
     }
  
     $block->content->item = $items->current();