When updating transactions in the store, if the username is numeric, the module thinks it is a uid and inserts improper values into the database. I have attached a diff that works for me, but I am not certain if I have broken anything else in the process.

Thank you,
Joshua Rubin

CommentFileSizeAuthor
store.diff_0.txt1.89 KBrubinj

Comments

sime’s picture

sime
any/any
Location Melbourne
commented

Problem verified.

sime’s picture

sime
any/any
Location Melbourne
commented
Priority: Normal » Critical

actually I think this is critical, because you could add transactions against the wrong customer.

brmassa’s picture

brmassa commented
Version: 4.7.x-1.x-dev » 5.x-3.x-dev
Status: Active » Needs review

Guys,

i confirm the problem too.
besides the patch (i didnt tested the patch), i believe all "uid" references should be changed to "username". Probably thats why the programmer used a is_numeric.

massa

brmassa’s picture

brmassa commented
Version: 5.x-3.x-dev » 5.x-4.x-dev
Status: Needs review » Active

Guys,

anyone can see if the bug is still there? specially for eC4

regards,

massa

brmassa’s picture

brmassa commented
Status: Active » Fixed

Guys,

its fixed on eC4

regards,

massa

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)