Перейти к содержимому


Фотография
* - - - - 1 Голосов

Фильтр не отображает "Нет в наличии"


Лучший Ответ SmetDenis , 12 April 2016 - 09:42

Я не понимаю в чем тут логический конфликт?
 

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

Сейчас мы безуспешно пытаемся добавить эту фичу. Перекраивать базу данных для это опции точно будет вредным и долгим шагом.

 

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

 

 

Список измененных файлов я писал выше. На данный момент это

media\zoo\applications\jbuniversal\elements\jbprice\jbprice.php

media\zoo\applications\jbuniversal\framework\render\pricefilter\element.balance.php

 

Я рекомендую вам рассмотреть альтернативные варианты решения задачи.

Например, сделать отдельное поле-checkbox, который будет показывать наличие товара и не зависеть от цены.

Перейти к сообщению


  • Закрытая тема Тема закрыта
Сообщений в теме: 27

#21 AidarT

AidarT

Отправлено 06 April 2016 - 10:54

Спасибо.

 

Нашел в чем была причина.

Пустые значения по умолчанию не попадают в поиск.

 

Добавил в исключения + переиндексация

media\zoo\applications\jbuniversal\elements\jbprice\jbprice.php

5bf4_200x0.png

 

Остальное все по сути тоже что и писал ранее.

Проверьте пожалуйста.

Спасибо, но не заработало. Фильтр теперь показывает всегда "Есть на складе". Переиндексацию еще раз сделал, почистил кеш, не помогает. Прикрепил скрин, как выглядит фильтр.

Прикрепленные изображения

  • Screenshot_1.png

Сообщение отредактировал AidarT: 06 April 2016 - 18:09

  • 0

Our democracy has been hacked


#22 AidarT

AidarT

Отправлено 08 April 2016 - 04:12

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


  • 0

Our democracy has been hacked


#23 SmetDenis

SmetDenis

Отправлено 08 April 2016 - 05:19

Добавил правку. Проверьте, пожалуйста.
  • 0
JBZoo v4.0 и новый чудный мир Open Source GPL
Отключайте проверку лицензий как можно скорее!



— Есть два типа людей: Кто еще не делает бекапы и кто уже делает бекапы.


#24 AidarT

AidarT

Отправлено 08 April 2016 - 10:36

Денис, теперь фильтр по умолчанию отображает "Нет в наличии",  должно по умолчанию стоять "-Все-". И неверно отбирает товары которые есть в наличии и которых нет. Можете проверить сами на сайте, ссылку и доступы я в личку направлял.

Прикрепленные изображения

  • Screenshot_2.png

  • 0

Our democracy has been hacked


#25 SmetDenis

SmetDenis

Отправлено 11 April 2016 - 05:45

Я вижу разные результаты поиска по обоим условиям.
С контентом не сравнивал, но похоже что работает фильтр верно. При переключении значение в поле сохраняется.

hje7_200x0.png

auy6_200x0.png

Значение "Все" (пустая строка) и "Нет в наличии" (0) - в обоих случаях php считает как пустое значение. Так же работает и JS.
Получается логический конфликт.
Я предлагаю по умолчанию сделать выбор "Есть на складе", а "Все" совсем убрать.
Думаю пользователи заинтересованы в том что сейчас есть в магазине.
  • 0
JBZoo v4.0 и новый чудный мир Open Source GPL
Отключайте проверку лицензий как можно скорее!



— Есть два типа людей: Кто еще не делает бекапы и кто уже делает бекапы.


#26 AidarT

AidarT

Отправлено 11 April 2016 - 16:56

Денис, при выборе значений "Есть в наличии" и "Нет в наличии" фильтр отбирает теже самые позиции, на ваших скриншотах это видно, т. е. неверно работает (товар же не может быть и в наличии и не в наличии одновременно). Я не понимаю в чем тут логический конфликт? Для меня логичная работа фильтра выглядит так: 1) Все материалы - "Все" 2) Нет в наличии - "0" 3) Есть на складе - "-1" (бесконечно) или 100500 (соответственно 100500 шт.) 4) Под заказ - "-2". Это я правильно понимаю? Возможно я ошибаюсь. Это все значения из поля "price_balance". Поле "price_balance" всегда имеет какое-то значение я думаю и не бывает пустым. Если моя логика не реализуема, то напишите пожалуйста какие файлы правили, чтобы я откатился на прежнюю версию и думаю из фильтра уберу эту фишку, чтобы не вводить в заблуждение пользователей. Спасибо в любом случае.


  • 0

Our democracy has been hacked


#27 SmetDenis

SmetDenis

Отправлено 12 April 2016 - 09:42   Лучший Ответ

Я не понимаю в чем тут логический конфликт?
 

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

Сейчас мы безуспешно пытаемся добавить эту фичу. Перекраивать базу данных для это опции точно будет вредным и долгим шагом.

 

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

 

 

Список измененных файлов я писал выше. На данный момент это

media\zoo\applications\jbuniversal\elements\jbprice\jbprice.php

media\zoo\applications\jbuniversal\framework\render\pricefilter\element.balance.php

 

Я рекомендую вам рассмотреть альтернативные варианты решения задачи.

Например, сделать отдельное поле-checkbox, который будет показывать наличие товара и не зависеть от цены.


  • 0
JBZoo v4.0 и новый чудный мир Open Source GPL
Отключайте проверку лицензий как можно скорее!



— Есть два типа людей: Кто еще не делает бекапы и кто уже делает бекапы.


#28 AidarT

AidarT

Отправлено 12 April 2016 - 09:58

 

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

Сейчас мы безуспешно пытаемся добавить эту фичу. Перекраивать базу данных для это опции точно будет вредным и долгим шагом.

 

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

 

 

Список измененных файлов я писал выше. На данный момент это

media\zoo\applications\jbuniversal\elements\jbprice\jbprice.php

media\zoo\applications\jbuniversal\framework\render\pricefilter\element.balance.php

 

Я рекомендую вам рассмотреть альтернативные варианты решения задачи.

Например, сделать отдельное поле-checkbox, который будет показывать наличие товара и не зависеть от цены.

 

Хорошо, спасибо Денис. Все понятно. Сейчас откачусь на старые файлы. Тему можно закрывать.


  • 0

Our democracy has been hacked





Click to return to top of page in style!