[jira] [Created] (CB-9446) Samsung device returns null from Cursor in Camera getRealPath

2015-08-04 Thread Charles Verge (JIRA)
Charles Verge created CB-9446:
-

 Summary: Samsung device returns null from Cursor in Camera 
getRealPath
 Key: CB-9446
 URL: https://issues.apache.org/jira/browse/CB-9446
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Camera, Plugin Media Capture
Affects Versions: 1.2.1
 Environment: Samsung 4.1.2 GT-N8010
Reporter: Charles Verge
Priority: Critical
 Fix For: 1.2.1


Using a Samsung Galaxy Note GT-N8010 with android 4.1.2 produces an exception 
in the Exif class. This is due to a filePath being null on line 66 of 
src/android/ExifHelper.java
{code} new ExifInterface(filePath){code}

This error has been introduced when the switch was made to using Cursors. This 
error did not happen in with the camera plugin bundled with Phone Gap 3.4.

This has been duplicated by other developers independently 
http://stackoverflow.com/questions/30616846/phonegap-app-crash-when-take-a-new-photo-with-camera-plugin



--
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-9446) Samsung device returns null from Cursor in Camera getRealPath

2015-08-04 Thread Charles Verge (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653545#comment-14653545
 ] 

Charles Verge commented on CB-9446:
---

Commit
https://github.com/charlesverge/cordova-plugin-camera/commit/d3987661aece89083f138c14c929e294949f2383
pull request
https://github.com/apache/cordova-plugin-camera/pull/112

 Samsung device returns null from Cursor in Camera getRealPath
 -

 Key: CB-9446
 URL: https://issues.apache.org/jira/browse/CB-9446
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Camera, Plugin Media Capture
Affects Versions: 1.2.1
 Environment: Samsung 4.1.2 GT-N8010
Reporter: Charles Verge
Priority: Critical
 Fix For: 1.2.1

 Attachments: realpath.diff


 Using a Samsung Galaxy Note GT-N8010 with android 4.1.2 produces an exception 
 in the Exif class. This is due to a filePath being null on line 66 of 
 src/android/ExifHelper.java
 {code} new ExifInterface(filePath){code}
 This error has been introduced when the switch was made to using Cursors. 
 This error did not happen in with the camera plugin bundled with Phone Gap 
 3.4.
 This has been duplicated by other developers independently 
 http://stackoverflow.com/questions/30616846/phonegap-app-crash-when-take-a-new-photo-with-camera-plugin



--
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] [Comment Edited] (CB-9446) Samsung device returns null from Cursor in Camera getRealPath

2015-08-04 Thread Charles Verge (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653532#comment-14653532
 ] 

Charles Verge edited comment on CB-9446 at 8/4/15 12:16 PM:


Attached patch to fix issue, check for null pointer before creating 
ExifInterface interface which prevents complete crash of app on Samsung devices 
and addition of getRealPathFallback function to use when Cursor method does not 
work. getRealPathFallback is from previous plugin versions.


was (Author: charlesverge):
Patch to fix issue, check for null pointer before creating ExifInterface 
interface which prevents complete crash of app on Samsung devices and addition 
of getRealPathFallback function to use when Cursor method does not work. 
getRealPathFallback is from previous plugin versions.

 Samsung device returns null from Cursor in Camera getRealPath
 -

 Key: CB-9446
 URL: https://issues.apache.org/jira/browse/CB-9446
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Camera, Plugin Media Capture
Affects Versions: 1.2.1
 Environment: Samsung 4.1.2 GT-N8010
Reporter: Charles Verge
Priority: Critical
 Fix For: 1.2.1

 Attachments: realpath.diff


 Using a Samsung Galaxy Note GT-N8010 with android 4.1.2 produces an exception 
 in the Exif class. This is due to a filePath being null on line 66 of 
 src/android/ExifHelper.java
 {code} new ExifInterface(filePath){code}
 This error has been introduced when the switch was made to using Cursors. 
 This error did not happen in with the camera plugin bundled with Phone Gap 
 3.4.
 This has been duplicated by other developers independently 
 http://stackoverflow.com/questions/30616846/phonegap-app-crash-when-take-a-new-photo-with-camera-plugin



--
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-9446) Samsung device returns null from Cursor in Camera getRealPath

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653543#comment-14653543
 ] 

ASF GitHub Bot commented on CB-9446:


GitHub user charlesverge opened a pull request:

https://github.com/apache/cordova-plugin-camera/pull/112

CB-9446 Fix real path returning null pointer on some Samsung devices

Fix for https://issues.apache.org/jira/browse/CB-9446

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

$ git pull https://github.com/charlesverge/cordova-plugin-camera 
CB-9446cordova-plugin-camera

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

https://github.com/apache/cordova-plugin-camera/pull/112.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 #112


commit d3987661aece89083f138c14c929e294949f2383
Author: Charles Verge charles.ve...@gmail.com
Date:   2015-08-04T12:23:55Z

CB-9446 Fix real path returning null pointer on some Samsung devices




 Samsung device returns null from Cursor in Camera getRealPath
 -

 Key: CB-9446
 URL: https://issues.apache.org/jira/browse/CB-9446
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Camera, Plugin Media Capture
Affects Versions: 1.2.1
 Environment: Samsung 4.1.2 GT-N8010
Reporter: Charles Verge
Priority: Critical
 Fix For: 1.2.1

 Attachments: realpath.diff


 Using a Samsung Galaxy Note GT-N8010 with android 4.1.2 produces an exception 
 in the Exif class. This is due to a filePath being null on line 66 of 
 src/android/ExifHelper.java
 {code} new ExifInterface(filePath){code}
 This error has been introduced when the switch was made to using Cursors. 
 This error did not happen in with the camera plugin bundled with Phone Gap 
 3.4.
 This has been duplicated by other developers independently 
 http://stackoverflow.com/questions/30616846/phonegap-app-crash-when-take-a-new-photo-with-camera-plugin



--
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] [Issue Comment Deleted] (CB-9446) Samsung device returns null from Cursor in Camera getRealPath

2015-08-04 Thread Charles Verge (JIRA)

 [ 
https://issues.apache.org/jira/browse/CB-9446?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Charles Verge updated CB-9446:
--
Comment: was deleted

(was: Commit
https://github.com/charlesverge/cordova-plugin-camera/commit/d3987661aece89083f138c14c929e294949f2383
pull request
https://github.com/apache/cordova-plugin-camera/pull/112)

 Samsung device returns null from Cursor in Camera getRealPath
 -

 Key: CB-9446
 URL: https://issues.apache.org/jira/browse/CB-9446
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Camera, Plugin Media Capture
Affects Versions: 1.2.1
 Environment: Samsung 4.1.2 GT-N8010
Reporter: Charles Verge
Priority: Critical
 Fix For: 1.2.1

 Attachments: realpath.diff


 Using a Samsung Galaxy Note GT-N8010 with android 4.1.2 produces an exception 
 in the Exif class. This is due to a filePath being null on line 66 of 
 src/android/ExifHelper.java
 {code} new ExifInterface(filePath){code}
 This error has been introduced when the switch was made to using Cursors. 
 This error did not happen in with the camera plugin bundled with Phone Gap 
 3.4.
 This has been duplicated by other developers independently 
 http://stackoverflow.com/questions/30616846/phonegap-app-crash-when-take-a-new-photo-with-camera-plugin



--
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-9237) Add cdvfile:// support to media plugin on windows platform

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653689#comment-14653689
 ] 

