"plugin" bugs with prefixed emails

Bug #1059561 reported by Lucas P
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Committed
Medium
OpenERP R&D Addons Team 1

Bug Description

Hello,

In the current implementation of the "plugin" addon, in the file "plugin_handler.py", the function "partner_get" handles the search with:
('email', 'like', address_email)

This causes the partner with the email '<email address hidden>' to match with an existing '<email address hidden>'.
simply replacing the previous search statement with
('email', '=', address_email)
fixes the problem.

Reproduce:
Clean install of OERP 6.1
Install a mailer plugin (TB or outlook)
Clean install of Thunderbird or outlook
Create a partner/contact in OERP whose email corresponds to a prefixed version of one of the emails you have in your inbox.
Export the said email as a partner

Expected:
Mailer window proposing you to create a contact
Instead:
Opens a browser window with the "prefixed partner" displayed.

Related branches

Lucas P (lucas33-3)
description: updated
Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 1 (openerp-dev-addons1)
importance: Undecided → Medium
status: New → Confirmed
Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
Nimesh Contractor(Open ERP) (nco-openerp) wrote :

Hello,

          It has been Fixed in https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-1059561-nco brach.
          Revno: 7780
          Revision ID: <email address hidden>

          It will be available in trunk soon.

Thanks,
 NCO.

Changed in openobject-addons:
status: In Progress → Fix Committed
Revision history for this message
Lucas P (lucas33-3) wrote :

Hello,

Acually I've been digging around a bit and just replacing with "=" doesn't exactly respect the norm. The norm states the the domain part of the email address should be case insensitive.
But that seems like an unreasonable amount of work for what you get out of it.
Just thought it was worth posting even if it's not worth implementing.

Thanks a lot for that fix.
I can't find an "accept the fix" functionality or a way to give you credit for fixing it so I'll assume it doesn't exist.
Please tell me if I missed it.

Regards

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.