From 55f9f4dc7470d3b54fd7562107c0310df0291323 Mon Sep 17 00:00:00 2001 From: "ajay javiya (OpenERP)" Date: Wed, 12 Dec 2012 10:58:11 +0530 Subject: [PATCH 01/12] [REM]: Remove warning bzr revid: aja@tinyerp.com-20121212052811-lfq1vyyu2bagx77f --- addons/web_tests_demo/__openerp__.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/addons/web_tests_demo/__openerp__.py b/addons/web_tests_demo/__openerp__.py index 7404ef6b4fc..e52707cdc5e 100644 --- a/addons/web_tests_demo/__openerp__.py +++ b/addons/web_tests_demo/__openerp__.py @@ -1,6 +1,10 @@ { 'name': "Demonstration of web/javascript tests", 'category': 'Hidden', + 'description': """ + OpenERP Web test demo suite. + ======================= + """, 'depends': ['web'], 'js': ['static/src/js/demo.js'], 'test': ['static/test/demo.js'], From 1ee6aa0000f697a9747a8c83614136c2ba4bbe64 Mon Sep 17 00:00:00 2001 From: Christophe Matthieu Date: Wed, 12 Dec 2012 10:05:54 +0100 Subject: [PATCH 02/12] [IMP] project: remove wrong div into the kanban view for a good rendering. apply groups='base.group_user' on links. bzr revid: chm@openerp.com-20121212090554-jyk60tt1snfqclqj --- addons/project/project_view.xml | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/addons/project/project_view.xml b/addons/project/project_view.xml index 853dac4e20c..e73fb0ec650 100644 --- a/addons/project/project_view.xml +++ b/addons/project/project_view.xml @@ -457,14 +457,12 @@
-
- - - - 7 - 7 - 7 -
+ + + + 7 + 7 + 7
From e96bfb0b04f6b589de87e13c1717b1a89bbf55a7 Mon Sep 17 00:00:00 2001 From: Christophe Matthieu Date: Wed, 12 Dec 2012 10:33:08 +0100 Subject: [PATCH 03/12] [IMP] project: revert to tag t of qweb with groups bzr revid: chm@openerp.com-20121212093308-w7riueqzvklufvt4 --- addons/project/project_view.xml | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/addons/project/project_view.xml b/addons/project/project_view.xml index e73fb0ec650..424034f1104 100644 --- a/addons/project/project_view.xml +++ b/addons/project/project_view.xml @@ -457,12 +457,14 @@
- - - - 7 - 7 - 7 + + + + + 7 + 7 + 7 +
From 814c7874e9bcb433ee85d22d35cfa8f8ceee2048 Mon Sep 17 00:00:00 2001 From: Arnaud Pineux Date: Wed, 12 Dec 2012 13:05:26 +0100 Subject: [PATCH 04/12] [FIX] product category access lp bug: https://launchpad.net/bugs/1086903 fixed bzr revid: api@openerp.com-20121212120526-g8wb3abd94feggfb --- addons/account/security/ir.model.access.csv | 1 + 1 file changed, 1 insertion(+) diff --git a/addons/account/security/ir.model.access.csv b/addons/account/security/ir.model.access.csv index d1f0bbab6b5..48617f2e46f 100644 --- a/addons/account/security/ir.model.access.csv +++ b/addons/account/security/ir.model.access.csv @@ -54,6 +54,7 @@ access_account_invoice_user,account.invoice.line user,model_account_invoice_line access_account_payment_term_partner_manager,account.payment.term partner manager,model_account_payment_term,base.group_user,1,0,0,0 access_account_payment_term_line_partner_manager,account.payment.term.line partner manager,model_account_payment_term_line,base.group_user,1,0,0,0 access_account_account_sale_manager,account.account sale manager,model_account_account,base.group_sale_manager,1,0,0,0 +access_product_category_sale_manager,product.category salemanager,product.model_product_category,base.group_sale_manager,1,1,1,1 access_account_fiscal_position_product_manager,account.fiscal.position account.manager,model_account_fiscal_position,account.group_account_manager,1,1,1,1 access_account_fiscal_position_tax_product_manager,account.fiscal.position.tax account.manager,model_account_fiscal_position_tax,account.group_account_manager,1,1,1,1 access_account_fiscal_position_account_product_manager,account.fiscal.position account.manager,model_account_fiscal_position_account,account.group_account_manager,1,1,1,1 From 76f2cf65c0019c04caa4d8883066a01a7e560d94 Mon Sep 17 00:00:00 2001 From: "ajay javiya (OpenERP)" Date: Wed, 12 Dec 2012 18:27:58 +0530 Subject: [PATCH 05/12] [FIX]: description bzr revid: aja@tinyerp.com-20121212125758-iv71s1bol2afbwe6 --- addons/web_tests_demo/__openerp__.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/addons/web_tests_demo/__openerp__.py b/addons/web_tests_demo/__openerp__.py index e52707cdc5e..d3075f093ba 100644 --- a/addons/web_tests_demo/__openerp__.py +++ b/addons/web_tests_demo/__openerp__.py @@ -2,8 +2,9 @@ 'name': "Demonstration of web/javascript tests", 'category': 'Hidden', 'description': """ - OpenERP Web test demo suite. + OpenERP Web demo of a test suite. ======================= + Holds the examples used in the testing documentation. """, 'depends': ['web'], 'js': ['static/src/js/demo.js'], From 6b74785450f00b890fc457e05723c824c703b5c3 Mon Sep 17 00:00:00 2001 From: Arnaud Pineux Date: Wed, 12 Dec 2012 13:59:12 +0100 Subject: [PATCH 06/12] [FIX] product category access lp bug: https://launchpad.net/bugs/1086903 fixed bzr revid: api@openerp.com-20121212125912-bnzhcaa1mwq153vt --- addons/account/security/ir.model.access.csv | 1 - addons/product/security/ir.model.access.csv | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/account/security/ir.model.access.csv b/addons/account/security/ir.model.access.csv index 48617f2e46f..d1f0bbab6b5 100644 --- a/addons/account/security/ir.model.access.csv +++ b/addons/account/security/ir.model.access.csv @@ -54,7 +54,6 @@ access_account_invoice_user,account.invoice.line user,model_account_invoice_line access_account_payment_term_partner_manager,account.payment.term partner manager,model_account_payment_term,base.group_user,1,0,0,0 access_account_payment_term_line_partner_manager,account.payment.term.line partner manager,model_account_payment_term_line,base.group_user,1,0,0,0 access_account_account_sale_manager,account.account sale manager,model_account_account,base.group_sale_manager,1,0,0,0 -access_product_category_sale_manager,product.category salemanager,product.model_product_category,base.group_sale_manager,1,1,1,1 access_account_fiscal_position_product_manager,account.fiscal.position account.manager,model_account_fiscal_position,account.group_account_manager,1,1,1,1 access_account_fiscal_position_tax_product_manager,account.fiscal.position.tax account.manager,model_account_fiscal_position_tax,account.group_account_manager,1,1,1,1 access_account_fiscal_position_account_product_manager,account.fiscal.position account.manager,model_account_fiscal_position_account,account.group_account_manager,1,1,1,1 diff --git a/addons/product/security/ir.model.access.csv b/addons/product/security/ir.model.access.csv index b5cd20854d9..d4c4261bb8c 100644 --- a/addons/product/security/ir.model.access.csv +++ b/addons/product/security/ir.model.access.csv @@ -17,3 +17,4 @@ access_product_pricelist_partner_manager,product.pricelist partner manager,model access_product_product_employee,product.product employee,model_product_product,base.group_user,1,0,0,0 access_product_template_sale_manager,product.template salemanager,model_product_template,base.group_sale_manager,1,1,1,1 access_product_product_sale_manager,product.product salemanager,model_product_product,base.group_sale_manager,1,1,1,1 +access_product_category_sale_manager,product.category salemanager,product.model_product_category,base.group_sale_manager,1,1,1,1 From 8f224d0914545291c15e79284428d3dbe5c588d7 Mon Sep 17 00:00:00 2001 From: niv-openerp Date: Wed, 12 Dec 2012 14:48:47 +0100 Subject: [PATCH 07/12] [FIX] problem with list views extending outside of sheets bzr revid: nicolas.vanhoren@openerp.com-20121212134847-1rbjhwk7vu1vmu1k --- addons/web/static/src/css/base.css | 12 +++++++++--- addons/web/static/src/css/base.sass | 4 ++++ 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/addons/web/static/src/css/base.css b/addons/web/static/src/css/base.css index 07614dc2c6b..6daf4ba9707 100644 --- a/addons/web/static/src/css/base.css +++ b/addons/web/static/src/css/base.css @@ -1198,7 +1198,7 @@ color: white; padding: 2px 4px; margin: 1px 6px 0 0; - border: 1px solid lightGray; + border: 1px solid lightgrey; text-shadow: 0 1px 1px rgba(0, 0, 0, 0.2); -moz-border-radius: 4px; -webkit-border-radius: 4px; @@ -1223,7 +1223,7 @@ transform: scale(1.1); } .openerp .oe_secondary_submenu .oe_active { - border-top: 1px solid lightGray; + border-top: 1px solid lightgrey; border-bottom: 1px solid #dedede; text-shadow: 0 1px 1px rgba(0, 0, 0, 0.2); -moz-box-shadow: inset 0 1px 3px rgba(0, 0, 0, 0.2), inset 0 -1px 3px rgba(40, 40, 40, 0.2); @@ -2037,6 +2037,9 @@ min-height: 330px; padding: 16px; } +.openerp .oe_form_sheet .oe_list { + overflow-x: auto; +} .openerp .oe_application .oe_form_sheetbg { background: url(/web/static/src/img/form_sheetbg.png); border-bottom: 1px solid #dddddd; @@ -2160,7 +2163,7 @@ } .openerp .oe_form .oe_form_label_help[for] span, .openerp .oe_form .oe_form_label[for] span { font-size: 80%; - color: darkGreen; + color: darkgreen; vertical-align: top; position: relative; top: -4px; @@ -3071,6 +3074,9 @@ div.ui-widget-overlay { border: 0px !important; box-shadow: 0px 0px 0px; } + .openerp .oe_application .oe_form_sheet .oe_list, .openerp .oe_application .oe_form_sheetbg .oe_list { + overflow-x: visible; + } .openerp .oe_view_manager_current > .oe_view_manager_header { border: 0px !important; box-shadow: 0px 0px 0px; diff --git a/addons/web/static/src/css/base.sass b/addons/web/static/src/css/base.sass index d9f3644135b..d30e8935a0c 100644 --- a/addons/web/static/src/css/base.sass +++ b/addons/web/static/src/css/base.sass @@ -1621,6 +1621,8 @@ $sheet-padding: 16px background: white min-height: 330px padding: 16px + .oe_list + overflow-x: auto // Sheet inline mode .oe_application .oe_form_sheetbg @@ -2437,6 +2439,8 @@ div.ui-widget-overlay .oe_form_sheet, .oe_form_sheetbg border: 0px !important box-shadow: 0px 0px 0px + .oe_list + overflow-x: visible .oe_view_manager_current > .oe_view_manager_header border: 0px !important box-shadow: 0px 0px 0px From 37a3ef575ebafbca919076421f5abe8160698dbc Mon Sep 17 00:00:00 2001 From: "Quentin (OpenERP)" Date: Wed, 12 Dec 2012 15:31:38 +0100 Subject: [PATCH 08/12] [FIX] base_calendar: virtual ids and misc fixes bzr revid: qdp-launchpad@openerp.com-20121212143138-r400ibtajigk0lns --- addons/base_calendar/base_calendar.py | 48 +++++++------------ .../security/ir.model.access.csv | 2 + 2 files changed, 20 insertions(+), 30 deletions(-) diff --git a/addons/base_calendar/base_calendar.py b/addons/base_calendar/base_calendar.py index 9cf07a4b73e..28264bea759 100644 --- a/addons/base_calendar/base_calendar.py +++ b/addons/base_calendar/base_calendar.py @@ -30,6 +30,7 @@ import pytz import re import time import tools +from openerp import SUPERUSER_ID months = { 1: "January", 2: "February", 3: "March", 4: "April", \ @@ -1001,12 +1002,15 @@ class calendar_event(osv.osv): if not isinstance(ids, list): ids = [ids] - for data in self.read(cr, uid, ids, ['id','byday','recurrency', 'month_list','end_date', 'rrule_type', 'select1', 'interval', 'count', 'end_type', 'mo', 'tu', 'we', 'th', 'fr', 'sa', 'su', 'exrule', 'day', 'week_list' ], context=context): - event = data['id'] + for id in ids: + #read these fields as SUPERUSER because if the record is private a normal search could return False and raise an error + data = self.read(cr, SUPERUSER_ID, id, ['interval', 'count'], context=context) if data.get('interval', 0) < 0: raise osv.except_osv(_('Warning!'), _('Interval cannot be negative.')) if data.get('count', 0) <= 0: raise osv.except_osv(_('Warning!'), _('Count cannot be negative or 0.')) + data = self.read(cr, uid, id, ['id','byday','recurrency', 'month_list','end_date', 'rrule_type', 'select1', 'interval', 'count', 'end_type', 'mo', 'tu', 'we', 'th', 'fr', 'sa', 'su', 'exrule', 'day', 'week_list' ], context=context) + event = data['id'] if data['recurrency']: result[event] = self.compute_rule_string(data) else: @@ -1339,40 +1343,24 @@ rule or repeating pattern of time to exclude from the recurring rule."), return data def search(self, cr, uid, args, offset=0, limit=0, order=None, context=None, count=False): - context = context or {} - args_without_date = [] - filter_date = [] + if context is None: + context = {} + new_args = [] for arg in args: - if arg[0] == "id": - new_id = self.remove_virtual_id(arg[2]) + new_arg = arg + if arg[0] in ('date', unicode('date'), 'date_deadline', unicode('date_deadline')): + if context.get('virtual_id', True): + new_args += ['|','&',('recurrency','=',1),('recurrent_id_date', arg[1], arg[2])] + elif arg[0] == "id": + new_id = get_real_ids(arg[2]) new_arg = (arg[0], arg[1], new_id) - args_without_date.append(new_arg) - elif arg[0] not in ('date', unicode('date'), 'date_deadline', unicode('date_deadline')): - args_without_date.append(arg) - else: - if context.get('virtual_id', True): - args_without_date.append('|') - args_without_date.append(arg) - if context.get('virtual_id', True): - args_without_date.append(('recurrency','=',1)) - filter_date.append(arg) + new_args.append(new_arg) - res = super(calendar_event, self).search(cr, uid, args_without_date, \ - 0, 0, order, context, count=False) - if context.get('virtual_id', True): - res = self.get_recurrent_ids(cr, uid, res, args, limit, context=context) - - if count: - return len(res) - elif limit: - return res[offset:offset+limit] - else: - return res + return super(calendar_event, self).search(cr, uid, new_args, offset=offset, limit=limit, order=order, context=context, count=count) def _get_data(self, cr, uid, id, context=None): - res = self.read(cr, uid, [id],['date', 'date_deadline']) - return res[0] + return self.read(cr, uid, id,['date', 'date_deadline']) def need_to_update(self, event_id, vals): split_id = str(event_id).split("-") diff --git a/addons/base_calendar/security/ir.model.access.csv b/addons/base_calendar/security/ir.model.access.csv index 54ab0590aca..7db870d9053 100644 --- a/addons/base_calendar/security/ir.model.access.csv +++ b/addons/base_calendar/security/ir.model.access.csv @@ -10,4 +10,6 @@ access_crm_meeting,crm.meeting,model_crm_meeting,base.group_sale_salesman,1,1,1, access_crm_meeting_all,crm.meeting_allll,model_crm_meeting,base.group_user,1,0,0,0 access_crm_meeting_partner_manager,crm.meeting.partner.manager,model_crm_meeting,base.group_partner_manager,1,1,1,1 access_crm_meeting_type_sale_manager,crm.meeting.type.manager,model_crm_meeting_type,base.group_sale_manager,1,1,1,0 +access_crm_meeting_type_sale_user,crm.meeting.type.user,model_crm_meeting_type,base.group_user,1,0,0,0 +access_crm_meeting_type_sale_user,crm.meeting.type.salesman,model_crm_meeting_type,base.group_sale_salesman,1,0,0,0 access_crm_meeting_type_manager,crm.meeting.type.manager,model_crm_meeting_type,base.group_system,1,1,1,1 From cf863ebd182e84ef0c41093c488e935e6d02c813 Mon Sep 17 00:00:00 2001 From: "Quentin (OpenERP)" Date: Wed, 12 Dec 2012 18:42:15 +0100 Subject: [PATCH 09/12] [IMP] base_calendar: improved yaml test to help debug bzr revid: qdp-launchpad@openerp.com-20121212174215-kclquypqib680zju --- addons/base_calendar/test/base_calendar_test.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/base_calendar/test/base_calendar_test.yml b/addons/base_calendar/test/base_calendar_test.yml index c84f3ce36c3..7afd2e49551 100644 --- a/addons/base_calendar/test/base_calendar_test.yml +++ b/addons/base_calendar/test/base_calendar_test.yml @@ -26,7 +26,7 @@ - !python {model: calendar.event}: | ids = self.search(cr, uid, [('date', '>=', '2011-04-30 16:00:00'), ('date', '<=', '2011-05-31 00:00:00')], context={'virtual_id': True} ) - assert len(ids) == 9 + assert len(ids) == 9, 'Wrong number of events found' - Now I will make All day event and test it - From 85c90838a860cdf4eab329a85688ee1eef5736e3 Mon Sep 17 00:00:00 2001 From: "Quentin (OpenERP)" Date: Wed, 12 Dec 2012 18:43:23 +0100 Subject: [PATCH 10/12] [REF] mail: splitted the search() function in smaller part in order to ease the inheritance (because some code needs to be overriden for virtual ids, for example) bzr revid: qdp-launchpad@openerp.com-20121212174323-3kw9vrzacfm3wvm7 --- addons/mail/mail_message.py | 22 +++++++++++++++------- 1 file changed, 15 insertions(+), 7 deletions(-) diff --git a/addons/mail/mail_message.py b/addons/mail/mail_message.py index 069e0118ec4..d0a73385f7f 100644 --- a/addons/mail/mail_message.py +++ b/addons/mail/mail_message.py @@ -550,6 +550,20 @@ class mail_message(osv.Model): if not cr.fetchone(): cr.execute("""CREATE INDEX mail_message_model_res_id_idx ON mail_message (model, res_id)""") + def _find_allowed_model_wise(self, cr, uid, doc_model, doc_dict, context=None): + doc_ids = doc_dict.keys() + allowed_doc_ids = self.pool.get(doc_model).search(cr, uid, [('id', 'in', doc_ids)], context=context) + return set([message_id for allowed_doc_id in allowed_doc_ids for message_id in doc_dict[allowed_doc_id]]) + + def _find_allowed_doc_ids(self, cr, uid, model_ids, context=None): + model_access_obj = self.pool.get('ir.model.access') + allowed_ids = set() + for doc_model, doc_dict in model_ids.iteritems(): + if not model_access_obj.check(cr, uid, doc_model, 'read', False): + continue + allowed_ids |= self._find_allowed_model_wise(cr, uid, doc_model, doc_dict, context=context) + return allowed_ids + def _search(self, cr, uid, args, offset=0, limit=None, order=None, context=None, count=False, access_rights_uid=None): """ Override that adds specific access rights of mail.message, to remove @@ -587,13 +601,7 @@ class mail_message(osv.Model): elif message.get('model') and message.get('res_id'): model_ids.setdefault(message.get('model'), {}).setdefault(message.get('res_id'), set()).add(message.get('id')) - model_access_obj = self.pool.get('ir.model.access') - for doc_model, doc_dict in model_ids.iteritems(): - if not model_access_obj.check(cr, uid, doc_model, 'read', False): - continue - doc_ids = doc_dict.keys() - allowed_doc_ids = self.pool.get(doc_model).search(cr, uid, [('id', 'in', doc_ids)], context=context) - allowed_ids |= set([message_id for allowed_doc_id in allowed_doc_ids for message_id in doc_dict[allowed_doc_id]]) + allowed_ids = self._find_allowed_doc_ids(cr, uid, model_ids, context=context) final_ids = author_ids | partner_ids | allowed_ids if count: From 7ef99890d82d3a51859b01867483b95c0f1721f1 Mon Sep 17 00:00:00 2001 From: "Quentin (OpenERP)" Date: Wed, 12 Dec 2012 18:44:15 +0100 Subject: [PATCH 11/12] [REV] base_calendar: reverted to a previous code because it broke one test bzr revid: qdp-launchpad@openerp.com-20121212174415-pllm4f5g7k6uyn3o --- addons/base_calendar/base_calendar.py | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/addons/base_calendar/base_calendar.py b/addons/base_calendar/base_calendar.py index 28264bea759..69baf57c1a9 100644 --- a/addons/base_calendar/base_calendar.py +++ b/addons/base_calendar/base_calendar.py @@ -1357,7 +1357,15 @@ rule or repeating pattern of time to exclude from the recurring rule."), new_arg = (arg[0], arg[1], new_id) new_args.append(new_arg) - return super(calendar_event, self).search(cr, uid, new_args, offset=offset, limit=limit, order=order, context=context, count=count) + #offset, limit and count must be treated separately as we may need to deal with virtual ids + res = super(calendar_event, self).search(cr, uid, new_args, offset=0, limit=0, order=order, context=context, count=False) + if context.get('virtual_id', True): + res = self.get_recurrent_ids(cr, uid, res, new_args, limit, context=context) + if count: + return len(res) + elif limit: + return res[offset:offset+limit] + return res def _get_data(self, cr, uid, id, context=None): return self.read(cr, uid, id,['date', 'date_deadline']) From 7b206d300458dd6c1b77f75e4af89e8ae593fb63 Mon Sep 17 00:00:00 2001 From: "Quentin (OpenERP)" Date: Wed, 12 Dec 2012 18:45:02 +0100 Subject: [PATCH 12/12] [FIX] base_calendar: opening a virtual event with the non-admin user was crashing bzr revid: qdp-launchpad@openerp.com-20121212174502-qj74lmso3a2yjpbx --- addons/base_calendar/crm_meeting.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/addons/base_calendar/crm_meeting.py b/addons/base_calendar/crm_meeting.py index 4745d6c5896..e9fe70f19ff 100644 --- a/addons/base_calendar/crm_meeting.py +++ b/addons/base_calendar/crm_meeting.py @@ -142,6 +142,12 @@ class mail_message(osv.osv): args[index][2] = get_real_ids(args[index][2]) return super(mail_message, self).search(cr, uid, args, offset=offset, limit=limit, order=order, context=context, count=count) + def _find_allowed_model_wise(self, cr, uid, doc_model, doc_dict, context=None): + if doc_model == 'crm.meeting': + for virtual_id in self.pool.get(doc_model).get_recurrent_ids(cr, uid, doc_dict.keys(), [], context=context): + doc_dict.setdefault(virtual_id, doc_dict[get_real_ids(virtual_id)]) + return super(mail_message, self)._find_allowed_model_wise(cr, uid, doc_model, doc_dict, context=context) + class ir_attachment(osv.osv): _inherit = "ir.attachment"