I noticed that the products report was not showing the first adjustment's SKU even if the product count was right.
It seems the way the array are merged in _uc_reports_product_get_skus() will discard the first adjustment SKU.

CommentFileSizeAuthor
#3 uc_reports.admin_.inc_products_report_sku.patch1015 bytesquadbyte
#1 _uc_reports_product_get_skus_1393000.patch1019 bytesquadbyte

Comments

quadbyte’s picture

quadbyte commented
StatusFileSize
new _uc_reports_product_get_skus_1393000.patch1019 bytes

Status: Needs review » Needs work

The last submitted patch, _uc_reports_product_get_skus_1393000.patch, failed testing.

quadbyte’s picture

quadbyte commented
Status: Needs work » Needs review
StatusFileSize
new uc_reports.admin_.inc_products_report_sku.patch1015 bytes

Re-rolled the patch toward latest dev version.

Status: Needs review » Needs work

The last submitted patch, uc_reports.admin_.inc_products_report_sku.patch, failed testing.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs work » Fixed

Confirmed and committed, thanks for the patch, apologies for taking so long to get around to reviewing it.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.