Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 235568 - www-misc/zoneminder-1.23.3 event timeline Undefined variable: filter_query
Summary: www-misc/zoneminder-1.23.3 event timeline Undefined variable: filter_query
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Web Application Packages Maintainers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-23 20:18 UTC by Michele Schiavo
Modified: 2008-09-10 07:52 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
zoneminder-1.23.3-r1.ebuild (zoneminder-1.23.3-r1.ebuild,3.65 KB, text/plain)
2008-08-23 20:19 UTC, Michele Schiavo
Details
zm_html_view.patch (zm_html_view.patch,362 bytes, patch)
2008-08-23 20:20 UTC, Michele Schiavo
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michele Schiavo 2008-08-23 20:18:37 UTC
on the event time line page, it's show this error : 
Notice: Undefined variable: filter_query in /var/www/zoneminder/htdocs/zm_html_view_timeline.php on line 1201

because is not checked if this variable exist.

Reproducible: Always

Steps to Reproduce:
1.go on page events, show timeline
2.
3.

Actual Results:  
$filter_query is used without verify if it exist

Expected Results:  
$filter_query set to "" if not exist
Comment 1 Michele Schiavo 2008-08-23 20:19:49 UTC
Created attachment 163673 [details]
zoneminder-1.23.3-r1.ebuild

add zm_html_view.php patch
Comment 2 Michele Schiavo 2008-08-23 20:20:50 UTC
Created attachment 163675 [details, diff]
zm_html_view.patch

$filter_query patch for zm_html_view_timeline.php
Comment 3 Michele Schiavo 2008-08-23 20:25:13 UTC
 emerge zoneminder -pv

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild   R   ] www-misc/zoneminder-1.23.3-r1  USE="ffmpeg -X10 -debug" 0 kB [1]

Total: 1 package (1 reinstall), Size of downloads: 0 kB
Portage tree and overlays:
 [0] /usr/portage
 [1] /usr/local/portage
Comment 4 Wormo (RETIRED) gentoo-dev 2008-08-24 00:10:39 UTC
Thanks for the patch!

Assigning to maintainers.
Comment 5 Gunnar Wrobel (RETIRED) gentoo-dev 2008-09-10 07:52:29 UTC
Thanks for the patch Michele but this is an upstream issue and you need to submit your patch there.