Opened on 02/19/2016 at 07:17:54 PM
Closed on 03/21/2017 at 07:37:46 PM
Last modified on 03/21/2017 at 07:39:19 PM
#3672 closed change (fixed)
Move antiadblockInit.js from adblockpluscore to adblockplusui
Reported by: | trev | Assignee: | wspee |
---|---|---|---|
Priority: | P2 | Milestone: | |
Module: | Core | Keywords: | goodfirstbug |
Cc: | kzar, sebastian | Blocked By: | |
Blocking: | #4715, #4716 | Platform: | Unknown / Cross platform |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
https://codereview.adblockplus.org/29366969/ |
Description
Background
We agreed that antiadblockInit.js is actually UI code and doesn't belong into core.
What to change
Move antiadblockInit.js and the corresponding strings from adblockpluscore repository to adblockplusui.
Attachments (0)
Change History (12)
comment:1 Changed on 02/19/2016 at 07:31:21 PM by sebastian
- Cc kzar sebastian added
- Keywords goodfirstbug added
comment:2 Changed on 12/06/2016 at 08:00:56 AM by wspee
- Owner set to wspee
comment:3 Changed on 12/07/2016 at 07:49:33 AM by wspee
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:4 Changed on 12/07/2016 at 01:49:23 PM by wspee
- Blocking 4715 added
comment:5 Changed on 12/07/2016 at 01:50:32 PM by wspee
- Blocking 4716 added
comment:6 Changed on 12/07/2016 at 01:57:16 PM by wspee
comment:8 Changed on 12/21/2016 at 08:33:39 AM by abpbot
A commit referencing this issue has landed:
Issue 3672 - Moved antiadblockInit.js from adblockpluscore to adblockplusui
comment:9 Changed on 12/21/2016 at 06:11:36 PM by abpbot
A commit referencing this issue has landed:
Issue 3672 - Moved antiadblockInit.js from adblockpluscore to adblockplusui
comment:10 Changed on 12/22/2016 at 08:48:37 AM by wspee
- Resolution set to fixed
- Status changed from reviewing to closed
comment:11 Changed on 02/16/2017 at 08:55:40 AM by wspee
- Resolution fixed deleted
- Review URL(s) modified (diff)
- Status changed from closed to reopened
During the review of https://codereview.adblockplus.org/29366966/ it was also
suggested to replace the usage of Utils.getString with ext.i18n.getMessage.
Unfortunately I screwed that one up by not testing the changes in Firefox, I
assumed ext.i18n was universally available.
Firefox doesn't have ext.i18n and there is no way (afaict) to provide ext.i18n
for antiadblockInit in a sane fashion.
Therefore I propose to revert the switch to ext.i18n.getMessage in
antiadblockInit, see https://codereview.adblockplus.org/29370594/.
comment:12 Changed on 03/21/2017 at 07:37:46 PM by sebastian
- Resolution set to fixed
- Status changed from reopened to closed
The review for the fix referred above is closed. Moreover, is this issue, while open, preventing dependent issues, i.e. #4715 which landed as well, from being closed. So I'm closing this issue now.
Please note #4715, #4716 which need to be fixed in conjunction.