Closed (outdated)
Project:
Util
Version:
6.x-3.x-dev
Component:
Dblog_extension
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
4 Nov 2009 at 23:18 UTC
Updated:
27 Sep 2016 at 18:45 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
arhak commenteddblog_clear:
dblog_time_filters:
dblog_host_filter:
dblog_user_filter:
finally
PS: almost all this issues have been pointed out by NancyDru, thanks a lot for your time to review these modules
Comment #2
arhak commentedall the above issues with available patches will be bundled/altogether at #642200: review dblog_ext/dblog_filters updates
Comment #3
nancydruOne little thing that I would find helpful is to show the current date/time on the filter page so I can see what's recent in the log.
Comment #4
arhak commented@#3 do you mean showing the current date/time at
admin/reports/dblog?what if I don't have any date/time filter enabled?
what when the page gets old and the time gets outdated? a JS timer?
recent/aged filters are based on "within/beyond the last X minutes", what reference would make having the current time there?
a website could have an administrative tool for showing current time, even with JS updating it every minute, or even it might have a site-wide clock
could you extend on the exactly feature you would like?
Comment #5
nancydruJust simply the current time when the page is generated; fancy updating is not important. It doesn't matter if a date/time filter is present.
Something like the attached (who needs sleep anyway).
Comment #6
nancydruBTW, rather than sleeping, I also did this.
Comment #7
arhak commented#5 & #6 are in the scope of
dblog's enhancements, but not in the scope ofdblog_filtersparticularly #6 has a performance impact on a
hook log_filtersthose features might be useful, but I think they should be in a
form_alterinsteadthey could be even in another sub-mod
dblog_enhancetotally independent fromdblog_ext/dblog_filtersI might give them a try in the form of
form_alterdo you agree with me on this?
Comment #8
nancydruI just did them where it was convenient last night. As for the time patch, I have no issue with where it is.
I agree that the count patch could be far better somewhere else and done in a better way, such as a core patch to the original query. If you have time to create a core patch to do that, feel free; I do not and will forget by the time I do. You can take full credit for it.
If you would prefer these done differently and have the time, please go ahead.
Comment #9
nancydru6.x no longer supported