6.0.0-RC7
6/19/26

[#13315] Imp mobile view does not work
Summary Imp mobile view does not work
Queue IMP
Queue Version Git master
Type Bug
State Duplicate
Priority 1. Low
Owners
Requester torben (at) dannhauer (dot) info
Created 7/3/14 (4369 days ago)
Due
Updated 7/20/14 (4352 days ago)
Assigned 7/4/14 (4368 days ago)
Resolved 7/20/14 (4352 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
327 Michael Slusarz State ⇒ Duplicate
 
355 torben (at) dannhauer (dot) info Comment #16 Reply to this comment

[Show Quoted Text - 11 lines]
Thanks for the hint, upgraded my debian squeez to dotdeb PHP packages 
(5.4) and now it works.



2312 Michael Slusarz Comment #15 Reply to this comment
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.
Looking at this more... this comment is a Duplicate of #13342.

Still don't know if this is the root of the issue reported by the OP though.
012 Michael Slusarz Comment #14 Reply to this comment
Is anybody here that sees the issue using PHP 5.3.3?
476 herde (at) tuhh (dot) de Comment #13 Reply to this 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,
511 Michael Slusarz Comment #12 Reply to this comment
The changes of the commit are available in my stadard installation, 
so I assume, the changes are already published in a stable release.
No.  The changes mentioned in this ticket were in Horde_Core.  So I 
have no idea whether you have updated that to the newest release.
Was someone else able to repoduce the error with the latest iOS release?
Nope.  At this point, you are probably going to have to trace the 
cause of the issue yourself locally.
137 torben (at) dannhauer (dot) info Comment #11 Reply to this comment
Michael, sorry for the delay, my child was born.
Congratulations!
Thanks! :)
Is the mentioned commit included in Activesync 2.16.11?
This issue has nothing to do with ActiveSync. Are you replying to 
the correct ticket?
Sorry, I was too tired:
The changes of the commit are available in my stadard installation, so 
I assume, the changes are already published in a stable release. 
Unfortunately the error still exists.

Was someone else able to repoduce the error with the latest iOS release?

Torben
563 Michael Rubinsky Comment #10 Reply to this comment
Michael, sorry for the delay, my child was born.
Congratulations!
Is the mentioned commit included in Activesync 2.16.11?
This issue has nothing to do with ActiveSync. Are you replying to the 
correct ticket?

88 torben (at) dannhauer (dot) info Comment #9 Reply to this comment

[Show Quoted Text - 25 lines]
Hm, neither the horde log nor the apache log throws an error.

However, horde thros an endless list of "EMERG: HORDE User is not 
authorized [pid 18167 on line 1546 of 
"/usr/share/php/Horde/Registry.php"]
" errors - but I do not know what they mean..

398 torben (at) dannhauer (dot) info Comment #8 Reply to this comment

[Show Quoted Text - 14 lines]
Michael, sorry for the delay, my child was born.

Is the mentioned commit included in Activesync 2.16.11?

If yes,it does not help, it still does not work with iOS 7.1.2.
I'll look in the logfiles to get a clue..

Torben

4810 Michael Slusarz Comment #7
Version ⇒ Git master
Reply to this comment
Maybe this fixes?

commit 61dff1f7855ab7243656ab71d776bdbec3764e9f
Author: Michael M Slusarz <slusarz at horde.org>
Date:   Tue Jul 8 15:27:24 2014 -0600

     [mms] Sanity checking in case smartmobile URL contains anchor 
that doesn't exist on the page.

  framework/Core/js/smartmobile.js |   11 +++++++++--
  framework/Core/package.xml       |    4 ++--
  2 files changed, 11 insertions(+), 4 deletions(-)

http://github.com/horde/horde/commit/61dff1f7855ab7243656ab71d776bdbec3764e9f
441 Michael Slusarz Comment #6 Reply to this comment
Can't reproduce either, although I only have Android devices to test on.
17 torben (at) dannhauer (dot) info Comment #5 Reply to this comment
Works fine with 7.1.1. You probably have some JS error. Try with a 
desktop browser and check the console log.
with desktop browser it works as expected.

I'll check it at the weekend..

07 Jan Schneider Comment #4 Reply to this comment
Works fine with 7.1.1. You probably have some JS error. Try with a 
desktop browser and check the console log.
436 torben (at) dannhauer (dot) info Comment #3 Reply to this comment
Cannot reproduce
I tested with iOS 7.1.2 (current stable)

"clicking" on the webmail app just shows the loading progress bar for 
some seconds but nothing happens afterwards..



246 Jan Schneider Comment #2
Priority ⇒ 1. Low
State ⇒ Feedback
Reply to this comment
Cannot reproduce
508 torben (at) dannhauer (dot) info Comment #1
Priority ⇒ 3. High
State ⇒ Unconfirmed
Patch ⇒ No
Milestone ⇒
Summary ⇒ Imp mobile view does not work
Type ⇒ Bug
Queue ⇒ IMP
Reply to this comment
After upgrading to latest horde/imp RC, it is possible to log into 
horde via smartphone, but "clicking" on the webmail link does not work.

Saved Queries