Uploaded image for project: 'PUBLIC - Liferay Portal Community Edition'
  1. PUBLIC - Liferay Portal Community Edition
  2. LPS-79763

portlet:renderURL adds liferay-util:include's parameters

    Details

      Description

      description

      When creating urls with <portlet:renderURL><portlet:param ..></portlet:renderURL> in an included JSP file, the generated url contains parameters from the <liferay-util:include ><liferay-util:param>...</liferay-util:include> tag.

      example

      A.jsp
      <liferay-util:include page="/B.jsp" servletContext="<%= application %>">
      <liferay-util:param name="x" value="v1" />
      </liferay-util:include>

      B.jsp
      <portlet:renderURL var="myurl">
      <portlet:param name="y" value="v2"/>
      </portlet:renderURL>

      myurl contains x and y.

      expected

      myurl should not contain parameter "x".

      hint0

      <portlet:renderURL var="myurl">
      </portlet:renderURL>

      When using without parameters, the created url does not have parameter "x". Could say, works well, but inconsistent. If adding extra parameters is deliberate than this would be an issue.

      hint1 (may be wrong)

      ParamAndPropertyAncestorTagImpl collects the parameters and put them into the request with _dynamicServletRequest. If it is null, it creates a new object, but based on the request, so the newly created _dynamicServletRequest object will contain the parameters in the request.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              neil.griffin Neil Griffin
              Reporter:
              peter.borkuti Peter Borkuti (Inactive)
              Participants of an Issue:
              Recent user:
              Neil Griffin
              Votes:
              1 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Days since last comment:
                3 years, 33 weeks, 1 day ago

                  Packages

                  Version Package