Uploaded image for project: 'Blossom'
  1. Blossom
  2. BLOSSOM-178

Add exposeModelAttributes property on UuidRedirectViewResolver

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Critical
    • 2.0.6, 3.0.3
    • 2.0.3, 3.0
    • None
    • None
    • Magnolia 4.5.11
      Blossom 2.0.3
      Spring 3.2.4.RELEASE
    • Yes

    Description

      We have the possibility in the Spring ViewResolver to set the attribute "exposeModelAttributes". When false, this will avoid to pass the model attributes to the request and when redirect have ugly string parameters visible in the url. In blossom, this option solves a big performance problem.

      I added this functionality by extending the UuidRedirectViewResolver in order to expose the "exposeModelAttributes" variable. (See java file attached)

      blossom-servlet.xml
        <bean class="com.aperto.bos.blossom.BosUuidRedirectViewResolver">
          <property name="order" value="1" />
          <property name="exposeModelAttributes" value="false" />
        </bean>
      

      Reason is: The parameters in url cause a huge performance problem.
      We have a page with lot of components (areas, components that contains other components) all are configured with blossom, and it seems that the request parameters will be transferred to every @controller mapping request, the load of the page is then extremely slow. (for example page.html?errors=true)
      Without paramters, the page is loaded regulary.
      The problem in editMode is not solved, because of the "mgnlPreview", etc..

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                tmattsson Tobias Mattsson
                jcrobert Jean-Charles Robert
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Task DoD