Details
-
Bug
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
4.4.3
-
None
-
None
Description
This is a mirror of the pull: https://github.com/apache/cordova-windows/pull/196
The issue is:
running prepare, the function setSplashBgColor on AppManifest.js throws exception when then splashscreen node is not found in the manifest.
The scenario:
Windows allows to have an app with NO splashscreen at all. You can either remove the node manually or, like me, use a hook to remove it because cordova actually doesn't support such a feature.
So, the code in that function crashes because it assumes a splashscreen exists, but the node is optional in the manifest.
The solution is easy: check if the node exists before doing stuff. No more than that. The pull request implements that simply if.
Please, don't delay because the missing feature. The issue is beyond that, the code is wrong because the assumption is false.
Attachments
Issue Links
- links to