Mailing-List: contact issues-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Date: Thu, 21 Jan 2016 21:15:39 +0000 (UTC)
From: "ASF GitHub Bot (JIRA)" <jira@apache.org>
To: issues@cordova.apache.org
Message-ID: <JIRA.12932529.1453237538000.175072.1453410939808@Atlassian.JIRA>
In-Reply-To: <JIRA.12932529.1453237538000@Atlassian.JIRA>
References: <JIRA.12932529.1453237538000@Atlassian.JIRA>
 <JIRA.12932529.1453237538707@arcas>
Subject: [jira] [Commented] (CB-10388) The correctOrientation method is not
 being called when the sourceType as PHOTOLIBRARY, the mediaType is ALLMEDIA
 and the selected file is an image (Only for Android devices)
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable


    [ https://issues.apache.org/jira/browse/CB-10388?page=3Dcom.atlassian.j=
ira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=3D151113=
13#comment-15111313 ]=20

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

GitHub user horozco opened a pull request:

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

    [CB-10388] - Added correctOrientation support for android 4.4

    Jira issue: https://issues.apache.org/jira/browse/CB-10388
   =20
    - Currently If You pass the correctOrientation param as true, the sourc=
eType as PHOTOLIBRARY or SAVEDPHOTOALBUM, the media type as ALLMEDIA and th=
e selected file is an image the correctOrientation function is not being ca=
lled. Also even if the function is called the code is not detecting the URI=
 and doesn=E2=80=99t rotate the picture. These changes fixed both issues.


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

    $ git pull https://github.com/horozco/cordova-plugin-camera master

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

    https://github.com/apache/cordova-plugin-camera/pull/158.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 #158
   =20
----
commit e2823f48d632e12f89960fe83ebc90ba7d0a5a6a
Author: Hernan Orozco <horozco15@gmail.com>
Date:   2016-01-21T21:02:11Z

    [CB-10388] - Added correctOrientation support for android 4.4
   =20
    - Currently If You pass the correctOrientation param as true, the
    sourceType as PHOTOLIBRARY or SAVEDPHOTOALBUM, the media type as
    ALLMEDIA and the selected file is an image the correctOrientation
    function is not being called. Also even if the function is called the
    code is not detecting the uri and doesn=E2=80=99t rotate the picture. T=
hese
    changes fixed both issues.

----


> The correctOrientation method is not being called when the sourceType as =
PHOTOLIBRARY, the mediaType is ALLMEDIA and the selected file is an image (=
Only for Android devices)
> -------------------------------------------------------------------------=
---------------------------------------------------------------------------=
-------------------------
>
>                 Key: CB-10388
>                 URL: https://issues.apache.org/jira/browse/CB-10388
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Android, Plugin Camera
>    Affects Versions: 1.2.0, 2.0.0, 2.1.0
>         Environment: Developing on MAC, android devices used: Xperia Z3, =
Dell venue7
>            Reporter: Hernan Orozco
>              Labels: android, camera, easyfix, easytest
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> The correctOrientation function is used to fix the orientation of a photo=
, if this method is not called and you take a picture in portrait mode, the=
 picture is going to be shown with a wrong orientation. To activate this fe=
ature we should send the correctOrientation param as true. The issue is tha=
t the current android code is only calling the correctOrientation method in=
 these cases:
> - When you pass the correctOrientation param as true and the sourceType a=
s CAMERA.
> - When you pass the correctOrientation param as true, the sourceType as P=
HOTOLIBRARY or SAVEDPHOTOALBUM and the media type as PICTURE.
> I think we should include this case:
> - You pass the correctOrientation param as true, the sourceType as PHOTOL=
IBRARY or SAVEDPHOTOALBUM, the media type as ALLMEDIA and the selected file=
 is a picture.
> Also I want to add that I was able to reproduce the issue CB-9012, that i=
ssue and this one are a little related so I think we can create a fix for b=
oth issues.=20


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