Skip to content

release 9.14#559

Open
aiiddqd wants to merge 1 commit intomasterfrom
r-9-14
Open

release 9.14#559
aiiddqd wants to merge 1 commit intomasterfrom
r-9-14

Conversation

@aiiddqd
Copy link
Collaborator

@aiiddqd aiiddqd commented Jul 20, 2024

Какие проблемы решаем? Что есть?

  • ...

Как надо? Как это должно работать? Как это будет?

  • ...

Чек лист

  • проверка локально
  • тест на стенде https://wooms.wpcraft.ru/
  • В changelog добавлено описание изменений

@darkvovich
Copy link

darkvovich commented Jul 22, 2024

Добавьте исправление опечатки в настройках:
Если включить опцию, то при обновлении продуктов категории не будут учтываться в соответствии с группами МойСклад.
учтываться

  • в файле ProductStocks.php функция update_stock(
    stock = это все остатки без уета резерва

@darkvovich
Copy link

В файле includes/ProductSingleSync.php несколько багов

  1. wooms_product_sinle_sync - название переменной с опечаткой, лучше бы исправить на single
  2. критичный баг в функции sync(
    строку if (empty($data['variantsCount'])) {
    нужно заменить на
    if (empty($row['variantsCount'])) {

иначе никаких синхронизаций не будет, variantsCount есть только в $row

@darkvovich
Copy link

darkvovich commented Sep 26, 2024

Файл ProductVariable.php функция update_variation(
строка if ( ! empty( $row["salePrices"][1]['value'] ) ) { // 0 - цена со скидкой, 1 - цена розница

вот тут не хватает функции - брать выбор из НАСТРОЕК как в файле productprices (if ($price_name = get_option('wooms_price_id')) {)

@darkvovich
Copy link

Я пока дебажил - заметил, что некоторые функции работают по 2 раза почему-то, хотя достаточно 1 раз.

@darkvovich
Copy link

Нужна подпись к настройке "Учитывать остатки с резервом" - мы ее выключили т.к. нам нужны только доступные остатки (без резерва), НО парадокс в том, что наоборот - только с ней берутся остатки без резерва, а без нее - все !
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants