Uploaded image for project: 'Velocity'
  1. Velocity
  2. VELOCITY-215

SimpleLog4JLogSystem property lookup should come from RuntimeConstants

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 1.3.1
    • 1.5
    • Engine
    • None
    • Operating System: Windows XP
      Platform: PC
    • 23676

    Description

      The SimpleLog4JLogSystem class looks at the
      property "runtime.log.logsystem.log4j.category" when trying to determine
      which "category" to log to [should really be logger as Log4j no longer uses the
      term 'category', but thats a different story...]

      The property name should come from the RuntimeConstants class like most of the
      other property keys.

      Attachments

        Activity

          Agreed. We'll fix after we punch out 1.4, which I 'm trying to do now

          geir Geir Magnusson Jr added a comment - Agreed. We'll fix after we punch out 1.4, which I 'm trying to do now
          dlr Daniel Rall added a comment -

          When changing this, I like moving to "runtime.log.logsystem.log4j.logger" (or
          something fo the sort), and supporting "runtime.log.logsystem.log4j.category"
          for a time-bounded period in a deprecated fashion.

          Geir, you thinking of this for a 1.4.1 release, or 1.5?

          dlr Daniel Rall added a comment - When changing this, I like moving to "runtime.log.logsystem.log4j.logger" (or something fo the sort), and supporting "runtime.log.logsystem.log4j.category" for a time-bounded period in a deprecated fashion. Geir, you thinking of this for a 1.4.1 release, or 1.5?

          Closing this issue - Log4JLogSystem was creating back in October 2003. The specified property is a public constant in rather than RuntimeConstants (which is more appropriate for an optional plugin).

          wglass William Glass-Husain added a comment - Closing this issue - Log4JLogSystem was creating back in October 2003. The specified property is a public constant in rather than RuntimeConstants (which is more appropriate for an optional plugin).

          Close all resolved issues for Engine 1.5 release.

          henning Henning Schmiedehausen added a comment - Close all resolved issues for Engine 1.5 release.

          People

            Unassigned Unassigned
            erik@threepointsoftware.com Erik G
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: