Therp Addons

Merge lp://staging/~therp-nl/therp-addons/7.0-fetchmail_inbox into lp://staging/~therp-nl/therp-addons/7.0

7.0-fetchmail_inbox
Merge into 7.0

Proposed by Holger Brunn (Therp) on 2014-01-03

Status:	Merged
Merged at revision:	96
Proposed branch:	lp://staging/~therp-nl/therp-addons/7.0-fetchmail_inbox
Merge into:	lp://staging/~therp-nl/therp-addons/7.0
Diff against target:	954 lines (+814/-0) 24 files modified fetchmail_inbox/__init__.py (+22/-0) fetchmail_inbox/__openerp__.py (+81/-0) fetchmail_inbox/model/__init__.py (+22/-0) fetchmail_inbox/model/fetchmail_inbox.py (+30/-0) fetchmail_inbox/model/mail_message.py (+127/-0) fetchmail_inbox/security/ir.model.access.csv (+2/-0) fetchmail_inbox/security/res_groups.xml (+8/-0) fetchmail_inbox/view/mail_message.xml (+33/-0) fetchmail_inbox/view/menu.xml (+21/-0) fetchmail_inbox/wizard/__init__.py (+22/-0) fetchmail_inbox/wizard/fetchmail_inbox_attach_existing_wizard.py (+84/-0) fetchmail_inbox/wizard/fetchmail_inbox_attach_existing_wizard.xml (+23/-0) fetchmail_inbox/wizard/fetchmail_inbox_create_wizard.py (+54/-0) fetchmail_inbox/wizard/fetchmail_inbox_create_wizard.xml (+21/-0) fetchmail_inbox_invoice/__init__.py (+22/-0) fetchmail_inbox_invoice/__openerp__.py (+60/-0) fetchmail_inbox_invoice/model/__init__.py (+22/-0) fetchmail_inbox_invoice/model/fetchmail_inbox_invoice.py (+28/-0) fetchmail_inbox_invoice/model/mail_message.py (+32/-0) fetchmail_inbox_invoice/security/ir.model.access.csv (+2/-0) fetchmail_inbox_invoice/view/mail_message.xml (+17/-0) fetchmail_inbox_invoice/view/menu.xml (+18/-0) fetchmail_inbox_invoice/wizard/__init__.py (+21/-0) fetchmail_inbox_invoice/wizard/fetchmail_inbox_attach_existing_wizard.py (+42/-0)
To merge this branch:	bzr merge lp://staging/~therp-nl/therp-addons/7.0-fetchmail_inbox
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Stefan Rijnhart (Opener) (community)		2014-01-03	Needs Information on 2014-01-13
Review via email: mp+200416@code.staging.launchpad.net

Commit message

[ADD] fetchmail_inbox, fetchmail_inbox_invoice

Description of the change

A base module to put incoming mails into an inbox for further (manual) processing.

Also a small usability module to do that specifically for invoices.

Revision history for this message

Stefan Rijnhart (Opener) (stefan-opener) wrote on 2014-01-13:

Thanks! I tested this, and it works great at first sight. A couple of remarks:

- Can permissions be succesfully applied to the inherited model fetchmail.inbox.invoice? I would rather make the processing of incoming invoices available to 'Account users', and I made the menu item available to them as well as added permissions on this model. But I still run into trouble when as an account user, I try processing the emails under the menu item.
- I appreciate the generic setup, but it poses the following challenge: in the case of invoices, when selecting an existing resource, I would like there to be a default filter on the partner_id to be the commercial_partner_id of the recognized author partner of the email, if any (I think this only works in the full search view when the user clicks on 'Search more...', but that is OK).

If this is at all possible to add, it would be great.

review: Needs Information

lp://staging/~therp-nl/therp-addons/7.0-fetchmail_inbox updated on 2014-01-14

104. By Holger Brunn (Therp) on 2014-01-14: [FIX] get pool model from mail.message's model to have more fine grained
control over permissions on different fetchmail.inbox.* models
[ADD] set default search filter to mail author's commercial partner for
invoices