Log4php
  1. Log4php
  2. LOG4PHP-81

Boolean values in .properties files and incorrectly parsed by LoggerConfiguratorIni

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.0
    • Component/s: Code
    • Labels:
      None

      Description

      Currently boolean values in log4php.properties have to be written as "true" and "false", i.e. as strings because they do not get parsed correctly else.

      This is quite irritating as the ini file format as well as the name of the configurator suggests that this file does behave like a PHP .ini file where true/false/on/off etc. can be used.
      Moreover it is no problem with log4j and the properties files should be interchangeable if possible.

      As I was bitten by this bug myself for the third time now I tried to fix it

      The patch currently only fixes LoggerLayoutTTCC and most probably breaks a whole lot of other stuff but it shows the right way, I guess.

      bye,

      christian

      1. LoggerOptionConverterTest.php
        2 kB
        Christian Brunotte
      2. layout_ttcc.properties
        0.3 kB
        Christian Brunotte
      3. layout_ttcc.php
        0.2 kB
        Christian Brunotte
      4. BooleansInProperties.diff
        6 kB
        Christian Brunotte

        Activity

        Ivan Habunek made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Christian Brunotte made changes -
        Assignee Christian Hammers [ lathspell ]
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Christian Brunotte made changes -
        Fix Version/s 2.0 [ 12313916 ]
        Christian Brunotte made changes -
        Attachment BooleansInProperties.diff [ 12419006 ]
        Christian Brunotte made changes -
        Attachment layout_ttcc.properties [ 12419005 ]
        Christian Brunotte made changes -
        Attachment layout_ttcc.php [ 12419004 ]
        Christian Brunotte made changes -
        Field Original Value New Value
        Attachment LoggerOptionConverterTest.php [ 12419003 ]
        Christian Brunotte created issue -

          People

          • Assignee:
            Christian Brunotte
            Reporter:
            Christian Brunotte
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development