Merge lp:~openerp-commiter/openobject-addons/trunk-rme into lp:~openerp-dev/openobject-addons/trunk-dev-addons1

Proposed by Ravindra Mekhiya(OpenERP)
Status: Rejected
Rejected by: tfr (Openerp)
Proposed branch: lp:~openerp-commiter/openobject-addons/trunk-rme
Merge into: lp:~openerp-dev/openobject-addons/trunk-dev-addons1
Diff against target: 316 lines (+53/-63)
15 files modified
anonymization/anonymization_view.xml (+2/-2)
auction/auction_view.xml (+1/-1)
base_calendar/base_calendar.py (+1/-1)
base_calendar/base_calendar_view.xml (+3/-0)
crm/crm_lead.py (+1/-1)
crm/crm_lead_view.xml (+24/-3)
crm/crm_opportunity_view.xml (+0/-16)
crm/crm_phonecall_view.xml (+1/-1)
document/security/ir.model.access.csv (+3/-7)
idea/idea.py (+1/-2)
project/project.py (+2/-11)
purchase/report/__init__.py (+1/-0)
resource/resource_view.xml (+8/-14)
subscription/subscription_view.xml (+4/-3)
survey/survey_view.xml (+1/-1)
To merge this branch: bzr merge lp:~openerp-commiter/openobject-addons/trunk-rme
Reviewer Review Type Date Requested Status
tfr (Openerp) Needs Information
Review via email: mp+46593@code.launchpad.net

Description of the change

Hello,
Changes applied in the following modules,

anonymization : usability issue [FIX],
idea : usability issue [FIX],
subscription : usability issue [FIX].

-Also fixes of lp bugs + IMP

-Bug : 718600 - Access rights having problem so that dash board not available in web-client.

-crm.case.stages search view implemented.

Thanks.

To post a comment you must log in.
Revision history for this message
tfr (Openerp) (tfr) wrote :

Can you complete the information about usability issues that are fix ?

review: Needs Information
Revision history for this message
Ravindra Mekhiya(OpenERP) (rme-openerp) wrote :

> Can you complete the information about usability issues that are fix ?

-buttons having wrong name or having wrong icon.
-In list view icon implementation wrong.
-Bit confusing implementation of icon between close & cancel.
-alignment problem.

Let me know if I am wrong !

Thanks.

Revision history for this message
tfr (Openerp) (tfr) wrote :

