[ZBX-5578] NaN, incorrect data and "strtotime() expects parameter 2 to be long" Created: 2012 Sep 12  Updated: 2017 May 30  Resolved: 2012 Sep 29

Status: Closed
Project: ZABBIX BUGS AND ISSUES
Component/s: Frontend (F)
Affects Version/s: None
Fix Version/s: 2.0.3rc2, 2.1.0

Type: Incident report Priority: Blocker
Reporter: richlv Assignee: Unassigned
Resolution: Fixed Votes: 0
Labels: screen
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

2.0 branch, rev 30205.


Attachments: PNG File 5578.png    

 Description   

open a screen in monitoring section that has two graphs (default screen might do).
do a drag-zoom on the left hand one, but do not release mouse button. move to the other graph, release mouse button there and click once in the graph.

time period from-to in the upper right corner of the scrollbar changes to "NaN.NaN.NaN NaN:NaN - NaN.NaN.NaN NaN:NaN"

with some creative clicking it is possible to get both graphs to change to error images about 'period' not being an integer, but i haven't got the exact steps down here (feel free to try it - click from one graph to another, eventually it will fail).

but that's not the worst yet

click on one of the graphs outside the graph wall area :
strtotime() expects parameter 2 to be long, string given [include/func.inc.php:311]

go back to monitoring -> screens again :
strtotime() expects parameter 2 to be long, string given [include/func.inc.php:311]

check the profile table :

web.screens.stime | 524 | 0 | 0 | NaNNaNNaNNaNNaNNaN

change the period - error message still persists and the incorrect value in profies table is not updated.

a) this should not happen, of course;
b) we should check what we pass to pages - even if we pass a string, we should not get php error;
c) we should check what we ave in the db. that is obviously not a correct value for this param.



 Comments   
Comment by richlv [ 2012 Sep 12 ]

oh, forgot to add - reproduced in opera 12.02, firefox 15.0 and konqueror 4.7.2

Comment by Eduards Samersovs (Inactive) [ 2012 Sep 13 ]

Fixed in ZBXNEXT-1398 development branch svn://svn.zabbix.com/branches/dev/ZBXNEXT-1398

Comment by richlv [ 2012 Sep 13 ]

discovered a few more problems that are regressions, and i believe are blockers as well

(1) drag-zoom, but do not release the mouse button. move mouse cursor outside the graph area, release the button - nothing happens. this is a regression, as mouse cursor outside graph borders worked as if against the border, and release event worked as well (it was explicitly developed that way)

<Eduard> RESOLVED

