Storage from @ionic/storage loses data

From @Lafaa on October 30, 2016 16:29

Hi guys! Am i the only one having problem with the new Storage since the ionic2 rc0?

It just forgets everything once the app is restarted. I use the Storage class from @ionic/storage to save username and password, but it’s like it get cleaned on app restart.

I’m using the Storage this way:

 public storage: Storage;
...
this.storage = new Storage();
...
this.storage.set(key, value);

I’ve also added the Storage in app.module.ts, both in the import section AND in the providers array. Did i get something wrong? In the running and building process i don’t have any error or warnings

Cordova CLI: 6.3.1 Ionic Framework Version: 2.0.0-rc.0-201610131811 Ionic CLI Version: 2.1.0 Ionic App Lib Version: 2.1.0-beta.1 OS: Node Version: v6.7.0

Copied from original issue: driftyco/ionic#8959

[paulovitorjp]

I am wrapping everything inside platform.ready() as you can see below, and the get returns null everytime.

  constructor(platform: Platform, public deploy: Deploy,public modalCtrl: ModalController, public authSrvc: AuthService,
    public localStorage:Storage, public chatSrvc:ChatService, public prefSrvc:PrefService) {
    platform.ready().then(() => {
      // Okay, so the platform is ready and our plugins are available.
      // Here you can do any higher level native things you might need.
      StatusBar.styleDefault();
      console.log('[App Component] SQLite length: ');
      localStorage.length().then((n)=>{console.log('[App Component] SQLite length: ' + n);});
      localStorage.set('test',true).then(()=>{
        localStorage.get('test').then((test)=>{
          console.log('testing of sqlite was ' + test); //this is always null, even though I just set it to true.
          //...
        });
      });
    });
  }

I’ve seen that @mirkonasato has submitted a pull request for this here #21 . Please @jgw96, could you take a look?

I am using ionic 2.1.8, cordova 6.3.1, ionic storage 1.1.6 and cordova-sqlite-storage 1.5.0.

Thanks!

[mlynch]

Fixed with #21