trunk-dev-addons1 is deprecated, so if the merge still important make it for the trunk

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'anonymization/anonymization_view.xml'
2--- anonymization/anonymization_view.xml 2011-01-07 15:31:06 +0000
3+++ anonymization/anonymization_view.xml 2011-02-16 11:28:13 +0000
4@@ -19,7 +19,7 @@
5 domain="[('model_id','=',model_id), ('ttype', 'not in', ['function', 'binary', 'many2many', 'many2one', 'one2many', 'reference'])]" />
6 <field name="field_name" select="1" on_change="onchange_field_name(field_name, model_name)" />
7 </group>
8- <field name="state" />
9+ <field name="state" string="State"/>
10 </form>
11 </field>
12 </record>
13@@ -34,7 +34,7 @@
14 <field name="model_name" />
15 <field name="field_id" />
16 <field name="field_name" />
17- <field name="state" />
18+ <field name="state" string="State"/>
19 </tree>
20 </field>
21 </record>
22
23=== modified file 'auction/auction_view.xml'
24--- auction/auction_view.xml 2011-01-14 00:11:01 +0000
25+++ auction/auction_view.xml 2011-02-16 11:28:13 +0000
26@@ -588,7 +588,7 @@
27 <field name="model">auction.deposit</field>
28 <field name="type">tree</field>
29 <field name="arch" type="xml">
30- <tree string="">
31+ <tree string="Deposits">
32 <field name="name"/>
33 <field name="partner_id"/>
34 <field name="date_dep"/>
35
36=== modified file 'base_calendar/base_calendar.py'
37--- base_calendar/base_calendar.py 2011-02-14 11:45:07 +0000
38+++ base_calendar/base_calendar.py 2011-02-16 11:28:13 +0000
39@@ -321,7 +321,7 @@
40 _columns = {
41 'cutype': fields.selection([('individual', 'Individual'), \
42 ('group', 'Group'), ('resource', 'Resource'), \
43- ('room', 'Room'), ('unknown', '') ], \
44+ ('room', 'Room'), ('unknown', 'Unknown') ], \
45 'Invite Type', help="Specify the type of Invitation"),
46 'member': fields.char('Member', size=124,
47 help="Indicate the groups that the attendee belongs to"),
48
49=== modified file 'base_calendar/base_calendar_view.xml'
50--- base_calendar/base_calendar_view.xml 2011-01-18 22:35:02 +0000
51+++ base_calendar/base_calendar_view.xml 2011-02-16 11:28:13 +0000
52@@ -78,6 +78,7 @@
53 <field name="sent_by_uid" string="Invitation From" />
54 <field name="role" string="My Role"/>
55 <field name="user_id" invisible="1"/>
56+ <field name="partner_address_id" string="Contact" invisible="1"/>
57 <field name="cutype" string="Invitation type"/>
58 <field name="state" />
59 <field name="rsvp" string="Required to Join"/>
60@@ -286,6 +287,8 @@
61 <field name="attendee_ids" colspan="4"
62 nolabel="1" widget="one2many" mode="tree,form">
63 <tree string="Invitation details" editable="top">
64+ <field name="sent_by_uid" string="From"/>
65+ <field name="user_id" string="To"/>
66 <field name="email" />
67 <field name="role" width="200" select="1" />
68 <field name="state" />
69
70=== modified file 'crm/crm_lead.py'
71--- crm/crm_lead.py 2011-01-27 12:58:29 +0000
72+++ crm/crm_lead.py 2011-02-16 11:28:13 +0000
73@@ -146,7 +146,7 @@
74 'state': fields.selection(crm.AVAILABLE_STATES, 'State', size=16, readonly=True,
75 help='The state is set to \'Draft\', when a case is created.\
76 \nIf the case is in progress the state is set to \'Open\'.\
77- \nWhen the case is over, the state is set to \'Done\'.\
78+ \nWhen the case is over, the state is set to \'Closed\'.\
79 \nIf the case needs to be reviewed then the state is set to \'Pending\'.'),
80 'message_ids': fields.one2many('mailgate.message', 'res_id', 'Messages', domain=[('model','=',_name)]),
81 }
82
83=== modified file 'crm/crm_lead_view.xml'
84--- crm/crm_lead_view.xml 2011-01-14 00:11:01 +0000
85+++ crm/crm_lead_view.xml 2011-02-16 11:28:13 +0000
86@@ -2,15 +2,36 @@
87 <openerp>
88 <data>
89
90- <!--Lead Stage Form view -->
91+
92+
93+
94+ <!-- CRM Satges Search View -->
95+
96+ <record id="crm_case_stage_search" model="ir.ui.view">
97+ <field name="name">stage.search</field>
98+ <field name="model">crm.case.stage</field>
99+ <field name="type">search</field>
100+ <field name="arch" type="xml">
101+ <search string="stage Search">
102+ <filter icon="terp-personal" name="lead"
103+ string="Lead" domain="[('type', '=', 'lead')]"/>
104+ <filter icon="terp-personal+" name="opportunity"
105+ string="Opportunity" domain="[('type', '=', 'opportunity')]"/>
106+ <separator orientation="vertical"/>
107+ <field name="name"/>
108+ </search>
109+ </field>
110+ </record>
111+
112+
113
114 <record id="crm_lead_stage_act" model="ir.actions.act_window">
115 <field name="name">Stages</field>
116 <field name="res_model">crm.case.stage</field>
117 <field name="view_type">form</field>
118 <field name="view_id" ref="crm.crm_case_stage_tree"/>
119- <field name="domain">[('type', '=', 'lead')]</field>
120- <field name="context">{'type':'lead'}</field>
121+ <field name="context">{'type':'lead','search_default_lead':1}</field>
122+ <field name="search_view_id" ref="crm_case_stage_search"/>
123 <field name="help">Add specific stages to leads and opportunities allowing your sales to better organise their sales pipeline. Stages will allow them to easily track how a specific lead or opportunity is positioned in the sales cycle.</field>
124 </record>
125
126
127=== modified file 'crm/crm_opportunity_view.xml'
128--- crm/crm_opportunity_view.xml 2011-01-14 00:11:01 +0000
129+++ crm/crm_opportunity_view.xml 2011-02-16 11:28:13 +0000
130@@ -1,22 +1,6 @@
131 <?xml version="1.0"?>
132 <openerp>
133 <data>
134-
135- <record id="crm_opportunity_stage_act" model="ir.actions.act_window">
136- <field name="name">Stages</field>
137- <field name="res_model">crm.case.stage</field>
138- <field name="view_type">form</field>
139- <field name="view_id" ref="crm.crm_case_stage_tree"/>
140- <field name="domain">[('type', '=', 'opportunity')]</field>
141- <field name="context">{'type':'opportunity'}</field>
142- <field name="help">Create specific stages that will help your sales better organise their sales pipeline by maintaining them to their sales opportunities. It will allow them to easily track how is positioned a specific opportunity in the sales cycle.</field>
143- </record>
144-
145-
146-
147- <menuitem action="crm_opportunity_stage_act" id="menu_crm_opportunity_stage_act" name="Stages"
148- groups="base.group_extended" sequence="0"
149- parent="base.menu_crm_config_opportunity" />
150
151 <!-- Opportunities Form View -->
152 <record model="ir.ui.view" id="crm_case_form_view_oppor">
153
154=== modified file 'crm/crm_phonecall_view.xml'
155--- crm/crm_phonecall_view.xml 2011-01-17 22:22:19 +0000
156+++ crm/crm_phonecall_view.xml 2011-02-16 11:28:13 +0000
157@@ -60,7 +60,7 @@
158 <form string="Phone Call">
159 <group colspan="6" col="7">
160 <field name="name" required="1"/>
161- <field name="partner_phone"/>
162+ <field name="partner_phone" required="1"/>
163 <field name="duration" widget="float_time" required="1"/>
164 <button string="Schedule a Meeting" name="action_make_meeting" icon="gtk-redo" type="object"/>
165
166
167=== modified file 'document/security/ir.model.access.csv'
168--- document/security/ir.model.access.csv 2011-01-14 00:11:01 +0000
169+++ document/security/ir.model.access.csv 2011-02-16 11:28:13 +0000
170@@ -2,10 +2,8 @@
171 "access_document_directory_all","document.directory all","model_document_directory",,1,0,0,0
172 "access_document_directory_group_document_manager","document.directory document manager","model_document_directory","base.group_system",1,1,1,1
173 "access_document_directory_group_knowledge","document.directory modify","model_document_directory","base.group_document_user",1,1,1,1
174-"access_document_directory_group_system","document.directory group system","model_document_directory","base.group_system",1,1,1,1
175 "access_document_directory_content_all","document.directory.content all","model_document_directory_content",,1,0,0,0
176 "access_document_directory_content_group_document_manager","document.directory.content document manager","model_document_directory_content","base.group_system",1,1,1,1
177-"access_document_directory_content_group_system","document.directory.content group system","model_document_directory_content","base.group_system",1,1,1,1
178 "access_document_directory_content_type_group_document_manager","document.directory.content.type document manager","model_document_directory_content_type","base.group_system",1,1,1,1
179 "access_document_directory_content_type_group_system","document.directory.content.type group system","model_document_directory_content_type","base.group_user",1,0,0,0
180 "access_document_storage_group_document_manager","document.storage document manager","model_document_storage","base.group_system",1,1,1,1
181@@ -16,13 +14,11 @@
182 "access_res_partner_group_user","res.partner user","base.model_res_partner","base.group_document_user",1,1,1,1
183 "access_document_directory_dctx_all","document.directory.dctx all","model_document_directory_dctx",,1,0,0,0
184 "access_document_directory_dctx_group_document_manager","document.directory.dctx document manager","model_document_directory_dctx","base.group_system",1,1,1,1
185-"access_document_directory_dctx_group_system","document.directory.dctx group system","model_document_directory_dctx","base.group_system",1,1,1,1
186 "access_report_document_user_group_document_manager","report.document.user document manager","model_report_document_user","base.group_system",1,0,0,0
187-"access_report_document_user_group_system","report.document.user group system","model_report_document_user","base.group_system",1,0,0,0
188 "access_report_files_partner_group_document_manager","report.files.partner document manager","model_report_files_partner","base.group_system",1,0,0,0
189-"access_report_files_partner_group_system","report.files.partner group system","model_report_files_partner","base.group_system",1,0,0,0
190 "access_report_document_file_group_document_manager","report.document.file document manager","model_report_document_file","base.group_system",1,0,0,0
191-"access_report_document_file_group_system","report.document.file group system","model_report_document_file","base.group_system",1,0,0,0
192 "access_report_document_wall_group_document_manager","report.document.wall document manager","model_report_document_wall","base.group_system",1,0,0,0
193-"access_report_document_wall_group_system","report.document.wall group system","model_report_document_wall","base.group_system",1,0,0,0
194+"access_report_files_partner_group_document","report.files.partner document manager","model_report_files_partner","base.group_document_user",1,0,0,0
195+"access_report_document_file_group_document","report.document.file document manager","model_report_document_file","base.group_document_user",1,0,0,0
196+"access_report_document_wall_group_document","report.document.wall document manager","model_report_document_wall","base.group_document_user",1,0,0,0
197 "access_report_document_user_knowledgeuser","report.document.user knowledgeuser","document.model_report_document_user","base.group_document_user",1,0,0,0
198
199=== modified file 'idea/idea.py'
200--- idea/idea.py 2011-01-14 00:11:01 +0000
201+++ idea/idea.py 2011-02-16 11:28:13 +0000
202@@ -169,8 +169,7 @@
203 ('close', 'Accepted'),
204 ('cancel', 'Refused')],
205 'State', readonly=True,
206- help='When the Idea is created the state is \'Draft\'.\n It is \
207- opened by the user, the state is \'Opened\'.\
208+ help='When the Idea is created the state is \'Draft\'.\nIt is opened by the user, the state is \'Opened\'.\
209 \nIf the idea is accepted, the state is \'Accepted\'.'
210 ),
211 'visibility':fields.boolean('Open Idea?', required=False),
212
213=== modified file 'project/project.py'
214--- project/project.py 2011-01-18 22:45:39 +0000
215+++ project/project.py 2011-02-16 11:28:13 +0000
216@@ -134,22 +134,13 @@
217 'project.project': (lambda self, cr, uid, ids, c={}: ids, ['tasks'], 10),
218 'project.task': (_get_project_task, ['planned_hours', 'effective_hours', 'remaining_hours', 'total_hours', 'progress', 'delay_hours','state'], 10),
219 }),
220- 'effective_hours': fields.function(_progress_rate, multi="progress", method=True, string='Time Spent', help="Sum of spent hours of all tasks related to this project and its child projects.",
221- store = {
222- 'project.project': (lambda self, cr, uid, ids, c={}: ids, ['tasks'], 10),
223- 'project.task': (_get_project_task, ['planned_hours', 'effective_hours', 'remaining_hours', 'total_hours', 'progress', 'delay_hours','state'], 10),
224- }),
225+ 'effective_hours': fields.function(_progress_rate, multi="progress", method=True, string='Time Spent', help="Sum of spent hours of all tasks related to this project and its child projects."),
226 'total_hours': fields.function(_progress_rate, multi="progress", method=True, string='Total Time', help="Sum of total hours of all tasks related to this project and its child projects.",
227 store = {
228 'project.project': (lambda self, cr, uid, ids, c={}: ids, ['tasks'], 10),
229 'project.task': (_get_project_task, ['planned_hours', 'effective_hours', 'remaining_hours', 'total_hours', 'progress', 'delay_hours','state'], 10),
230 }),
231- 'progress_rate': fields.function(_progress_rate, multi="progress", method=True, string='Progress', type='float', group_operator="avg", help="Percent of tasks closed according to the total of tasks todo.",
232- store = {
233- 'project.project': (lambda self, cr, uid, ids, c={}: ids, ['tasks'], 10),
234- 'project.task': (_get_project_task, ['planned_hours', 'effective_hours', 'remaining_hours', 'total_hours', 'progress', 'delay_hours','state'], 10),
235- 'project.task.work': (_get_project_work, ['hours'], 10),
236- }),
237+ 'progress_rate': fields.function(_progress_rate, multi="progress", method=True, string='Progress', type='float', group_operator="avg", help="Percent of tasks closed according to the total of tasks todo."),
238 'warn_customer': fields.boolean('Warn Partner', help="If you check this, the user will have a popup when closing a task that propose a message to send by email to the customer.", states={'close':[('readonly',True)], 'cancelled':[('readonly',True)]}),
239 'warn_header': fields.text('Mail Header', help="Header added at the beginning of the email for the warning message sent to the customer when a task is closed.", states={'close':[('readonly',True)], 'cancelled':[('readonly',True)]}),
240 'warn_footer': fields.text('Mail Footer', help="Footer added at the beginning of the email for the warning message sent to the customer when a task is closed.", states={'close':[('readonly',True)], 'cancelled':[('readonly',True)]}),
241
242=== modified file 'purchase/report/__init__.py'
243--- purchase/report/__init__.py 2011-01-14 00:11:01 +0000
244+++ purchase/report/__init__.py 2011-02-16 11:28:13 +0000
245@@ -22,6 +22,7 @@
246 import order
247 import request_quotation
248 import purchase_report
249+import test
250
251 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
252
253
254=== modified file 'resource/resource_view.xml'
255--- resource/resource_view.xml 2011-01-14 00:11:01 +0000
256+++ resource/resource_view.xml 2011-02-16 11:28:13 +0000
257@@ -218,20 +218,14 @@
258 <field name="type">form</field>
259 <field name="arch" type="xml">
260 <form string="Leave Detail">
261- <group colspan="4" col="6">
262- <group colspan="2" col="2">
263- <field name="resource_id" on_change="onchange_resource(resource_id)"/>
264- <field name="name" string="Reason"/>
265- </group>
266- <group colspan="2" col="2">
267- <field name="date_from"/>
268- <field name="calendar_id" string="Working Period" readonly='1'/>
269- <field name="company_id" widget="selection" groups="base.group_multi_company"/>
270- </group>
271- <group colspan="2" col="2">
272- <field name="date_to"/>
273- </group>
274- </group>
275+ <field name="resource_id" on_change="onchange_resource(resource_id)"/>
276+ <field name="name" string="Reason"/>
277+ <newline/>
278+ <field name="date_from"/>
279+ <field name="date_to" />
280+ <newline/>
281+ <field name="calendar_id" string="Working Period" readonly='1'/>
282+ <field name="company_id" widget="selection" groups="base.group_multi_company" />
283 </form>
284 </field>
285 </record>
286
287=== modified file 'subscription/subscription_view.xml'
288--- subscription/subscription_view.xml 2011-01-14 00:11:01 +0000
289+++ subscription/subscription_view.xml 2011-02-16 11:28:13 +0000
290@@ -62,9 +62,10 @@
291 <field name="active" groups="base.group_extended"/>
292 <field name="user_id" select="1"/>
293 <field name="state"/>
294- <button name="set_process" states="draft" string="Process" type="object" icon="terp-check"/>
295- <button name="set_done" states="running" string="Done" type="object" icon="terp-gtk-stop"/>
296- <button name="set_draft" states="done" string="Set to Draft" type="object" icon="terp-document-new"/>
297+ <button name="set_process" states="draft" string="Process" type="object" icon="gtk-go-forward"/>
298+ <button name="set_done" states="running" string="Stop" type="object" icon="gtk-stop"/>
299+ <button name="set_draft" states="done" string="Set to Draft" type="object" icon="gtk-convert"/>
300+
301 </tree>
302 </field>
303 </record>
304
305=== modified file 'survey/survey_view.xml'
306--- survey/survey_view.xml 2011-01-14 00:11:01 +0000
307+++ survey/survey_view.xml 2011-02-16 11:28:13 +0000
308@@ -1026,7 +1026,7 @@
309 <field name="in_visible_menu_choice" invisible="1"/>
310 <group colspan="4" attrs="{'invisible':[('in_visible_rating_weight','!=',False)]}">
311 <field name="rating_weight"/>
312- <label align="0.0" string="Use if question type is rating_scale" />
313+ <label align="0.0" string="Use if question type is rating scale" />
314 </group>
315 <group colspan="4" attrs="{'invisible':[('in_visible_menu_choice','!=',False)]}">
316 <separator string="Menu Choices (each choice on separate lines)" colspan="4"/>

Subscribers

People subscribed via source and target branches