MonitorWare Knowledge Base

[sire]

Hi All,

I have enhanced phpLogCon to be able to search by time interval in addition to date interval.
I have tested it in Opera browser, it works
Patch and screenshot follow.

Regards,
Sergey Sireskin

[bwinchell]

Will this work with V2.8? Do you have a version for 2.8?
Thanks

[sire]

Will this work with V2.8? Do you have a version for 2.8?
Thanks

Didn't notice 2.8 is out. I'll try this later.

[sire]

Will this work with V2.8? Do you have a version for 2.8?
Thanks

Didn't notice 2.8 is out. I'll try this later.

According to 2.8's changelog there are no much changes from 2.7.3, so this patch should work.

[alorbach]

Thanks for your contribution, I will integrate this patch into 2.8.1

best regards,
Andre Lorbach

[sire]

Thanks for your contribution, I will integrate this patch into 2.8.1

Excellent!

Regards,
Sergey

[sire]

Thanks for your contribution, I will integrate this patch into 2.8.1

I have found a mistake in my patch in lines 176 and 186 of the patch file .

Code: Select all

>if ( isset($_GET['filter_daterange_to_hour']) ) {
--><------>$filters['filter_daterange_to_hour'] = intval($_GET['filter_daterange_to_hour']);
--><------>if ($filters['filter_daterange_to_hour'] < 10)
--><------>    $filters['filter_daterange_to_hour'] = '0' . $filters['filter_daterange_to_hour'] = 0;
-->}

Code: Select all

>if ( isset($_GET['filter_daterange_to_second']) ) {
--><------>$filters['filter_daterange_to_second'] = intval($_GET['filter_daterange_to_second']);
--><------>if ($filters['filter_daterange_to_second'] < 10)
--><------>    $filters['filter_daterange_to_second'] = '0' . $filters['filter_daterange_to_second'] = 0;
-->}


There should be no

Code: Select all

= 0

at the end of these lines. I think it appeared there because of my cut and paste programming style

[alorbach]

thanks for the hint, added the fix as well

best regards,
Andre Lorbach