summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--modules/gallery/helpers/MY_url.php6
1 files changed, 4 insertions, 2 deletions
diff --git a/modules/gallery/helpers/MY_url.php b/modules/gallery/helpers/MY_url.php
index 129efb91..1ca9a58f 100644
--- a/modules/gallery/helpers/MY_url.php
+++ b/modules/gallery/helpers/MY_url.php
@@ -26,9 +26,11 @@ class url extends url_Core {
$query = "";
}
- $parts = explode("/", $uri, 3);
// @todo if we're only doing this for Item_Model, why not just put this
- // all into Item_Model::url()? It'd make url::site() faster.
+ // all into Item_Model::url()? It'd make url::site() faster. Downside is that
+ // anywhere we refer to an item by id, eg url::site("albums/123") would have
+ // to load the item and do $item->url();
+ $parts = explode("/", $uri, 3);
if ($parts[0] == "albums" || $parts[0] == "photos" || $parts[0] == "movies") {
$uri = model_cache::get("item", $parts[1])->relative_url();
}