PUBLIC - Liferay Portal Community Edition
  1. PUBLIC - Liferay Portal Community Edition
  2. LPS-44313

MethodHandler should not have caller to control whether or not to create a new instance to perform the method invocation

    Details

    • Type: Story Story
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 7.0.0 M1
    • Component/s: Core Infrastructure
    • Labels:
      None
    • Branch Version/s:
      6.2.x
    • Backported to Branch:
      Committed
    • Similar Issues:
      Show 5 results 

      Description

      If the target method is a static one, create or not create a instance of the target class makes no difference.

      If the target method is a non-static one, not creating a instance of the target class will lead to NPE failure which is not acceptable.

      So the MethodHandler should always create instance automatically for non static methods, always not to create instance for static methods. This should be not decided by caller.

        Activity

        Hide
        Justin Choi added a comment -

        Closing as fixed because code has been committed on:
        Portal master GIT ID: 084de0380aae10bd2fde1d7e8c29e43b7975a9b3.

        Per Shuyang, it is not testable on the front end.

        Show
        Justin Choi added a comment - Closing as fixed because code has been committed on: Portal master GIT ID: 084de0380aae10bd2fde1d7e8c29e43b7975a9b3. Per Shuyang, it is not testable on the front end.

          People

          • Assignee:
            Justin Choi
            Reporter:
            Shuyang Zhou
            Recent user:
            Randy Zhu
            Participants of an Issue:
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development

                Structure Helper Panel