FuelPHPの html_tag 関数をコントローラで使いたい

Question

状況

FuelPHPのhtml_tag関数を使ってHTML要素を作成したいです。

ビューで使用する方法は分かったのですが、コントローラで同様に記述すると文字列として出力されてしまいました
具体例を示すと、下のキャプチャでid=elem1のdiv要素が""で囲まれて文字列とみなされています。一方、id=elem2の要素はビューでhtml_tag関数を使ったものです。

質問

FuelPHPのコントローラでhtml_tag関数を使う場合、HTMLタグとしてビューに渡すにはどのように記述すればよいでしょうか？
もしくは、ビューで文字列をHTMLタグとして解釈する方法を知りたいです。

コード

コントローラ

<?php

class Controller_Home extends Controller
{
  public function action_index()
  {
    // HTMLタグを生成
    $elem_ctr = html_tag('div', array(
      'id' => 'elem1',
      'class' => 'test'
    ), 'コントローラで生成した要素');
    
    // ビューを生成
    $view = View::forge('home');
    $view->set_global('elem_ctr', $elem_ctr);
    return $view;
  }
}

ビュー

<?php echo $elem_ctr; ?>
<?php
  $elem_view = html_tag('div', array(
    'id' => 'elem2',
    'class' => 'test'
  ), 'ビューで生成した要素');
  echo $elem_view;
?>

Comments

[退会済みユーザー]

htmlspecialchars()使ってみるとか？

[m.ts10806]

出力時に使うものを出力よりだいぶ前で使うメリットはないかと思いますが…

[AkihiroIshii]

>m6u さん
コントローラ側で使っても変化はありませんでした。ビュー側で使うと次のようになりましたが、""で囲まれるのは変わりなかったです。
"<div id="elem1" class="test">コントローラで生成した要素</div>"

[AkihiroIshii]

>m.ts10806 さん
本来やりたいのは、DBから取得した値を属性に付与したdiv要素を一括生成することでした。そのためコントローラでdiv要素を作れないかと考えてhtml_tag関数を試していた次第です。

ただ、そもそもhtml_tag関数がビューで使うことを想定したものならば、方針を変えるべきかもしれません(_ _;)

[m.ts10806]

コントローラはあくまでデータ処理のためのものでhtml生成するものではないので確かにやり方はかえたほうが良いと思います。

[AkihiroIshii]

>m.ts10806 さん
なるほど、コントローラとビューの切り分けが不適切でしたね。方針を再検討してみます。

[m.ts10806]

はい。なるべく処理結果だけを渡してビュー側でやりくりする方法を考えた方が良さそうです。
Ajaxでもデータだけ返してDOM構築はJavaScriptで行うことの方が多いですし役割分担が明確になります

[AkihiroIshii]

>m.ts10806 さん
生成するdiv要素に付与する属性のリストをコントローラで用意してビューに渡し、ビューのループ処理にてHTMLタグを生成する方針に変更しました。
冒頭の質問に対してはkawaxさんの回答をベストアンサーとさせていただきましたが、m.ts10806さんのご指摘も大変勉強になりました。ありがとうございます。

↓div要素をhtml_tag関数で一括生成するための代替案
---------- コントローラ ----------
// div要素にセットする属性のリストを設定（実際はモデルから取得する予定）
$attrs = array(
array('western'=>'1987', 'japanese'=>'昭和62年'),
array('western'=>'1988', 'japanese'=>'昭和63年'),
array('western'=>'1989', 'japanese'=>'平成元年'),
array('western'=>'1990', 'japanese'=>'平成2年'),
array('western'=>'1991', 'japanese'=>'平成3年'),
);

// ビューを生成
$view = View::forge('home');
$view->set_global('attrs', $attrs);
return $view;

---------- ビュー ----------
<?php foreach($attrs as $attr): ?>
<?php echo html_tag('div', array('japanese'=>$attr['japanese']), $attr['western']); ?>
<?php endforeach; ?>

[m.ts10806]

↑で試されたやり方の方が後々のメンテナンスせい含めてスッキリした印象が持てます

Answer 1

fuelは知らないけどset_global()だとエスケープされてるから。
http://fuelphp.jp/docs/1.7/classes/view.html

この場合はそもそもset_safe()使うのでは。

$view->set_safe('elem_ctr', $elem_ctr);

Comments

[退会済みユーザー]

言われてみれば、そのとおり。