Tickets :: [#11386] curr_row is undefined

6.0.0-beta1

7/7/25

Summary	curr_row is undefined
Queue	IMP
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	jan (at) horde (dot) org
Created	08/30/2012 (4694 days ago)
Due
Updated	10/15/2012 (4648 days ago)
Assigned	09/05/2012 (4688 days ago)
Resolved	09/06/2012 (4687 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

10/15/2012 08:37:51 PM	Git Commit	Comment #5	Reply to this comment
Changes have been made in Git (develop): commit 862a20747f33485e0366f8db62f9484d6cf4bd27 Author: Michael M Slusarz <slusarz@horde.org> Date: Thu Sep 6 09:05:52 2012 -0600 ~~Bug #11386~~: Correctly update selected state if not current view imp/js/viewport.js \| 9 ++++----- 1 files changed, 4 insertions(+), 5 deletions(-) http://git.horde.org/horde-git/-/commit/862a20747f33485e0366f8db62f9484d6cf4bd27

09/06/2012 03:10:45 PM	Michael Slusarz	Comment #4 State ⇒ Resolved	Reply to this comment
Looking through the viewport code, I did verify that it would not update the selected state if the view was not currently active. That is incorrect: we should update the cached DB and just not trigger any UI update events in this case. I believe I have fixed this so marking as resolved. Reopen if seen again.

09/06/2012 03:06:49 PM	Git Commit	Comment #3	Reply to this comment
Changes have been made in Git (master): commit 862a20747f33485e0366f8db62f9484d6cf4bd27 Author: Michael M Slusarz <slusarz@horde.org> Date: Thu Sep 6 09:05:52 2012 -0600 ~~Bug #11386~~: Correctly update selected state if not current view imp/js/viewport.js \| 9 ++++----- 1 files changed, 4 insertions(+), 5 deletions(-) http://git.horde.org/horde-git/-/commit/862a20747f33485e0366f8db62f9484d6cf4bd27

09/05/2012 09:37:25 PM	Michael Slusarz	Comment #2 State ⇒ Feedback	Reply to this comment
I have the vague idea that this happens if you have a row selected, delete it, move to another mailbox before the ajax response returned, and then return back to the original mailbox. This doesn't sound right. Because this code is in moveSelected(), and the delete message request would only be triggered after this code is executed. Someone will need to find a reliable way for me to reproduce.

08/30/2012 01:39:57 PM	Jan Schneider	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ curr_row is undefined Queue ⇒ IMP Assigned to Michael Slusarz Milestone ⇒ Patch ⇒ No State ⇒ Assigned	Reply to this comment
Line 154 of dimpbase.js. I have the vague idea that this happens if you have a row selected, delete it, move to another mailbox before the ajax response returned, and then return back to the original mailbox.