ASF GitHub Bot commented on CB-9237:


Github user sgrebnov commented on the pull request:


https://github.com/apache/cordova-plugin-media/pull/57#issuecomment-127625076
  
reviewed, lgtm :+1: 


 Add cdvfile:// support to media plugin on windows platform
 --

 Key: CB-9237
 URL: https://issues.apache.org/jira/browse/CB-9237
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Media
 Environment: windows
Reporter: Sergey Shakhnazarov

 cdvfile:// protocol is not currently supported on Windows:
 Audio Error: {code:1} is returned



--
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-9238) Media plugin cannot record audio on windows

2015-08-04 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653695#comment-14653695
 ] 

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

Commit 8cf9a96d9bbceb1141b7638347a2ee6bc685a4b6 in cordova-plugin-media's 
branch refs/heads/master from daserge
[ https://git-wip-us.apache.org/repos/asf?p=cordova-plugin-media.git;h=8cf9a96 ]

CB-9238 Media plugin cannot record audio on windows

Reworked Windows proxy to support ms-appdata file paths (temporaryFolder and 
localFolder)
Fixed release method to avoid Access is denied exception
Added MEDIA_RUNNING and MEDIA_ERROR states firing to record-related methods
Reworked Windows proxy to not to create Audio object in case of record mode
Adds releaseAudio to record manual test to avoid Access denied errors

github close #57
github close #58


 Media plugin cannot record audio on windows
 ---

 Key: CB-9238
 URL: https://issues.apache.org/jira/browse/CB-9238
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Media, Windows
Reporter: Alexander Sorokin

 Steps to reproduce:
 - create mobile spec app with media plugin
 - run mobile spec app on windows store or windows phone
 - go to media manual tests
 - click Record Audio 10 sec
 The result is: RecordAudio():Audio Error: 1



--
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-9237) Add cdvfile:// support to media plugin on windows platform

2015-08-04 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653696#comment-14653696
 ] 

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

Commit b992df38da74d7abc67bde3615edce034003c98f in cordova-plugin-media's 
branch refs/heads/master from daserge
[ https://git-wip-us.apache.org/repos/asf?p=cordova-plugin-media.git;h=b992df3 ]

CB-9237 Add cdvfile:// support to media plugin on windows platform

Mentioned cdvfile support in the documentation


 Add cdvfile:// support to media plugin on windows platform
 --

 Key: CB-9237
 URL: https://issues.apache.org/jira/browse/CB-9237
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Media
 Environment: windows
Reporter: Sergey Shakhnazarov

 cdvfile:// protocol is not currently supported on Windows:
 Audio Error: {code:1} is returned



--
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-9238) Media plugin cannot record audio on windows

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653698#comment-14653698
 ] 

ASF GitHub Bot commented on CB-9238:


Github user asfgit closed the pull request at:

https://github.com/apache/cordova-plugin-media/pull/58


 Media plugin cannot record audio on windows
 ---

 Key: CB-9238
 URL: https://issues.apache.org/jira/browse/CB-9238
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Media, Windows
Reporter: Alexander Sorokin

 Steps to reproduce:
 - create mobile spec app with media plugin
 - run mobile spec app on windows store or windows phone
 - go to media manual tests
 - click Record Audio 10 sec
 The result is: RecordAudio():Audio Error: 1



--
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-9237) Add cdvfile:// support to media plugin on windows platform

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653697#comment-14653697
 ] 

ASF GitHub Bot commented on CB-9237:


Github user asfgit closed the pull request at:

https://github.com/apache/cordova-plugin-media/pull/57


 Add cdvfile:// support to media plugin on windows platform
 --

 Key: CB-9237
 URL: https://issues.apache.org/jira/browse/CB-9237
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Media
 Environment: windows
Reporter: Sergey Shakhnazarov

 cdvfile:// protocol is not currently supported on Windows:
 Audio Error: {code:1} is returned



--
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-9406) Cordova-iOS Platform Release July 28, 2015

2015-08-04 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653476#comment-14653476
 ] 

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

Commit 32dedd67ddc285dd853fcdc873b933adc020f595 in cordova-ios's branch 
refs/heads/master from sgrebnov
[ https://git-wip-us.apache.org/repos/asf?p=cordova-ios.git;h=32dedd6 ]

CB-9406 updated RELEASENOTES


 Cordova-iOS Platform Release July 28, 2015
 --

 Key: CB-9406
 URL: https://issues.apache.org/jira/browse/CB-9406
 Project: Apache Cordova
  Issue Type: Task
Reporter: Sergey Grebnov
Assignee: Sergey Grebnov

 Following steps at 
 https://github.com/apache/cordova-coho/blob/master/docs/platforms-release-process.md



--
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] [Resolved] (CB-9406) Cordova-iOS Platform Release July 28, 2015

2015-08-04 Thread Sergey Grebnov (JIRA)

 [ 
https://issues.apache.org/jira/browse/CB-9406?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sergey Grebnov resolved CB-9406.

Resolution: Fixed
  Assignee: (was: Sergey Grebnov)

Vote has passed and updated package has been published to npm as 
cordova-ios@3.9.0

 Cordova-iOS Platform Release July 28, 2015
 --

 Key: CB-9406
 URL: https://issues.apache.org/jira/browse/CB-9406
 Project: Apache Cordova
  Issue Type: Task
Reporter: Sergey Grebnov

 Following steps at 
 https://github.com/apache/cordova-coho/blob/master/docs/platforms-release-process.md



--
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-9442) Camera crashes multiple pictures taken wp8

2015-08-04 Thread Elodie Corbel (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9442?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653255#comment-14653255
 ] 

Elodie Corbel commented on CB-9442:
---

The OS version is 8.10.14203.306. I have tested on a nokia 530 and it doesn't 
happen.

 Camera crashes multiple pictures taken wp8
 --

 Key: CB-9442
 URL: https://issues.apache.org/jira/browse/CB-9442
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Camera
Affects Versions: 5.1.1
 Environment: Windows Phone 8.1
