Struts 2
  1. Struts 2
  2. WW-1814

Default TypeConverters always created with default ObjectFactory

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.6
    • Fix Version/s: 2.0.7, 2.1.0
    • Component/s: XML Configuration
    • Labels:
      None
    • Flags:
      Patch

      Description

      XWorkConverter uses ObjectFactory.getObjectFactory() when loading default TypeConvertors in its constructor. However, currently in struts-default.xml, the ObjectFactory has its static injections after the XWorkConverter does, so when the XWorkConverter is constructed, it ends up using the default ObjectFactory (as set in DefaultConfiguration.reload()).

      The fix is easy, just move:
      <bean class="com.opensymphony.xwork2.ObjectFactory" static="true" />
      to be the first static injection.

      Also, while you are there,
      <bean class="com.opensymphony.xwork2.util.XWorkConverter" static="true" />
      should also be moved to be before
      <bean class="com.opensymphony.xwork2.util.OgnlValueStack" static="true" />

      This is because XWorkMapPropertyAccessor staticly creates an instance of XWorkMapPropertyAccessor which in turn staticlly gets an instance of the XWorkConverter. (Necessary until the XWork 2 issue is resolved: http://jira.opensymphony.com/browse/XW-492)

        Activity

        Lukasz Lenart made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Jeff Turner made changes -
        Project Import Mon Feb 01 01:17:42 UTC 2010 [ 1264987062082 ]
        Antonio Petrelli made changes -
        Workflow Struts - editable closed status (temporary) [ 47218 ] Struts - editable closed status [ 52474 ]
        Antonio Petrelli made changes -
        Workflow Struts - editable closed status [ 42487 ] Struts - editable closed status (temporary) [ 47218 ]
        Jeff Turner made changes -
        Workflow Struts [ 41121 ] Struts - editable closed status [ 42487 ]
        Ted Husted made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Ted Husted [ husted ]
        Fix Version/s 2.1.0 [ 21794 ]
        Ted Husted made changes -
        Field Original Value New Value
        Fix Version/s 2.0.7 [ 21796 ]
        Jasper Rosenberg created issue -

          People

          • Assignee:
            Ted Husted
            Reporter:
            Jasper Rosenberg
          • Votes:
            2 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development