データのsanitizeをどうしたらよいのか教えてください。

Question

みなさんお疲れ様です。
現在、データを保存する際のサニタイズを行っているのですが、
remove_htmlをしたところ、正しい挙動でもエラーが発生してしまっていて困っています。
saveするデータが数値でしているので、そのせいだとは思うのですが、どう書いたら正しいかわかりません。
周りに気軽に聞けるcakephpをよく理解している方がいないので、どなたかご教授お願いします。

以下はその部分のコードになります。

View/Details/index.ctp

   <div id="status">
      <?php
         echo "●処理状態 : ";
         if ($reportstatus == 0){
            echo "未処理";
         } else if($reportstatus == 1){
            echo "処理済み";
         } else {
            echo "不明な処理";
         }
      ?>
   </div>

   <form action="/details/update" method="post">
      <input type="radio" name="report_status" value=0><?php echo "未処理" ?>  //ここの部分
      <input type="radio" name="report_status" value=1><?php echo "処理済み" ?>  //ここの部分
      <input type="hidden" name="report_id" value=<?php echo $reportId ?>>
      <input id="reportstatus_submit" type="submit" value="変更">
   </form>

   <div id="status">
      <?php
         echo "●表示状態 : ";
         if($requeststatus == 9){
            echo "非表示処理";
         } else {
            echo "表示";
         }
      ?>
   </div>

   <form action="/details/update" method="post">
      <input type="radio" name="request_status" value=0><?php echo "表示" ?>  //ここの部分
      <input type="radio" name="request_status" value=9><?php echo "非表示" ?>  //ここの部分
      <input type="hidden" name="report_id" value=<?php echo $reportId ?>>
      <input type="hidden" name="request_id" value=<?php echo $requestId ?>>
      <input id="requeststatus_submit" type="submit" value="変更">
   </form>

   <div id="comment">
      <?php echo "●管理側コメント" ?><br />
      <?php
         if (isset($comment)) {
            echo "$comment";
         } else {
            echo "";
         }
      ?>
   </div>

   <form action="/details/update" method="post">
      <textarea name="report_comment" cols=40 rows=4 maxlength=66></textarea>
      <input type="hidden" name="report_id" value=<?php echo $reportId ?>>
      <input id="reportcomment_submit" type="submit" value="更新"><input id="reportcomment_reset" type="reset" value="リセット">
   </form>

Controller/DetailsController

    public function update(){

        $this->autoRender = false;
        //処理状態の更新--------------------------------------------------------------

        if($this->request->is('post')){

            if(isset($this->request->data['report_status'])){
                $reportId = $this->request->data['report_id'];
                $report = $this->Report->find('all', array('conditions' => array("id" => $reportId)));
                $report_obj = $report[0]["Report"];
                $report_obj["status"] = $this->request->data['report_status'];
                $report_obj["modified"] = array('modified' => date('Y-m-d H:i:s'));
                $data = $report_obj;

                $data = Sanitize::clean($data, array('remove_html' => true));  //このままだとエラーになる

                if($this->Report->save($data)){
                    //save成功
                } else {
                    //save失敗エラー画面
                    $this->Session->setFlash('処理状態の更新に失敗しました');
                    $this->Session->flash();
                }

                $this->redirect($_SERVER['HTTP_REFERER']);

            }
            //表示状態の更新--------------------------------------------------------------

            if(isset($this->request->data['request_status'])){
                $requestId = $this->request->data['request_id'];
                $request = $this->Request->find('all', array('conditions' => array("id" => $requestId)));
                $request_obj = $request[0]["Request"];
                $request_obj["status"] = $this->request->data['request_status'];
                $data = $request_obj;

                $data = Sanitize::clean($data, array('remove_html' => true));  //このままだとエラーになる

                if($this->Request->save($data)){
                    // save成功 この後の処理に続く
                } else {
                    // save失敗 エラー画面
                    $this->Session->setFlash('表示状態の更新に失敗しました');
                    $this->Session->flash();
                }

                $reportId = $this->request->data['report_id'];
                $report = $this->Report->find('all', array('conditions' => array("id" => $reportId)));
                $report_obj = $report[0]["Report"];
                $report_obj["modified"] = array('modified' => date('Y-m-d H:i:s'));
                $data = $report_obj;

                $data = Sanitize::clean($data, array('remove_html' => true));  //このままだとエラーになる

                if($this->Report->save($data)){
                    //save成功 この後の処理に続く
                } else {
                    //save失敗エラー画面
                    $this->Session->setFlash('時間の更新に失敗しました');
                    $this->Session->flash();
                }

                $this->redirect($_SERVER['HTTP_REFERER']);

            }
            //コメントの更新--------------------------------------------------------------

            if(isset($this->request->data['report_comment'])){
                $reportId = $this->request->data['report_id'];
                $report = $this->Report->find('all', array('conditions' => array("id" => $reportId)));
                $report_obj = $report[0]["Report"];
                $report_obj["comment"] = $this->request->data['report_comment'];
                $report_obj["modified"] = array('modified' => date('Y-m-d H:i:s'));
                $data = $report_obj;

                $data = Sanitize::clean($data, array('remove_html' => true));  //このままだとエラーになる

                if($this->Report->save($data)){
                    // save成功 この後の処理に続く
                } else {
                    // save失敗 エラー画面表示
                    $this->Session->setFlash('コメントの更新に失敗しました');
                    $this->Session->flash();
                }

                $this->redirect($_SERVER['HTTP_REFERER']);

            }

        } else {
            throw new NotFoundException('このGetは見つかりませんでした。');
        }

        $this->redirect($_SERVER['HTTP_REFERER']);  

    }

