[dev] [cvs] commit: framework/Kolab_Filter/lib/Horde/Kolab Resource.php

[Jan Schneider]

Zitat von Gunnar Wrobel <p at rdus.de>:

> Quoting Jan Schneider <jan at horde.org>:
>
>> Zitat von Gunnar Wrobel <p at rdus.de>:
>>
>>> Hi Jan,
>>>
>>> Quoting Jan Schneider <jan at horde.org>:
>>>
>>>> Zitat von Jan Schneider <jan at horde.org>:
>>>>
>>>>> jan         2008-09-18 05:59:44 EDT
>>>>>
>>>>> Modified files:
>>>>> Kolab_Filter/lib/Horde/Kolab Resource.php
>>>>> Log:
>>>>> No double quotes, less wrapping, fix gettext strings, coding style.
>>>>>
>>>>> Revision  Changes    Path
>>>>> 1.3       +77 -91    framework/Kolab_Filter/lib/Horde/Kolab/Resource.php
>>>>
>>>> Gunnar, it would also help translators if you could reuse as many  
>>>> strings from the other iTip code in Horde, especially the iTip  
>>>> MIME viewer in IMP.
>>>
>>> I looked at this and saw that the Kolab code has been originally  
>>> copied from the iTip library within IMP. There have been some  
>>> changes to the strings and I can fix some of them. But there are  
>>> some where I believe the change on the Kolab side mide be the  
>>> preferred message.
>>>
>>> Kolab uses "%s has declined the invitation to the following event:  
>>> %s" instead of "%s has declined." used within Horde. While the  
>>> event summary information appended for the Kolab iTip message is  
>>> also present in the message subject I believe it is actually more  
>>> readable for the user.
>>>
>>> Do you agree or are there reasons why we should keep the Horde version?
>>
>> The summary is already part of any rendered iTip message, adding  
>> this again for each attendee doesn't make much sense.
>
> For the dummy event "Cologne"
>
> 1) Horde sends:
>
> Subject: Declined: Cologne
>
> wrobel at example.com has declined.
>
> 2) Kolab sends:
>
> Subject: Declined: Cologne
>
> wrobel at example.com has declined the invitation to the following event:
>
> Cologne
>
>
> Version 2) seems to be much more readable and understandable to me.  
> Version 1) does not tell me that the message is referring to an  
> event at all. This may cause difficulties if a delegate of mine  
> (e.g. secretary) is also accessing my mail account and cannot judge  
> what the message might be about.

That took me quite some time to find out where you actually see this.  
In the text part of invitation replies, right? This part is not even  
visible in IMP.
Anyway, it's fine to change it there.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/