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

        Hide
        higa higa added a comment -

        修正しました。

        Show
        higa higa added a comment - 修正しました。
        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()を呼び出さないように修正します。
        Hide
        higa higa added a comment -

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

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