PHP foreach文を使用するとページが移動しなくなる

Question

お世話になります。

webサイト内で、phpを使用しデータベースとデータをやりとりしています。
今回は、データベースの中に複数レコードが存在し、そこから日付を呼び出してweb上のメニューに反映させ、クリックした日付を持ってtest.phpに移動し、該当のレコードの情報を呼び出すようにしたいと思っています。

<ul>
<?php
    #メニューの日付
    foreach ($records_menu as $record_menu) {?>
    <FORM NAME="test" METHOD="POST" ACTION="test.php">
    <A HREF="javascript:document.test.submit()">
        <li>
        <?php
            echo date('Y年m月d日',$record_menu->getFieldAsTimeStamp('年月日'));
            echo '<input type="hidden" name="menuhiduke" value="' . $record->getField('年月日') . '">';
            ?>
        </li>
    </A>
    </Form>
<?php } ?>
</ul>

以上のソースなのですが、foreach文を使用しない場合問題なくプログラムが動くのですが（メニューには日付が１つのみ表示されている）、foreachを使って繰り返すと、日付をクリックしても無反応の状態です。

すいませんが何が間違っているのでしょうか？
ご教授よろしくお願いいたします。

Answer 1

formの名前がみんなtestなので名前の競合が発生しているのでは？
カウンタをつかってtest1,test2・・・のようにユニークにしていくのが妥当
もしくはスタイルシートで隠したサブミットボタンにidを降っておき
document.getElementById(ボタンのid).click()
で処理するとか

Answer 2

質問の内容とはそれますが、HTMLの文法上<ul> 要素の直下には<li> 要素しか置けないため、下記のような順にするほうが良いです。

<ul>
  <li>
    <form>
      <a>
        <!-- 内容 -->
      </a>
    </form>
  </li>
</ul>

Answer 3

formが同じ名前で複数作られてしまうからでしょう。
foreachの外に出しましょう。
hiddenも同じ名前なので、こちらはループごとに名前を変えるか、定義は１か所にしてクリックのタイミングで値を入れ替えるとかしないとダメですね。

Comments

[satoshi501]

ありがとうございます。
質問ですが、hiddenのnameも可変にした場合 受け取る側ではどのように受け取ればいいでしょうか？
現在は　$_POST['menuhiduke'];　として受け取っています

[ttyp03]

自分ならそういう作りにはしないですが…
もしやるならnameを連番（menuhiduke1など）にしておいて、PHP側でループで回して値があるか確認する、でしょうか。
for($i = 1; ; $i++){
　if(isset($_POST['menuhiduke'.$i])===TRUE){
　　$menuhiduke = $_POST['menuhiduke'.$i];
　　break;
　}
}
みたいな。