Can't download my addon

RESOLVED FIXED in Builder 1.0

Status

P2
normal
RESOLVED FIXED
7 years ago
4 years ago

People

(Reporter: erikvvold, Assigned: smcarthur)

Tracking

unspecified
Builder 1.0

Details

(Reporter)

Description

7 years ago
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]
First file contains a "var {UserScript} ="
I haven't seen this construct before ... Are you sure it's a valid JS?
(Assignee)

Comment 3

7 years ago
@zalun: It is (Firefox only). It's used through the SDK, even.
(Reporter)

Comment 4

7 years ago
(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)
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

7 years ago
Assignee: nobody → smcarthur
Priority: -- → P2
Target Milestone: --- → Builder 1.0
Is this still an issue under new SDK?
(Assignee)

Comment 7

7 years ago
Reopen if it's still an issue.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.