Reporter: Elodie Corbel
Assignee: Jesse MacFadyen
Priority: Blocker
  Labels: Windows

 The camera and the app crash when I try to take pictures multiple times. It 
 tries to open up the camera but the camera stays black and we only see the 
 button to take pictures that doesn't work. If I wait few seconds, it closes 
 the camera and the app.
 It doesn't happen at a fixed number of times. Sometimes, the second time, 
 sometimes later but never the first. I have nothing in the log that indicates 
 this error, no exception.
 I call getPicture with this :
 {code:javascript}
 {
   quality: 50,
   destinationType: Camera.DestinationType.DATA_URL,
   allowEdit: false,
   saveToPhotoAlbum: false,
   correctOrientation: false
 }
 {code}



--
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] [Comment Edited] (CB-9412) Unnatural translated expression in Japanese cordova-cli document

2015-08-04 Thread Victor Adrian Sosa Herrera (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9412?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14647897#comment-14647897
 ] 

Victor Adrian Sosa Herrera edited comment on CB-9412 at 8/4/15 1:11 PM:


hello [~jollyjoester]
For the English strings, you can create a Pull Request (PR) in GitHub


was (Author: sosah.victor):
helo [~jollyjoester]
For the English strings, you can create a Pull Request (PR) in GitHub

 Unnatural translated expression in Japanese cordova-cli document
 

 Key: CB-9412
 URL: https://issues.apache.org/jira/browse/CB-9412
 Project: Apache Cordova
  Issue Type: Task
  Components: CLI
Reporter: Hideyuki Nanashima
Assignee: Victor Adrian Sosa Herrera
Priority: Trivial
   Original Estimate: 24h
  Remaining Estimate: 24h

 There're some unnatural translated expression in following Japanese 
 cordova-cli document.
 https://github.com/apache/cordova-cli/blob/master/doc/ja/
 - README.md
 - bash.md
 Can I fix and send pull request?



--
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-9412) Unnatural translated expression in Japanese cordova-cli document

2015-08-04 Thread Victor Adrian Sosa Herrera (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9412?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653598#comment-14653598
 ] 

Victor Adrian Sosa Herrera commented on CB-9412:


hello [~jollyjoester]
I cannot find myself that document, can you tell me which file is it?

 Unnatural translated expression in Japanese cordova-cli document
 

 Key: CB-9412
 URL: https://issues.apache.org/jira/browse/CB-9412
 Project: Apache Cordova
  Issue Type: Task
  Components: CLI
Reporter: Hideyuki Nanashima
Assignee: Victor Adrian Sosa Herrera
Priority: Trivial
   Original Estimate: 24h
  Remaining Estimate: 24h

 There're some unnatural translated expression in following Japanese 
 cordova-cli document.
 https://github.com/apache/cordova-cli/blob/master/doc/ja/
 - README.md
 - bash.md
 Can I fix and send pull request?



--
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] [Comment Edited] (CB-9412) Unnatural translated expression in Japanese cordova-cli document

2015-08-04 Thread Victor Adrian Sosa Herrera (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9412?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653598#comment-14653598
 ] 

Victor Adrian Sosa Herrera edited comment on CB-9412 at 8/4/15 1:13 PM:


hello [~jollyjoester]
I cannot find myself that document, can you tell me which file is it?

BTW,  I am not sure about this, but I have the feeling that any time there is a 
modification in a document, it gets a new identifier, that's probably why you 
get that 404 (need to confirm this with CrowdIn team though). My advise is that 
you don't store direct link, rather learn the file you are in the middle of 
translation and use the UI to locate it


was (Author: sosah.victor):
hello [~jollyjoester]
I cannot find myself that document, can you tell me which file is it?

 Unnatural translated expression in Japanese cordova-cli document
 

 Key: CB-9412
 URL: https://issues.apache.org/jira/browse/CB-9412
 Project: Apache Cordova
  Issue Type: Task
  Components: CLI
Reporter: Hideyuki Nanashima
Assignee: Victor Adrian Sosa Herrera
Priority: Trivial
   Original Estimate: 24h
  Remaining Estimate: 24h

 There're some unnatural translated expression in following Japanese 
 cordova-cli document.
 https://github.com/apache/cordova-cli/blob/master/doc/ja/
 - README.md
 - bash.md
 Can I fix and send pull request?



--
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] [Created] (CB-9447) Cordova-Windows Platform Release August 5, 2015

2015-08-04 Thread Rob Paveza (JIRA)
Rob Paveza created CB-9447:
--

 Summary: Cordova-Windows Platform Release August 5, 2015
 Key: CB-9447
 URL: https://issues.apache.org/jira/browse/CB-9447
 Project: Apache Cordova
  Issue Type: Task
  Components: Windows
Affects Versions: 4.1.0
Reporter: Rob Paveza
Assignee: Rob Paveza


Following steps at 
https://github.com/apache/cordova-coho/blob/master/docs/platforms-release-process.md



--
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] [Created] (CB-9452) RTSP Streams are considered local resources on playback

2015-08-04 Thread John C. Bland II (JIRA)
John C. Bland II created CB-9452:


 Summary: RTSP Streams are considered local resources on playback
 Key: CB-9452
 URL: https://issues.apache.org/jira/browse/CB-9452
 Project: Apache Cordova
  Issue Type: Bug
  Components: Android
Reporter: John C. Bland II
Priority: Blocker


Cordova fails to play when attempting to play an RTSP stream. The stream is 
treated as a local resource so in AudioPlayer.java:305.

