Мой первый компонент - Общие вопросы веб разработки

require_once(JPATH_ADMINISTRATOR.DS.'components'.DS.'com_virtuemart'.DS.'helpers'.DS.'config.php');
$vmlang = VmConfig::$vmlang; //получаем код языка
 
$join[] = ' LEFT JOIN `#__virtuemart_products` AS p ON p.`virtuemart_product_id` = pl.`virtuemart_product_id`';
        $join[] = ' LEFT JOIN `#__vmxg_lt` AS v ON v.id = p.`virtuemart_product_id`';
 
$query  = ' SELECT SQL_CALC_FOUND_ROWS v.*, pl.product_name FROM `#__virtuemart_products_' . $vmlang . '` AS pl '.implode(' ', $join);
        $query .= ' WHERE '.implode(' AND ', $where).' '.$order;

Table 'sitebase.#__virtuemart_products_' doesn't exist SQL=SELECT SQL_CALC_FOUND_ROWS v.*, pl.product_name FROM `#__virtuemart_products_` AS pl LEFT JOIN `#__virtuemart_products` AS p ON p.`virtuemart_product_id` = pl.`virtuemart_product_id` LEFT JOIN `#__vmxg_lt` AS v ON v.id = p.`virtuemart_product_id` WHERE p.published = 1 ORDER BY product_name ASC LIMIT 0, 20

`#__virtuemart_products_' . $vmlang . '`

`#__virtuemart_products_ru_ru`

ничего не изменилось

Внимательнее читайте ошибку
Table 'sitebase.#__virtuemart_products_' doesn't exist
Таблица virtuemart_products_ не найдена.

У тебя в коде обращение тут
$query  = ' SELECT SQL_CALC_FOUND_ROWS v.*, pl.product_name FROM `#__virtuemart_products_'

Что-то мне подсказывает что таблица называется #__virtuemart_products без '_' в конце

P.S За такой код оторвал бы руки.

Код: php-brief

//$vmlang = VmConfig::$vmlang; //получаем код языка
 
$join[] = ' LEFT JOIN `#__virtuemart_products` AS p ON p.`virtuemart_product_id` = pl.`virtuemart_product_id`';
$join[] = ' LEFT JOIN `#__vmxg_lt` AS v ON v.id = p.`virtuemart_product_id`';
 
$query  = ' SELECT SQL_CALC_FOUND_ROWS v.*, pl.product_name FROM `#__virtuemart_products_' . VmConfig::$vmlang . '` AS pl '.implode(' ', $join);
$query .= ' WHERE '.implode(' AND ', $where).' '.$order;

я беру текущий язык сайта, пишу его в переменную $vmlang и пытаюсь вставить и имя таблицы.

VmConfig::$vmlang

Потише. И не торопитесь кому-то руки отрывать - нужна таблица #__virtuemart_products_ru_RU.[/spoiler]

Руки оторвал бы не за таблицу( хотя смотря какая задача возможно и не нужно тут 3 таблицы, а достаточно модифицировать основную.) а за написание кода коннекта. Элемента зачем запихивать 2 join в массив и потом через две строки использовать implode

Ну и читать куда удобнее
$query->select
$query->from
$query->join
ну и т.п
А не sql в одну строку.
Ссылка на документацию JDatabase https://docs.joomla.org/Selecting_data_using_JDatabase

Дополнение

Спойлер

[свернуть]

Ну и читать куда удобнее
$query->select
$query->from
$query->join
ну и т.п
А не sql в одну строку.

Это еще большой вопрос что удобнее читать. Прямой запрос выполнится быстрее. Я хоть и пользуюсь JDatabaseQuery оберткой в моделях но каких то особых преимуществ ее использования не вижу. Абсолютно абстрагироваться от sql запросов она не позволяет.

Ну и читать куда удобнее
$query->select
$query->from
$query->join
ну и т.п
А не sql в одну строку.
Ссылка на документацию JDatabase https://docs.joomla.org/Selecting_data_using_JDatabase

$db = JFactory::getDBO();
$query = 'SELECT virtuemart_category_id as category_id, category_parent_id, category_name
$query .= FROM #__virtuemart_categories_'.VMLANG.', #__virtuemart_category_categories
$query .= WHERE #__virtuemart_categories_'.VMLANG.'.virtuemart_category_id = #__virtuemart_category_categories.category_child_id ';
$db->setQuery($query);
$catlist = $db->loadObjectList();

$db =JFactory::getDBO();
$query = $db->getQuery(true);
$query -> select($db->quoteName(array('virtuemart_category_id AS category_id', 'category_parent_id', 'category_name')));
$query -> from($db->quoteName('#__virtuemart_categories_'.VmConfig::$vmlang.', #__virtuemart_category_categories'));
$query->where($db->quoteName('#__virtuemart_categories_'.VmConfig::$vmlang.'.virtuemart_category_id')." = ". $db->quoteName('#__virtuemart_category_categories.category_child_id '));
$db->setQuery($query);
$catlist = $db->loadObjectList();

Ошибка 0 Identifier name '#__virtuemart_categories_ru_ru, #__virtuemart_category_categories' is too long SQL=SELECT `virtuemart_category_id`,`category_parent_id`,`category_name` FROM `#__virtuemart_categories_ru_ru, #__virtuemart_category_categories` WHERE `#__virtuemart_categories_ru_ru`.`virtuemart_category_id` = `#__virtuemart_category_categories`.`category_child_id `

Например вы можете написать такую систему которую будете расширять плагинами!

