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

Robin Zeggelaar commented on CB-5634:
-------------------------------------

I can confirm this issue. I tried to find the source of the problem, which 
brought me to configParser.js, line 88.

{code}if (preferences[i].attrib.name.toLowerCase() === name) {
    ret = a.attrib.value;
}{code}

Here, the variable a is not defined, generating the error. I'd say changing 
this line to {code}ret = preferences[i].attrib.value{code} would fix it.

Then again, i'm no cordova expert. But that's what it looks like to me.

> Orientation Preference is ignored in Android, should be parsed by CLI
> ---------------------------------------------------------------------
>
>                 Key: CB-5634
>                 URL: https://issues.apache.org/jira/browse/CB-5634
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Amazon FireOS, Android, CLI, Docs, iOS
>            Reporter: Joe Bowser
>            Assignee: Andrew Grieve
>
> The docs say that an Orientation preference is supposed to work across all 
> platforms.  However, Android's activities can only be locked to an 
> orientation by modifying the AndroidManifest.xml. 
> To get past this, we either change the docs, or we support this in the CLI.  
> I feel that adding an XML parser to the scripts is complete overkill, since 
> we don't have any tests for our platform scripts, and I personally think 
> they're a giant pile of technical debt.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to