Velocity
  1. Velocity
  2. VELOCITY-493

RuntimeInstance.getProperty() should check overridingProperties when not yet initialized

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.5 beta1
    • Fix Version/s: 1.5
    • Component/s: Engine
    • Labels:
      None
    • Environment:
      all

      Description

      For consistency, RuntimeInstance.getProperty() should return the values previously set with setProperty() event when it hasn't already been initialized.

      runtimeInstance.setProperty("foo","bar");
      System.out.println(runtimeInstance.getProperty("foo")); // output "null"
      runtimeInstance.init();
      System.out.println(runtimeInstance.getProperty("foo")); // output "bar"

      1. runtimeinstance.patch
        0.8 kB
        Claude Brisson

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        19m 16s 1 Will Glass-Husain 11/Nov/06 16:57
        Closed Closed Reopened Reopened
        1d 17h 26m 1 Henning Schmiedehausen 11/Jan/07 17:26
        Reopened Reopened Resolved Resolved
        22s 1 Henning Schmiedehausen 11/Jan/07 17:27
        Resolved Resolved Closed Closed
        114d 13h 40m 2 Henning Schmiedehausen 08/Mar/07 00:04
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12551438 ] jira [ 12552125 ]
        Mark Thomas made changes -
        Workflow jira [ 12389130 ] Default workflow, editable Closed status [ 12551438 ]
        Henning Schmiedehausen made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Henning Schmiedehausen added a comment -

        Close all resolved issues for Engine 1.5 release.

        Show
        Henning Schmiedehausen added a comment - Close all resolved issues for Engine 1.5 release.
        Henning Schmiedehausen made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Fix Version/s 1.5 beta2 [ 12312142 ]
        Fix Version/s 1.5 [ 12310253 ]
        Hide
        Henning Schmiedehausen added a comment -

        Patch applied

        Show
        Henning Schmiedehausen added a comment - Patch applied
        Henning Schmiedehausen made changes -
        Resolution Fixed [ 1 ]
        Assignee Henning Schmiedehausen [ henning ]
        Status Closed [ 6 ] Reopened [ 4 ]
        Hide
        Henning Schmiedehausen added a comment -

        Please don't close issues, resolve them.

        Show
        Henning Schmiedehausen added a comment - Please don't close issues, resolve them.
        Claude Brisson made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Will Glass-Husain made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.5 beta2 [ 12312142 ]
        Resolution Fixed [ 1 ]
        Hide
        Will Glass-Husain added a comment -

        Patch applied. Thanks Claude for the patch and Mike for raising the issue!

        (minor note: remember our code style re: braces when submitting patches, please).

        Show
        Will Glass-Husain added a comment - Patch applied. Thanks Claude for the patch and Mike for raising the issue! (minor note: remember our code style re: braces when submitting patches, please).
        Claude Brisson made changes -
        Field Original Value New Value
        Attachment runtimeinstance.patch [ 12344807 ]
        Hide
        Claude Brisson added a comment -

        This patch solves this issue.

        Note: the issue was initially raised by Mike Kienenberger while trying to integrate Velosurf in VPP.

        Show
        Claude Brisson added a comment - This patch solves this issue. Note: the issue was initially raised by Mike Kienenberger while trying to integrate Velosurf in VPP.
        Claude Brisson created issue -

          People

          • Assignee:
            Henning Schmiedehausen
            Reporter:
            Claude Brisson
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development