Uploaded image for project: 'Log4php'
  1. Log4php
  2. LOG4PHP-81

Boolean values in .properties files and incorrectly parsed by LoggerConfiguratorIni

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 2.0.0
    • Code
    • 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

      Attachments

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

        Activity

          People

            lathspell Christian Brunotte
            lathspell Christian Brunotte
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment