Exceptions in actions of xquantity_stock

Created on 24 November 2023, about 1 year ago

Updated 21 May 2024, 7 months 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 7 months 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 about 1 year ago1:16pm 1 December 2023
Comment about 1 year ago →
🇩🇪Germany danielspeicher Steisslingen
I just handled the case of the variable $variations being NULL.
First commit to issue fork.
Comment 7 months 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.71.5 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