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

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            higa higa added a comment -

            修正しました。

            Show
            higa higa added a comment - 修正しました。
            Hide
            Permalink
            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
            Permalink
            higa higa added a comment -

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

            Show
            higa higa added a comment - 二重にWrapperUtil.convert()を呼び出さないように修正します。
            Hide
            Permalink
            higa higa added a comment -

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

            Show
            higa higa added a comment - 修正しました。 試していただけるでしょうか。
            Hide
            Permalink
            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