You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/10/19 15:56:58 UTC

[jira] [Commented] (CB-12036) Exception on prepare when no SplashScreen node not found in manifest

    [ https://issues.apache.org/jira/browse/CB-12036?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15589114#comment-15589114 ] 

ASF GitHub Bot commented on CB-12036:
-------------------------------------

Github user giuseppelt commented on the issue:

    https://github.com/apache/cordova-windows/pull/196
  
    Jira issue https://issues.apache.org/jira/browse/CB-12036 set as Minor.
    
    But, please, don't delay because the missing feature. The issue is beyond that, the code is wrong because the assumption is false.
    



> Exception on prepare when no SplashScreen node not found in manifest
> --------------------------------------------------------------------
>
>                 Key: CB-12036
>                 URL: https://issues.apache.org/jira/browse/CB-12036
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Windows
>    Affects Versions: 4.4.3
>            Reporter: Giuseppe La Torre
>            Priority: Minor
>
> 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, I use 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.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org