6.0.0-alpha12
6/8/25

[#6097] Queue name not displaying in ticket history
Summary Queue name not displaying in ticket history
Queue Whups
Type Bug
State No Feedback
Priority 1. Low
Owners
Requester paul.g.roy (at) uottawa (dot) ca
Created 01/09/2008 (6360 days ago)
Due
Updated 01/16/2008 (6353 days ago)
Assigned 01/09/2008 (6360 days ago)
Resolved 01/16/2008 (6353 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
01/16/2008 10:52:18 PM Jan Schneider State ⇒ No Feedback
 
01/10/2008 03:57:25 PM Chuck Hagenbuch Comment #6 Reply to this comment
It's exactly the same logic - that makes no sense, and isn't broken 
for anyone else. Are you by chance using the suhosin patch?
01/10/2008 01:44:22 PM paul (dot) g (dot) roy (at) uottawa (dot) ca Comment #5 Reply to this comment
Gotcha.



There's definitely history for the ticket that was fetched.



I fixed the problem by changing line 951 from



    for ($i = 0, $iMax = count($history); $i < $iMax; ++$i) {



to



        $count = count($history);

         for ($i = 0; $i < $count; $i++) {





which is basically the same logic except for the $i++ portion.



Queue name and all other history info display fine now.






01/10/2008 04:49:56 AM Chuck Hagenbuch Comment #4 Reply to this comment
The date you last ran cvs update was what I had in mind. Sounds pretty 
recent if your sql driver was updated 7 days ago.



If the for loop doesn't execute that means no history was fetched, 
which probably means there was a database error just above - is there 
anything in your Horde log? Have you run the update scripts from 
whups/scripts/upgrades/ ?
01/09/2008 09:52:11 PM paul (dot) g (dot) roy (at) uottawa (dot) ca Comment #3 Reply to this comment
Fair enough.



As to version info, Whups has been 1.0-cvs for as long as I can 
remember so you'll have to educate me on where I can find a build # 
for the application.



With respect to the offending whups/lib/Driver/sql.php file, it's 
version 1.282 dated 2008/01/02.



Cheers!





Paul


01/09/2008 09:06:19 PM Chuck Hagenbuch Comment #2
State ⇒ Feedback
Reply to this comment
Whups isn't released for Horde 3.1.x, so you may want to try Horde 
3.2-RC1. And please be more specific than "recent" when talking about 
a HEAD application.
01/09/2008 09:04:03 PM paul (dot) g (dot) roy (at) uottawa (dot) ca Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Queue name not displaying in ticket history
Queue ⇒ Whups
State ⇒ Unconfirmed
Reply to this comment
The name of the queue does not display in the History section when 
viewing an existing ticket: the area after Queue=> is blank.



The problem is in file /whups/lib/Driver/sql.php at line 951. The for 
loop is not executing.



         for ($i = 0, $iMax = count($history); $i < $iMax; ++$i) {



Using Horde 3.1.5 and a recent Whups build from CVS.



Thanks much



Paul

Saved Queries