Uploaded image for project: '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

    • Branch Version/s:
      6.2.x
    • Backported to Branch:
      Committed

      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.

        Issue Links

          Activity

          Hide
          justin.choi 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 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 Justin Choi
              Reporter:
              shuyang.zhou Shuyang Zhou
              Recent user:
              Shalaka Tendulkar
              Participants of an Issue:
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development

                  Subcomponents