6.0.0-beta1
7/12/25

[#2257] Search error for 'All Past' and 'All Future'
Summary Search error for 'All Past' and 'All Future'
Queue Kronolith
Queue Version 2.0.3
Type Bug
State Resolved
Priority 2. Medium
Owners
Requester csacca (at) thecsl (dot) org
Created 07/12/2005 (7305 days ago)
Due
Updated 07/12/2005 (7305 days ago)
Assigned 07/12/2005 (7305 days ago)
Resolved 07/12/2005 (7305 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
07/12/2005 08:28:52 PM Chuck Hagenbuch Comment #4
State ⇒ Resolved
Reply to this comment
Committed, thanks!
07/12/2005 07:43:02 PM csacca (at) thecsl (dot) org Comment #3
New Attachment: search.patch Download
Reply to this comment
Ok, hacked something up against 2.0.3, but from what I've looked at of 
head it should apply there too.  Wasn't acutally as bad as I though it 
was going to be first.  I did some testing on my changes (basic event, 
recuring event, all day event) and they were good, but a second check 
is always useful.
07/12/2005 07:09:02 PM Chuck Hagenbuch Comment #2
State ⇒ Feedback
Reply to this comment
A patch would be great. I'd put in logic to handle the NULL values, 
personally.
07/12/2005 06:59:27 PM csacca (at) thecsl (dot) org Comment #1
State ⇒ Unconfirmed
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ Search error for 'All Past' and 'All Future'
Queue ⇒ Kronolith
Reply to this comment
When performing a search in kronolith, and using the time frames all 
past or all future, I get the following error:

Fatal error: Call to a member function on a non-object in 
/usr/share/horde3/kronolith/lib/Driver.php on line 98 (or 97)



Through var_dump()'ing $query, I found that it's because $query->start 
or $query->end is set to null.  In HEAD, this happend on lines 74-79 
of search.php.  It seems that you either need to set the values for 
->start and ->end to real, though almost infinite times, or build 
logic into lib/Driver.php to deal with the case that one of them is 
null.



I'd be willing to patch this, but I'm not sure on which way to go.



Thanks,



Chris Sacca

Saved Queries