prepare fails on unknown splashscreen

[goya]

If targetImg is null it blows up and fails the prepare action. it should just return.

cordova-windows/template/cordova/lib/prepare.js

Line 513 in d424e99

if (!targetImg.targetProject) {

[brodybits]

I just raised PR #301 which should resolve this issue.

Long term solution (in a major release) may be to drop the lines after "Fall back on find by size for old non-MRT syntax" in getTargetForImage.

[brodybits]

P.S. This issue was introduced in 973fe6d.

[janpio]

@goya Do you still know how to recreate this? It would be best if we could add a test case on our side to avoid this popping up again.

[goya]

you can repro it now using:

<?xml version='1.0' encoding='utf-8'?>
<widget id="io.cordova.hellocordova" version="1.0.0">
    <name>HelloCordova</name>
    <splash src="splash.jpg" width="7000" height="1200" />
</widget>

n.b. the splash file must exist

[janpio]

Confirmed fixed:

E:\Projects\throwaway\cordovaWindows299  (helloworld@1.0.0)                                                                                       
λ cordova platform add windows                                                                                                                    
Using cordova-fetch for cordova-windows@^7.0.0                                                                                                    
Adding windows project...                                                                                                                         
Creating Cordova Windows Project:                                                                                                                 
        Path: platforms\windows                                                                                                                   
        Namespace: io.cordova.hellocordova                                                                                                        
        Name: HelloCordova                                                                                                                        
cp: no such file or directory: E:\Projects\throwaway\cordovaWindows299\node_modules\cordova-windows\node_modules\winjs\js\base.js                 
                                                                                                                                                  
cp: no such file or directory: E:\Projects\throwaway\cordovaWindows299\node_modules\cordova-windows\node_modules\winjs\js\base.js                 
                                                                                                                                                  
Windows project created with cordova-windows@7.0.0                                                                                                
Plugin 'cordova-plugin-whitelist' found in config.xml... Migrating it to package.json                                                             
Discovered saved plugin "cordova-plugin-whitelist". Adding it to the project                                                                      
Installing "cordova-plugin-whitelist" for windows                                                                                                 
Adding cordova-plugin-whitelist to package.json                                                                                                   
                                                                                                                                                  
E:\Projects\throwaway\cordovaWindows299  (helloworld@1.0.0)                                                                                       
λ cordova prepare windows                                                                                                                         
The following image was skipped because it has an unsupported size (7000x1200): splash.png                                                        
Cannot read property 'targetProject' of null                                                                                                      
                                                                                                                                                  
E:\Projects\throwaway\cordovaWindows299  (helloworld@1.0.0)                                                                                       
λ cordova platform rm windows                                                                                                                     
Removing windows from cordova.platforms array in package.json                                                                                     
                                                                                                                                                  
E:\Projects\throwaway\cordovaWindows299  (helloworld@1.0.0)                                                                                       
λ cordova platform add github:apache/cordova-windows                                                                                              
Using cordova-fetch for github:apache/cordova-windows                                                                                             
Warning: using prerelease platform windows@7.1.0-dev.                                                                                             
Use 'cordova platform add windows@latest' to add the latest published version instead.                                                            
Adding windows project...                                                                                                                         
Creating Cordova Windows Project:                                                                                                                 
        Path: platforms\windows                                                                                                                   
        Namespace: io.cordova.hellocordova                                                                                                        
        Name: HelloCordova                                                                                                                        
Windows project created with cordova-windows@7.1.0-dev                                                                                            
The following image was skipped because it has an unsupported size (7000x1200): splash.png                                                        
Installing "cordova-plugin-whitelist" for windows                                                                                                 
The following image was skipped because it has an unsupported size (7000x1200): splash.png                                                        
                                                                                                                                                  
E:\Projects\throwaway\cordovaWindows299  (helloworld@1.0.0)                                                                                       
λ cordova prepare windows                                                                                                                         
The following image was skipped because it has an unsupported size (7000x1200): splash.png                                                        
                                                                                                                                                  
E:\Projects\throwaway\cordovaWindows299  (helloworld@1.0.0)                                                                                       
λ                                                                                                                                                 

But tbh no idea how to turn this into a test.