Как сохранить все выбранные чекбоксы в базу?

Question

[alex88a]

Не сохраняет все чекбоксы, только один. В интернете читал, что нужно, что бы был массив, но ничего не понял. форма такая

<div class="scrollbox">
							<?php $class = 'odd'; ?>
							<?php foreach ($categories as $category) { ?>
									<?php $class = ($class == 'even' ? 'odd' : 'even'); ?>
									 
									<div class="<?php echo $class; ?> <?php echo ($category['disabled'] ? 'disabled' : ''); ?>">
									<?php if ($seller['ms.categ'] == $category['category_id']) { ?>
										<input type="checkbox" name="seller[categ]" checked="checked" value="<?php echo $category['category_id']; ?>"/>
										
										<?php echo $category['name']; ?>
										<?php } else { ?>
					<input type="checkbox" name="seller[categ]" value="<?php echo $category['category_id']; ?>"/>

										<?php echo $category['name']; ?>
										<?php } ?>
									</div>
								<?php } ?>
							</div>

часть кода для записи в базу

$sql = "UPDATE " . DB_PREFIX . "ms_seller
				SET description = '" . $this->db->escape($data['description']) . "',
					company = '" . $this->db->escape($data['company']) . "',
					adress = '" . $this->db->escape($data['adress']) . "',
					sites = '" . $this->db->escape($data['sites']) . "',
					datas = '" . $this->db->escape($data['datas']) . "',
					ports = '" . $this->db->escape($data['ports']) . "',
					rubrs = '" . $this->db->escape($data['rubrs']) . "',
					categ = '" . $this->db->escape($data['categ']) . "',
					nickname = '" . $this->db->escape($data['nickname']) . "',
					country_id = " . (int)$data['country'] . ",
					zone_id = " . (int)$data['zone'] . ","
					. (isset($data['status']) ? "seller_status=  " .  (int)$data['status'] . "," : '')
					. (isset($data['approved']) ? "seller_approved=  " .  (int)$data['approved'] . "," : '')
					. "paypal = '" . $this->db->escape($data['paypal']) . "',
					banner = '" . $this->db->escape($banner) . "',
					avatar = '" . $this->db->escape($avatar) . "'
				WHERE seller_id = " . (int)$seller_id;
		
		$this->db->query($sql);

Подскажите, в какую сторону капать?

Answer 1

[ThunderCat]

Базовые знания - потрачено...
У вас инпуты с одинаковыми именами, соответственно передаются они один за другим на сервер, где воспринимаются как переменные с соответствующим именем. Логично что последний зачеканый чекбокс тупо затрет предыдущее значение переменной с тем же именем.
Самое простое решение - добавить "индивидуальности" именам полей.
так в скрипте будет выводиться как многомерный массив c ключом категорией:

<input type="checkbox" name="seller[categ][<?php echo $category['category_id']; ?>]" checked="checked" value="<?php echo $category['category_id']; ?>"/>

так будет выводиться как многомерный массив c автоинкрементным ключом:

<input type="checkbox" name="seller[categ][]" checked="checked" value="<?php echo $category['category_id']; ?>"/>

а так как обычный массив:

<input type="checkbox" name="seller_categ_<?php echo $category['category_id']; ?>" checked="checked" value="<?php echo $category['category_id']; ?>"/>

Comments

[alex88a]

к сожалению не работает. пробовал второй вариант уже. и другие тоже. нашел в интернете такой вариант

<select name="seller[categ][]" multiple>
							<?php foreach ($categories as $category) { ?>
							<option value="seller[categ][<?php echo $category['category_id']; ?>]"><?php echo $category['name']; ?></option>
							<?php } ?>
							</select>

$sql = "UPDATE " . DB_PREFIX . "ms_seller
				SET description = '" . $this->db->escape($data['description']) . "',
					company = '" . $this->db->escape($data['company']) . "',
					adress = '" . $this->db->escape($data['adress']) . "',
					sites = '" . $this->db->escape($data['sites']) . "',
					datas = '" . $this->db->escape($data['datas']) . "',
					ports = '" . $this->db->escape($data['ports']) . "',
					rubrs = '" . $this->db->escape($data['rubrs']) . "',
					categ = '".implode(",", $data['categ'])."',
					nickname = '" . $this->db->escape($data['nickname']) . "',
					country_id = " . (int)$data['country'] . ",
					zone_id = " . (int)$data['zone'] . ","
					. (isset($data['status']) ? "seller_status=  " .  (int)$data['status'] . "," : '')
					. (isset($data['approved']) ? "seller_approved=  " .  (int)$data['approved'] . "," : '')
					. "paypal = '" . $this->db->escape($data['paypal']) . "',
					banner = '" . $this->db->escape($banner) . "',
					avatar = '" . $this->db->escape($avatar) . "'
				WHERE seller_id = " . (int)$seller_id;
		
		$this->db->query($sql);

во всех случаях пишет PHP Warning: mysqli::real_escape_string() expects parameter 1 to be string, array given in
строка ошибки

public function escape($value) {
		return $this->link->real_escape_string($value);
	}

изменять его крайне не желательно, т.к. это стандартный файл опенкарт.
все равно спасибо за ответ, буду копаться дальше.

[ThunderCat]

alex88a,

В интернете читал, что нужно, что бы был массив, но ничего не понял.

нашел в интернете такой вариант

я вам сразу скажу, что бы вы там в форме не вставляли - работать не будет, так как у вас на стороне сервера нет обработки этого массива, вы как мартышка и очки - не понимаете что пишете, по этому РАБОТАТЬ НЕ БУДЕТ. Либо разберитесь как работает взаимодействие форм с сервером, перебор/обработка параметров на сервере, циклы, короче поймите что и как делать, либо отдайте работу кому-то с пониманием что нужно делать. По тому как проблема не в форме конечно же, любой из написанных вариантов (и мои и ваш) работают как надо, просто на сервере их надо соответствующе обработать.

[alex88a]

ThunderCat, спасибо за подсказку. После того, как написали, что форма рабочая, проверил и понял, что проблема была в базе. Точнее в поле. Тип поля был int, изменил на varchar и записывает верно.
Еще раз спасибо!