diff options
author | Bartek Przybylski <bart.p.pl@gmail.com> | 2012-06-07 00:10:09 +0400 |
---|---|---|
committer | Bartek Przybylski <bart.p.pl@gmail.com> | 2012-06-10 15:15:51 +0400 |
commit | c67de113c61faa27c819f928578f4d793e7d3123 (patch) | |
tree | a6cdc1bb570f82a27d1b5148649b3c3ab922f467 /apps | |
parent | 63c02fd595b28065de8f328bd4733ca2a5a9995b (diff) |
aviod incorrect image size returning in gallery listing
Diffstat (limited to 'apps')
-rw-r--r-- | apps/gallery/lib/managers.php | 7 | ||||
-rw-r--r-- | apps/gallery/templates/index.php | 18 |
2 files changed, 14 insertions, 11 deletions
diff --git a/apps/gallery/lib/managers.php b/apps/gallery/lib/managers.php index 6cb9b420ac6..2444659d0a4 100644 --- a/apps/gallery/lib/managers.php +++ b/apps/gallery/lib/managers.php @@ -32,9 +32,10 @@ class DatabaseManager { \OCP\DB::beginTransaction(); $stmt = \OCP\DB::prepare('INSERT INTO *PREFIX*pictures_images_cache (uid_owner, path, width, height) VALUES (?, ?, ?, ?)'); $stmt->execute(array(\OCP\USER::getUser(), $path, $image->width(), $image->height())); - unset($image); \OCP\DB::commit(); - return $this->getFileData($path); + $ret = array('filepath' => $path, 'width' => $image->width(), 'height' => $image->height()); + unset($image); + return $ret; } private function __construct() {} @@ -93,4 +94,4 @@ class ThumbnailsManager { private function __construct() {} } -?>
\ No newline at end of file +?> diff --git a/apps/gallery/templates/index.php b/apps/gallery/templates/index.php index 3dc722f0666..39e3bbf47b3 100644 --- a/apps/gallery/templates/index.php +++ b/apps/gallery/templates/index.php @@ -42,7 +42,7 @@ function o(element) { function openNewGal(album_name) { root = root + album_name + "/"; var url = window.location.toString().replace(window.location.search, ''); - url = url + "?app=gallery&root="+root; + url = url + "?app=gallery&root="+encodeURIComponent(root); window.location = url; } @@ -85,13 +85,15 @@ for($i = 0; $i < count($images); $i++) { $dir_arr = explode('/', $previous_element); -if (count($dir_arr)==0) { - $tl->addTile(new \OC\Pictures\TileSingle($previous_element)); -} else if (count($dir_arr) && $ts->getCount() == 0){ - $ts = new \OC\Pictures\TileStack(array($root.$previous_element), $dir_arr[0]); -} else { - $arr[] = $previous_element; - $ts->addTile($arr); +if (count($images)>1) { + if (count($dir_arr)==0) { + $tl->addTile(new \OC\Pictures\TileSingle($previous_element)); + } else if (count($dir_arr) && $ts->getCount() == 0){ + $ts = new \OC\Pictures\TileStack(array($root.$previous_element), $dir_arr[0]); + } else { + $arr[] = $previous_element; + $ts->addTile($arr); + } } if ($ts->getCount() != 0) { |