Fix: 
{code}
public boolean isStreaming(String file) {
if (file.contains(http://;) || file.contains(https://;) || 
file.contains(rtsp://)) {
return true;
}
else {
return false;
}
}
{code}



--
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] [Updated] (CB-9452) RTSP Streams are considered local resources on playback

2015-08-04 Thread John C. Bland II (JIRA)

 [ 
https://issues.apache.org/jira/browse/CB-9452?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

John C. Bland II updated CB-9452:
-
Description: 
Cordova Media plugin fails to play when attempting to play an RTSP stream. The 
stream is treated as a local resource so in AudioPlayer.java:305.

Fix: 
{code}
public boolean isStreaming(String file) {
if (file.contains(http://;) || file.contains(https://;) || 
file.contains(rtsp://)) {
return true;
}
else {
return false;
}
}
{code}

  was:
Cordova fails to play when attempting to play an RTSP stream. The stream is 
treated as a local resource so in AudioPlayer.java:305.

Fix: 
{code}
public boolean isStreaming(String file) {
if (file.contains(http://;) || file.contains(https://;) || 
file.contains(rtsp://)) {
return true;
}
else {
return false;
}
}
{code}


 RTSP Streams are considered local resources on playback
 ---

 Key: CB-9452
 URL: https://issues.apache.org/jira/browse/CB-9452
 Project: Apache Cordova
  Issue Type: Bug
  Components: Android
Reporter: John C. Bland II
Priority: Blocker

 Cordova Media plugin fails to play when attempting to play an RTSP stream. 
 The stream is treated as a local resource so in AudioPlayer.java:305.
 Fix: 
 {code}
 public boolean isStreaming(String file) {
 if (file.contains(http://;) || file.contains(https://;) || 
 file.contains(rtsp://)) {
 return true;
 }
 else {
 return false;
 }
 }
 {code}



--
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-9452) RTSP Streams are considered local resources on playback

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654816#comment-14654816
 ] 

ASF GitHub Bot commented on CB-9452:


GitHub user johncblandii opened a pull request:

https://github.com/apache/cordova-plugin-media/pull/63

CB-9452: Treat RTSP streams as remote URLs

Cordova Media plugin fails to play when attempting to play an RTSP stream. 
The stream is treated as a local resource so in AudioPlayer.java:305.

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

$ git pull https://github.com/johncblandii/cordova-plugin-media 
rtsp_streaming

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

https://github.com/apache/cordova-plugin-media/pull/63.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 #63


commit 0bfa72636e707655fe94783289aa0da533c18216
Author: John C. Bland II johncblan...@gmail.com
Date:   2015-08-05T04:55:27Z

CB-9452: Treat RTSP streams as remote URLs




 RTSP Streams are considered local resources on playback
 ---

 Key: CB-9452
 URL: https://issues.apache.org/jira/browse/CB-9452
 Project: Apache Cordova
  Issue Type: Bug
  Components: Android
Reporter: John C. Bland II
Priority: Blocker

 Cordova Media plugin fails to play when attempting to play an RTSP stream. 
 The stream is treated as a local resource so in AudioPlayer.java:305.
 Fix: 
 {code}
 public boolean isStreaming(String file) {
 if (file.contains(http://;) || file.contains(https://;) || 
 file.contains(rtsp://)) {
 return true;
 }
 else {
 return false;
 }
 }
 {code}



--
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-9452) RTSP Streams are considered local resources on playback

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654871#comment-14654871
 ] 

ASF GitHub Bot commented on CB-9452:


Github user johncblandii commented on the pull request:


https://github.com/apache/cordova-plugin-media/pull/63#issuecomment-127881080
  
I will happily write tests if I can see how to run them. :) I didn't see 
the normal culprits for a test runner or anything in the README.


 RTSP Streams are considered local resources on playback
 ---

 Key: CB-9452
 URL: https://issues.apache.org/jira/browse/CB-9452
 Project: Apache Cordova
  Issue Type: Bug
  Components: Android
Reporter: John C. Bland II
Priority: Blocker

 Cordova Media plugin fails to play when attempting to play an RTSP stream. 
 The stream is treated as a local resource so in AudioPlayer.java:305.
 Fix: 
 {code}
 public boolean isStreaming(String file) {
 if (file.contains(http://;) || file.contains(https://;) || 
 file.contains(rtsp://)) {
 return true;
 }
 else {
 return false;
 }
 }
 {code}



--
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-9436) Refactor browserify to get rid of `require-tr` transform

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654553#comment-14654553
 ] 

ASF GitHub Bot commented on CB-9436:


Github user stevengill commented on the pull request:

https://github.com/apache/cordova-lib/pull/275#issuecomment-127799658
  
Make sense. Thanks! Lets merge it in!


 Refactor browserify to get rid of `require-tr` transform
 

 Key: CB-9436
 URL: https://issues.apache.org/jira/browse/CB-9436
 Project: Apache Cordova
  Issue Type: Improvement
  Components: CordovaJS, CordovaLib
Reporter: Vladimir Kotikov
Assignee: Vladimir Kotikov





--
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-9436) Refactor browserify to get rid of `require-tr` transform

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654552#comment-14654552
 ] 

ASF GitHub Bot commented on CB-9436:


Github user stevengill commented on the pull request:

https://github.com/apache/cordova-js/pull/124#issuecomment-127799631
  
Merge away! Thanks again. Love the refactor. Way simpler to read and follow.


 Refactor browserify to get rid of `require-tr` transform
 

 Key: CB-9436
 URL: https://issues.apache.org/jira/browse/CB-9436
 Project: Apache Cordova
  Issue Type: Improvement
  Components: CordovaJS, CordovaLib
Reporter: Vladimir Kotikov
Assignee: Vladimir Kotikov





--
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-9436) Refactor browserify to get rid of `require-tr` transform

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654487#comment-14654487
 ] 

ASF GitHub Bot commented on CB-9436:


Github user vladimir-kotikov commented on the pull request:

https://github.com/apache/cordova-lib/pull/275#issuecomment-127782310
  
 Wondering why you had to add require('codrdova/init') to the bundle? 
MSOpenTech@5105161#diff-28a16640f3807e6ee2300ea7ce8c2f1dR196. How was that 
getting added previously?

Previously this was being added by 
`cordova-js/tasks/lib/bundle-browserify.js` as `cordova/bootstrap` module. The 
problem is when you adding plugin modules into bundle after bootstrap is added, 
they only get loaded _after_ cordova initialization and `deviceready` event. 
The only way i found to fix this, is to add `bootstrap` as the last file in the 
bundle.


 Refactor browserify to get rid of `require-tr` transform
 

 Key: CB-9436
 URL: https://issues.apache.org/jira/browse/CB-9436
 Project: Apache Cordova
  Issue Type: Improvement
  Components: CordovaJS, CordovaLib
Reporter: Vladimir Kotikov
Assignee: Vladimir Kotikov





--
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] [Created] (CB-9451) Fetching images for Google Photos by local URL fails when the image isn't downloaded

2015-08-04 Thread Julian Frumar (JIRA)
Julian Frumar created CB-9451:
-

 Summary: Fetching images for Google Photos by local URL fails when 
the image isn't downloaded
 Key: CB-9451
 URL: https://issues.apache.org/jira/browse/CB-9451
 Project: Apache Cordova
  Issue Type: Improvement
  Components: Android, Plugin Camera
Affects Versions: 3.8.0
Reporter: Julian Frumar


Attempting to fetch an image that the user has picked from Google Photos seems 
doomed for failure if we attempt to access it via a local file URL. It seems 
the only reliable way to retrieve an image (which could hosted remotely by 
Google) is to use:

MediaStore.Images.Media.getBitmap()

That way the system will be responsible for retrieving the Bitmap. The bitmap 
can then be written to the filesystem if necessary and the URL of that file can 
be returned for passing back across the Cordova JS bridge.

Thoughts?



--
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] [Updated] (CB-9451) Fetching images for Google Photos by local URL fails when the image isn't downloaded

