Revert "Create theme::get_var(), theme::set_var() methods to set the options of the active site theme. Change all refrences to theme options to use these methods. Update the version number of Gallery to 20 and move any them related options to the be stored under the active theme."

This reverts commit 26114972c3.
This commit is contained in:
Tim Almdal
2009-11-19 11:44:01 -08:00
parent 6fd1e2af1f
commit 57adefc5ba
11 changed files with 31 additions and 71 deletions

View File

@@ -35,8 +35,8 @@ class Admin_g2_import_Controller extends Admin_Controller {
if (g2_import::is_initialized()) {
$view->content->g2_stats = $g2_stats;
$view->content->g2_sizes = $g2_sizes;
$view->content->thumb_size = theme::get_var("thumb_size");
$view->content->resize_size = theme::get_var("resize_size");
$view->content->thumb_size = module::get_var("gallery", "thumb_size");
$view->content->resize_size = module::get_var("gallery", "resize_size");
}
print $view;
}

View File

@@ -671,8 +671,8 @@ class g2_import_Core {
$g2_item_id = self::$current_g2_item->getId();
$derivatives = g2(GalleryCoreApi::fetchDerivativesByItemIds(array($g2_item_id)));
$target_thumb_size = theme::get_var("thumb_size");
$target_resize_size = theme::get_var("resize_size");
$target_thumb_size = module::get_var("gallery", "thumb_size");
$target_resize_size = module::get_var("gallery", "resize_size");
if (!empty($derivatives[$g2_item_id])) {
foreach ($derivatives[$g2_item_id] as $derivative) {
if ($derivative->getPostFilterOperations()) {

View File

@@ -23,7 +23,7 @@ class Albums_Controller extends Items_Controller {
* @see REST_Controller::_show($resource)
*/
public function _show($album) {
$page_size = theme::get_var("page_size", 9);
$page_size = module::get_var("gallery", "page_size", 9);
if (!access::can("view", $album)) {
if ($album->id == 1) {
$view = new Theme_View("page.html", "other", "login");

View File

@@ -226,9 +226,9 @@ class gallery_installer {
module::set_var("gallery", "active_site_theme", "wind");
module::set_var("gallery", "active_admin_theme", "admin_wind");
module::set_var("wind", "page_size", 9);
module::set_var("wind", "thumb_size", 200);
module::set_var("wind", "resize_size", 640);
module::set_var("gallery", "page_size", 9);
module::set_var("gallery", "thumb_size", 200);
module::set_var("gallery", "resize_size", 640);
module::set_var("gallery", "default_locale", "en_US");
module::set_var("gallery", "image_quality", 75);
module::set_var("gallery", "image_sharpen", 15);
@@ -265,10 +265,10 @@ class gallery_installer {
module::set_var("gallery", "date_format", "Y-M-d");
module::set_var("gallery", "date_time_format", "Y-M-d H:i:s");
module::set_var("gallery", "time_format", "H:i:s");
module::set_var("wind", "show_credits", 1);
module::set_var("gallery", "show_credits", 1);
// @todo this string needs to be picked up by l10n_scanner
module::set_var("gallery", "credits", "Powered by <a href=\"%url\">Gallery %version</a>");
module::set_version("gallery", 20);
module::set_version("gallery", 19);
}
static function upgrade($version) {
@@ -432,17 +432,6 @@ class gallery_installer {
module::clear_var("gallery", "blocks_site.sidebar");
module::set_version("gallery", $version = 19);
}
// Move the theme related variables into the current theme
if ($version == 19) {
foreach (array("page_size", "thumb_size", "resize_size", "header_text",
"footer_text", "show_credits") as $var) {
$value = module::get_var("gallery", $var);
theme::set_var($var, $value);
module::clear_var("gallery", $var);
}
module::set_version("gallery", $version = 20);
}
}
static function uninstall() {

View File

@@ -57,33 +57,4 @@ class theme_Core {
return $theme_info;
}
/**
* Get a variable from the active theme
* @param string $name
* @param string $default_value
* @return the value
*/
static function get_var($name, $default_value=null) {
return module::get_var(self::$site, $name, $default_value);
}
/**
* Store a variable for active theme
* @param string $module_name
* @param string $name
* @param string $value
*/
static function set_var($name, $value) {
module::set_var(self::$site, $name, $value);
}
/**
* Remove a variable for this module.
* @param string $module_name
* @param string $name
*/
static function clear_var($name) {
module::clear_var(self::$site, $name);
}
}

View File

@@ -65,7 +65,7 @@ class Theme_View_Core extends Gallery_View {
public function thumb_proportion() {
// @TODO change the 200 to a theme supplied value when and if we come up with an
// API to allow the theme to set defaults.
return theme::get_var("thumb_size", 200) / 200;
return module::get_var("gallery", "thumb_size", 200) / 200;
}
public function item() {

View File

@@ -1,3 +1,3 @@
name = "Gallery 3"
description = "Gallery core application"
version = 20
version = 19

View File

@@ -19,7 +19,7 @@
*/
class Search_Controller extends Controller {
public function index() {
$page_size = theme::get_var("page_size", 9);
$page_size = module::get_var("gallery", "page_size", 9);
$q = $this->input->get("q");
$page = $this->input->get("page", 1);
$offset = ($page - 1) * $page_size;

View File

@@ -21,7 +21,7 @@ class Tags_Controller extends REST_Controller {
protected $resource_type = "tag";
public function _show($tag) {
$page_size = theme::get_var("page_size", 9);
$page_size = module::get_var("gallery", "page_size", 9);
$page = (int) $this->input->get("page", "1");
$children_count = $tag->items_count();
$offset = ($page-1) * $page_size;

View File

@@ -24,50 +24,50 @@ class wind {
$group = $form->group("edit_theme");
$group->input("page_size")->label(t("Items per page"))->id("g-page-size")
->rules("required|valid_digit")
->value(module::get_var("wind", "page_size"));
->value(module::get_var("gallery", "page_size"));
$group->input("thumb_size")->label(t("Thumbnail size (in pixels)"))->id("g-thumb-size")
->rules("required|valid_digit")
->value(module::get_var("wind", "thumb_size"));
->value(module::get_var("gallery", "thumb_size"));
$group->input("resize_size")->label(t("Resized image size (in pixels)"))->id("g-resize-size")
->rules("required|valid_digit")
->value(module::get_var("wind", "resize_size"));
->value(module::get_var("gallery", "resize_size"));
$group->textarea("header_text")->label(t("Header text"))->id("g-header-text")
->value(module::get_var("wind", "header_text"));
->value(module::get_var("gallery", "header_text"));
$group->textarea("footer_text")->label(t("Footer text"))->id("g-footer-text")
->value(module::get_var("wind", "footer_text"));
->value(module::get_var("gallery", "footer_text"));
$group->checkbox("show_credits")->label(t("Show site credits"))->id("g-footer-text")
->checked(module::get_var("wind", "show_credits"));
->checked(module::get_var("gallery", "show_credits"));
$group->submit("")->value(t("Save"));
return $form;
}
static function update_options($form) {
module::set_var("wind", "page_size", $form->edit_theme->page_size->value);
module::set_var("gallery", "page_size", $form->edit_theme->page_size->value);
$thumb_size = $form->edit_theme->thumb_size->value;
$thumb_dirty = false;
if (module::get_var("wind", "thumb_size") != $thumb_size) {
if (module::get_var("gallery", "thumb_size") != $thumb_size) {
graphics::remove_rule("gallery", "thumb", "gallery_graphics::resize");
graphics::add_rule(
"gallery", "thumb", "gallery_graphics::resize",
array("width" => $thumb_size, "height" => $thumb_size, "master" => Image::AUTO),
100);
module::set_var("wind", "thumb_size", $thumb_size);
module::set_var("gallery", "thumb_size", $thumb_size);
}
$resize_size = $form->edit_theme->resize_size->value;
$resize_dirty = false;
if (module::get_var("wind", "resize_size") != $resize_size) {
if (module::get_var("gallery", "resize_size") != $resize_size) {
graphics::remove_rule("gallery", "resize", "gallery_graphics::resize");
graphics::add_rule(
"gallery", "resize", "gallery_graphics::resize",
array("width" => $resize_size, "height" => $resize_size, "master" => Image::AUTO),
100);
module::set_var("wind", "resize_size", $resize_size);
module::set_var("gallery", "resize_size", $resize_size);
}
module::set_var("wind", "header_text", $form->edit_theme->header_text->value);
module::set_var("wind", "footer_text", $form->edit_theme->footer_text->value);
module::set_var("wind", "show_credits", $form->edit_theme->show_credits->value);
module::set_var("gallery", "header_text", $form->edit_theme->header_text->value);
module::set_var("gallery", "footer_text", $form->edit_theme->footer_text->value);
module::set_var("gallery", "show_credits", $form->edit_theme->show_credits->value);
}
}

View File

@@ -77,7 +77,7 @@
<?= $theme->site_status() ?>
<div id="g-header" class="ui-helper-clearfix">
<div id="g-banner">
<? if ($header_text = theme::get_var("header_text")): ?>
<? if ($header_text = module::get_var("gallery", "header_text")): ?>
<?= $header_text ?>
<? else: ?>
<a id="g-logo" class="g-left" href="<?= item::root()->url() ?>" title="<?= t("go back to the Gallery home")->for_html_attr() ?>">
@@ -129,11 +129,11 @@
</div>
<div id="g-footer" class="ui-helper-clearfix">
<?= $theme->footer() ?>
<? if ($footer_text = theme::get_var("footer_text")): ?>
<? if ($footer_text = module::get_var("gallery", "footer_text")): ?>
<?= $footer_text ?>
<? endif ?>
<? if (theme::get_var("show_credits")): ?>
<? if (module::get_var("gallery", "show_credits")): ?>
<ul id="g-credits" class="g-inline">
<?= $theme->credits() ?>
</ul>