[Tickets #13315] Re: Imp mobile view does not work

[noreply at bugs.horde.org]

BITTE NICHT AUF DIESE NACHRICHT ANTWORTEN. NACHRICHTEN AN DIESE  
E-MAIL-ADRESSE WERDEN NICHT GELESEN.

Ticket-URL: http://bugs.horde.org/ticket/13315
------------------------------------------------------------------------------
  Ticket           | 13315
  Aktualisiert Von | herde at tuhh.de
  Zusammenfassung  | Imp mobile view does not work
  Warteschlange    | IMP
  Version          | Git master
  Typ              | Bug
  Status           | Feedback
  Priorität        | 1. Low
  Milestone        |
  Patch            |
  Zuständige       |
------------------------------------------------------------------------------


herde at tuhh.de (2014-07-18 06:28) hat geschrieben:

We had the same or very similar problem - and we developed a hotfix  
for our special case. This could not be the final solution but its a  
workaround. I think we have do investigate the reason why ...

I found that horde/services/portal/smartmobile.php might produce a  
broken link if the smartmobile link contains a '#' or something like  
this. The imp initial_page is  
/horde/imp/smartmobile.php#mailbox?mbox=SU5CT1g and after building a  
Horde_Core_Smartmobile_Url from this it has the value of just  
/horde/#mailbox?mbox=SU5CT1g - no 'imp' nor 'smartmobile.php'  
contained in the resulting link.

This broken link causing  302 Errors until we get the error message of  
to many recursive redirections.

The patch attached solved the Problem for us.  Probably this is caused  
by our special configuration here - or it is caused by the constructor  
of the Horde_Core_Smartmobile_Url

[root at horde-dev horde/services/portal]# diff smartmobile.o.php smartmobile.php
27a28,29
>     if ($app != 'imp') { $my_smartmobile_url = new  
> Horde_Core_Smartmobile_Url(Horde::url($registry->getInitialPage($app)));  
> }
>     else { $my_smartmobile_url = new  
> Horde_Core_Smartmobile_Url(Horde::url('imp/smartmobile.php')); }
29c31,32
< 	new  
Horde_Core_Smartmobile_Url(Horde::url($registry->getInitialPage($app))),
---
> 	// new  
> Horde_Core_Smartmobile_Url(Horde::url($registry->getInitialPage($app))),
> 	$my_smartmobile_url,