2015-08-04 Thread Julian Frumar (JIRA)

 [ 
https://issues.apache.org/jira/browse/CB-9451?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Julian Frumar updated CB-9451:
--
Description: 
Attempting to fetch an image that the user has picked from Google Photos seems 
doomed for failure if we attempt to access it via a local file URL. It seems 
the only reliable way to retrieve an image (which could be hosted remotely by 
Google) is to use:

MediaStore.Images.Media.getBitmap()

That way the system will be responsible for retrieving the Bitmap. The bitmap 
can then be written to the filesystem if necessary and the URL of that file can 
be returned for passing back across the Cordova JS bridge.

Thoughts?

  was:
Attempting to fetch an image that the user has picked from Google Photos seems 
doomed for failure if we attempt to access it via a local file URL. It seems 
the only reliable way to retrieve an image (which could hosted remotely by 
Google) is to use:

MediaStore.Images.Media.getBitmap()

That way the system will be responsible for retrieving the Bitmap. The bitmap 
can then be written to the filesystem if necessary and the URL of that file can 
be returned for passing back across the Cordova JS bridge.

Thoughts?


 Fetching images for Google Photos by local URL fails when the image isn't 
 downloaded
 

 Key: CB-9451
 URL: https://issues.apache.org/jira/browse/CB-9451
 Project: Apache Cordova
  Issue Type: Improvement
  Components: Android, Plugin Camera
Affects Versions: 3.8.0
Reporter: Julian Frumar

 Attempting to fetch an image that the user has picked from Google Photos 
 seems doomed for failure if we attempt to access it via a local file URL. It 
 seems the only reliable way to retrieve an image (which could be hosted 
 remotely by Google) is to use:
 MediaStore.Images.Media.getBitmap()
 That way the system will be responsible for retrieving the Bitmap. The bitmap 
 can then be written to the filesystem if necessary and the URL of that file 
 can be returned for passing back across the Cordova JS bridge.
 Thoughts?



--
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-9449) Impossible to use blank strings as dialog titles

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654412#comment-14654412
 ] 

ASF GitHub Bot commented on CB-9449:


Github user purplecabbage commented on the pull request:


https://github.com/apache/cordova-plugin-dialogs/pull/55#issuecomment-127771911
  
good additions here! Thanks


 Impossible to use blank strings as dialog titles
 

 Key: CB-9449
 URL: https://issues.apache.org/jira/browse/CB-9449
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Dialogs
Affects Versions: 3.5.0
Reporter: Ian Paterson
Priority: Minor
  Labels: dialog, patch

 Currently there is no way to create a dialog in the Cordova Dialogs plugin 
 without a title. Providing an empty string as the title for any of {{alert}}, 
 {{confirm}}, or {{prompt}} results in a non-localized default title. This 
 makes it impossible to show message-only dialogs on platforms that support 
 such a thing, like iOS. Currently, all dialogs must have a title and 
 additionally prompts must have a message, whether they are provided by the 
 developer or forced by the plugin.
 h4. No workarounds
 There are no known workarounds. One that came to mind was passing a space 
 character {{navigator.notification.alert('message', null, ' ');}} but on iOS 
 this results in a blank space for the title at the top of the alert. Sending 
 an object that will evaluate to true but return a blank string when 
 stringified causes the app to crash {{navigator.notification.alert('message', 
 null, {toString: function() { return ''; }});}}
 h4. Solution
 Rather than testing whether {{title}} is falsy to set the default, test 
 whether it is a string, regardless of whether it is a falsy empty string. 
 h4. Breaking change?
 Requiring the title to be a string prevents numeric and other values from 
 being used, while previously they were passed through to the native plugin. 
 This is a _good thing_ because sending anything other than a string causes 
 the plugin to crash the app (at least on iOS), so despite being more 
 restrictive about the data type, I do not consider it a breaking change.



--
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-9436) Refactor browserify to get rid of `require-tr` transform

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654499#comment-14654499
 ] 

ASF GitHub Bot commented on CB-9436:


Github user vladimir-kotikov commented on a diff in the pull request:

https://github.com/apache/cordova-js/pull/124#discussion_r36251972
  
--- Diff: tasks/lib/packager-browserify.js ---
@@ -46,25 +37,13 @@ module.exports = function generate(platform, 
useWindowsLineEndings, platformVers
 outReleaseFile = path.join('pkg', 'cordova.' + platform + '.js');
 outReleaseFileStream = fs.createWriteStream(outReleaseFile);
 
-var symbolList = null;
-var addSymbolList = through.obj(function(row, enc, next) {
-if(symbolList === null) {
-symbolList = requireTr.getModules(platform);
-this.push(util.format('var symbolList = %s;\n%s\n', 
JSON.stringify(symbolList, null, 4), row));
-} else {
-this.push(row);
-}
-next();
-});
-
-libraryRelease.pipeline.get('wrap').push(addSymbolList);
-
 // write license header
 writeLicenseHeader(outReleaseFileStream, platform, commitId, 
platformVersion);
 
-releaseBundle = libraryRelease.bundle();
-
-releaseBundle.pipe(outReleaseFileStream);
+bundle(platform, false, commitId, platformVersion, platformPath)
+  .add(path.resolve(__dirname, '..', '..', 
'src/scripts/bootstrap.js'))
--- End diff --

Yes, they both needed. This was moved from `bundle-browserify.js` direcly 
to `cordova-lib/prepare-browserify` due to some reasons (see 
https://github.com/apache/cordova-lib/pull/275#issuecomment-127782310) but we 
still need to add this file when building clean cordova-js bundle (without 
plugins) from grunt task. 


 Refactor browserify to get rid of `require-tr` transform
 

 Key: CB-9436
 URL: https://issues.apache.org/jira/browse/CB-9436
 Project: Apache Cordova
  Issue Type: Improvement
  Components: CordovaJS, CordovaLib
Reporter: Vladimir Kotikov
Assignee: Vladimir Kotikov





--
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-8936) Surface platform-specific logs in buildbot

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-8936?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653922#comment-14653922
 ] 

ASF GitHub Bot commented on CB-8936:


GitHub user alsorokin opened a pull request:

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

CB-8936 Added logging functionality

https://issues.apache.org/jira/browse/CB-8936

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

$ git pull https://github.com/MSOpenTech/cordova-windows CB-8936

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

https://github.com/apache/cordova-windows/pull/107.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 #107


commit d833994e12de6cdd74b497ef24ed4c6e38811598
Author: alsorokin alexander.soro...@akvelon.com
Date:   2015-08-04T16:26:11Z

CB-8936 Added logging functionality




 Surface platform-specific logs in buildbot
 --

 Key: CB-8936
 URL: https://issues.apache.org/jira/browse/CB-8936
 Project: Apache Cordova
  Issue Type: Task
  Components: Medic
Reporter: Alexander Sorokin
Assignee: Alexander Sorokin

 Platform specific logs (e.g. logcat for android, stderr.log and stdin.log for 
 iOS etc.) should be gathered and displayed in buildbot.



--
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-8936) Surface platform-specific logs in buildbot

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-8936?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653944#comment-14653944
 ] 

ASF GitHub Bot commented on CB-8936:


Github user nikhilkh commented on the pull request:

https://github.com/apache/cordova-windows/pull/107#issuecomment-127668967
  
@alsorokin Looks like there are jshint failures


 Surface platform-specific logs in buildbot
 --

 Key: CB-8936
 URL: https://issues.apache.org/jira/browse/CB-8936
 Project: Apache Cordova
  Issue Type: Task
  Components: Medic
