This patch sets a message when user is adding an item to the cart that is already there.

This ended up touching more places than I wanted, but I'm posting it here anyway for review.

See screenshot.

CommentFileSizeAuthor
commerce_cart_ux_duplicates.patch3.11 KBrecidive
Captura de tela 2010-11-18 às 18.08.28.png17.65 KBrecidive

Comments

rszrama’s picture

rszrama commented

For reference, this is essentially the same thing Amazon does:

Only local images are allowed.

rszrama’s picture

rszrama commented
Status: Needs review » Needs work

This patch is out of date, as it still references the old "product" field (now commerce_product). Additionally, instead of loading the order and looping over its line items like this patch does, we could just use a single EntityFieldQuery. I'm also not sure this needs to be a core feature. This could probably even be implemented using Rules as it is now.

recidive’s picture

recidive commented
Status: Needs work » Closed (won't fix)

Yeah. This is just stupid!

I'll check if I can do this with rules and post the results here.

rszrama’s picture

rszrama commented

lol I didn't think it was that bad of an idea. It would probably make for a handy contrib to have. : )

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Closed (won't fix) » Postponed

Setting as postponed (as it would suit in contrib) maybe we should build a list for people wanting to do some contrib work with all this stuff

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Component: User experience » Contributed modules
bojanz’s picture

bojanz commented
Issue summary: View changes
Status: Postponed » Closed (won't fix)