Closed
Bug 693217
Opened 14 years ago
Closed 13 years ago
Can't download my addon
Categories
(addons.mozilla.org Graveyard :: Add-on Builder, defect, P2)
addons.mozilla.org Graveyard
Add-on Builder
Tracking
(Not tracked)
RESOLVED
FIXED
Builder 1.0
People
(Reporter: erikvvold, Assigned: smcarthur)
Details
When I try to download https://builder.addons.mozilla.org/addon/1019777/latest/ I get a spinner for a few minutes then the spinner stops and nothing happens.
When I try to download https://builder.addons.mozilla.org/addon/1004755/latest/ it works perfectly.
Same thing here.
After I click "Download", the web console's net tab shows a bunch of calls like
[19:22:22.623] GET https://builder.addons.mozilla.org/xpi/check_download/1019777r4/ [HTTP/1.1 200 OK 104ms]
Clicking the "Test" button, builder pops up a huge red error box, and the web console logged
[19:23:11.003] GET https://builder.addons.mozilla.org/xpi/test/1019777r4/ [HTTP/1.1 404 NOT FOUND 106ms]
Comment 2•14 years ago
|
||
First file contains a "var {UserScript} ="
I haven't seen this construct before ... Are you sure it's a valid JS?
Assignee | ||
Comment 3•14 years ago
|
||
@zalun: It is (Firefox only). It's used through the SDK, even.
(In reply to Piotr Zalewa [:zalun] from comment #2)
> First file contains a "var {UserScript} ="
> I haven't seen this construct before ... Are you sure it's a valid JS?
Yeah, it's called a destructuring assignment.
https://developer.mozilla.org/en/New_in_JavaScript_1.7#Destructuring_assignment_(Merge_into_own_page.2Fsection)
Comment 5•14 years ago
|
||
Hey Erik,
I think it is a module scoping issue. I modified your Userscripts library via a copy of your add-on (https://builder.addons.mozilla.org/addon/1019947/latest/) and now it both test installs and downloads fine.
When I first tried your add-on via test, there was an error thrown out of the greasemonkey-api.js file indicating that a module was not found when it was being constructed. What was happened is that when you required the tabs and clipboard APIs in the greasemonkey-api.js file, the string used was simply "tabs" and "clipboard", I changed this to "addon-kit/tabs" and "addon-kit/clipboard" and the error was corrected.
I will look into the path stuff and see if we can't do the same thing with addon-kit scoping that we do with api-utils.
Updated•14 years ago
|
Assignee: nobody → smcarthur
Priority: -- → P2
Target Milestone: --- → Builder 1.0
Comment 6•14 years ago
|
||
Is this still an issue under new SDK?
Assignee | ||
Comment 7•13 years ago
|
||
Reopen if it's still an issue.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•