From 74e44765b890bdbcc616b0d0facd816f01f8ac9e Mon Sep 17 00:00:00 2001 From: pdontthink Date: Fri, 30 Jul 2004 12:36:36 +0000 Subject: [PATCH 1/1] Custom option page values now repopulate correctly when save action goes back to same options page git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@7780 7612ce4b-ef26-0410-bec9-ea0150e637f0 --- ChangeLog | 1 + functions/options.php | 11 +++++++++++ 2 files changed, 12 insertions(+) diff --git a/ChangeLog b/ChangeLog index 7831dc2f..5c4239c8 100644 --- a/ChangeLog +++ b/ChangeLog @@ -72,6 +72,7 @@ Version 1.5.1 -- CVS - Removed html_top and html_bottom hooks. No longer used/needed. - Added "trailing text" for options built by SquirrelMail (text placed after text and select list inputs on options pages) + - Custom option page values now repopulate correctly Version 1.5.0 -------------------- diff --git a/functions/options.php b/functions/options.php index 7fadcbc6..5ae717a0 100644 --- a/functions/options.php +++ b/functions/options.php @@ -154,6 +154,12 @@ class SquirrelOption { function createHTMLWidget() { global $javascript_on; + // Use new value if available + if (!empty($this->new_value)) { + $tempValue = $this->value; + $this->value = $this->new_value; + } + /* Get the widget for this option type. */ switch ($this->type) { case SMOPT_TYPE_STRING: @@ -192,6 +198,11 @@ class SquirrelOption { /* Add the "post script" for this option. */ $result .= $this->post_script; + // put correct value back if need be + if (!empty($this->new_value)) { + $this->value = $tempValue; + } + /* Now, return the created widget. */ return ($result); } -- 2.25.1