<richlv> thanks, it works to the left and right of a graph now, but not above/below the graph wall area - it should work also there (see how it's implemented in 2.0.2)

<Eduard> Moved to ZBXNEXT-1351 (19) CLOSED

<Sasha> Seems it fixed.

<richlv> yay, indeed seems to be fixed in dev branch rev 30374.
tested with opera 12.02, konqueror 4.7.2 and firefox 15.0

CLOSED

Comment by richlv [ 2012 Sep 13 ]

(2) if i have a screen with two columns, i can't do drag-zoom on the second column graphs

<Eduard> RESOLVED

<richlv> CLOSED, but see (4)

Comment by richlv [ 2012 Sep 13 ]

(3) screen period is also remembered in config.
open screen in monitoring, set some time period, other than one hour. open same screen in config - that time & period are used. in config screens should always show one hour at "now" (as we have no way to change displayed period there)

<Eduard> RESOLVED

<Toms> CLOSED

Comment by richlv [ 2012 Sep 14 ]

(4) while drag-zooming in graphs in the second column works now, there's something wrong with the area positioning.
open a screen with two columns and a graph in each (i used different graphs to have slightly different layout, not sure whether that's related).
repeatedly start dragging from the middle of one graph to the very edge of it. do this on both graphs in both directions a few times, and at some point the highlighted blue area will start covering different area than graph wall - it will overflow graph wall area on one side, not go far enough on the other, go outside the vertical graph wall area etc.
not sure about exact steps to reproduce it, but for me usually drag-zooming some 3 times causes this to happen, and then it happens constantly.

<Toms> Zooming in second column doesn't work in IE 9.0.8112
<Eduard> RESOLVED

<Toms> CLOSED

Comment by Toms (Inactive) [ 2012 Sep 17 ]

(5) In Monitoring > Screens, when changing period, it is possible to select 1y 12m 3d, though it should be 2y 3d

<Eduard> Moved to ZBXNEXT-1351 (15) CLOSED

Comment by Toms (Inactive) [ 2012 Sep 17 ]

(6) Monitoring > Screens, when selecting period less than 1h, alignment is done by right side, I believe more convenient would be to align to left side
or set according to language settings

<Toms> Moved to https://support.zabbix.com/browse/ZBXNEXT-1351 as Nr. 16

<Toms> CLOSED

Comment by Toms (Inactive) [ 2012 Sep 17 ]

(7) Don't know if this related to this issue, but in graph on horizontal axis date with time 00:00 is missing time part

see 5578.png

<Eduard> Moved to ZBXNEXT-1351 (18) CLOSED

Comment by Toms (Inactive) [ 2012 Sep 17 ]

(8) gtlc.js line 248 overrides function param objid

<Eduard> RESOLVED

<Toms> Same in line 280
<Eduard> RESOLVED

tomtom CLOSED

Comment by Toms (Inactive) [ 2012 Sep 17 ]

(9)
I offer to write function:
function mouseupSBoxes(e) {
for (var sbid in ZBX_SBOX) {
if (empty(ZBX_SBOX[sbid]) || !isset('sbox', ZBX_SBOX[sbid]))

{ continue; }

ZBX_SBOX[sbid].sbox.mouseup(e);
}
}

like:
function mouseupSBoxes(e) {
for (var sbid in ZBX_SBOX) {
if (!empty(ZBX_SBOX[sbid]) && isset('sbox', ZBX_SBOX[sbid]))

{ ZBX_SBOX[sbid].sbox.mouseup(e); }

}
}

<Eduard> RESOLVED, thanks for suggesting!

<Toms> same for moveSBoxes(), line 201, isNow() line227, line 249, line 281, line 299, ...
<Eduard> RESOLVED

tomtom CLOSED

Comment by Toms (Inactive) [ 2012 Sep 17 ]

(10) trailing comma in flickerscreen.js line 324

<Eduard> RESOLVED

<Toms> CLOSED

Comment by Toms (Inactive) [ 2012 Sep 19 ]

(11) Problems with graph selection, if graph width is changed in screen edit form

<Eduard> Moved to ZBXNEXT-1351 (17) CLOSED

Comment by Eduards Samersovs (Inactive) [ 2012 Sep 19 ]

Refixed in development branch svn://svn.zabbix.com/branches/dev/ZBX-5578

Comment by Pavels Jelisejevs (Inactive) [ 2012 Sep 20 ]

(12) A couple of bugs in IE8.

I have a screen with two graphs.
1. When I select an interval on the first graph, I'm redirected to the graph page.
2. When I try to select an interval on the second graph, the selection goes up to the end by itself.

<Eduard> RESOLVED

<pavels> CLOSED.

Comment by Pavels Jelisejevs (Inactive) [ 2012 Sep 24 ]

(13) In IE8 when you select a time interval and let the mouse button go outside of the graph, the interval selection is not applied.

<Eduard> In IE8 it's hard to fix. My last changes fix event when user click separately outside the graph - then interval selection will be applied. Moved into ZBXNEXT-1351 (21) CLOSED

<Sasha> REOPENED
Before implementation of flicker-free screens we could change interval outside the graph. Now - can not.

<Eduard> RESOLVED

jelisejev CLOSED.

<Sasha> REOPENED. In IE9 when you select a time interval and let the mouse button go outside of the graph, the interval selection is applied but selection any more doesn't work

<Sasha> CLOSED

Comment by Pavels Jelisejevs (Inactive) [ 2012 Sep 24 ]

(14) The shouldn't be any padding in screens with graphs in IE.

<Eduard> Moved into ZBXNEXT-1351 (20)

Comment by Eduards Samersovs (Inactive) [ 2012 Sep 24 ]

Fixed in versions pre-2.1.0 (beta) r30326, pre-2.0.3 r30325

Comment by Pavels Jelisejevs (Inactive) [ 2012 Sep 26 ]

(15) Another bug with interval selection. When I open the screen I have an interval of 1h selected. Then I zoom out to 10 months and try to select a time period on the graph. The selection still thinks, that the current interval is just 1 hour.

Eduards RESOLVED! Now screens must fly! 8-)

<Sasha> CLOSED

Comment by Alexander Vladishev [ 2012 Sep 26 ]

(16) Simple graphs disappeared.

Eduards RESOLVED

jelisejev CLOSED.

Comment by Pavels Jelisejevs (Inactive) [ 2012 Sep 27 ]

(17) When I try to select an interval on a graph in a subscreen in IE9, I get a JS error.

Eduards Please re-test, after last fix can't reproduce this issue

jelisejev Still reproducible.

Eduards RESOLVED

jelisejev CLOSED.

Comment by Pavels Jelisejevs (Inactive) [ 2012 Sep 27 ]

(18) In IE9, when you start to select an interval and let the selection go over a graph, you are redirected to the graph page.

Eduards Seems RESOLVED..

jelisejev Now clicking on graphs does not work in all IEs.

jelisejev CLOSED.

Comment by Pavels Jelisejevs (Inactive) [ 2012 Sep 27 ]

(19) When I open a screen, select a time interval and click on one of the graphs on the screen, the timebar is displayed with incorrect time. It's always set to 1h at the beginning.

Eduards RESOLVED

jelisejev CLOSED.

Comment by Alexander Vladishev [ 2012 Sep 27 ]

(20) Incorrect displaying of time line in "Web monitoring" screen. The same problem with an "Events" screen.

Eduards RESOLVED

<Sasha> CLOSED

Comment by Alexander Vladishev [ 2012 Sep 27 ]

(21) "Last hour graph" doesn't show latest data.

Steps to reproduce:

  • open Monitoring -> Overview -> Type: [Data]
  • click on numeric value and select "Last month graph"
  • return to Monitoring -> Overview
  • click on the same numeric value and select "Last hour graph"

Eduards RESOLVED

<Sasha> CLOSED

Comment by Pavels Jelisejevs (Inactive) [ 2012 Sep 28 ]

TESTED.

Comment by Eduards Samersovs (Inactive) [ 2012 Sep 28 ]

Fixed in versions pre-2.1.0 (beta) r30416, pre-2.0.3 r30415

Comment by richlv [ 2012 Sep 29 ]

reopen to change "fix version"

Comment by richlv [ 2012 Nov 27 ]

this resulted in a regression : ZBX-5899

Generated at Fri Mar 29 03:20:23 EET 2024 using Jira 9.12.4#9120004-sha1:625303b708afdb767e17cb2838290c41888e9ff0.