Uploaded image for project: 'Struts 2'
  1. Struts 2
  2. WW-4119

Wasted work in RefreshModelBeforeResult.beforeResult()

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.3.14.3
    • Fix Version/s: 2.3.16
    • Component/s: None
    • Labels:
    • Environment:

      any

    • Flags:
      Patch, Important

      Description

      The problem appears in version 2.3.14.3 and in revision 1495522. I
      attached a one-line patch that fixes it. This problem and the
      attached patch are similar to the previously fixed WW-4115 and its
      patch.

      In method "RefreshModelBeforeResult.beforeResult", the loop over
      "root" should break immediately after "needsRefresh" is set to
      "false". All the iterations after "needsRefresh" is set to "false" do
      not perform any useful work, at best they just set "needsRefresh"
      again to "false".

      1. patch.diff
        0.6 kB
        Adrian Nistor

        Activity

        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit 1502452 from Lukasz Lenart
        [ https://svn.apache.org/r1502452 ]

        WW-4119 Improves loop

        Show
        jira-bot ASF subversion and git services added a comment - Commit 1502452 from Lukasz Lenart [ https://svn.apache.org/r1502452 ] WW-4119 Improves loop
        Hide
        lukaszlenart Lukasz Lenart added a comment -

        Patch applied, thanks!

        Show
        lukaszlenart Lukasz Lenart added a comment - Patch applied, thanks!
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Struts2-JDK6 #749 (See https://builds.apache.org/job/Struts2-JDK6/749/)
        WW-4119 Improves loop (lukaszlenart: rev 1502452)

        • /struts/struts2/trunk/xwork-core/src/main/java/com/opensymphony/xwork2/interceptor/ModelDrivenInterceptor.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Struts2-JDK6 #749 (See https://builds.apache.org/job/Struts2-JDK6/749/ ) WW-4119 Improves loop (lukaszlenart: rev 1502452) /struts/struts2/trunk/xwork-core/src/main/java/com/opensymphony/xwork2/interceptor/ModelDrivenInterceptor.java

          People

          • Assignee:
            lukaszlenart Lukasz Lenart
            Reporter:
            adriannistor Adrian Nistor
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development