[VELOCITY-2] another-s2velocityのマージ Created: 30/May/06  Updated: 03/Oct/06  Resolved: 03/Oct/06

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 [ 02/Oct/06 ]

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だった場合にはコンテナマネージド(即席単語)になるようにしてみました。

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

Comment by tanigon [ 02/Oct/06 ]

しまった、

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

でしたね...

Comment by tadayosi [ 02/Oct/06 ]

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

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

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

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

Comment by tanigon [ 02/Oct/06 ]

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

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

Comment by tanigon [ 02/Oct/06 ]

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

Comment by tadayosi [ 02/Oct/06 ]

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

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

Comment by tanigon [ 03/Oct/06 ]

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

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

Generated at Wed Dec 11 00:39:15 JST 2019 using JIRA 7.9.2#79002-sha1:3bb15b68ecd99a30eb364c4c1a393359bcad6278.