Uploaded image for project: 'SAStruts'
  1. SAStruts
  2. SASTRUTS-116

ActionFormのプロパティをリクエストにセットするときにActionFormWrapper経由にしました

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0.4-sp6
    • Component/s: SAStruts
    • Labels:
      None

      Description

      ActionFormのプロパティをリクエストにセットするときにActionFormWrapper経由にしました。
      これは、生のActionForm(POJO)をセッションから削除したときも、
      次のJSPにActionFormの内容を渡すことができるようにするためです。

        Activity

        higa higa created issue -
        Hide
        higa higa added a comment -

        修正しました。

        Show
        higa higa added a comment - 修正しました。
        higa higa made changes -
        Field Original Value New Value
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Repository Revision Date User Message
        sastruts repository #832 Fri Apr 24 13:59:12 JST 2009 higa [SASTRUTS-116]ActionFormのプロパティをリクエストにセットするときにActionFormWrapper経由にしました。
        Files Changed
        DEL /trunk/sa-struts-tutorial/src/main/webapp/WEB-INF/lib/sa-struts-1.0.4-sp4.jar
        MODIFY /trunk/sa-struts/src/main/java/org/seasar/struts/customizer/ActionCustomizer.java
        ADD /trunk/sa-struts-tutorial/src/main/webapp/WEB-INF/lib/sa-struts-1.0.4-sp5.jar
        DEL /trunk/sa-struts-tutorial/src/main/webapp/WEB-INF/libsrc/sa-struts-1.0.4-sp4-sources.jar
        MODIFY /trunk/sa-struts/src/main/java/org/seasar/struts/action/S2RequestProcessor.java
        MODIFY /trunk/sa-struts-tutorial/build.xml
        MODIFY /trunk/sa-struts-tutorial/.classpath
        ADD /trunk/sa-struts-tutorial/src/main/webapp/WEB-INF/libsrc/sa-struts-1.0.4-sp5-sources.jar
        Hide
        shinsuke shinsuke added a comment -

        この修正で今まで BeanWrapper でリクエストにセットされたものが、BeanWrapper がさらに MapWrapper でラップされた状態でリクエストにセットされるようになりました(WrapperUtil#convertが2回通るようになったためと思われます)。ですので、sp4 と動きが変わってしまっています。
        再現状況としては、アクションフォームに Dto があって、jsp でリクエストスコープからそれを取得するような場合、BeanWrapper でなく、MapWrapper で得ることになります。できれば、今までどおり BeanWrapper で返ってくるのが良い気がしますがいかがでしょうか?

        Show
        shinsuke shinsuke added a comment - この修正で今まで BeanWrapper でリクエストにセットされたものが、BeanWrapper がさらに MapWrapper でラップされた状態でリクエストにセットされるようになりました(WrapperUtil#convertが2回通るようになったためと思われます)。ですので、sp4 と動きが変わってしまっています。 再現状況としては、アクションフォームに Dto があって、jsp でリクエストスコープからそれを取得するような場合、BeanWrapper でなく、MapWrapper で得ることになります。できれば、今までどおり BeanWrapper で返ってくるのが良い気がしますがいかがでしょうか?
        Hide
        higa higa added a comment -

        二重にWrapperUtil.convert()を呼び出さないように修正します。

        Show
        higa higa added a comment - 二重にWrapperUtil.convert()を呼び出さないように修正します。
        higa higa made changes -
        Status Resolved [ 5 ] Reopened [ 4 ]
        Resolution Fixed [ 1 ]
        higa higa made changes -
        Fix Version/s 1.0.4-sp5 [ 10874 ]
        Fix Version/s 1.0.4-sp6 [ 10940 ]
        Hide
        higa higa added a comment -

        修正しました。
        試していただけるでしょうか。

        Show
        higa higa added a comment - 修正しました。 試していただけるでしょうか。
        higa higa made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        sastruts repository #849 Thu Jun 04 17:09:10 JST 2009 higa [SASTRUTS-116]WrapperUtilが二重に呼び出されていたのを修正しました。
        Files Changed
        MODIFY /trunk/sa-struts/pom.xml
        MODIFY /trunk/sa-struts/src/test/java/org/seasar/struts/action/S2RequestProcessorTest.java
        MODIFY /trunk/sa-struts/src/main/java/org/seasar/struts/action/S2RequestProcessor.java
        Hide
        shinsuke shinsuke added a comment -

        対応いただきありがとうございます!
        BeanWrapper で取れることを確認しました。

        Show
        shinsuke shinsuke added a comment - 対応いただきありがとうございます! BeanWrapper で取れることを確認しました。

          People

          • Assignee:
            higa higa
            Reporter:
            higa higa
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development