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

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

Commit a15b24da0fc32cfc6bb09fa30ea09452e95ee680 in cordova-lib's branch 
refs/heads/common-1.4.x from [~vladimir.kotikov]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-lib.git;h=a15b24d ]

CB-11636 Handle attributes with quotes correctly

Avoid using complex queries in et.findall to prevent
issues w/ quoted values. Previous approach w/ crafting
complex query from attributes names and values was faulty
when any of the attributes had double quotes or "

 This closes #470


> Merging config.xml w/ attributes containing " fails.
> ---------------------------------------------------------
>
>                 Key: CB-11636
>                 URL: https://issues.apache.org/jira/browse/CB-11636
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: CordovaCommon
>    Affects Versions: 1.4.0
>            Reporter: Vladimir Kotikov
>            Assignee: Vladimir Kotikov
>              Labels: reproduced, triaged, xml
>             Fix For: 1.4.1
>
>
> Repro steps:
> - Create an empty project
> - Add {{<preference name="name" value="my &quot;quoted&quot; value">}} to 
> config.xml (notice that JIRA replaces &quot; by " symbol)
> - Add any of ios, windows or android platform
> - Call {{cordova prepare}}
> Expected:
> prepare succeds and preference copied to platform's config.xml
> Actual:
> prepare fails with unrelevant stacktrace:
> {noformat}
> No scripts found for hook "before_prepare".
> Checking config.xml for saved platforms that haven't been added to the project
> Checking for any plugins added to the project that have not been installed in 
> android platform
> No differences found between plugins added to project and installed in 
> android platform. Continuing...
> Generating config.xml from defaults for platform "android"
> net.js:647
>     throw new TypeError(
>     ^
> TypeError: Invalid data, chunk must be a string or buffer, not undefined
>     at WriteStream.Socket.write (net.js:647:11)
>     at WriteStream.stream.write 
> (D:\cordova\cordova-lib\cordova-common\node_modules\ansi\lib\newlines.js:36:21)
>     at Cursor.write 
> (D:\cordova\cordova-lib\cordova-common\node_modules\ansi\lib\ansi.js:157:23)
>     at CordovaLogger.log 
> (D:\cordova\cordova-lib\cordova-common\src\CordovaLogger.js:102:12)
>     at process.<anonymous> (D:\cordova\cordova-cli\src\cli.js:253:16)
>     at emitOne (events.js:96:13)
>     at process.emit (events.js:188:7)
>     at process._fatalException (bootstrap_node.js:254:26)
> {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

Reply via email to