Issue 6389

Title
Add a test to ensure that all buttons are registered in ir.model.button
Priority
feature
Status
resolved
Nosy list
ced, pokoli, reviewbot, roundup-bot
Assigned to
pokoli
Keywords
review

Created on 2017-03-22.13:36:52 by pokoli, last changed 41 months ago by roundup-bot.

Messages

New changeset e15594edbcc5 by Sergi Almacellas Abellana in branch 'default':
Do not show confirm dialog for buttons with empty value
http://hg.tryton.org/sao/rev/e15594edbcc5
New changeset cb87f4bb161f by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/web_user/rev/cb87f4bb161f
New changeset 22da6f650b18 by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/stock_split/rev/22da6f650b18
New changeset 34a9612c2e6a by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/stock_package_shipping/rev/34a9612c2e6a
New changeset 038ea2ff97ec by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/stock_forecast/rev/038ea2ff97ec
New changeset f5e02ae4fd6d by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/stock/rev/f5e02ae4fd6d
New changeset 7ec6f0035e3c by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/sale_supply_drop_shipment/rev/7ec6f0035e3c
New changeset 543991600421 by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/sale_subscription/rev/543991600421
New changeset 086f805b892e by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/sale_opportunity/rev/086f805b892e
New changeset 1d04c484b6fa by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/sale_complaint/rev/1d04c484b6fa
New changeset 8d9a7f10cf5e by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/sale/rev/8d9a7f10cf5e
New changeset 4de32c8e1568 by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/purchase_requisition/rev/4de32c8e1568
New changeset e6ac7151e40a by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/purchase_request/rev/e6ac7151e40a
New changeset ca15a59c14d1 by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/purchase/rev/ca15a59c14d1
New changeset 62fecb4a4a3f by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/project_invoice/rev/62fecb4a4a3f
New changeset 78e7b71ce9d4 by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/production_work/rev/78e7b71ce9d4
New changeset b1264b80cf81 by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/production_split/rev/b1264b80cf81
New changeset ba3c1c1e4d0b by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/production/rev/ba3c1c1e4d0b
New changeset 242b81644187 by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/product_price_list_dates/rev/242b81644187
New changeset b321fabd7d3c by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/commission/rev/b321fabd7d3c
New changeset c8724dbd90ca by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/account_stock_landed_cost/rev/c8724dbd90ca
New changeset 6586237b5f8a by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/account_statement/rev/6586237b5f8a
New changeset f8805ab5e29d by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/account_payment_stripe/rev/f8805ab5e29d
New changeset 4ed8e029c139 by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/account_payment_sepa/rev/4ed8e029c139
New changeset 1176da9202bd by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/account_payment_clearing/rev/1176da9202bd
New changeset 5a5e84a302d5 by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/account_payment/rev/5a5e84a302d5
New changeset cc3b7e404866 by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/account_invoice_correction/rev/cc3b7e404866
New changeset 5cf3640ae145 by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/account_invoice/rev/5cf3640ae145
New changeset 1a209a178206 by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/account_deposit/rev/1a209a178206
New changeset bfd7f75ca0cb by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/account_asset/rev/bfd7f75ca0cb
New changeset 75f7d4cb0fbf by Sergi Almacellas Abellana in branch 'default':
Register all buttons are registered in ir.model.button
http://hg.tryton.org/modules/account/rev/75f7d4cb0fbf
New changeset 83757dc87808 by Sergi Almacellas Abellana in branch 'default':
Ensure that all buttons are registered in ir.model.button
http://hg.tryton.org/trytond/rev/83757dc87808
Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2017-09-23.23:17:34
> Could you submit a review for trytond only, it will be easier to review.
Here is review36621002
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2017-09-23.23:03:43
Could you submit a review for trytond only, it will be easier to review.
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2017-07-27.22:33:29
Yes I think module should reflect good practice so if we decide that string, help and confirm should be defined in ir.model.button, modules should do that.
Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2017-05-05.11:22:28
I updated the review by adding the new fields on ir.model.button. 

I'm wondering if we should review all the existing buttons, to move this attributes from views to the 'ir.model.button' table
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2017-04-03.14:30:10
On 2017-04-03 13:29, Sergi Almacellas Abellana wrote:
> > I think if we make button registration mandatory, we should use it to store the default label of them on the ir.model.button and put it as default string attribute on the views.
> 
> Then we should also add: 
> 
> - Help
> - confirm
> - Icon (as pyson)
> - States
> - Prevalidate domain
> 
> And probably move the button menu outside of the access group. 
> 
> And if we go this way, I think that we can remove the _buttons in favour of the ir.button table

This would be a bad move because it will make buttons un-extendable.
So only help and confirm seem to be good extra.
Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2017-04-03.13:29:32
> I think if we make button registration mandatory, we should use it to store the default label of them on the ir.model.button and put it as default string attribute on the views.

Then we should also add: 

- Help
- confirm
- Icon (as pyson)
- States
- Prevalidate domain

And probably move the button menu outside of the access group. 

And if we go this way, I think that we can remove the _buttons in favour of the ir.button table
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2017-04-02.15:24:53
I think if we make button registration mandatory, we should use it to store the default label of them on the ir.model.button and put it as default string attribute on the views.
Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2017-03-23.10:33:42
Review is ready for testing
Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2017-03-22.13:36:52
History
Date User Action Args
2018-03-06 16:45:09roundup-botsetmessages: + msg38824
2018-03-01 13:01:37roundup-botsetmessages: + msg38729
2018-03-01 13:01:29roundup-botsetmessages: + msg38728
2018-03-01 13:01:24roundup-botsetmessages: + msg38727
2018-03-01 13:01:18roundup-botsetmessages: + msg38726
2018-03-01 13:01:15roundup-botsetmessages: + msg38725
2018-03-01 13:01:11roundup-botsetmessages: + msg38724
2018-03-01 13:01:08roundup-botsetmessages: + msg38723
2018-03-01 13:01:02roundup-botsetmessages: + msg38722
2018-03-01 13:00:59roundup-botsetmessages: + msg38721

Showing 10 items. Show all history (warning: this could be VERY long)