savoirfairelinux-openerp team mailing list archive

[Jan-Philipp Fischer <jan-philipp.fischer@xxxxxxxxxxxxxx>]

Jan-Philipp Fischer has proposed merging lp:~jan-philipp-fischer/e-commerce-addons/7.0-payment-name-required into lp:e-commerce-addons.

Requested reviews:
  extra-addons-commiter (extra-addons-commiter)

For more details, see:
https://code.launchpad.net/~jan-philipp-fischer/e-commerce-addons/7.0-payment-name-required/+merge/210768

The name field in payment_method should be required to prevent an empty data row.
-- 
https://code.launchpad.net/~jan-philipp-fischer/e-commerce-addons/7.0-payment-name-required/+merge/210768
Your team extra-addons-commiter is requested to review the proposed merge of lp:~jan-philipp-fischer/e-commerce-addons/7.0-payment-name-required into lp:e-commerce-addons.

=== modified file 'sale_payment_method/payment_method.py'
--- sale_payment_method/payment_method.py	2013-12-05 14:46:55 +0000
+++ sale_payment_method/payment_method.py	2014-03-13 10:27:23 +0000
@@ -29,7 +29,8 @@
 
     _columns = {
         'name': fields.char('Name',
-                            help="The name of the method on the backend"),
+                            help="The name of the method on the backend",
+                            required=True),
         'journal_id': fields.many2one(
             'account.journal',
             'Journal',