From 872323e44c68703e6f4e803010c4620567d87c89 Mon Sep 17 00:00:00 2001 From: Imanol Losada Date: Mon, 16 Mar 2015 16:09:43 -0500 Subject: [PATCH] Fix advanced subscription settings updating and error message feedback - refs BT#9578 --- main/admin/configure_plugin.php | 4 ++++ main/inc/lib/api.lib.php | 5 +++++ 2 files changed, 9 insertions(+) diff --git a/main/admin/configure_plugin.php b/main/admin/configure_plugin.php index d4e2548c41..1c5325aa42 100755 --- a/main/admin/configure_plugin.php +++ b/main/admin/configure_plugin.php @@ -89,6 +89,10 @@ if (isset($form)) { header("Location: $currentUrl"); exit; + } else { + foreach ($form->_errors as $error) { + $message .= Display::return_message($error, 'error'); + } } } diff --git a/main/inc/lib/api.lib.php b/main/inc/lib/api.lib.php index 8fca9356d1..d3288a35df 100644 --- a/main/inc/lib/api.lib.php +++ b/main/inc/lib/api.lib.php @@ -5235,6 +5235,11 @@ function api_add_setting($val, $var, $sk = null, $type = 'textfield', $c = null, $res = Database::query($select); if (Database::num_rows($res) > 0) { // Found item for this access_url. $row = Database::fetch_array($res); + Database::update( + $t_settings, + array('selected_value' => $val), + array('id = ?' => array($row['id'])) + ); return $row['id']; }