[kronolith] kronolith: two post-install snags

[Brent]

Quoting cjdl01 <cjdl01 at brokensolstice.com>:

>>> Since there doesn't seem to be an issue with my c library or my  
>>> php, should I file a bug for this?  Or is there something else we  
>>> can do?
>>
>> Since I cannot reproduce this, a bug report won't help either. I'm  
>> afraid you have to track this down yourself.
>>
>> -- 
>> Jan Schneider
>
>
> OK.... I'm not really a programmer though.  Can you point me in the  
> right direction?  Is there a way to force horde to get something  
> other than %r?
>
> I had the same problem with the weather block, and someone posted a  
> fix that involved a minor code tweak.  In horde/config/prefs.php I  
> changed:
>
> // Time format
> $_prefs['time_format'] = array(
>    'value' => '%X',
>    'type' => 'enum',
>    'enum' => array(
>        '%X' => strftime('%X') . ' (' . _("Default") . ')',
>        '%H:%M:%S' => strftime('%H:%M:%S') . ' (' . _("24-hour format") . ')',
>        '%l:%M:%S %p' => strftime('%l:%M:%S %p'),
>        '%R' => strftime('%R') . ' (' . _("24-hour format") . ')',
>        '%l:%M %p' => strftime('%l:%M %p'),
>    ),
>    'desc' => _("Choose how to display times:")
> );
>
> to
>
> // Time format
> $_prefs['time_format'] = array(
>     'value' => '%H:%M:%S',
>     'type' => 'enum',
>     'enum' => array(
>         '%X' => strftime('%X') . ' (' . _("Default") . ')',
>         '%H:%M:%S' => strftime('%H:%M:%S') . ' (' . _("24-hour  
> format") . ')',
>         '%l:%M:%S %p' => strftime('%l:%M:%S %p'),
>         '%R' => strftime('%r') . ' (' . _("24-hour format") . ')',
>         '%l:%M %p' => strftime('%l:%M %p'),
>     ),
>     'desc' => _("Choose how to display times:")
> );
>
> And that fixed it (actually I made a prefs.local.php like a good  
> boy).  I had really hoped it would carry over into Kronolith, but it  
> didn't.  Is there something similar I can do for Kronolith?
>
> Any help you can give me here would be greatly appreciated.  Thanks.

I would be the one that helped fixed your %r in Weather.  It seems  
that the following function is somehow not working correctly in the  
kronolith config.  If you run the strftime in a standalone php, it is  
correct.  However, if you view source on the resulting .html that is  
sent to users, the time is coming as %r instead.  The .ics file  
generated is correct, so something gets munged in the .html  
generation.  No idea what.

The fix/hack:

Edit:  <hordedir>/kronolith/templates/itip/notification.html.php

Search for:  strftime('%x %X')

Change to:   strftime('%c')

Note:  leaving this alone and reading mail via Horde, you do not get a  
%r.  You only get the %r if you send to something like Gmail or an  
Exchange server.  The %r is in the .html generated, but not in the  
.ics file.

I note that %r in the strftime format means "time in am and pm  
notation".  Somehow, the %X (preferred time designation without the  
date) is returning %r.  Could it be something is not setting the  
preferred timed designation correctly somewhere?  A standalone .php  
script returns the correct time when using %x %X.

brent