Reporter: Alexander Sorokin
Assignee: Alexander Sorokin

 Platform specific logs (e.g. logcat for android, stderr.log and stdin.log for 
 iOS etc.) should be gathered and displayed in buildbot.



--
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] [Created] (CB-9448) 'aaptOptions' does not work in Cordova Gradle building

2015-08-04 Thread Christopher Mindus (JIRA)
Christopher Mindus created CB-9448:
--

 Summary: 'aaptOptions' does not work in Cordova Gradle building
 Key: CB-9448
 URL: https://issues.apache.org/jira/browse/CB-9448
 Project: Apache Cordova
  Issue Type: Bug
  Components: Android
Affects Versions: 4.1.0
 Environment: Windows
Reporter: Christopher Mindus


The aaptOptions configuration for Gradle building does not work. It can be made 
to work only with Ant (using another mechanism). The build-extras.gradle file 
does not help (in the platforms/android) directory. The build.gradle file 
cannot be used. Environment variables do not work either. In projects using 
e.g. the Dojo Framework that contains directory names that start with an 
underscore _ are all removed with any Gradle build, but can be made to work 
in Ant build.

There are some reports that can be found on google about this problem, and 
nobody has apparently reported it.



--
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-9278) Restoring multiple platforms fails

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654048#comment-14654048
 ] 

ASF GitHub Bot commented on CB-9278:


Github user asfgit closed the pull request at:

https://github.com/apache/cordova-lib/pull/276


 Restoring multiple platforms fails
 --

 Key: CB-9278
 URL: https://issues.apache.org/jira/browse/CB-9278
 Project: Apache Cordova
  Issue Type: Bug
  Components: CordovaLib
Affects Versions: Master
 Environment: OS X
Reporter: Dmitry Blotsky
Assignee: Omar Mefire
  Labels: cordova-lib, platform, restore, save
   Original Estimate: 48h
  Remaining Estimate: 48h

 Steps to reproduce:
 1. Create a blank app: {{cordova create test com.app.test test}}
 2. Add and save a platform: {{cordova platform add --save ios}}
 3. Add and save another platform: {{cordova platform add --save android}}
 4. Remove the platforms: {{cordova platform remove android ios}}
 5. Try to restore platforms automatically: {{cordova prepare}}
 Expected: both platforms installed and prepared
 Actual:
 {noformat}
 Error: Trying to initialize npm when settings have not been restored from a 
 previous initialization.
 {noformat}
 Running {{cordova prepare}} again fixes the issue.



--
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-9278) Restoring multiple platforms fails

2015-08-04 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654047#comment-14654047
 ] 

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

Commit 2edf1df741b844f6cc4e6e65cd804c4cb07b0665 in cordova-lib's branch 
refs/heads/master from [~omefire]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-lib.git;h=2edf1df ]

CB-9278: Tests to prevent future regressions while restoring multiple 
platforms. This closes #276


 Restoring multiple platforms fails
 --

 Key: CB-9278
 URL: https://issues.apache.org/jira/browse/CB-9278
 Project: Apache Cordova
  Issue Type: Bug
  Components: CordovaLib
Affects Versions: Master
 Environment: OS X
Reporter: Dmitry Blotsky
Assignee: Omar Mefire
  Labels: cordova-lib, platform, restore, save
   Original Estimate: 48h
  Remaining Estimate: 48h

 Steps to reproduce:
 1. Create a blank app: {{cordova create test com.app.test test}}
 2. Add and save a platform: {{cordova platform add --save ios}}
 3. Add and save another platform: {{cordova platform add --save android}}
 4. Remove the platforms: {{cordova platform remove android ios}}
 5. Try to restore platforms automatically: {{cordova prepare}}
 Expected: both platforms installed and prepared
 Actual:
 {noformat}
 Error: Trying to initialize npm when settings have not been restored from a 
 previous initialization.
 {noformat}
 Running {{cordova prepare}} again fixes the issue.



--
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-8936) Surface platform-specific logs in buildbot

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-8936?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653941#comment-14653941
 ] 

ASF GitHub Bot commented on CB-8936:


Github user nikhilkh commented on the pull request:

https://github.com/apache/cordova-windows/pull/107#issuecomment-127668674
  
@robpaveza @purplecabbage Please review and see if it can be merged for the 
current release.


 Surface platform-specific logs in buildbot
 --

 Key: CB-8936
 URL: https://issues.apache.org/jira/browse/CB-8936
 Project: Apache Cordova
  Issue Type: Task
  Components: Medic
Reporter: Alexander Sorokin
Assignee: Alexander Sorokin

 Platform specific logs (e.g. logcat for android, stderr.log and stdin.log for 
 iOS etc.) should be gathered and displayed in buildbot.



--
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-8936) Surface platform-specific logs in buildbot

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-8936?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14653955#comment-14653955
 ] 

ASF GitHub Bot commented on CB-8936:


Github user alsorokin commented on the pull request:

https://github.com/apache/cordova-windows/pull/107#issuecomment-127672899
  
@nikhilkh thanks for instant reply! Fixed.


 Surface platform-specific logs in buildbot
 --

 Key: CB-8936
 URL: https://issues.apache.org/jira/browse/CB-8936
 Project: Apache Cordova
  Issue Type: Task
  Components: Medic
Reporter: Alexander Sorokin
Assignee: Alexander Sorokin

 Platform specific logs (e.g. logcat for android, stderr.log and stdin.log for 
 iOS etc.) should be gathered and displayed in buildbot.



--
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] [Resolved] (CB-9397) cordova requirements android on a new Mac machine has issues

2015-08-04 Thread Nikhil Khandelwal (JIRA)

 [ 
https://issues.apache.org/jira/browse/CB-9397?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Nikhil Khandelwal resolved CB-9397.
---
Resolution: Fixed

 cordova requirements android on a new Mac machine has issues
 

 Key: CB-9397
 URL: https://issues.apache.org/jira/browse/CB-9397
 Project: Apache Cordova
  Issue Type: Bug
  Components: Android
Affects Versions: 4.1.0
Reporter: Vladimir Kotikov
Assignee: Vladimir Kotikov

 cordova requirements android on a Mac with no Android SDK, below is what I 
 get. I see 3 problems:
 - For some reason all the errors are in square brackets - should we fix that?
 - Also, why do we have a 'Error' prefix, are there other options here other 
 than Error
 - There is a TyperError which looks like a bug.
 {noformat}
 Requirements check results for android:
 Java JDK: not installed 
 [Error: Failed to find 'JAVA_HOME' environment variable. Try setting setting 
 it manually.]
 Android SDK: not installed 
 [Error: Failed to find 'ANDROID_HOME' environment variable. Try setting 
 setting it manually.
 Failed to find 'android' command in your 'PATH'. Try update your 'PATH' to 
 include path to valid SDK directory.]
 Android target: not installed 
 [Error: Android SDK not found. Make sure that it is installed. If it is not 
 at the default location, set the ANDROID_HOME environment variable.]
 Gradle: not installed 
 [TypeError: Arguments to path.join must be strings]
 Some of requirements check failed
 {noformat}



--
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-8084) Push Notifications code included by default

