Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
dblog.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
5 Jan 2010 at 00:18 UTC
Updated:
3 Jan 2014 at 01:08 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
arhak commentedNote that this bug goes away if #582622: provide hook for dblog_filters gets in first, since it uses another approach for determining whether the fieldset should be collapsed/expanded (taking into account every available filter)
Comment #2
arhak commentedComment #3
mr.baileysCan you roll a patch for review/testing?
Comment #4
mr.baileysMarked #721282: show Filter log messages if it selected as a duplicate.
Comment #5
arhak commentedI have no CVS copy of Drupal
from the time I download the dev tarball to the time it gets reviewed... around three months pass in between..
nevertheless, I would do it if you insist
Comment #6
jersu commentedHere's the patch that removes the unknown variable $session and replaces with $_SESSION['dblog_overview_filter'].
Comment #8
scott falconer commented#6: 674352-6.dblog_session_variable_update.patch queued for re-testing.
Tested locally against current head and this passed. Not sure why filter.test failed, but there was an hour+ delay in testing after submit...so something might have gotten out of date during the wait.
Comment #9
jody lynnLooks good.
Comment #10
dries commentedCommitted to CVS HEAD. Thanks.