Closed
Bug 163570
Opened 22 years ago
Closed 22 years ago
Search.pm calls time2str which is not found in the file
Categories
(Bugzilla :: Query/Bug List, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: jussi, Assigned: jussi)
References
()
Details
Attachments
(1 file, 1 obsolete file)
399 bytes,
patch
|
bbaetz
:
review+
bbaetz
:
review+
|
Details | Diff | Splinter Review |
When using a '10d' type date in the 'were changed between' field, bugzilla reports the following error: Undefined subroutine &Bugzilla::Search::time2str called at Bugzilla/Search.pm line 835. I could not duplicate this in http://landfill.bugzilla.org/bugzilla-tip/query.cgi.
Assignee | ||
Comment 1•22 years ago
|
||
Fixes the error by adding &:: in front of the two time2str calls.
Comment 2•22 years ago
|
||
Comment on attachment 95948 [details] [diff] [review] patch |use Date::Format;| at the top (just under the package line) instead
Attachment #95948 -
Flags: review-
Assignee | ||
Comment 3•22 years ago
|
||
Add the use statement instead of &::
Attachment #95948 -
Attachment is obsolete: true
Comment 4•22 years ago
|
||
Hmm. While this is definately the right thing to do, I can't reproduce this. What perl version are you running?
Assignee | ||
Comment 5•22 years ago
|
||
I'm running perl 5.6.1.
Comment 6•22 years ago
|
||
Hmm - so do I Can you give a url which would trigger this, that I can copy-and-paste to my local installation? (ie something without products/components selected)
Assignee | ||
Comment 7•22 years ago
|
||
In my installation, the query in url field causes the error.
Updated•22 years ago
|
Attachment #95956 -
Flags: review+
Comment 8•22 years ago
|
||
Comment on attachment 95956 [details] [diff] [review] patch v2 Hmm. No idea why I couldn't reproduce this. r=bbaetz x2, I'll check it in under the Bugzilla::Util include (for style reasons)
Comment 10•22 years ago
|
||
checked in.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Target Milestone: --- → Bugzilla 2.18
Updated•11 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•