From e7435df04aa38f478fc93d662346c7155ee0f6d2 Mon Sep 17 00:00:00 2001 From: Andrew Dolgov Date: Wed, 17 Nov 2010 16:11:13 +0300 Subject: [PATCH] prefs tab reloading fixes --- modules/pref-filters.php | 5 ----- prefs.js | 32 ++++++++++++++------------------ 2 files changed, 14 insertions(+), 23 deletions(-) diff --git a/modules/pref-filters.php b/modules/pref-filters.php index d796d84e..7b64b2a8 100644 --- a/modules/pref-filters.php +++ b/modules/pref-filters.php @@ -208,11 +208,6 @@ action_param = '$action_param', filter_param = '$filter_param' WHERE id = '$filter_id' AND owner_uid = " . $_SESSION["uid"]); - - if (db_affected_rows($link, $result) != 0) { - print_notice(T_sprintf("Saved filter %s", htmlspecialchars($reg_exp))); - } - } if ($subop == "remove") { diff --git a/prefs.js b/prefs.js index db84a210..de19eaeb 100644 --- a/prefs.js +++ b/prefs.js @@ -16,8 +16,8 @@ function feedlist_callback2(transport) { try { - var container = $('feedConfigTab'); - container.innerHTML=transport.responseText; + dijit.byId('feedConfigTab').attr('content', transport.responseText); + selectTab("feedConfig", true); if (caller_subop) { @@ -37,8 +37,7 @@ function feedlist_callback2(transport) { } function filterlist_callback2(transport) { - var container = $('filterConfigTab'); - container.innerHTML=transport.responseText; + dijit.byId('filterConfigTab').attr('content', transport.responseText); notify(""); remove_splash(); } @@ -70,14 +69,13 @@ function labellist_callback2(transport) { try { - var container = $('labelConfigTab'); - closeInfoBox(); - container.innerHTML=transport.responseText; + dijit.byId('labelConfigTab').attr('content', transport.responseText); + closeInfoBox(); - init_label_inline_editor(); + init_label_inline_editor(); - notify(""); - remove_splash(); + notify(""); + remove_splash(); } catch (e) { exception_error("labellist_callback2", e); @@ -86,12 +84,10 @@ function labellist_callback2(transport) { function userlist_callback2(transport) { try { - var container = $('userConfigTab'); - if (transport.readyState == 4) { - container.innerHTML=transport.responseText; - notify(""); - remove_splash(); - } + dijit.byId('userConfigTab').attr('content', transport.responseText); + + notify(""); + remove_splash(); } catch (e) { exception_error("userlist_callback2", e); } @@ -99,8 +95,8 @@ function userlist_callback2(transport) { function prefslist_callback2(transport) { try { - var container = $('genConfigTab'); - container.innerHTML=transport.responseText; + dijit.byId('genConfigTab').attr('content', transport.responseText); + notify(""); remove_splash(); } catch (e) { -- 2.39.5