У всех неправильно position в таблице s_images ставится?

[granata005]

Заметил у себя такую картину, что в таблице изображений колонка position заполняется не по счётчику+1, а по id изображения.

 

Причём, раньше всё было ок, а с какого-то момента пошло не так.

Это только я себе в коде нахулиганил, или это в одном из обновлений было такое нововведение? 

 

 

Хотя, с виду должно заливать правильно:

 

public function add_image($product_id, $filename, $name = '')
	{
		$query = $this->db->placehold("SELECT id FROM __images WHERE product_id=? AND filename=?", $product_id, $filename);
		$this->db->query($query);
		$id = $this->db->result('id');
		if(empty($id))
		{
			$query = $this->db->placehold("INSERT INTO __images SET product_id=?, filename=?", $product_id, $filename);
			$this->db->query($query);
			$id = $this->db->insert_id();
			$query = $this->db->placehold("UPDATE __images SET position=id WHERE id=?", $id);
			$this->db->query($query);
		}
		return($id);
	}

 

[Kors]

Все правильно, так и было, так и должно быть.

[granata005]

Все правильно, так и было, так и должно быть.

Почему? Раньше было "0,1,2", в зависимости от позиции изображения внутри этого товара...

[Kors]

Зависит от того, что понимать под "раньше".

В версии 2.0 указанный фрагмент такой же, то есть присутствует

			$query = $this->db->placehold("UPDATE __images SET position=id WHERE id=?", $id);
			$this->db->query($query);

Если сильно интересно, смотрите дистрибутивы разных версий, может, где что и менялось...

[granata005]

Просто не пойму почему ты говоришь что это правильно и так и должно быть?

У тебя тоже в БД такие цифры?

Это ж вроде как-то нелогично...

[Kors]

При добавлении изображения при импорте выполняется SET position=id.

А при сохранении товара, у ранее имевшихся изображений position меняется на 0,1,2,...

То есть имеется такой двойной подход к формированию position.

 

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

[granata005]

ну хз, не считаю это правильным...