In our site, we have a lot of "Adjusted" SKUs (for various sizes and other attributes). The Stock view (node/nn/edit/stock) sometimes misses a SKU or two. I've tracked it down to this line in uc_product.module (aroudn line 1585)

models[$node->model] = $node->model;

I'm not much of a PHP programmer, but this seems broken to me. Instead, doing something like

$models[] = $node->model;

(i.e. append the base SKU).

seems to solve the problem.

CommentFileSizeAuthor
#1 479856.sku_fix.2.x.patch543 bytescha0s

Comments

cha0s’s picture

Assigned: Unassigned » cha0s
Status: Active » Needs review
StatusFileSize
new543 bytes

Yeah, dunno why that code was left like that (was my mistake).

Your fix is correct, thanks! =)

Island Usurper’s picture

Status: Needs review » Fixed

Thanks, guys. Committed.

Status: Fixed » Closed (fixed)

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