コメントアウトした箇所が問題のところになります。
その他にもこれはこうした方がいいんじゃないかというのがあればぜひご助言お願いいたします。

Answer 1

エラーメッセージを教えて頂いていいでしょうか?

それと、view(.ctp)のフォームはFormヘルパーを使用しないとCakePHPを使用する意味が半減します。

<form>、<input>の箇所です。

Comments

[amakusa]

Error: Call to undefined method SessionComponent::flash()
とでていました。
$this->Session->flash();の部分でエラーがでています。

[rik]

$this->Session->flash(); は、$this->Session->setFlash();でセットしたメッセージを表示する物ですので、
$this->Session->flash(); はview内に書く必要があります。コントローラ内に書くものではありません。
view内で、
<?php print $this->Session->flash(); ?>
です。

[amakusa]

そうだったのですね。勉強不足でした。ありがとうございます。
viewの方に書き直しました。
しかしまた次のようなエラーが出ました。
Notice (1024): non-utf8 string: � [APP/Plugin/Mongodb/Model/Datasource/MongodbSource.php, line 540]

[amakusa]

public function create(Model $Model, $fields = null, $values = null) {
if (!$this->isConnected()) {
return false;
}

if ($fields !== null && $values !== null) {
$data = array_combine($fields, $values);
} else {
$data = $Model->data;
}

if($Model->primaryKey !== '_id' && isset($data[$Model->primaryKey]) && !empty($data[$Model->primaryKey])) {
$data['_id'] = $data[$Model->primaryKey];
unset($data[$Model->primaryKey]);
}

if (!empty($data['_id'])) {
$this->_convertId($data['_id']);
}

$this->_prepareLogQuery($Model); // just sets a timer
$table = $this->fullTableName($Model);
try{
if ($this->_driverVersion >= '1.3.0') {
$return = $this->_db
->selectCollection($table)
->insert($data, array('safe' => true));
} else {
$return = $this->_db
->selectCollection($table)
->insert($data, true);
}
} catch (MongoException $e) {
$this->error = $e->getMessage();
trigger_error($this->error); //ここが540行目
}
if ($this->fullDebug) {
$this->logQuery("db.{$table}.insert( :data , true)", compact('data'));
}

if (!empty($return) && $return['ok']) {

$id = $data['_id'];
if($this->config['set_string_id'] && is_object($data['_id'])) {
$id = $data['_id']->__toString();
}
$Model->setInsertID($id);
$Model->id = $id;
return true;
}
return false;
}

該当箇所はこちらです

[rik]

http://php.net/manual/ja/class.mongoexception.php
こちらを参照ください。
「このエラーが発生するのは、非 utf8 文字列をデータベースに送信しようとしたときです。 データベースに送る文字列はすべて UTF8 でなければなりません。 」
となっていますので、この通りだと思います。

[amakusa]

とすると、どのように対応するのがよいのでしょうか。
・データベースに送る文字列をutf8に変える（変え方がわかりません）
・エラーが出ないようにする
このどちらかでutf8に変えるのがよいと思うのですが、僕の場合だとどのように書き換えたらよいのでしょうか。

[rik]

保存前のデータを mb_convert_encoding() なりを使用してUTF-8に変換すればいいと思います。一部でShiftJISを使用しているなどの場合は全面的にUTF-8に統一されることをお勧めします。