chrome.storage.local.get results in “Undefined” when called

Question

I\'m building a chrome extension, and I needed to save some data locally; so I used the Storage API . I got to run the simple example and save the data, but when I integrate

Answer 1

First of All thanks to Rob and BreadFist and all you guys. I found out why my code wasn't working. Storage.Set doesn't accept the key to be an 'integer' and even if you try to convert that key to be a 'string' it won't work too. So I've added a constant character before each key and it worked. Here's my code.

function saveResults(Id, resultsArray) {
    var key = Id.toString();
    key = 'a'.key;
    chrome.storage.local.set({key : resultsArray});
}

function Load(Id) {
    var key = Id.toString();
    key = 'a'.key;
    chrome.storage.local.get(key, function(result){
        console.debug('result: ', result.key);
    });
}

Answer 2

I am not certain what type of data you are saving or how much, but it seems to me that there may be more than one newsId and a resultsArray of varying length for each one. Instead of creating keys for each element of resultsArarry have you considered just storing the entire thing as is. An example of this would be:

chrome.storage.local.set({'results':[]});

function saveResults(newsId, resultsArray) {
  // first combine the data into one object
  var result = {'newsId':newsId, 'resultsArray':resultsArray};

  // next we will push each individual results object into an array
  chrome.storage.get('results',function(item){
    item.results.push(result);
    chrome.storage.set({'results':item.results});
  });
}

function getResults(newsId){
  chrome.storage.get('results', function(item){
    item.results.forEach(function(v,i,a){
      if(v.newsId == newsId){
        // here v.resultsArray is the array we stored
        // we can remove any part of it such as
        v.resultsArray.splice(0,1);
        // or
        a.splice(i,1);
        // to remove the whole object, then simply set it again
        chrome.storage.local.set({'results':a});
      }
    });
  });
}

This way you don't need to worry about dynamically naming any fields or keys.