From b0b2fea4f88a8bdd92f6339a51481ead8317aa93 Mon Sep 17 00:00:00 2001
From: Thomas <thomas@roundcube.net>
Date: Thu, 17 Oct 2013 03:11:47 -0400
Subject: [PATCH] Revert "Allow sysadmins to define static responses which are immutable for the user"
---
skins/larry/templates/responseedit.html | 2
program/steps/settings/responses.inc | 8 +--
skins/larry/settings.css | 4 --
program/include/rcmail.php | 15 -------
skins/classic/settings.css | 5 --
config/main.inc.php.dist | 6 ---
skins/classic/templates/responseedit.html | 2
program/steps/settings/edit_response.inc | 19 +++++----
program/js/app.js | 2
9 files changed, 17 insertions(+), 46 deletions(-)
diff --git a/config/main.inc.php.dist b/config/main.inc.php.dist
index 2bd9491..186335e 100644
--- a/config/main.inc.php.dist
+++ b/config/main.inc.php.dist
@@ -406,12 +406,6 @@
// Note: useful when SMTP server stores sent mail in user mailbox
$rcmail_config['no_save_sent_messages'] = false;
-// A static list of canned responses which are immutable for the user
-$rcmail_config['compose_responses_static'] = array(
-// array('name' => 'Canned Response 1', 'text' => 'Static Response One'),
-// array('name' => 'Canned Response 2', 'text' => 'Static Response Two'),
-);
-
// ----------------------------------
// PLUGINS
// ----------------------------------
diff --git a/program/include/rcmail.php b/program/include/rcmail.php
index 343f479..0ae5734 100644
--- a/program/include/rcmail.php
+++ b/program/include/rcmail.php
@@ -346,24 +346,11 @@
* These are stored in local config and user preferences.
*
* @param boolean True to sort the list alphabetically
- * @param boolean True if only this user's responses shall be listed
* @return array List of the current user's stored responses
*/
- public function get_compose_responses($sorted = false, $user_only = false)
+ public function get_compose_responses($sorted = false)
{
$responses = array();
-
- if (!$user_only) {
- foreach ($this->config->get('compose_responses_static', array()) as $response) {
- if (empty($response['key']))
- $response['key'] = substr(md5($response['name']), 0, 16);
- $response['static'] = true;
- $response['class'] = 'readonly';
- $k = $sorted ? '0000-' . strtolower($response['name']) : $response['key'];
- $responses[$k] = $response;
- }
- }
-
foreach ($this->config->get('compose_responses', array()) as $response) {
if (empty($response['key']))
$response['key'] = substr(md5($response['name']), 0, 16);
diff --git a/program/js/app.js b/program/js/app.js
index 44962f1..0396470 100644
--- a/program/js/app.js
+++ b/program/js/app.js
@@ -440,7 +440,7 @@
this.responses_list = new rcube_list_widget(this.gui_objects.responseslist, {multiselect:false, draggable:false, keyboard:false});
this.responses_list.addEventListener('select', function(list){
var win, id = list.get_single_selection();
- p.enable_command('delete', !!id && $.inArray(id, p.env.readonly_responses) < 0);
+ p.enable_command('delete', !!id);
if (id && (win = p.get_frame_window(p.env.contentframe))) {
p.set_busy(true);
p.location_href({ _action:'edit-response', _key:id, _framed:1 }, win);
diff --git a/program/steps/settings/edit_response.inc b/program/steps/settings/edit_response.inc
index 4985677..26f7e6e 100644
--- a/program/steps/settings/edit_response.inc
+++ b/program/steps/settings/edit_response.inc
@@ -24,6 +24,8 @@
// edit-response
if (($key = get_input_value('_key', RCUBE_INPUT_GPC))) {
foreach ($responses as $i => $response) {
+ if (empty($response['key']))
+ $response['key'] = substr(md5($response['name']), 0, 16);
if ($response['key'] == $key) {
$RESPONSE_RECORD = $response;
$RESPONSE_RECORD['index'] = $i;
@@ -33,9 +35,12 @@
}
// save response
-if ($RCMAIL->action == 'save-response' && isset($_POST['_name']) && !$RESPONSE_RECORD['static']) {
+if ($RCMAIL->action == 'save-response' && isset($_POST['_name'])) {
$name = trim(get_input_value('_name', RCUBE_INPUT_POST));
$text = trim(get_input_value('_text', RCUBE_INPUT_POST));
+
+ if (!empty($_REQUEST['_framed']))
+ $RCMAIL->output->framed = 1;
if (!empty($name) && !empty($text)) {
$dupes = 0;
@@ -57,11 +62,9 @@
$responses[] = $response;
}
- $responses = array_filter($responses, function($item){ return empty($item['static']); });
- if ($RCMAIL->user->save_prefs(array('compose_responses' => array_values($responses)))) {
+ if ($RCMAIL->user->save_prefs(array('compose_responses' => $responses))) {
$RCMAIL->output->show_message('successfullysaved', 'confirmation');
- $RCMAIL->output->command('parent.update_response_row', $response, $key);
- $RCMAIL->overwrite_action('edit-response');
+ $RCMAIL->output->command('update_response_row', $response, $key);
$RESPONSE_RECORD = $response;
}
}
@@ -76,7 +79,6 @@
global $RCMAIL, $OUTPUT, $RESPONSE_RECORD;
// Set form tags and hidden fields
- $disabled = !empty($RESPONSE_RECORD['static']);
$key = $RESPONSE_RECORD['key'];
list($form_start, $form_end) = get_form_tags($attrib, 'save-response', $key, array('name' => '_key', 'value' => $key));
unset($attrib['form'], $attrib['id']);
@@ -88,10 +90,10 @@
$label = rcube_label('responsename');
$table->add('title', html::label('ffname', Q(rcube_label('responsename'))));
- $table->add(null, rcube_output::get_edit_field('name', $RESPONSE_RECORD['name'], array('id' => 'ffname', 'size' => $attrib['size'], 'disabled' => $disabled), 'text'));
+ $table->add(null, rcube_output::get_edit_field('name', $RESPONSE_RECORD['name'], array('id' => 'ffname', 'size' => $attrib['size']), 'text'));
$table->add('title', html::label('fftext', Q(rcube_label('responsetext'))));
- $table->add(null, rcube_output::get_edit_field('text', $RESPONSE_RECORD['text'], array('id' => 'fftext', 'size' => $attrib['textareacols'], 'rows' => $attrib['textarearows'], 'disabled' => $disabled), 'textarea'));
+ $table->add(null, rcube_output::get_edit_field('text', $RESPONSE_RECORD['text'], array('id' => 'fftext', 'size' => $attrib['textareacols'], 'rows' => $attrib['textarearows']), 'textarea'));
$out .= $table->show($attrib);
$out .= $form_end;
@@ -99,7 +101,6 @@
return $out;
}
-$OUTPUT->set_env('readonly', !empty($RESPONSE_RECORD['static']));
$OUTPUT->add_handler('responseform', 'rcube_response_form');
$OUTPUT->set_pagetitle(rcube_label(($RCMAIL->action=='add-response' ? 'savenewresponse' : 'editresponse')));
diff --git a/program/steps/settings/responses.inc b/program/steps/settings/responses.inc
index cfc4148..330b4fd 100644
--- a/program/steps/settings/responses.inc
+++ b/program/steps/settings/responses.inc
@@ -26,7 +26,7 @@
if (!empty($name) && !empty($text)) {
$dupes = 0;
- $responses = $RCMAIL->get_compose_responses(false, true);
+ $responses = $RCMAIL->get_compose_responses();
foreach ($responses as $resp) {
if (strcasecmp($name, preg_replace('/\s\(\d+\)$/', '', $resp['name'])) == 0)
$dupes++;
@@ -54,7 +54,7 @@
if ($RCMAIL->action == 'delete-response') {
if ($key = get_input_value('_key', RCUBE_INPUT_GPC)) {
- $responses = $RCMAIL->get_compose_responses(false, true);
+ $responses = $RCMAIL->get_compose_responses();
foreach ($responses as $i => $response) {
if (empty($response['key']))
$response['key'] = substr(md5($response['name']), 0, 16);
@@ -67,7 +67,7 @@
}
if ($deleted) {
- $RCMAIL->output->command('display_message', rcube_label('deletedsuccessfully'), 'confirmation');
+ $RCMAIL->output->command('display_message', rcube_label('successfullydeleted'), 'confirmation');
$RCMAIL->output->command('remove_response', $key);
}
@@ -99,8 +99,6 @@
// set client env
$OUTPUT->add_gui_object('responseslist', $attrib['id']);
- $OUTPUT->set_env('readonly_responses', array_values(array_map(function($rec){ return $rec['key']; },
- array_filter($plugin['list'], function($item){ return !empty($item['static']); }))));
return $out;
}
diff --git a/skins/classic/settings.css b/skins/classic/settings.css
index 336422e..8758c57 100644
--- a/skins/classic/settings.css
+++ b/skins/classic/settings.css
@@ -34,11 +34,6 @@
height: 18px;
}
-#identities-table tbody tr.readonly td
-{
- font-style: italic;
-}
-
#subscription-table tr.virtual td
{
color: #666;
diff --git a/skins/classic/templates/responseedit.html b/skins/classic/templates/responseedit.html
index 67ba35b..fbc5f66 100644
--- a/skins/classic/templates/responseedit.html
+++ b/skins/classic/templates/responseedit.html
@@ -15,7 +15,7 @@
<div id="formfooter">
<div class="footerindent">
- <roundcube:button command="save" type="input" class="button mainaction" label="save" condition="!env:readonly" />
+ <roundcube:button command="save" type="input" class="button mainaction" label="save" />
</div>
</div>
</div>
diff --git a/skins/larry/settings.css b/skins/larry/settings.css
index 7b9fe36..d71e5b5 100644
--- a/skins/larry/settings.css
+++ b/skins/larry/settings.css
@@ -194,10 +194,6 @@
text-overflow: ellipsis;
}
-#identities-table tbody tr.readonly td {
- font-style: italic;
-}
-
#folder-details,
#identity-details {
position: absolute;
diff --git a/skins/larry/templates/responseedit.html b/skins/larry/templates/responseedit.html
index 8f180fe..d2f031b 100644
--- a/skins/larry/templates/responseedit.html
+++ b/skins/larry/templates/responseedit.html
@@ -13,7 +13,7 @@
</div>
<div class="footerleft formbuttons">
- <roundcube:button command="save" type="input" class="button mainaction" label="save" condition="!env:readonly" />
+ <roundcube:button command="save" type="input" class="button mainaction" label="save" />
</div>
<roundcube:include file="/includes/footer.html" />
--
Gitblit v1.9.1