[
https://issues.apache.org/jira/browse/LOG4PHP-56?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12735311#action_12735311
]
Christian Grobmeier commented on LOG4PHP-56:
--------------------------------------------
Thanks, will replace this var.
About DI: I implemented such a lib within PIWI: piwi.googlecode.com
I am not sure if that would be good, since PIWI isn't an Apache Project (yet -
i am working on it). However, this should be considered.
Thanks!
> Replace define('LOG4PHP_CONFIGURATION' by static variable
> ---------------------------------------------------------
>
> Key: LOG4PHP-56
> URL: https://issues.apache.org/jira/browse/LOG4PHP-56
> Project: Log4php
> Issue Type: Improvement
> Components: Code
> Affects Versions: 2.2
> Reporter: Christian Hammers
> Priority: Minor
>
> As you're currently switching to PHP5 OOP code, I would suggest to replace
> this ugly global constant by a static attribute of the LogManager class i.e.
> LogManager::$configFile = ... Or maybe even a constructor/factory argument
> This would have the additional benefit that it could be set by some kind of
> Dependency Injection framework like Spring for Java, although I don't know of
> such is even existing for PHP :) But it might come in the future.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
