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

* Your Email Address
* Spam protection	Enter the letters below: .___.__.. .. ,. . [__ [__]\|__\| \./ \|\/\| \| \| \|\| \| \| \| \|
Comment	> 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@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, >
Attachment
Watch this ticket

Saved Queries