summaryrefslogtreecommitdiff
path: root/modules/gallery/tests/File_Proxy_Controller_Test.php
diff options
context:
space:
mode:
authorshadlaws <shad@shadlaws.com>2013-02-26 18:39:59 +0100
committershadlaws <shad@shadlaws.com>2013-02-26 18:39:59 +0100
commit251e9d5c8f727b886676e010481a6090ddac028c (patch)
treef7dd7fa36887727961abd56ff11c25b239327f62 /modules/gallery/tests/File_Proxy_Controller_Test.php
parent81011785aa8bbaad6dcfe5bb44a247cad778ed75 (diff)
#2010 - Revise item::find_by_path to search for jpg-converted items.
- added extra $var_subdir argument to item::find_by_path. - changed item::find_by_path to use $var_subdir to detect if we should look for a jpg-converted item or not (e.g. movie thumbs) - moved the album thumb detection to item::find_by_path to ensure it knows to look for an exact album match. - added more sanity checks to item::find_by_path (now has fewer false positive possibilities). - updated file_proxy to remove the need to guess different movie files. - updated File_Proxy_Controller - new sanity checks catch previously undetected bug. - added additional unit tests for item::find_by_path.
Diffstat (limited to 'modules/gallery/tests/File_Proxy_Controller_Test.php')
-rw-r--r--modules/gallery/tests/File_Proxy_Controller_Test.php2
1 files changed, 1 insertions, 1 deletions
diff --git a/modules/gallery/tests/File_Proxy_Controller_Test.php b/modules/gallery/tests/File_Proxy_Controller_Test.php
index 562100e4..06068d62 100644
--- a/modules/gallery/tests/File_Proxy_Controller_Test.php
+++ b/modules/gallery/tests/File_Proxy_Controller_Test.php
@@ -66,7 +66,7 @@ class File_Proxy_Controller_Test extends Gallery_Unit_Test_Case {
public function movie_thumbnails_are_jpgs_test() {
$movie = test::random_movie();
$name = legal_file::change_extension($movie->name, "jpg");
- $_SERVER["REQUEST_URI"] = url::file("var/thumbs/{$movie->name}");
+ $_SERVER["REQUEST_URI"] = url::file("var/thumbs/$name");
$controller = new File_Proxy_Controller();
$this->assert_same($movie->thumb_path(), $controller->__call("", array()));
}