Jump to content
Официальный форум поддержки Simpla

Товары скрытых категорий

Wizard

By Wizard,
in Готовые решения

 Share Followers 1

Recommended Posts

Wizard

Wizard

Posted
Posted

Как то не обращал внимания и наткнулся на не совсем логичную штуку. Товары скрытых категорий участвуют в выводе. Зачем функция отключения каталога, для того что бы убрать ее с менюшки?

Вообщем в каталоге решил этот момент следующим образом

// Выберем текущую категорию
if (!empty($category_url))
{
  $category = $this->categories->get_category((string)$category_url);
  if (empty($category) || (!$category->visible && empty($_SESSION['admin'])))
  return false;
  $this->design->assign('category', $category);

  $categories = $this->categories->get_categories();
  foreach($category->children as $id){
       if($categories[$id]->visible>0)
           $filter['category_id'][] = $id;
   }

}



Скрываем бренды которые относятся к товарам находящихся в скрытой категории. 

// Выбираем бренды, они нужны нам в шаблоне	if(!empty($category)){   $brands = $this->brands->get_brands(array('category_id'=>$filter['category_id']));   $category->brands = $brands;}


Обновленная версия http://simpla.cms-update.ru/catalog/bytovaya-tehnika

Стандарт http://demo.simplacms.ru/catalog/bytovaya-tehnika - если в меню есть категория пылесосы - отключите ее в админке для того что бы увидеть разницу.
Link to post
Share on other sites
Novecheg

Novecheg

Posted
Posted

Многие, как я уже понял, сталкивались. Где-то видел эту тему. Пикусов на этот вопрос ответил "скрыта категория, а не товар, значит нужно отобразить товар, а категорию скрать".
Ещё что-то говорил про то, что товар может принадлежать 2 категориям (сейчас не нашёл этой темы)

Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted

Если товар принадлежит двум категориям при этом ода из категорий скрыта то товар будет отображаться в той категории которая доступна, а в той которая скрыта – не будет. это логично. Под функционалам "Скрыть категорию" - предполагается скрыть не только категорию но и товары, которые относятся к ней.

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

Получается - я отключаю каталог "пылесосы", захожу в категорию бытовая техника и что я там вижу? Я вижу пылесосы висящие в воздухе, их не найти в дочерних каталогах данной категории!!! И тут понимание логики Пикусова у меня заканчивается)))

Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted

а что с запросом не так? выложенный пример очень даже работает и ничего сложного в нем нет. Запрос к товарам приходит по ид категории - просто не отправляем ид-ы скрытых категорий. этого вполне достаточно.

Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted

kors, я немного не могу вьехать в то что вы пытаетесь сказать.

вот этой функцией мы отбираем только видимые категории (мы и без того одним запросов выбираем всю иерархию каталога - чего добру пропадать.)

$categories = $this->categories->get_categories();   foreach($category->children as $id){        if($categories[$id]->visible>0)            $filter['category_id'][] = $id;    }


, и в дальнейшем по отобранным идам делаем запрос к товарам. не знаю как еще объяснить... Не обязательно выборку по доступным категориям делать через sql запрос, хотя и там нет ну прям ничего сложного!!
Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted

Зачем нам отправлять в запрос выборки товаров Иды скрытых категорий? Для того что бы дополнительным запросов убедится в том что они скрыты?))))

Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted

если товар присутствует в другой категории и она не скрыта то естественно надо показать этот товар!

Какова суть категории? Она необходима для того что бы прикрепить к ней товары. Если категория доступна - выводим товары, если категория не доступна - не выводим товары, если хотим что бы товары были не доступны - отключаем товары.

что в предложенном мною коде не логично или не соответствует функциональности - отключить категорию? Есть моменты которые надо доработать, но это уже ближе к истине нежели существующая функциональность.! Выбрать ID-ы скрытых категорий можно на стадии формирования дерева каталога, это ведь тоже не слоджно - добавить массив ID-ы - visible_ids

У нас есть пылесосы которые прикреплены к двум категориям, давайте посмотрим что получается с моим не совершенным кодом:
http://simpla.cms-update.ru/catalog/tehnika-dlya-uborki
http://simpla.cms-update.ru/catalog/bytovaya-tehnika - в этой категории есть пылесосы, и они отключены. И О БОЖЕ - товары не выводятся!!! Не вероятно, это надо осмыслить!

