Skip to content

Commit 4cadc8d

Browse files
Change variable name
Make createContextMenu always handle the same structure.
1 parent 0fa4657 commit 4cadc8d

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

background.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,11 @@ function getDefaultProvidersClone() {
2525
return defaultProviders.map(p => p.clone());
2626
}
2727

28-
// Set up context menu for images
29-
function createContextMenu(storageProviders) {
28+
// Set up context menu for images, always get whole storage
29+
function createContextMenu(storedSettings) {
3030
const title = chrome.i18n.getMessage('contextMenuTitle');
3131

32-
const selectedProviders = storageProviders.storageProviders.filter(p => p.selected);
32+
const selectedProviders = storedSettings.storageProviders.filter(p => p.selected);
3333

3434
/* If there is only one search provider, do not create a submenu */
3535
if (selectedProviders.length === 1) {
@@ -87,7 +87,7 @@ const defaultSettings = {
8787
If we don't, then store the default settings. */
8888
function checkStoredSettings(storedSettings) {
8989
if (Object.getOwnPropertyNames(storedSettings).length) {
90-
chrome.storage.sync.get('storageProviders', createContextMenu);
90+
chrome.storage.sync.get(null, createContextMenu);
9191
} else {
9292
chrome.storage.sync.set(defaultSettings);
9393
createContextMenu(defaultSettings);

0 commit comments

Comments
 (0)