Skip to content
This repository has been archived by the owner on Sep 4, 2019. It is now read-only.

Escape double quotes. PR 308748 #25

Closed
wants to merge 1 commit into from
Closed

Escape double quotes. PR 308748 #25

wants to merge 1 commit into from

Conversation

haahmad
Copy link
Contributor

@haahmad haahmad commented Apr 24, 2013

@bryanhiggins
Copy link
Contributor

Is there a test case associated with this?

@jeffheifetz
Copy link
Contributor

@DanielAudino There was a specific test case, wasn't there?

@haahmad
Copy link
Contributor Author

haahmad commented May 9, 2013

rebased

@tracyli
Copy link

tracyli commented May 9, 2013

special characters including quote can be parsed without problem.
however, there's one observation:
1.suppose there're 2 contacts with special chars:
"Test2" "Hello2"
Test!-3 Hello!=3
2.search '2' by first name, the following contact was returned: "Test2" "Hello2"
3.search 'h' by last name, the following 2 contacts were returned: "Test2" "Hello2" and Test!-3 Hello!=3
=> please advise if behavior 2&3 are what expected?

@bryanhiggins
Copy link
Contributor

Issue logged for @tracyli 's finding: http://github.rim.net/webworks/BB10-WebWorks-Cordova/issues/183

@haahmad haahmad mentioned this pull request May 23, 2013
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@haahmad @bryanhiggins @jeffheifetz @tracyli