2015-08-04 Thread Filipe (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-8084?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654017#comment-14654017
 ] 

Filipe commented on CB-8084:


I also got cordova iOS platform version 3.8.0 and the issue is still there.

 Push Notifications code included by default
 ---

 Key: CB-8084
 URL: https://issues.apache.org/jira/browse/CB-8084
 Project: Apache Cordova
  Issue Type: Bug
  Components: iOS
Affects Versions: 3.5.0
Reporter: Jacob Weber
Assignee: Shazron Abdullah
 Fix For: 3.8.0


 When creating a new Cordova app using the CLI, it adds code to AppDelegate.m 
 for Push Notifications. This causes Apple to issue a warning when you submit 
 the app, if your provisioning profile doesn't have Push Notifications enabled:
 {quote}
 Missing Push Notification Entitlement - Your app appears to include API used 
 to register with the Apple Push Notification service, but the app signature's 
 entitlements do not include the aps-environment entitlement. If your app 
 uses the Apple Push Notification service, make sure your App ID is enabled 
 for Push Notification in the Provisioning Portal, and resubmit after signing 
 your app with a Distribution provisioning profile that includes the 
 aps-environment entitlement. See Provisioning and Development in the 
 Local and Push Notification Programming Guide for more information. If your 
 app does not use the Apple Push Notification service, no action is required. 
 You may remove the API from future submissions to stop this warning. If you 
 use a third-party framework, you may need to contact the developer for 
 information on removing the API.
 {quote}



--
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-9443) Windows camera picks incorrect maxResolution

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9443?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654114#comment-14654114
 ] 

ASF GitHub Bot commented on CB-9443:


Github user purplecabbage commented on a diff in the pull request:


https://github.com/apache/cordova-plugin-camera/pull/111#discussion_r36225312
  
--- Diff: src/windows/CameraProxy.js ---
@@ -696,21 +696,22 @@ function 
takePictureFromCameraWindows(successCallback, errorCallback, args) {
 // decide which max pixels should be supported by targetWidth or 
targetHeight.
 var maxRes = null;
 var UIMaxRes = WMCapture.CameraCaptureUIMaxPhotoResolution;
+var totalPixels = targetWidth * targetHeight;
 switch (true) {
-case (targetWidth = 1280 || targetHeight = 960) :
-cameraCaptureUI.photoSettings.maxResolution = UIMaxRes.large3M;
+case (totalPixels = 320 * 240):
+cameraCaptureUI.photoSettings.maxResolution = 
UIMaxRes.verySmallQvga;
 break;
-case (targetWidth = 1024 || targetHeight = 768) :
-maxRes = UIMaxRes.mediumXga;
+case (totalPixels = 640 * 480):
+maxRes = UIMaxRes.smallVga;
 break;
-case(targetWidth = 800 || targetHeight = 600) :
+case (totalPixels = 1024 * 768):
 maxRes = UIMaxRes.mediumXga;
 break;
-case  (targetWidth = 640 || targetHeight = 480) :
-maxRes = UIMaxRes.smallVga;
+case (totalPixels = 3 * 1000 * 1000):
+maxRes = UIMaxRes.large3M;
 break;
-case(targetWidth = 320 || targetHeight = 240) :
-maxRes = UIMaxRes.verySmallQvga;
+case (totalPixels = 5 * 1000 * 1000):
+maxRes = UIMaxRes.veryLarge5M;
 break;
 default :
 maxRes = UIMaxRes.highestAvailable;
--- End diff --

Yes, the 'faster' argument is not particularly strong in this case
considering how often it will be called.



My team is hiring!
@purplecabbage
risingj.com

On Tue, Aug 4, 2015 at 8:08 AM, Rob Paveza notificati...@github.com wrote:

 In src/windows/CameraProxy.js
 
https://github.com/apache/cordova-plugin-camera/pull/111#discussion_r36199186
 :

   break;
  -case(targetWidth = 320 || targetHeight = 240) :
  -maxRes = UIMaxRes.verySmallQvga;
  +case (totalPixels = 5 * 1000 * 1000):
  +maxRes = UIMaxRes.veryLarge5M;
   break;
   default :
   maxRes = UIMaxRes.highestAvailable;

 switch(true) would only be faster in the event that your cases were
 constants and you were doing explicit equality checks (e.g., case 10:).
 They're not, though, and because you're using greater-than comparisons, 
you
 lose the ability of the runtime compiler to optimize using jump-tables. If
 we're really concerned about perf, I'd suggest adding profiling code to
 order the most-common to the least-common cases; but I think this code 
will
 run so infrequently, it's not a perf-critical path.

 —
 Reply to this email directly or view it on GitHub
 https://github.com/apache/cordova-plugin-camera/pull/111/files#r36199186
 .




 Windows camera picks incorrect maxResolution
 

 Key: CB-9443
 URL: https://issues.apache.org/jira/browse/CB-9443
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Camera
 Environment: Windows Desktop, Windows Phone 10
Reporter: Murat Sutunc
Assignee: Murat Sutunc
  Labels: Windows





--
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] [Created] (CB-9449) Impossible to use blank strings as dialog titles

2015-08-04 Thread Ian Paterson (JIRA)
Ian Paterson created CB-9449:


 Summary: Impossible to use blank strings as dialog titles
 Key: CB-9449
 URL: https://issues.apache.org/jira/browse/CB-9449
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Dialogs
Affects Versions: 3.5.0
Reporter: Ian Paterson
Priority: Minor


Currently there is no way to create a dialog in the Cordova Dialogs plugin 
without a title. Providing an empty string as the title for any of {{alert}}, 
{{confirm}}, or {{prompt}} results in a non-localized default title. This makes 
it impossible to show message-only dialogs on platforms that support such a 
thing, like iOS. Currently, all dialogs must have a title and additionally 
prompts must have a message, whether they are provided by the developer or 
forced by the plugin.

h4. No workarounds

There are no known workarounds. One that came to mind was passing a space 
character {{navigator.notification.alert('message', null, ' ');}} but on iOS 
this results in a blank space for the title at the top of the alert. Sending an 
object that will evaluate to true but return a blank string when stringified 
causes the app to crash {{navigator.notification.alert('message', null, 
{toString: function() { return ''; }});}}

h4. Solution

Rather than testing whether {{title}} is falsy to set the default, test whether 
it is a string, regardless of whether it is a falsy empty string. 

h4. Breaking change?

Requiring the title to be a string prevents numeric and other values from being 
used, while previously they were passed through to the native plugin. This is a 
_good thing_ because sending anything other than a string causes the plugin to 
crash the app (at least on iOS), so despite being more restrictive about the 
data type, I do not consider it a breaking change.



--
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-9449) Impossible to use blank strings as dialog titles

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-9449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654310#comment-14654310
 ] 

