* Mail: allow to enter name+mail eg. "Ralf Becker <rb@stylite.de>" in compose, automatic fix unquoted commas in entered mail addresses

This commit is contained in:
Ralf Becker 2014-10-08 09:09:02 +00:00
parent a19fe17f45
commit 065f772302
4 changed files with 144 additions and 23 deletions

View File

@ -68,7 +68,39 @@ class etemplate_widget_date extends etemplate_widget_transformer
if($this->type != 'date-duration' && $value)
{
// string with formatting letters like for php's date() method
$value = $this->format_date($value);
}
}
/**
* Perform any needed data manipulation on each row
* before sending it to client.
*
* This is used by etemplate_widget_nextmatch on each row to do any needed
* adjustments. If not needed, don't implement it.
*
* @param type $cname
* @param array $expand
* @param array $data Row data
* @return type
*/
public function set_row_value($cname, Array $expand, Array &$data)
{
if($this->type == 'date-duration') return;
$form_name = self::form_name($cname, $this->id, $expand);
$value =& $this->get_array($data, $form_name, true);
$value = $this->format_date($value);
}
/**
* Put date in the proper format for sending to client
* @param string|int $value
* @param string $format
*/
public function format_date($value)
{
if ($this->attrs['dataformat'] && !is_numeric($value))
{
$date = date_create_from_format($this->attrs['dataformat'], $value, egw_time::$user_timezone);
@ -86,7 +118,7 @@ class etemplate_widget_date extends etemplate_widget_transformer
// postfix date-string with "Z" so javascript doesn't add/subtract anything
$value = $date->format('Y-m-d\TH:i:s\Z');
}
}
return $value;
}
/**

View File

@ -357,6 +357,24 @@ class etemplate_widget_nextmatch extends etemplate_widget
$row_id = isset($value['row_id']) ? $value['row_id'] : 'id';
$row_modified = $value['row_modified'];
if($template && $template->attrs['template'])
{
$row_template = $template->getElementById($template->attrs['template']);
if(!$row_template)
{
$row_template = etemplate_widget_template::instance($template->attrs['template']);
}
// Try to find just the repeating part
$repeating_child = null;
// First child should be a grid, we want last row
foreach($row_template->children[0]->children[1]->children as $child)
{
if($child->type == 'row') $repeating_child = $child;
}
$row_template = $repeating_child ? $repeating_child : null;
}
foreach($rows as $n => $row)
{
$kUkey = false;
@ -373,8 +391,21 @@ class etemplate_widget_nextmatch extends etemplate_widget
if (!$row_id || !$knownUids || ($kUkey = array_search($id, $knownUids)) === false ||
!$lastModified || !isset($row[$row_modified]) || $row[$row_modified] > $lastModified)
{
if($row_template)
{
// Change anything by widget for each row ($row set to 1)
$_row = array(1 => &$row);
$row_template->run('set_row_value', array('',array('row' => 1), &$_row), true);
$result['data'][$id] = $row;
}
else if (!$template && !get_class($template) != 'etemplate_widget_historylog')
{
// Fallback based on widget names
error_log(self::$request->template['name'] . ' had to fallback to run_beforeSendToClient() because it could not find the row' );
$result['data'][$id] = self::run_beforeSendToClient($row);
}
}
if ($kUkey !== false) unset($knownUids[$kUkey]);
}
else // non-row data set by get_rows method

View File

@ -99,7 +99,13 @@ var et2_taglist = et2_selectbox.extend(
// Selectbox attributes that are not applicable
"multiple": { ignore: true},
"rows": { ignore: true},
"tags": { ignore: true}
"tags": { ignore: true},
useCommaKey: {
name: "comma will start validation",
type: "boolean",
"default": true,
description: "Set to false to allow comma in entered content"
}
},
// Allows sub-widgets to override options to the library
@ -160,6 +166,7 @@ var et2_taglist = et2_selectbox.extend(
required: this.options.required,
allowFreeEntries: this.options.allowFreeEntries,
useTabKey: true,
useCommaKey: this.options.useCommaKey,
disabled: this.options.disabled || this.options.readonly,
editable: !(this.options.disabled || this.options.readonly),
selectionRenderer: jQuery.proxy(this.options.tagRenderer || this.selectionRenderer,this),
@ -474,6 +481,12 @@ var et2_taglist_email = et2_taglist.extend(
description:"Include mailing lists in search results",
default: false,
type: "boolean"
},
useCommaKey: {
name: "comma will start validation",
type: "boolean",
"default": false,
description: "Set to false to allow comma in entered content"
}
},
lib_options: {
@ -507,6 +520,48 @@ var et2_taglist_email = et2_taglist.extend(
// We check free entries for valid email, and render as invalid if it's not.
var valid = item.id != item.label || et2_url.prototype.EMAIL_PREG.test(item.id || '');
if (!valid && item.id)
{
// automatic quote 'Becker, Ralf <rb@stylite.de>' as '"Becker, Ralf" <rb@stylite.de>'
var matches = item.id.match(/^(.*) ?<(.*)>$/);
if (matches && et2_url.prototype.EMAIL_PREG.test('"'+matches[1].trim()+'" <'+matches[2].trim()+'>'))
{
item.id = item.label = '"'+matches[1].trim()+'" <'+matches[2].trim()+'>';
valid = true;
}
// automatic insert multiple comma-separated emails like "rb@stylite.de, hn@stylite.de"
if (!valid)
{
var parts = item.id.split(/, */);
if (parts.length > 1)
{
valid = true;
for(var i=0; i < parts.length; ++i)
{
parts[i] = parts[i].trim();
if (!et2_url.prototype.EMAIL_PREG.test(parts[i]))
{
valid = false;
break;
}
}
if (valid)
{
item.id = item.label = parts.shift();
// insert further parts into taglist, after validation first one
var taglist = this.taglist;
window.setTimeout(function()
{
for(var i=0; i < parts.length; ++i)
{
taglist.addToSelection({id: parts[i], label: parts[i]});
}
}, 10);
}
}
}
}
var label = jQuery('<span>').text(item.label);
if (item.class) label.addClass(item.class);
if (typeof item.title != 'undefined') label.attr('title', item.title);

View File

@ -1291,7 +1291,9 @@
}
break;
case 188: // comma
if(!cfg.useCommaKey) break;
if(e.shiftKey) break; // Shift + , = < on some keyboards
if(e.originalEvent && e.originalEvent.keyIdentifier && e.originalEvent.keyIdentifier != "U+002C") break;
case 9: // tab
case 13: // enter
e.preventDefault();
@ -1348,7 +1350,8 @@
break;
case 13:case 9:case 188:// enter, tab, comma
// Shift + comma = < on English keyboard
if(e.keyCode !== 188 || (cfg.useCommaKey === true && !e.shiftKey)) {
if(e.keyCode !== 188 || (cfg.useCommaKey === true &&
!(e.shiftKey || e.originalEvent && e.originalEvent.keyIdentifier && e.originalEvent.keyIdentifier != "U+002C"))) {
e.preventDefault();
if(cfg.expanded === true){ // if a selection is performed, select it and reset field
selected = ms.combobox.find('.ms-res-item-active:first');