или вот

http://demo.simplacms.ru/catalog/bytovaya-tehnika если категория пылесосы отключена, то товары все равно выводятся. Это куда более сложно осмыслить... Я как покупатель перехожу в категорию бытовая техника, в меню вижу Миксеры, а в списке товаров Пылесосы Dyson. - Откинувшись на спинку кресла подумал - "а может они еще и кофе варят??", смотрю на ценник и понимаю - "за такую цену должны, не просто кофе варить но и в постель приносить"

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

Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted

Можно открыть через хлебные крошки, если и дальше оставить этот баг таким какой он есть

http://simpla.cms-update.ru/products/pylesos-dyson-dc23-pink

Что вы видете в "хлебных крошках" данного товара? Мы там видим Техника для уборки. Вы ожидали увидеть там что то другое?

А ведь эта категрия действительно доступна! Блин как же так, все должно быть сложнее, мы же должны были увидеть там пылесосы! Вот блин...

Вот еще пример того как можно получить видимые категории всех дочерних категорий, но вы ведь и сами об этом знаете

if($pointers[$id]->visible>0){
$pointers[$id]->visible_children[] = $id;

if(isset($pointers[$pointers[$id]->parent_id]->visible_children))
$pointers[$pointers[$id]->parent_id]->visible_children = array_merge($pointers[$id]->visible_children, $pointers[$pointers[$id]->parent_id]->visible_children);
else
$pointers[$pointers[$id]->parent_id]->visible_children = $pointers[$id]->visible_children;
}



но в этом нет потребности потому что вот этим

$categories = $this->categories->get_categories();   foreach($category->children as $id){        if($categories[$id]->visible>0)            $filter['category_id'][] = $id;    }


мы делаем то же самое! У на уже есть иды дочерних категорий, нам осталось понять видимые они или нет
Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted

Как я понял вы считаете что данный пример не может работать корректно. Можно конкретный пример подтверждающий это?

1. Создаю категорию Пылесосы2 в категории Пылесосы.
2. Создаю в категории Пылесосы2 товар ппппппппп.
2. Категорию Пылесосы2 включаю.
3. Категорию Пылесосы выключаю.

Этот пример только подтверждает недоработку этого момента и не опровергает работоспособность моего примера.

Тут осталось дополнить функцию - выключаем видимость всех дочерних категорий, если отключаем материнский каталог. Потому что это логично! Категория у нас ДРЕВОВИДНАЯ - отключая каталог мы должны отключить всех ее потомков. Иначе зачем нам такая структура каталога? Зачем нам дочерние категории если материнские на них не влияют?

Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted



Вот посмотрите что получается - я потянул категория и все дочерние каталоги тоже потянулись! А почему? То есть по мнению автора логично перетянуть все дочерние категории вместе с материнским но не логично отключать видимость дочерних категорий если мы отключаем материнскую.

Бррр... я вообще теряю нить логики автора))))

Древовидная структура необходима для зависимости дочерних категорий. И все манипуляции связанные с категориями верхнего уровня должны отображаться на дочерних.

Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted

А это о чем? "подтверждающий это" - чему хотите подтверждение?
Может, это имеете в виду http://demo.simplacms.ru/catalog/pylesosy

А в чем тут проблема?

if(visible==0)  return false


В чем проблема дописать? Это еще один минус в копилку багов имеющейся функциональности.

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

получается нужна еще одна кнопочка

- выключить
- отключить видимость на сайте

Дааа... это глобально! На месяц работы!)))
Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted

Вообщем не хочу продолжать данную тему. Я наглядно показал что на данный момент, с имеющейся функциональностью можно исправить этот косяк. В Симпле по мимо имеющихся багов, к сожалению присутствую и такие, которые вызваны ленью...

Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted

Я думаю, что автор просто не подумал про все возможные варианты

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

Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted

Глобально - не глобально, а работа тут нужна.

Конечно нужно, люди деньги иногда платят за симплу.

Если считаете, что за мизерное время можете легко поправить...

Ничто не избавит человека от нелогичности и ошибок. Но ничто так не логично как бездействие зная о не наличии нелогичностей и ошибок, и нежелания их исправить.

А разве кто в этом сомневался и просил это показать наглядно?

