← Back to team overview

openerp-dev-web team mailing list archive

lp:~openerp-dev/openobject-addons/addons-training-fix-readonly-task-2125-ysa into lp:~openobject-training/openobject-addons/training

 

Yogesh(Open ERP) has proposed merging lp:~openerp-dev/openobject-addons/addons-training-fix-readonly-task-2125-ysa into lp:~openobject-training/openobject-addons/training.

Requested reviews:
  OpenObject Training (openobject-training)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/addons-training-fix-readonly-task-2125-ysa/+merge/55871

training, training_room :- remove readonly attribute in stackholder requests object and fix return value of default get method in training.seances object. task id :- 2125
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/addons-training-fix-readonly-task-2125-ysa/+merge/55871
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/addons-training-fix-readonly-task-2125-ysa.
=== modified file 'training/training.py'
--- training/training.py	2011-03-30 12:59:07 +0000
+++ training/training.py	2011-04-01 07:32:28 +0000
@@ -3878,15 +3878,15 @@
         return res
 
     _columns = {
-        'reference': fields.char('Reference', size=16, readonly=True, required=True),
-        'session_id': fields.many2one('training.session', 'Session', select=1, required=True, readonly=True),
+        'reference': fields.char('Reference', size=16, required=True),
+        'session_id': fields.many2one('training.session', 'Session', required=True),
         'date': fields.function(_date_compute, method=True, string='First Seance Date', type='datetime',
                                 store={
                                     'training.participation.stakeholder': (_store_get_requests, None, 25),
                                 }),
-        'kind' : fields.related('session_id', 'offer_id', 'kind', type='char', readonly=True, select=1, string="Offer's Kind"),
-        'job_id': fields.many2one('res.partner.job', 'Contact', required=True, select=1, **WRITABLE_ONLY_IN_DRAFT),
-        'email' : fields.char('Email', size=128, select=1, **WRITABLE_ONLY_IN_DRAFT),
+        'kind' : fields.related('session_id', 'offer_id', 'kind', type='char', string="Offer's Kind"),
+        'job_id': fields.many2one('res.partner.job', 'Contact', required=True, **WRITABLE_ONLY_IN_DRAFT),
+        'email' : fields.char('Email', size=128, **WRITABLE_ONLY_IN_DRAFT),
         'payment_mode' : fields.selection([('contract', 'Contract'),
                                            ('invoice', 'Invoice')
                                           ], 'Payment Mode'),
@@ -3902,7 +3902,7 @@
                                   ], 'State', readonly=True, required=True),
         'purchase_order_id': fields.many2one('purchase.order', 'Purchase Order', readonly=True),
         'purchase_paid': fields.related('purchase_order_id', 'invoice_ids', 'reconciled', type='boolean', string='Invoice Paid', readonly=True,),
-        'amount_to_pay': fields.function(_amount_to_pay, string='Amount to pay', type='char', size='20', readonly=True, method=True),
+        'amount_to_pay': fields.function(_amount_to_pay, string='Amount to pay', type='char', size='20', method=True),
         'price' : fields.function(_price_compute, method=True, string='Remuneration', type='float', digits_compute=dp.get_precision('Account'),
                                   store={
                                       'training.participation.stakeholder': (_store_get_requests, None, 25),

=== modified file 'training_room/training_room.py'
--- training_room/training_room.py	2011-03-21 09:18:22 +0000
+++ training_room/training_room.py	2011-04-01 07:32:28 +0000
@@ -121,7 +121,7 @@
         location = cr.fetchone()
         if not location or (len(location) > 0 and not location[0]):
             location = False
-        return location
+        return location and len(location) and location[0] or False
 
     _defaults = {
         'reserved' : lambda *a: 0,