Permalink
Browse files

- fixed bug could not set "name" attribute of form tag. (thanks: shutta)

  • Loading branch information...
1 parent 0fe06ea commit bd72207257a0d1616f9a6dfe3df724e4e76dd1de @mumumu mumumu committed Oct 5, 2009
View
3 CHANGES
@@ -36,13 +36,14 @@
- Ethna_Plugin::includePlugin メソッドの実装が動作するものではなかったので変更
- Ethna_Plugin_Cachemanager のクラスのプロパティに指定する $namespace が意味をなしていなかったので修正 (#17753)
- PROJECT_DIR/lib/Ethna/extlib 以下にファイルを設置するタイプのプラグインを pear-local などでインストールすると、それ以後
- ethnaコマンドが使えなくな問題を修正
+ ethnaコマンドが使えなくなる問題を修正
- 新しいプラグインの命名規則に従っていない古いプラグインを別物として読み込もうとしてクラス名がかぶる問題を修正(#17875) thanks: id:okonomi
- 存在しない(or 削除された) ethnaコマンドを指定すると Fatal Error が起きるバグを修正 (thanks:kondo_)
-- http://sourceforge.jp/ticket/browse.php?group_id=1343&tid=17894
- Ethna_Plugin_Logwriter の debug_backtrace の一部が取得できず、E_NOTICE が出るバグを修正 (thanks: http://www.remix.gr.jp/)
- cli 環境で Ethna_Session::start を叩いたときに $_SERVER 変数がないために E_NOTICE が出る問題を修正
- Smartyのデリミタを変更している場合にi18nコマンドが機能しないバグを修正 (#18668)
+- formタグのname属性が設定できなくなっていたバグを修正 (thanks: shutta) (#19037)
** 2.5.0-preview5
View
4 class/Plugin/Smarty/block.form.php
@@ -64,8 +64,10 @@ function smarty_block_form($params, $content, &$smarty, &$repeat)
// {form name=... }
// 指定された場合は、submitされた {form}を識別する
// id をhiddenタグで指定する
+ //
+ // $params['name'] は formタグのnameタグになるため
+ // unset してはいけない
$name = $params['name'];
- unset($params['name']);
if ($name != 'default') {
$name_hidden = sprintf('<input type="hidden" name="ethna_fid" value="%s" />',
htmlspecialchars($name, ENT_QUOTES)
View
25 test/Ethna_ViewClass_FormHelper_Test.php
@@ -88,5 +88,30 @@ function test_formhelper_Textarea()
$this->assertPattern('#<textarea name="textarea_setactval">input</textarea>#', $result);
$this->assertPattern('#<textarea value="foo" name="textarea_settplval">foo</textarea>#', $result);
}
+
+ function test_formhelper_FormName()
+ {
+ $action_name = $tpl_name = 'formnametest';
+ $action_skel = ETHNA_TEST_SKELDIR . 'skel.action.formhelper.php';
+ $this->project->runCmd('add-action',
+ array(
+ '-s',
+ $action_skel,
+ $action_name,
+ )
+ );
+ $tpl_skel = ETHNA_TEST_SKELTPLDIR . 'skel.template.blockform.tpl';
+ $this->project->runCmd('add-template',
+ array(
+ '-s',
+ $tpl_skel,
+ $tpl_name,
+ )
+ );
+ $result = $this->project->runMain($action_name, array());
+
+ $this->assertPattern('#<form name="hoge" method="post"><input type="hidden" name="ethna_fid" value="hoge" />#', $result);
+ }
+
}
View
3 test/skel/template/skel.template.blockform.tpl
@@ -0,0 +1,3 @@
+{form name="hoge" ethna_action="formnametest"}
+ {form_submit}
+{/form}

0 comments on commit bd72207

Please sign in to comment.