Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow system log to be triggered from a controller or model #5

Open
nohponex opened this issue Apr 27, 2016 · 1 comment
Open

Allow system log to be triggered from a controller or model #5

nohponex opened this issue Apr 27, 2016 · 1 comment
Assignees

Comments

@nohponex
Copy link
Member

  • One approach is to define the phramework callback step in register method and allow the modification of $additionalParameters argument

The modification of $additionalParameters might be impossible due to immediate definition of the callbacks.

We might have to call the register method the moment we need to defined the additional log information or it might be possible to change the callback definitions to use the provided arguments later in execution

@nohponex
Copy link
Member Author

@alkallio there is an issue with the solution we discussed, "changing phramework's callback step in register method"

The definition of callback also require invoked controller and method which are used in addition with matrix settings. Although we need to figure out a way to have triggered system log entries in a uniform way.

This might require additional modifications and restructuring of SystemLog class, but it wont affect the current projects that are using SystemLog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants