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

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

GitHub user sanjeewa894 opened a pull request:

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

    CB-11085 (Android) Fixed AudioHandler logs should use system log, not 
println

    <!--
    Please make sure the checklist boxes are all checked before submitting the 
PR. The checklist
    is intended as a quick reference, for complete details please see our 
Contributor Guidelines:
    
    http://cordova.apache.org/contribute/contribute_guidelines.html
    
    Thanks!
    -->
    
    ### Platforms affected
    Android
    
    ### What does this PR do?
    This is a fix for https://issues.apache.org/jira/browse/CB-11085.
    It fixes the println to LOG (android.util.Log use in 
org.apache.cordova.LOG).
    
    ### What testing has been done on this change?
    Testing was unable to do.
    
    ### Checklist
    - [ ] [ICLA](http://www.apache.org/licenses/icla.txt) has been signed and 
submitted to secret...@apache.org.
    - [ ] [Reported an issue](http://cordova.apache.org/contribute/issues.html) 
in the JIRA database
    - [ ] Commit message follows the format: "CB-3232: (android) Fix bug with 
resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform 
affected.
    - [ ] Added automated test coverage as appropriate for this change.


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

    $ git pull https://github.com/sanjeewa894/cordova-plugin-media master

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

    https://github.com/apache/cordova-plugin-media/pull/110.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 #110
    
----
commit 9084576e1a6853c82457e2d379fb3d2a0e3364f6
Author: sanjeewa894 <sanjeewa...@gmail.com>
Date:   2016-08-23T17:44:29Z

    Fix error output using println to Log.e

commit fac888da1b8cf88eb1fc44ac3a0e94b6d0030dbb
Author: sanjeewa894 <sanjeewa...@gmail.com>
Date:   2016-08-23T17:46:05Z

    Merge remote-tracking branch 'origin/master'

commit c1c01b1834e1a37158b6de247ac444a1d49fbdbf
Author: sanjeewa894 <sanjeewa...@gmail.com>
Date:   2016-08-23T17:53:12Z

    Fix error output using println to LOG.e

----


> AudioHandler logs should use system log, not println
> ----------------------------------------------------
>
>                 Key: CB-11085
>                 URL: https://issues.apache.org/jira/browse/CB-11085
>             Project: Apache Cordova
>          Issue Type: Improvement
>          Components: Plugin Media
>    Affects Versions: 1.0.0, 1.1.0, 1.2.0, 1.3.0, 1.4.0, 1.5.0, 1.6.0, 1.6.1, 
> 1.7.0, 1.8.0, 1.8.1, 1.9.0, 2.0.0, 2.1.0, 2.2.0, 2.3.0, 2.4.0, 2.5.0, 2.6.0, 
> 2.7.0, 2.8.0, 2.9.0, 3.0.0, 3.1.0, 3.2.0, 3.3.0, 3.4.0, 2.9.1, 3.5.0
>            Reporter: Fabrice Lebas
>            Priority: Minor
>              Labels: Android, easyfix, test
>   Original Estimate: 0.5h
>  Remaining Estimate: 0.5h
>
> AudioHandler.java should use android.util.Log rather than println.



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

Reply via email to