[Seasar-user:3001] [S2Struts][要望] POJO Actionのreturn null時の挙動をstruts互換に

[Hikaru Taniguchi]

たにぐちです。

S2Struts1.2.0 + POJO Actionを使っていますが、POJO Actionが return
null;をしたときに 500 (S2StrutsContextUtil.setPath()でnullpointer)が出
てしまいます。

で、Struts標準というか　ActionForwardを返す Actionクラスのときに
return nullをした場合には例外が出るのではなく responseなしで200で終了
するみたいなので、ここを同じような仕様にしていただくことは可能でしょう
か。(次バージョンあたりで)

パッチ案としては・・・setPathを囲んでしまえば(安直)いいのではないかと.

ご検討いただければ幸いです

--- AcceptorImpl.java
+++ AcceptorImpl.java
@@ -119,7 +119,9 @@
             forward = processor.processException(request, response, e, form, mapping);
         }
 
-        processor.processSetPath(forward);
+        if (forward != null) {
+            processor.processSetPath(forward);
+        }
         
         // Process the returned ActionForward instance
         processor.processForwardConfig(request, response, forward);

--
Hikaru Taniguchi <taniguchi ＠ furyu.omron.co.jp>