Exceptions in actions of xquantity_stock

Created on 24 November 2023, 7 months ago

Updated 21 May 2024, about 1 month ago

Problem/Motivation

In both action VariationXQuantityStockAdjust and VariationXQuantityStockSet in the function, buildConfigurationForm it happens that the query to commerce_product_variations returns an array with one empty element.
The loadMultiple method returns an empty array then, which is allowed. And the following method reset in the for loop returns FALSE. This results in the exception:

Call to member function on getFieldDefinition() on bool (In the for loop).

This needs to be considered. I will provide a patch soon.

Steps to reproduce

Add products and variations and trigger the actions.

🐛 Bug report

Status

Needs review

Version

2.0

Component

Xquantity Stock

Created by

🇩🇪Germany danielspeicher Steisslingen

Live updates comments and jobs are added and updated live.

Merge Requests

!4Exceptions in actions of xquantity_stock
Open
🇩🇪Germany danielspeicher
updated about 1 month ago

Comments & Activities

Issue created by @danielspeicher
Merge request !4Issue #3404050: Exceptions in actions of xquantity_stock → (Open) created by danielspeicher
Status changed to Needs review 7 months ago1:16pm 1 December 2023
Comment 7 months ago →
🇩🇪Germany danielspeicher Steisslingen
I just handled the case of the variable $variations being NULL.
First commit to issue fork.
Comment about 1 month ago →
🇩🇪Germany jurgenhaas Gottmadingen
I have added a few more fixes, since we get more exceptions being thrown.

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024

Exceptions in actions of xquantity_stock

Problem/Motivation

Steps to reproduce

Merge Requests

!4Exceptions in actions of xquantity_stockOpen

Comments & Activities

!4Exceptions in actions of xquantity_stock
Open