Closed Bug 1455766 Opened 6 years ago Closed 6 years ago

Langpacks don't handle versions properly


(WebExtensions :: General, enhancement, P5)

57 Branch


(firefox61 fixed)

Tracking Status
firefox61 --- fixed


(Reporter: zombie, Assigned: zbraniecki)


(Whiteboard: typescript-check)


(1 file)

TypeScript check found that `.version` is not available here:

and I didn't follow enough to see if the same is true for `.id`.
Hi Zibi, can you please look into this?
Flags: needinfo?(gandalf)
Hmm, I can see that we're not reading the version, but I'm not sure if we should do anything specific for Langpacks...

Every addon has a version, right? Andrew - can you advise on what I should do here?
Flags: needinfo?(gandalf) → needinfo?(aswan)
Yes, we use the version for langpacks eg here:

Just grab that and store it in this.version I guess?
Flags: needinfo?(aswan)
Priority: -- → P5
Flags: needinfo?(gandalf)
Assignee: nobody → gandalf
Flags: needinfo?(gandalf)
Comment on attachment 8971387 [details]
Bug 1455766 - Handle langpacks versions properly.

It looks like `Langpack.readLocaleFile()` is not reachable, so lets just remove it altogether?
Comment on attachment 8971387 [details]
Bug 1455766 - Handle langpacks versions properly.

r=me with the commit message revised to describe the change
Attachment #8971387 - Flags: review?(aswan) → review+
Pushed by
Handle langpacks versions properly. r=aswan
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
I have devised 2 tests for this:
1. I modify the id of the langpack and install it from file, this will result in corrupted file warning and the langpack is not installed, this behavior is not changed by this fix.
2. I have modified the version number of the langpack, this resulted in a caution unverified langpack for beta 60b13 and a corrupted file warning for beta 61b03.

Is this the desired behavior, for both version and id?
Flags: needinfo?(gandalf)
This bug was just removing some unreachable code, I don't think it needs manual testing.
Flags: needinfo?(gandalf) → qe-verify-
Whiteboard: typescript-check
Product: Toolkit → WebExtensions
You need to log in before you can comment on or make changes to this bug.