Если данная тема обсуждается, значит в этом есть потребность. Я так рад что сам автор не отписывается таким вот образом

- "Автор, у вас тут баг, он идет в разрез логике функциональности!"
- "А разве вас кто то просил покупать симплу? Я ведь все го лиш человек. Да, я знаю что я там не доработал, теперь и вы об этом знаете - давайте оставим все как есть..."

Link to post
Share on other sites
pikusov

pikusov

Posted
Posted

Еще один способ сделать невидимыми товары из невидимой категории:
api/Products.php в функции get_products() изменить фильтр по видимости на:

if(!empty($filter['visible']))     $visible_filter = $this->db->placehold('AND p.visible=? AND (SELECT count(*) FROM __categories, __products_categories WHERE __categories.id = __products_categories.category_id AND __categories.visible=1 AND p.id=__products_categories.product_id) > 0', intval($filter['visible']));
Link to post
Share on other sites
pikusov

pikusov

Posted
Posted

Я рассматриваю категории исключительно для структурирования товаров, ближайший аналог чтоб было понятно - это как "теги" товаров. Более того, у товара вообще может не быть категории.
Я могу принять аргумент в том что показ товаров из скрытой категории это не совсем логично, но и в вашем случае логика пропадает, если товары имеют по несколько категорий, часть из которых скрыты. Или даже если все категории скрыты - может быть мы хотим продавать товары по брендам, и категории нам вообще не нужны?

Link to post
Share on other sites
Wizard

Wizard

Posted
  • Author
Posted

Денис, бывает разные случаи, разные потребности. Будем мы продавать по брендам, нужны ли нам категории или нет и для чего мы скрываем категории в которым прикреплены товары - это дело индивидуальное. Все случае не перечислить и всего не учесть.

Просто по моему если есть категория и к ней по какой то причине прикреплены товары а админ решил отключить категорию (возможно временно) - то в этом случае товары не должны выгружаться. Это соответствует функциональности - "Отключить категорию"

Link to post
Share on other sites
Promo

Promo

Posted
Posted

Я конечно может чего не понимаю, но вот это

Wizard 6 Август
Зачем нам отправлять в запрос выборки товаров Иды скрытых категорий? Для того что бы дополнительным запросов убедится в том что они скрыты?))))

И вот это


pikusov 17:27
Еще один способ сделать невидимыми товары из невидимой категории:
api/Products.php в функции get_products() изменить фильтр по видимости на:

if(!empty($filter['visible']))
$visible_filter = $this->db->placehold('AND p.visible=? AND (SELECT count(*) FROM __categories, __products_categories WHERE __categories.id = __products_categories.category_id AND __categories.visible=1 AND p.id=__products_categories.product_id) > 0', intval($filter['visible']));

Не одна ли и таже тема? Для чего делать запрос по тем категориям которые скрыты?

Link to post
Share on other sites
Promo

Promo

Posted
Posted

pikusov 18:04
Я рассматриваю категории исключительно для структурирования товаров, ближайший аналог чтоб было понятно - это как "теги" товаров. Более того, у товара вообще может не быть категории.



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

Дальше лучше, может мы и фильтры делать начнем древовидными?

Link to post
Share on other sites
  • 4 years later...
Skynet

Skynet

Posted
Posted (edited)

Добрый день!

 

Версия Simpla 2.0.2

 

После исправления Wizard-a и pikusov-a, категории отключаются.

 

Но есть несколько замеченных ошибок, которые так же хотелось бы исправить.

 

Если выключить все имеющиеся категории, то на сайте:

 

1. По прямой ссылке на товар, страница товара открывается.

пример: http://demo.simplacms.ru/products/samsung-s5570-galaxy-mini

 

Как можно отключить так же доступность товара по прямой ссылке?

 

2. Отмеченные хитами товары отображаются на главной странице, если они находятся в отключеной категорий.

 

Как можно исправить данную ситуацию?

 

3. На сайте при поиске товара по названию или артикулу, высвечиваются выпадающий список предлагаемых товаров, если перейти на выбранный товар, он не доступен.

 

Как исправить, что при вводе названия товара или артикула, товар не высвечивался, если он находится в выключенной категории?

Edited by delcc
Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...
×
 Share
Followers 1
Go to topic listing
×
×
  • Create New...