Closed (fixed)
Project:
Rules
Version:
8.x-3.x-dev
Component:
Rules Core
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
16 Dec 2019 at 06:33 UTC
Updated:
25 Jan 2020 at 22:34 UTC
Jump to comment: Most recent
As outlined in the parent issue, we need a D8 replacement for the D7 RulesLog class, which was a transient singleton object used to collect (in memory) Rules debug messages so they could be optionally rendered as a debug log on the screen and optionally forwarded to the core dblog, controlled by Rules system variables, for storage in the database.
Steps that need to be taken:
RulesLog class. #3103439: Create RulesDebugLog logger to hold temporary debug logs Assigned to: trRulesDebugLoggerChannel logger channel created by #3101194: Rename logger.channel.rules to logger.channel.rules_debug to use the new logger described above and to preprocess the variables supplied via log() so as to compute everything that needs to be stored for use in the on-screen log. #3103555: Update RulesDebugLoggerChannel Assigned to: tr
Comments
Comment #2
tr commentedThis is going to have to be done in a number of stages, as there are many parts to it. Correspondingly, I've updated the issue summary and turned this into a meta. I'm assigning this to myself since I've already done the work; I just have to polish it and post the patches.
Comment #3
tr commentedAdded #3103439: Create RulesDebugLog logger to hold temporary debug logs
Comment #4
tr commentedAdded #3103555: Update RulesDebugLoggerChannel
Comment #5
tr commentedConsolidated last two steps because they have a lot of commonality and you can't do one without the other.
Comment #6
tr commentedAdded #3104109: Implement rendering of debug logs to screen and database
Comment #7
tr commentedSwitching this to "Needs review"
The only step left is in #3104109: Implement rendering of debug logs to screen and database. Please review that issue.
Comment #8
tr commentedComment #9
tr commented#3104109: Implement rendering of debug logs to screen and database has been committed, so this meta is complete. If you find any problems with this please open a new issue as a follow-up.
Comment #10
jonathan1055 commentedGood work. Thank you.