[jira] [Commented] (CB-11579) Windows: "cordova clean windows" command fails due to undefined variable

2016-07-29 Thread ASF subversion and git services (JIRA)

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

ASF subversion and git services commented on CB-11579:
--

Commit 38ec99c8049ea84e8c4c199ef61cd7b053bbe9aa in cordova-windows's branch 
refs/heads/master from [~Nikita Matrosov]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-windows.git;h=38ec99c ]

CB-11579 windows: fix bug with 'cordova clean windows'

This closes #189


> Windows: "cordova clean windows" command fails due to undefined variable
> 
>
> Key: CB-11579
> URL: https://issues.apache.org/jira/browse/CB-11579
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: Windows
>Affects Versions: 4.4.1
>Reporter: James White
>  Labels: windows
>
> Running the *cordova clean windows* command in a Windows Cordova platform 
> project fails with the following error:
> {color:red}
> Error: TypeError: Cannot read property 'root' of undefined
>   at cleanImages 
> (C:\project\myapp\platforms\windows\cordova\lib\prepare.js:396:71)
> {color}
> It looks like the *cleanImages* function in *prepare.js* isn't being passed 
> the *locations* parameter on line 484.  
> If I change line 484 from
> {code:Javascript}
> cleanImages(projectRoot, projectConfig);
> {code}
> to 
> {code:Javascript}
> cleanImages(projectRoot, projectConfig, self.locations);
> {code}
> it completes the icon and splash screen image removal as expected.



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

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



[jira] [Commented] (CB-11579) Windows: "cordova clean windows" command fails due to undefined variable

2016-07-29 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CB-11579:
-

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-windows/pull/189


> Windows: "cordova clean windows" command fails due to undefined variable
> 
>
> Key: CB-11579
> URL: https://issues.apache.org/jira/browse/CB-11579
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: Windows
>Affects Versions: 4.4.1
>Reporter: James White
>  Labels: windows
>
> Running the *cordova clean windows* command in a Windows Cordova platform 
> project fails with the following error:
> {color:red}
> Error: TypeError: Cannot read property 'root' of undefined
>   at cleanImages 
> (C:\project\myapp\platforms\windows\cordova\lib\prepare.js:396:71)
> {color}
> It looks like the *cleanImages* function in *prepare.js* isn't being passed 
> the *locations* parameter on line 484.  
> If I change line 484 from
> {code:Javascript}
> cleanImages(projectRoot, projectConfig);
> {code}
> to 
> {code:Javascript}
> cleanImages(projectRoot, projectConfig, self.locations);
> {code}
> it completes the icon and splash screen image removal as expected.



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

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



[jira] [Commented] (CB-11579) Windows: "cordova clean windows" command fails due to undefined variable

2016-07-27 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CB-11579:
-

Github user codecov-io commented on the issue:

https://github.com/apache/cordova-windows/pull/189
  
## [Current coverage][cc-pull] is 73.30% (diff: 100%)
> Merging [#189][cc-pull] into [master][cc-base-branch] will increase 
coverage by **0.25%**

```diff
@@ master   #189   diff @@
==
  Files14 14  
  Lines  1937   1937  
  Methods 360360  
  Messages  0  0  
  Branches390390  
==
+ Hits   1415   1420 +5   
+ Misses  522517 -5   
  Partials  0  0  
```

> Powered by [Codecov](https://codecov.io?src=pr). Last update 
[7f379ff...6a60c60][cc-compare]
[cc-base-branch]: 
https://codecov.io/gh/apache/cordova-windows/branch/master?src=pr
[cc-compare]: 
https://codecov.io/gh/apache/cordova-windows/compare/7f379ff33494ed40ba2315178f31dc973d0f958d...6a60c6060b1a09f98260d48bc707dbecd0d2ad0d?src=pr
[cc-pull]: https://codecov.io/gh/apache/cordova-windows/pull/189?src=pr


> Windows: "cordova clean windows" command fails due to undefined variable
> 
>
> Key: CB-11579
> URL: https://issues.apache.org/jira/browse/CB-11579
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: Windows
>Affects Versions: 4.4.1
>Reporter: James White
>  Labels: windows
>
> Running the *cordova clean windows* command in a Windows Cordova platform 
> project fails with the following error:
> {color:red}
> Error: TypeError: Cannot read property 'root' of undefined
>   at cleanImages 
> (C:\project\myapp\platforms\windows\cordova\lib\prepare.js:396:71)
> {color}
> It looks like the *cleanImages* function in *prepare.js* isn't being passed 
> the *locations* parameter on line 484.  
> If I change line 484 from
> {code:Javascript}
> cleanImages(projectRoot, projectConfig);
> {code}
> to 
> {code:Javascript}
> cleanImages(projectRoot, projectConfig, self.locations);
> {code}
> it completes the icon and splash screen image removal as expected.



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

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



[jira] [Commented] (CB-11579) Windows: "cordova clean windows" command fails due to undefined variable

2016-07-27 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CB-11579:
-

GitHub user matrosovN opened a pull request:

https://github.com/apache/cordova-windows/pull/189

CB-11579 windows: fix bug with 'cordova clean windows' command



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/matrosovN/cordova-windows master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-windows/pull/189.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #189






> Windows: "cordova clean windows" command fails due to undefined variable
> 
>
> Key: CB-11579
> URL: https://issues.apache.org/jira/browse/CB-11579
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: Windows
>Affects Versions: 4.4.1
>Reporter: James White
>  Labels: windows
>
> Running the *cordova clean windows* command in a Windows Cordova platform 
> project fails with the following error:
> {color:red}
> Error: TypeError: Cannot read property 'root' of undefined
>   at cleanImages 
> (C:\project\myapp\platforms\windows\cordova\lib\prepare.js:396:71)
> {color}
> It looks like the *cleanImages* function in *prepare.js* isn't being passed 
> the *locations* parameter on line 484.  
> If I change line 484 from
> {code:Javascript}
> cleanImages(projectRoot, projectConfig);
> {code}
> to 
> {code:Javascript}
> cleanImages(projectRoot, projectConfig, self.locations);
> {code}
> it completes the icon and splash screen image removal as expected.



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

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