Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

don't match res.partner.title on substrings #68

Merged
merged 1 commit into from
Nov 27, 2014

Conversation

tremlin
Copy link

@tremlin tremlin commented Nov 26, 2014

I would like to suggest to use the =ilike operator for title matching because at least in our installations we want to distinguish between "Herr" and "Herr Dr.", for instance.

@guewen
Copy link
Member

guewen commented Nov 26, 2014

👍 but needs a rebase it seems

Thanks

@tremlin tremlin force-pushed the 7.0-title_exact_match branch from a0770fa to 0eb8637 Compare November 26, 2014 15:01
@tremlin
Copy link
Author

tremlin commented Nov 26, 2014

I rebased my patch.

@OSguard
Copy link

OSguard commented Nov 26, 2014

👍
builds fails to pylint

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 0eb8637 on initOS:7.0-title_exact_match into * on OCA:7.0*.

@pedrobaeza
Copy link
Member

👍

@guewen
Copy link
Member

guewen commented Nov 27, 2014

I restarted the build and it is now green

guewen added a commit that referenced this pull request Nov 27, 2014
don't match res.partner.title on substrings
@guewen guewen merged commit 4c1633b into OCA:7.0 Nov 27, 2014
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants