Author: ralfbecker
New Revision: 55926
URL: http://svn.stylite.de/viewvc/egroupware?rev=55926&view=rev
Log:
- Mail/Addressbook/Calendar: add and use charset when sending/receiving mails with vCard/iCals, change default vCard charset preference to utf-8
Modified:
trunk/addressbook/inc/class.addressbook_hooks.inc.php
trunk/addressbook/inc/class.addressbook_vcal.inc.php
trunk/addressbook/js/app.js
trunk/calendar/inc/class.calendar_ical.inc.php
trunk/egroupware/api/src/CalDAV/IcalIterator.php
trunk/egroupware/api/src/Mail.php
trunk/egroupware/api/src/Mailer.php
trunk/infolog/inc/class.infolog_ical.inc.php
trunk/mail/inc/class.mail_compose.inc.php
trunk/mail/inc/class.mail_ui.inc.php
— trunk/addressbook/inc/class.addressbook_hooks.inc.php (original)
+++ trunk/addressbook/inc/class.addressbook_hooks.inc.php Fri Apr 29 09:51:13 2016
@@ -305,7 +305,7 @@
‘help’ => ‘Which charset should be used for the vCard import and export.’,
‘xmlrpc’ => True,
‘admin’ => false,
— trunk/addressbook/inc/class.addressbook_vcal.inc.php (original)
+++ trunk/addressbook/inc/class.addressbook_vcal.inc.php Fri Apr 29 09:51:13 2016
@@ -12,9 +12,10 @@
+use EGroupware\Api;
+
/**
- Addressbook - vCard parser
-
*/
class addressbook_vcal extends addressbook_bo
{
@@ -555,11 +556,17 @@
$container = false;
$vCard = Horde_Icalendar::newComponent(‘vcard’, $container);
-
if (!$vCard->parsevCalendar($_vcard, 'VCARD', $charset))
-
if ($charset && $charset != 'utf-8')
-
{
-
$_vcard = Api\Translation::convert($_vcard, $charset, 'utf-8');
-
}
-
if (!$vCard->parsevCalendar($_vcard, 'VCARD'))
{
return False;
}
— trunk/addressbook/js/app.js (original)
+++ trunk/addressbook/js/app.js Fri Apr 29 09:51:13 2016
@@ -230,7 +230,7 @@
extras.owner.push(‘c’+ids);
}
}
@@ -704,13 +704,13 @@
adb_mail_vcard: function(_action, _elems)
{
var app_registry = egw.link_get_registry(‘mail’);
-
if (typeof app_registry['view'] == 'undefined') app_registry = egw.link_get_registry('felamimail');
var link = egw().link("/index.php","menuaction="+app_registry['add']['menuaction']);
for (var i = 0; i < _elems.length; i++)
{
var idToUse = _elems[i].id;
var idToUseArray = idToUse.split('::');
idToUse = idToUseArray[1];
-
link += "&preset[type][]="+encodeURIComponent("text/vcard; charset="+(egw.preference('vcard_charset', 'addressbook') || 'utf-8'));
link += "&preset[file][]="+encodeURIComponent("vfs://default/apps/addressbook/"+idToUse+"/.entry");
}
if (typeof app_registry['view'] != 'undefined' && typeof app_registry['view_popup'] != 'undefined' )
@@ -903,7 +903,7 @@
break;
}
},
- /**
- Get title in order to set it as document title
-
@returns {string}
— trunk/calendar/inc/class.calendar_ical.inc.php (original)
+++ trunk/calendar/inc/class.calendar_ical.inc.php Fri Apr 29 09:51:13 2016
@@ -2264,7 +2264,11 @@
$events = array();
$vcal = new Horde_Icalendar;
-
if ($charset && $charset != 'utf-8')
-
{
-
$_vcalData = Api\Translation::convert($_vcalData, $charset, 'utf-8');
-
}
-
if (!$vcal->parsevCalendar($_vcalData, 'VCALENDAR'))
{
if ($this->log)
{
— trunk/egroupware/api/src/CalDAV/IcalIterator.php (original)
+++ trunk/egroupware/api/src/CalDAV/IcalIterator.php Fri Apr 29 09:51:13 2016
@@ -253,8 +253,12 @@
return;
//return PEAR::raiseError(“Unable to create object for type $type”);
}
-
//error_log(__METHOD__."() about to call parsevCalendar('".substr($data,0,100)."...','$type','$this->charset')");
-
$this->component->parsevCalendar($data, $type, $this->charset);
-
if ($this->charset && $this->charset != 'utf-8')
-
{
-
$data = Api\Translation::convert($data, $this->charset, 'utf-8');
-
}
-
//error_log(__METHOD__."() about to call parsevCalendar('".substr($data,0,100)."...','$type')");
-
$this->component->parsevCalendar($data, $type);
// VTIMEZONE components are NOT returned, they are only processed internally
if ($type == 'VTIMEZONE')
— trunk/egroupware/api/src/Mail.php (original)
+++ trunk/egroupware/api/src/Mail.php Fri Apr 29 09:51:13 2016
@@ -5708,11 +5708,10 @@
if ($part && ($partDisposition==‘attachment’ || $partDisposition==‘inline’ || ($part->getPrimaryType() == ‘text’ && $part->getSubType() == ‘calendar’)))
{
//$headerObject=$part->getAllDispositionParameters();//not used anywhere around here
-
$structure_bytes = $part->getBytes();
-
$structure_mime=$part->getType();
-
$structure_partID=$part->getMimeId();
-
$filename=$part->getName();
-
//error_log(__METHOD__.__LINE__." fetchPartContents(".array2string($_uid).", $structure_partID, $_stream, $_preserveSeen,$structure_mime)" );
-
$structure_mime = $part->getType();
-
$filename = $part->getName();
-
$charset = $part->getContentTypeParameter('charset');
-
//$structure_bytes = $part->getBytes(); $structure_partID=$part->getMimeId(); error_log(__METHOD__.__LINE__." fetchPartContents(".array2string($_uid).", $structure_partID, $_stream, $_preserveSeen,$structure_mime)" );
$this->fetchPartContents($_uid, $part, $_stream, $_preserveSeen=true,$structure_mime);
if ($_returnPart) return $part;
}
@@ -5727,8 +5726,8 @@
}
$attachmentData = array(
‘type’ => $structure_mime,
— trunk/egroupware/api/src/Mailer.php (original)
+++ trunk/egroupware/api/src/Mailer.php Fri Apr 29 09:51:13 2016
@@ -321,19 +321,18 @@
* “text/calendar; method=…” get automatic detected and added as highest priority alternative
*
* @param string|resource $data Path to the attachment or open file-descriptor
-
-
@param string $name The file name to use for the attachment.
-
-
@param string $type The content type of the file.
-
-
@param string $charset The character set of the part, only relevant for text parts.
-
-
@param string $name =null file name to use for the attachment
-
-
@param string $type =null content type of the file, incl. parameters eg. “text/plain; charset=utf-8”
-
-
@param string $old_type =null used to support phpMailer signature (deprecated)
-
@return integer part-number
-
@throws Exception\NotFound if $file could not be opened for reading
*/
- public function addAttachment($data, $name = null, $type = null, $charset = ‘us-ascii’)
- public function addAttachment($data, $name = null, $type = null, $old_type=null)
{
// deprecated PHPMailer::AddAttachment($path, $name = ‘’, $encoding = ‘base64’, $type = ‘application/octet-stream’) call
if ($type === ‘base64’)
{
-
$type = $charset;
-
$charset = 'us-ascii';
@@ -358,6 +357,14 @@
$part = new Horde_Mime_Part();
$part->setType($type);
-
// set content-type parameters, which get ignored by setType
-
if (preg_match_all('/;\s*([^=]+)=([^;]*)/', $type, $matches))
-
{
-
foreach($matches[1] as $n => $label)
-
{
-
$part->setContentTypeParameter($label, $matches[2][$n]);
-
}
-
}
$part->setContents($resource);
// setting name, also sets content-disposition attachment (!), therefore we have to do it after "text/calendar; method=" handling
@@ -365,7 +372,7 @@
// this should not be necessary, because binary data get detected by mime-type,
// but at least Cyrus complains about NUL characters
-
if (substr($type, 0, 5) != 'text/') $part->setTransferEncoding('base64', array('send' => true));
$part->setDisposition('attachment');
return $this->addMimePart($part);
— trunk/infolog/inc/class.infolog_ical.inc.php (original)
+++ trunk/infolog/inc/class.infolog_ical.inc.php Fri Apr 29 09:51:13 2016
@@ -405,7 +405,7 @@
foreach((array)$value as $compvData)
{
$comp = Horde_Icalendar::newComponent(substr($name,3), $vevent);
@@ -654,7 +654,11 @@
}
$vcal = new Horde_Icalendar;
-
if ($charset && $charset != 'utf-8')
-
{
-
$_vcalData = Api\Translation::convert($_vcalData, $charset, 'utf-8');
-
}
-
if (!($vcal->parsevCalendar($_vcalData, 'VCALENDAR')))
{
if ($this->log)
{
@@ -1096,7 +1100,11 @@
case 'text/x-vnote':
$vnote = new Horde_Icalendar;
-
if ($charset && $charset != 'utf-8')
-
{
-
$_data = Api\Translation::convert($_data, $charset, 'utf-8');
-
}
-
if (!$vnote->parsevCalendar($_data, 'VCALENDAR')) return false;
$components = $vnote->getComponent();
foreach ($components as $component)
— trunk/mail/inc/class.mail_compose.inc.php (original)
+++ trunk/mail/inc/class.mail_compose.inc.php Fri Apr 29 09:51:13 2016
@@ -913,6 +913,11 @@
{
$name .= ‘.’.mime_magic::mime2ext($type);
}
-
// use type specified by caller, if Vfs reports only default, or contains specified type (eg. "text/vcard; charset=utf-8")
-
if (!empty($types[$k]) && ($type == 'application/octet-stream' || stripos($types[$k], $type) === 0))
-
{
-
$type = $types[$k];
-
}
$path = str_replace('+','%2B',$path);
$formData = array(
'name' => $name,
— trunk/mail/inc/class.mail_ui.inc.php (original)
+++ trunk/mail/inc/class.mail_ui.inc.php Fri Apr 29 09:51:13 2016
@@ -2383,7 +2383,7 @@
$eventid = $calendar_ical->search($attachment[‘attachment’],-1);
//error_log(METHOD.array2string($eventid));
if (!$eventid) $eventid = -1;
-
$event = $calendar_ical->importVCal($attachment['attachment'],(is_array($eventid)?$eventid[0]:$eventid),null,true);
-
$event = $calendar_ical->importVCal($attachment['attachment'],(is_array($eventid)?$eventid[0]:$eventid),null,true,0,'',null,$attachment['charset']);
//error_log(__METHOD__.$event);
if ((int)$event > 0)
{
@@ -2401,7 +2401,7 @@
// double \r\r\n seems to end a vcard prematurely, so we set them to \r\n
//error_log(METHOD.LINE.$attachment[‘attachment’]);
$attachment[‘attachment’] = str_replace("\r\r\n", “\r\n”, $attachment[‘attachment’]);
@@ -2412,7 +2412,7 @@
// if there are not enough fields in the vcard (or the parser was unable to correctly parse the vcard (as of VERSION:3.0 created by MSO))
if ($contact || count($vcard)>2)
{
-
$contact = $addressbook_vcal->addVCard($attachment['attachment'],(is_array($contact)?array_shift($contact):$contact),true,$attachment['charset']);
}
if ((int)$contact > 0)
{
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
eGroupWare-cvs mailing list
eGroupWare-cvs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/egroupware-cvs
