Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
Fix crash for regional adblock packaged builds
Browse files Browse the repository at this point in the history
Fix #4491

Auditors: @bsclifton

Test Plan: Make sure regional adblock works from packaged builds, see the issue #4491.
  • Loading branch information
bbondy committed Oct 4, 2016
1 parent 94b4185 commit b65291d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions js/stores/appStore.js
Original file line number Diff line number Diff line change
Expand Up @@ -708,7 +708,7 @@ const handleAppAction = (action) => {
appState = appState.setIn(['menu', 'template'], action.menubarTemplate)
break
case AppConstants.APP_UPDATE_ADBLOCK_DATAFILES:
const adblock = require('../../app/adblock')
const adblock = require('../../app/adBlock')
adblock.updateAdblockDataFiles(action.uuid, action.enable)
handleAppAction({
actionType: AppConstants.APP_CHANGE_SETTING,
Expand All @@ -717,7 +717,7 @@ const handleAppAction = (action) => {
})
return
case AppConstants.APP_UPDATE_ADBLOCK_CUSTOM_RULES: {
const adblock = require('../../app/adblock')
const adblock = require('../../app/adBlock')
adblock.updateAdblockCustomRules(action.rules)
handleAppAction({
actionType: AppConstants.APP_CHANGE_SETTING,
Expand Down

2 comments on commit b65291d

@bsclifton
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bbondy Is there a way to force case sensitivity? (ex: like by using strict mode?)

@bbondy
Copy link
Member Author

@bbondy bbondy commented on b65291d Oct 5, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was wondering how best to lint that as well, this bit us a few times.

Please sign in to comment.