You are viewing a plain text version of this content. The canonical link for it is here.

Posted to log4php-dev@logging.apache.org by "Christian Grobmeier (JIRA)" <ji...@apache.org> on 2009/07/26 13:47:14 UTC

[jira] Updated: (LOG4PHP-56) Replace define('LOG4PHP_CONFIGURATION' by static variable

     [ https://issues.apache.org/jira/browse/LOG4PHP-56?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Christian Grobmeier updated LOG4PHP-56:
---------------------------------------

    Fix Version/s: 2.0

> 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
>             Fix For: 2.0
>
>
> 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.