Closed
Bug 767981
Opened 12 years ago
Closed 12 years ago
mozmarket.js fails on receiptverifier.js: "NewState.name is read-only"
Categories
(Marketplace Graveyard :: Payments/Refunds, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cvan, Unassigned)
References
()
Details
http://cl.ly/2y0C400Q1F423a3s3A0gt https://marketplace-dev-cdn.allizom.org/mozmarket.js includes https://github.com/mozilla/receiptverifier/blob/master/receiptverifier.js which fails on "NewState.name is read-only"
|
Reporter | |
Comment 1•12 years ago
|
||
http://cl.ly/2y0C400Q1F423a3s3A0g/o
|
||
Comment 2•12 years ago
|
||
I was running into this too. Ian do you have any ideas? It's only happening when running the verifier from mozmarket.js; the line of code seems to be https://github.com/mozilla/receiptverifier/blob/master/receiptverifier.js#L74 You can reproduce it at https://marketplace-dev-cdn.allizom.org/mozmarket.js (but we may need to disable it if someone has to use in-app payments on -dev)
|
|
||
Updated•12 years ago
|
Summary: In-app tester fails on JS receiptverifier.js: "NewState.name is read-only" → mozmarket.js fails on receiptverifier.js: "NewState.name is read-only"
|
||
Comment 3•12 years ago
|
||
This was a side effect of "use strict", which the code combining caused to be applied to receiptverifier.js. Fixed in https://github.com/mozilla/receiptverifier/commit/3b805412d5103d87fd1d555253c55f37ac3ea64b (still requires mozmarket.js rebuild)
|
|
||
Comment 4•12 years ago
|
||
aha, thanks. After the code update it's working for me.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•