[VELOCITY-2] another-s2velocityのマージ Created: 2006-05-30  Updated: 2006-10-03  Resolved: 2006-10-03

Status: Resolved
Project: S2Velocity
Component/s: None
Affects Version/s: None
Fix Version/s: 1.2

Type: New Feature Priority: Major
Reporter: tanigon Assignee: tanigon
Resolution: Fixed Votes: 0
Labels: None


 Description   

another-s2velocity は toolbox.xml 自体をなくして s2container配下のコンポーネントを
そのままVelocityのコンテキストで参照できるようにする VelocityViewServlet 実装。

このソースを S2Velocityのtrunkマージする。
ついでに addData()などのVelocityTools1.2への対応も行う必要あり



 Comments   
Comment by tanigon [ 2006-10-03 ]

exampleの整備は必要であれば別課題として、1.2で統合予定ということで
課題をクローズします。

リリースしたら他のResolvedも含めて、一気にクローズしちゃいましょう

Comment by tadayosi [ 2006-10-02 ]

valueが container の時は「コンテナマネージド」で、 /WEB-INF/toolbox.xml とか従来の値なら、
toolbox.xmlベース+S2管理というS2Velocity1.1相当のまま使える

あ、なるほど。ボケておりました(^^;
そうですね、org.apache.velocity.toolboxパラメータの方が断然いいですね。
谷口さんの仕様に賛成です。

Comment by tanigon [ 2006-10-02 ]

また言葉足らずでした。 valueが container の時は「コンテナマネージド」で、 /WEB-INF/toolbox.xml とか従来の値なら、
toolbox.xmlベース+S2管理というS2Velocity1.1相当のまま使える、という状態が理想と思っています。
(いまコミットされているコードはそのつもりです)

Comment by tanigon [ 2006-10-02 ]

個人的には org.apache.velocity.toolbox のパラメータ名のほうがいいような気がします。
velocity-toolsの標準のVelocityViewServletが参照しているのがこのパラメータで、
/WEB-INF/toolbox.xml とか書いてあるわけですが、S2Velocityに乗り換えるユーザの利用像は
velocity-toolsをS2化!というパターンかな、と思うので。

web.xmlのキーは一緒のほうがいいのかなあ、とぼんやり思いました。

Comment by tadayosi [ 2006-10-02 ]

web.xmlで設定できるの賛成です。コンテナマネージドもいいと思います。パラメータ名ですが、

<param-name>org.apache.velocity.toolbox</param-name>

<param-name>org.seasar.velocity.toolbox</param-name>

どっちがいいでしょう。(後者ですかね・・・)

Comment by tanigon [ 2006-10-02 ]

しまった、

    <servlet-class&gt;org.seasar.velocity.tools.S2VelocityViewServlet</servlet-class&gt;

でしたね...

Comment by tanigon [ 2006-10-02 ]

ContainerBasedToolboxManager というクラスを追加しました.
で、「ユーザーが使い分けられるように」という意見を前に交換していたので、
S2VelocityViewServletを改造して、

  <servlet>
    <servlet-name>velocity</servlet-name>
    <servlet-class&gt;org.apache.velocity.tools.view.servlet.VelocityViewServlet</servlet-class&gt;
    <init-param>
      <param-name>org.apache.velocity.toolbox</param-name>
      <param-value>container</param-value>
    </init-param>
    <load-on-startup>10</load-on-startup>
  </servlet>

というweb.xmlだった場合にはコンテナマネージド(即席単語)になるようにしてみました。

佐藤さん:意見ください~

Generated at Thu Mar 28 22:10:58 JST 2024 using Jira 9.13.0#9130001-sha1:94297e63fec900085df7b42441d17146fe025bb5.