Uploaded image for project: 'Bamboo Data Center'
  1. Bamboo Data Center
  2. BAM-13005

Improve abstract class ResultLabelEvent

XMLWordPrintable

    • Icon: Suggestion Suggestion
    • Resolution: Fixed
    • 5.0
    • None
    • None

    • Our product teams collect and evaluate feedback from a number of different sources. To learn more about how we use customer feedback in the planning process, check out our new feature policy.

      The class com.atlassian.bamboo.event.ResultLabelEvent takes a com.atlassian.user.User in the constructor.

      However, the class members only store the :

      private final String username;

      with a getUsername() method.

      With this setup the ResultLabelEvent essentially drops the other User-related fields passed to the constructor. If you are writing plugins and write a Listener that catches the ResultLabelEvent it would be REALLY useful to get the other fields in the User object other than just the Username without having to inject a user manager and re-lookup the User object using the username.

      I would vote for a private member :

      private final User user;

      with a getter of :

      public User getUser(){}

      This would allow the subclasses ResultLabelAddedEvent and ResultLabelRemovedEvent access to the User object instead of JUST the Username String.

        1. ResultLabelEvent.java
          2 kB

            [BAM-13005] Improve abstract class ResultLabelEvent

            PiotrA added a comment -

            note to self: merged to 5.0 branch with 18b1d35

            PiotrA added a comment - note to self: merged to 5.0 branch with 18b1d35

            Adam Myatt added a comment -

            See attached for example.

            Adam Myatt added a comment - See attached for example.

              Unassigned Unassigned
              ef602377c686 Adam Myatt
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved: