CHCP reload index.html page on screen wake

[sckoh]

CHCP reload the index.html page everytime i turn off screen and turn it back.

the log show:

D/CHCP﹕ Loading external page: /data/user/0/com.example.package/files/cordova-hot-code-push-plugin/2016-01-18T09:33:25.621Z/www/index.html

[nikDemyankov]

My guess that this is on Android?

[sckoh]

yes, is this a known issue?

[nikDemyankov]

Thanks for noticing, will fix that.

[sckoh]

The reason it reload everytime on wake is on

https://github.com/nordnet/cordova-hot-code-push/blob/master/src/android/src/com/nordnetab/chcp/main/HotCodePushPlugin.java

line 115

it redirect to index.html in the onStart function which will be called everytime on screen wake

[nikDemyankov]

Yes, that is correct. Found it when posted the first answer. But thanks for the help :)

Will add check, that if we already loaded correct index page - don't do it again.

[johnmiroki]

@nikDemyankov I noticed that the app is reloaded on every resume (on Android, haven't tested on iOS), even if I set {"update": "start"} in chcp.json. I'm wondering if this is affected by the same code you've mentioned.

[nikDemyankov]

@johnmiroki yes, it doesn't depend on the chcp.json configuration. Page just get's reloaded, when application is awaken. Should be fine on iOS.

[johnmiroki]

@nikDemyankov I see. Thanks! Good to know it's gonna be fixed on next version bump. chcp.json.update only controls when the downloaded update actually replaces the old file. When the plugin checks for an update or reload the page is another story. right?

[nikDemyankov]

@johnmiroki yes, chcp.json.update controls, when plugin installs the update. The reload problem, described in this issue, is just a bug, which is not related to that preference.

[sckoh]

@johnmiroki I added a check in my fork to only reload page during app start or app update
https://github.com/sckoh/cordova-hot-code-push

[johnmiroki]

@sckoh Thanks a lot. Just tried on one android device quickly, the plugin doesn't reload the app on every resume any more at the very least. consider a pull-request?

[nikDemyankov]

@sckoh good one :) Thanks for sharing.

But I should say, that it will not work properly for the update, that happens on the very first start. When application launched for the first time (or after being updated from the store) - it will exit on line 112. So the reload flag is false. Then update is installed - page gets reloaded from the external page. When user goes to home screen and resume the app - this time it will hit line 119 , but the page will be reloaded, although it was already loaded from external storage. But on every next start it should be fine.