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

### What does this PR do?
Resolves an issue where a regex was overly greedy when comparing plist entries 
for merging and could result in collapsing several of them into a single entry.

### What testing has been done on this change?
Added a test that fails with current master and passes with this change, based 
on the details provided by @knight9999 in the JIRA bug. It sounds like this 
might be the cause of GH-44 as well (/fyi @donnie-jp).

Now, my fix assumes that we're always looking for `$(something)` and not just 
`$something`, which *might* not be a safe assumption. Unfortunately I'm not 
well-versed enough in iOS stuff to know :(


### Checklist
- [x] Reported an issue in the JIRA database
- [x] Commit message follows the format
- [x] Added automated test coverage as appropriate for this change.


[ Full content available at: https://github.com/apache/cordova-common/pull/45 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to