861192 - gcli addon command shows [object Object] instead of a list of addons

Status: RESOLVED FIXED

(DevTools Graveyard :: Graphic Commandline and Toolbar, defect)

Description

[Frederik Braun [:freddy]]

I opened the gcli (Shift+F2) and typed "addon list".

Expected Result:
a list of all addons.

Actual Result:
The string "[object Object]" was displayed

Comment 1

[Frederik Braun [:freddy]]

Still exists on nightly.

Comment 2

[Josh Marchán]

Attachment: reviving and modernizing addon list

This should revive the command -- I took the chance to split it into a command and a converter, and putting a bit of spit and polish on the listing itself. Once bug 855124 lands, I'd like to also add a Prefs button to the listing, which should be simple and quick. A simpler fix could've been to just add returnType: "view" to the command, but that would be boring! :) (but doable if requested)

Joe: I might owe you test cases for this. Still trying to figure out how tests work, and try access is still not quite there (relevant bug reopened, hopefully it was just my mistake)

Comment 3

[Joe Walker [:jwalker]]

Comment on attachment 737812 [details] [diff] [review]
reviving and modernizing addon list

Review of attachment 737812 [details] [diff] [review]:
-----------------------------------------------------------------

I'm holding Mike up on a bug, and he originally wrote the add-on command, so I'm going to bounce the r? to him.

Comment 4

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Comment on attachment 737812 [details] [diff] [review]
reviving and modernizing addon list

Review of attachment 737812 [details] [diff] [review]:
-----------------------------------------------------------------

This looks excellent and the refactoring was very welcome.

The test still passes fine so there is nothing for you to do there:
./mach mochitest-browser browser/devtools/commandline/test/browser_cmd_addon.js

The one problem is that if you use the enable / disable buttons on an addon that has no version number (e.g. Shockwave Flash) then the addon name in the commandline has an extra _ so it fails to disable:
addon disable Shockwave_Flash_

Once you fix that I will give you an r+.

Comment 5

[Josh Marchán]

Attachment: v2

Oops! Good catch, and thanks for the mochitest incantation, I meant to ask how to do that. :)

Comment 6

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Comment on attachment 738448 [details] [diff] [review]
v2

Review of attachment 738448 [details] [diff] [review]:
-----------------------------------------------------------------

Magnifico, r+

Comment 7

[Panos Astithas (he/him) [:past] (please ni?)]

https://hg.mozilla.org/integration/fx-team/rev/a7e584370ab9

Comment 8

[Tim Taubert [:ttaubert] (inactive)]

https://hg.mozilla.org/mozilla-central/rev/a7e584370ab9