ASF GitHub Bot commented on CB-9449:


GitHub user idpaterson opened a pull request:

https://github.com/apache/cordova-plugin-dialogs/pull/55

CB-9449 - Allow a blank string for dialog titles and messages.

Currently there is no way to create a dialog in the Cordova Dialogs plugin 
without a title. Providing an empty string as the title for any of alert, 
confirm, or prompt results in a non-localized default title. This makes it 
impossible to show message-only dialogs on platforms that support such a thing, 
like iOS. Currently, all dialogs must have a title and additionally prompts 
must have a message, whether they are provided by the developer or forced by 
the plugin.

### No workarounds

There are no known workarounds. One that came to mind was passing a space 
character `navigator.notification.alert('message', null, ' ');` but on iOS this 
results in a blank space for the title at the top of the alert. Sending an 
object that will evaluate to true but return a blank string when stringified 
causes the app to crash `navigator.notification.alert('message', null, 
{toString: function() { return ''; }});`

### Solution

Rather than testing whether title is falsy to set the default, test whether 
it is a string, regardless of whether it is a falsy empty string.

### Breaking change?

Requiring the title to be a string prevents numeric and other values from 
being used, while previously they were passed through to the native plugin. 
This is a *good thing* because sending anything other than a string causes the 
plugin to crash the app (at least on iOS), so despite being more restrictive 
about the data type, I do not consider it a breaking change.

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

$ git pull https://github.com/Newstex/cordova-plugin-dialogs 
pull-requests/blank-titles

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

https://github.com/apache/cordova-plugin-dialogs/pull/55.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 #55


commit 03444fcfe204d0576313702b5fe5ac2c6732798d
Author: ipaterson ipater...@newstex.com
Date:   2015-08-04T19:33:40Z

Allow a blank string for dialog titles and messages.




 Impossible to use blank strings as dialog titles
 

 Key: CB-9449
 URL: https://issues.apache.org/jira/browse/CB-9449
 Project: Apache Cordova
  Issue Type: Bug
  Components: Plugin Dialogs
Affects Versions: 3.5.0
Reporter: Ian Paterson
Priority: Minor
  Labels: dialog, patch

 Currently there is no way to create a dialog in the Cordova Dialogs plugin 
 without a title. Providing an empty string as the title for any of {{alert}}, 
 {{confirm}}, or {{prompt}} results in a non-localized default title. This 
 makes it impossible to show message-only dialogs on platforms that support 
 such a thing, like iOS. Currently, all dialogs must have a title and 
 additionally prompts must have a message, whether they are provided by the 
 developer or forced by the plugin.
 h4. No workarounds
 There are no known workarounds. One that came to mind was passing a space 
 character {{navigator.notification.alert('message', null, ' ');}} but on iOS 
 this results in a blank space for the title at the top of the alert. Sending 
 an object that will evaluate to true but return a blank string when 
 stringified causes the app to crash {{navigator.notification.alert('message', 
 null, {toString: function() { return ''; }});}}
 h4. Solution
 Rather than testing whether {{title}} is falsy to set the default, test 
 whether it is a string, regardless of whether it is a falsy empty string. 
 h4. Breaking change?
 Requiring the title to be a string prevents numeric and other values from 
 being used, while previously they were passed through to the native plugin. 
 This is a _good thing_ because sending anything other than a string causes 
 the plugin to crash the app (at least on iOS), so despite being more 
 restrictive about the data type, I do not consider it a breaking change.



--
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] [Resolved] (CB-9394) Cordova-Android Platform Release July 20, 2015

2015-08-04 Thread Nikhil Khandelwal (JIRA)

 [ 
https://issues.apache.org/jira/browse/CB-9394?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Nikhil Khandelwal resolved CB-9394.
---
Resolution: Fixed

 Cordova-Android Platform Release July 20, 2015
 --

 Key: CB-9394
 URL: https://issues.apache.org/jira/browse/CB-9394
 Project: Apache Cordova
  Issue Type: Task
  Components: Android
Reporter: Nikhil Khandelwal
Assignee: Nikhil Khandelwal

 Following steps at 
 https://github.com/apache/cordova-coho/blob/master/docs/platforms-release-process.md



--
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] [Closed] (CB-9394) Cordova-Android Platform Release July 20, 2015

2015-08-04 Thread Nikhil Khandelwal (JIRA)

 [ 
https://issues.apache.org/jira/browse/CB-9394?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Nikhil Khandelwal closed CB-9394.
-

 Cordova-Android Platform Release July 20, 2015
 --

 Key: CB-9394
 URL: https://issues.apache.org/jira/browse/CB-9394
 Project: Apache Cordova
  Issue Type: Task
  Components: Android
Reporter: Nikhil Khandelwal
Assignee: Nikhil Khandelwal

 Following steps at 
 https://github.com/apache/cordova-coho/blob/master/docs/platforms-release-process.md



--
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-8804) https://github.com/apache/cordova-plugin-camera#android-quirks

2015-08-04 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-8804?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14654670#comment-14654670
 ] 

ASF GitHub Bot commented on CB-8804:


Github user adamduren commented on the pull request:


https://github.com/apache/cordova-plugin-camera/pull/97#issuecomment-127817980
  
+1 on this bugfix. I see this happen a lot.


 https://github.com/apache/cordova-plugin-camera#android-quirks
 --

 Key: CB-8804
 URL: https://issues.apache.org/jira/browse/CB-8804
 Project: Apache Cordova
  Issue Type: Improvement
  Components: Android, Plugin Camera
 Environment: Android
Reporter: Serge Huijben

 Android Quirks
 Android uses intents to launch the camera activity on the device to capture 
 images, and on phones with low memory, the Cordova activity may be killed. In 
 this scenario, the image may not appear when the Cordova activity is restored.
 It can be solved bij implementing this PR: 
 https://github.com/apache/cordova-plugin-camera/pull/97



--
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] [Created] (CB-9450) During Build, set publisherId according to packaging configuration (build.json)

2015-08-04 Thread Rob Paveza (JIRA)
Rob Paveza created CB-9450:
--

 Summary: During Build, set publisherId according to packaging 
configuration (build.json)
 Key: CB-9450
 URL: https://issues.apache.org/jira/browse/CB-9450
 Project: Apache Cordova
  Issue Type: Bug
  Components: Windows
Reporter: Rob Paveza
Assignee: Rob Paveza


Presently, the `Identity Publisher` attribute is not set by either build or 
prepare.  The value of this attribute should be coming from publisherId 
parameter or build.json, but because build.json parameters are only set during 
build, but we don't update the .appxmanifest during build, it never gets set.

This change will be to set the appxmanifest publisher if the publisher is 
updated during build.



--
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