フォームからの入力で未入力の場合はnullでデータベースに登録したい

Question

FuelPHPを独学で学びながらシステムの構築をしています。

まずは練習としてフォームに入力した値を送信し入力した値をチェックしてフォームに再セットして表示するページを作ってみました。

困っているのは「定価」という項目のデータベースへの登録で、この項目はテーブル定義では数値型でNULLが許可されています。(MySQL)

何も入力がなかった場合はNULLとしてデータベースに登録したいのですが、このソースでModelのSaveメソッドを実行すると''で登録しようとしてSQLエラーになってしまいます。(ORMモデルです)

Validationのチェックが通ったらSaveメソッドも成功するようにする様に作りたいです。''をNULLに変える処理をどこに書くのが適当でしょうか？

■Controller側

public function action_create(){

	$data = [];
	$data['item_cd'] = '';
	$data['item_name'] = '';
	$data['op_flag'] = '';
	$data['catalog_price'] = '';
	$data['stock_type'] = '';

	$data['message'] = '';

	//POSTの場合
	if(Input::method() == 'POST'){

		$val = Validation::forge();
		//ValidationクラスにFieldを登録
		$val->add('item_cd','商品コード')
		    ->add_rule('required')
		    ->add_rule('max_length',100);

		$val->add('item_name','商品名')
			->add_rule('required')
			->add_rule('max_length',300);

		$val->add('op_flag','オープン価格')
			->add_rule('required')
			->add_rule('max_length',1)
			->add_rule('valid_string',array('numeric'));

		$val->add('catalog_price','定価')
			->add_rule('max_length',8)
			->add_rule('valid_string',array('numeric'));

		$val->add('stock_type','在庫タイプ')
			->add_rule('required')
			->add_rule('max_length',1)
			->add_rule('valid_string',array('numeric'));

		$val->run()

		//postされた値を受け取る
		$data['item_cd'] = Input::post('item_cd');
		$data['item_name'] = Input::post('item_name');
		$data['op_flag'] = Input::post('op_flag',0);
		$data['catalog_price'] = Input::post('list_price');
		$data['stock_type'] = Input::post('stock_type');

		if($out===''){

			$item = Model_Item::forge();
			$item->item_cd = $data['item_cd'];
			$item->item_name = $data['item_name'];
			$item->op_flag = $data['op_flag'];
			$item->catalog_price = $data['catalog_price'];
			$item->stock_type = $data['stock_type'];

			if($item->save()){
				Response::redirect('item/view/' . $item->id);
			}
		}
	}

	$this->template->content = View::forge('item/create',$data);
}

■View側

<table>
<?php echo Form::open('item/create'); ?>
<tr>
  <td><?php echo Form::label('商品コード','item_cd'); ?></td>
  <td><?php echo Form::input('item_cd',$item_cd,array('size' => 20)); ?></td>
</tr>
<tr>
  <td><?php echo Form::label('商品名','item_name'); ?></td>
  <td><?php echo Form::input('item_name',$item_name,array('size'=>20)); ?></td>
</tr>
  <tr>
  <td><?php echo Form::label('オープン価格','op_flag'); ?>
</td>

<?php if($op_flag === '1'){
	$op_flag = ['id' => 'form_op_flag', 'checked' => ''];
}else{
	$op_flag = ['id' => 'form_op_flag' ];
}
?>
<td><?php echo Form::checkbox('op_flag','1',$op_flag); ?></td>
</tr>
<tr>
  <td><?php echo Form::label('定価','catalog_price'); ?></td>
  <td><?php echo Form::input('catalog_price',$list_price,array('size' => 20)); ?></td>
</tr><tr>
<td>在庫タイプ</td>
  <td>
<?php
$form_stock_type_1 = ['id' => 'form_stock_type_1'];
$form_stock_type_2 = ['id' => 'form_stock_type_2'];
$form_stock_type_3 = ['id' => 'form_stock_type_3'];
if(isset($stock_type)){
	switch(true)
		case $stock_type === '1':
			$form_stock_type_1 = ['id' => 'form_stock_type_1','checked'=>''];
			break;
		case $stock_type === '2':
			$form_stock_type_2 = ['id' => 'form_stock_type_2','checked'=>''];
			break;
		case $stock_type === '3':
			$form_stock_type_3 = ['id' => 'form_stock_type_3','checked'=>''];
			break;
		default:
	}
}
?>
    <?php echo Form::radio('stock_type','1',$form_stock_type_1), Form::label('設定なし','stock_type_1'); ?>
   <?php echo Form::radio('stock_type','2',$form_stock_type_2), Form::label('単品在庫','stock_type_2'); ?>
   <?php echo Form::radio('stock_type','3',$form_stock_type_3), Form::label('バリエーション在庫','stock_type_3'); ?>
  </td>
</tr>
<tr>
  <td><?php echo Form::submit('submit','チェック'); ?></td>
</tr>
</table>
<?php echo Form::close();?>

class Model_Item extends \Orm\Model
{
	protected static $_properties = array(
		'id',
		'item_cd' => array(
				'data_type' => 'varchar',
				'label' => '商品コード',
				'validation' => array('required','max_length'=>array(100)),
				'form' => array('type' => 'text'),
		),
		'item_name' => array(
				'data_type' => 'varchar',
				'label' => '商品名',
				'validation' => array('required','max_length'=>array(300)),
				'form' => array('type' => 'text'),
		),
		'catalog_price' => array(
				'data_type' => 'int',
				'label' => '定価',
				'validation' => array('valid_string'=>array(array('numeric')),'max_length'=>array(8),'intnull'),
				'form' => array('type' => 'text'),
		),
		'op_flag' => array(
				'data_type' => 'int',
				'label' => 'オープン価格',
				'form' => array('type' => 'checkbox', 'options' => array(1 => 'aaa'), 'value' => 0 ),
		),

		'stock_type' => array(
				'data_type' => 'int',
				'label' => '在庫タイプ',
				'validation' => array('required'),
				'form' => array('type' => 'radio', 'options' => array(0 => '設定なし', 1 => '単品在庫' , 2 =>'バリエーション在庫' )),
		),
	);
	protected static $_observers = array(
		'Orm\Observer_CreatedAt' => array(
			'events' => array('before_insert'),
			'mysql_timestamp' => false,
		),
		'Orm\Observer_UpdatedAt' => array(
			'events' => array('before_update'),
			'mysql_timestamp' => false,
		),
	);
	protected static $_table_name = 'item';
}

Comments

[liply]

Model_Itemも提示したほうが有用な回答が得られやすいかと思います。

[msx2]

ご指摘ありがとうございます。
Model_Itemも追加しました。

Answer 1

Model_Itemにプロパティをチェックするメソッドを追加しデータベースでエラーにならない値に変換、saveメソッドをオーバーライドしてチェックメソッドを実行するようにしました。

とりあえずこれで思っている感じにはなったので解決です。

private function chk_prop(){
	//プロパティーのチェック
	$this->catalog_price = ctype_digit($this->catalog_price) ? $this->catalog_price : null;
	$this->op_flag =  ctype_digit($this->op_flag ) ? $this->op_flag : 0;
	$this->stock_type =  ctype_digit($this->stock_type) ? $this->stock_type : 1;
}

public function save($cascade = NULL, $use_transaction = false){
	$this->chk_prop();
	parent::save($cascade,$use_transaction);
}