Как правильно написать?

Question

[mrzgt]

Подскажите,как правильно написать условие
$query = $this->db->query("SELECT * FROM " . DB_PREFIX . "news n LEFT JOIN " . DB_PREFIX . "news_description nd ON (n.news_id = nd.news_id) LEFT JOIN " . DB_PREFIX . "news_to_store n2s ON (n.news_id = n2s.news_id) WHERE nd.language_id = '" . (int)$this->config->get('config_language_id') . "' AND n2s.store_id = '" . (int)$this->config->get('config_store_id') . "' AND n.status = '1' ORDER BY n.sort_order DESC LIMIT " . (int)$limit);

ORDER BY n.sort_order DESC LIMIT " . (int)$limit)
как я понимаю в этом куске ошибка

Answer 1

[333Serega333 333Serega333]

Напишите так
ORDER BY n.sort_order DESC LIMIT " .$limit)

Comments

[mrzgt]

так sort_order не работает, а если убрать DESC LIMIT " .$limit , то sort_order работает)

[333Serega333 333Serega333]

Что значит не работает? Вы получаете ошибку или неправильный результат?

[mrzgt]

не сортирует по индексу

[333Serega333 333Serega333]

покажите результат

[mrzgt]

на денвере

если так ORDER BY n.sort_order "); сортирует все ок, если
ORDER BY n.sort_order DESC LIMIT 2 "); сортировка уже не работает по индексу

[mrzgt]

полный код ниже

[333Serega333 333Serega333]

код не нужен. нужен результат запроса

Answer 2

[mrzgt]

полный код

<?php
class ModelCatalogNews extends Model {

	public function updateViewed($news_id) { 
		$this->db->query("UPDATE " . DB_PREFIX . "news SET viewed = (viewed + 1) WHERE news_id = '" . (int)$news_id . "'");
	}

	public function getNewsStory($news_id) {
		$query = $this->db->query("SELECT DISTINCT * FROM " . DB_PREFIX . "news n LEFT JOIN " . DB_PREFIX . "news_description nd ON (n.news_id = nd.news_id) LEFT JOIN " . DB_PREFIX . "news_to_store n2s ON (n.news_id = n2s.news_id) WHERE n.news_id = '" . (int)$news_id . "' AND nd.language_id = '" . (int)$this->config->get('config_language_id') . "' AND n2s.store_id = '" . (int)$this->config->get('config_store_id') . "' AND n.status = '1'");

		return $query->row;
	}

	public function getNews($data = array()) {
		$sql = "SELECT * FROM " . DB_PREFIX . "news n LEFT JOIN " . DB_PREFIX . "news_description nd ON (n.news_id = nd.news_id) LEFT JOIN " . DB_PREFIX . "news_to_store n2s ON (n.news_id = n2s.news_id) WHERE nd.language_id = '" . (int)$this->config->get('config_language_id') . "' AND n2s.store_id = '" . (int)$this->config->get('config_store_id') . "'  AND n.status = '1' ORDER BY n.sort_order ";

		if (!empty($data['filter_name'])) {
			$sql .= " AND LCASE(nd.title) LIKE '" . $this->db->escape(utf8_strtolower($data['filter_name'])) . "%'";
		}


		$query = $this->db->query($sql);

		return $query->rows;
	}

	public function getNewsShort($limit) {
		$news_data = $this->cache->get('news.short.' . (int)$this->config->get('config_language_id') . '.' . (int)$this->config->get('config_store_id') . '.' . (int)$limit);

		if (!$news_data) {
			$news_data = array();

				$query = $this->db->query("SELECT * FROM " . DB_PREFIX . "news n LEFT JOIN " . DB_PREFIX . "news_description nd ON (n.news_id = nd.news_id) LEFT JOIN " . DB_PREFIX . "news_to_store n2s ON (n.news_id = n2s.news_id) WHERE nd.language_id = '" . (int)$this->config->get('config_language_id') . "' AND n2s.store_id = '" . (int)$this->config->get('config_store_id') . "' AND n.status = '1'  ORDER BY n.sort_order DESC LIMIT 1 ");

			foreach ($query->rows as $result) {
				$news_data[$result['news_id']] = $this->getNewsStory($result['news_id']);
			}

			$this->cache->set('news.short.' . (int)$this->config->get('config_language_id') . '.' . (int)$this->config->get('config_store_id') . '.' . (int)$limit, $news_data);
		}

		return $news_data;
	}

	public function getTotalNews() {
     	$query = $this->db->query("SELECT COUNT(*) AS total FROM " . DB_PREFIX . "news n LEFT JOIN " . DB_PREFIX . "news_to_store n2s ON (n.news_id = n2s.news_id) WHERE n2s.store_id = '" . (int)$this->config->get('config_store_id') . "' AND n.status = '1'");

		if ($query->row) {
			return $query->row['total'];
		} else {
			return false;
		}
	}
}
?>