Я так понимаю, что ему не нравится слишком длинное имя идентификатора. Но в первоначальном варианте все работало.

Об этом я не думал. Но это наверное единственное преимущество. Впрочем если мне нужно будет писать что нибудь расширяемое плагинами то я могу и самостоятельно добавить события.

Потому что вы фигню написали в коде. Вы не можете выбирать сразу из двух таблиц. Используйте join метод раз уж решили через JDatabaseQuery работать.

Вопросы веб разработки - офигенно. Человек даже sql запросы простейшие писать не умеет.

$db =JFactory::getDBO();
$query = $db->getQuery(true);
$query -> select($db->quoteName(array('virtuemart_category_id', 'category_parent_id', 'category_name')));
$query -> from($db->quoteName('#__virtuemart_categories_' . VmConfig::$vmlang, 'a'));
$query -> join('INNER', $db->quoteName('#__virtuemart_category_categories', 'b'). 'ON (' . $db->quoteName('a.virtuemart_category_id'). ' = ' . $db->quoteName('b.category_child_id'). ')');
$db->setQuery($query);
$catlist = $db->loadObjectList();

то честно http://www.mysql.ru/docs/man/Multiple_tables.html  ?)) думаю вам лучше удалить последний абзац )))

Я просто с ходу не могу придумать преимущества, но факт в том что запрос можно собрать в любой последовательности, и как угодно, в любом месте!

Об этом я не думал. Но это наверное единственное преимущество. Впрочем если мне нужно будет писать что нибудь расширяемое плагинами то я могу и самостоятельно добавить события.

Потому что вы фигню написали в коде. Вы не можете выбирать сразу из двух таблиц

Вопросы веб разработки - офигенно. Человек даже sql запросы простейшие писать не умеет.

ну да, еще учусь. а вы, наверное, с рождения все знали и умели?

Разница в скорости если и есть то минимальна(ну сколько надо чтобы преобразовать код в запрос), и компенсируется удобством в использовании, не только плагинов, но и условий, и разумеется чтением, легкостью написание и корректировки.

$db->setQuery('SELECT field from #__table WHERE id = 1')->loadResult();

$query = $db->getQuery(true);
$query->select('field');
$query->from('#__table');
$query->where('id = 1');
$db->setQuery($query)->loadResult();

Ну вообще-то можно, я где-то встречал такой пример, там как  union пойдет вроде. Могу порыться в закромах если интрестно.

Каждый учится по своему. Я лично учился по принципу сначала копируешь, потом понимаешь как работает методом проб и ошибок, а потом пишешь свое. Правда документацию по php и sql всегда держал открытой

Где же тут удобство и легкость написания о котором вы говорите если вместо:

Код: php

$db->setQuery('SELECT field from #__table WHERE id = 1')->loadResult();

Вам нужно написать:

Код: php

$query = $db->getQuery(true);
$query->select('field');
$query->from('#__table');
$query->where('id = 1');
$db->setQuery($query)->loadResult();

$query = $query = $db->getQuery(true)->select('field')->from('#__table')->where('id=1');
$db->setQuery($query)->loadResult();

В плане читаемости по мне так удобнее прочесть запрос когда он в одном месте а не разбросан по коду кусками. Чтобы прочесть запрос который в $query мне приходится вызывать $query->__toString();

Как бы если сделать echo $query то выведется запрос который вам надо прочесть! Вам не надо вызывать $query->__toString()!

echo $query->dump();

Где же тут удобство и легкость написания о котором вы говорите если вместо:
В плане читаемости по мне так удобнее прочесть запрос когда он в одном месте а не разбросан по коду кусками. Чтобы прочесть запрос который в $query мне приходится вызывать $query->__toString(); Даже в моем примере читаемость лучше в простом запросе. Для более менее сложных запросов применение вполне разумно, я писал уже что я и сам в моделях использую эту обертку на всякий случай.

union это вообще из другой Opera. Речь идет о выборке из двух таблиц.

Вы тут сами обещали ТС оторвать руки недавно) Причем за практически те же операции что делает метод where и join в JDatabaseQuery. Они там тоже формируют массивы из которых потом собирают запрос.

Если уж на то пошло, то лучше всего

Код: php-brief

echo $query->dump();

Вот сами не знаете основ ООП в PHP, а человека в чем то обвиняете!  Grin
Как бы если сделать echo $query то выведется запрос который вам надо прочесть! Вам не надо вызывать $query->__toString()! прочтите что делает http://php.net/manual/ru/language.oop5.magic.php#object.tostring

А вот может не надо лукавить?! Ведь запрос можно написать и так:
$query = $query = $db->getQuery(true)->select('field')->from('#__table')->where('id=1');
$db->setQuery($query)->loadResult();
и оно будет более читаемо! потому что ключевые елементы запроса это методы класса, а что брать в скобочках строкой! это вы написали "'SELECT field from #__table WHERE id = 1" а многие не замарачиваются и пишут так: select field from table where id=1 ... и попробуй почитать эту стену текста!

Uion и есть выборка из нескольких таблиц. А Inner Lef Right и т.п это объединение.

Я обычно делал print_r ($query) тогда получаешь все операции с базой. количество результатов и прочие, когда с аяксом вожусь так частенько делаю, если запрос не работает как надо.

Писанины все равно больше. И где же читаемость? В упор не вижу лучшую читаемость. Совершенно бесполезная, длинная цепочка вызовов. Да и кстати где там плагины подключаются, можете показать? Я чего-то не могу найти в исходниках.

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

+ Так и есть.