Merge lp:~openerp-dev/openobject-addons/acct_prj_hr_logs into lp:~openerp-dev/openobject-addons/trunk-dev-addons3

[pap(openerp)]

added logs for account, account_*, hr, hr_* and project modules

[Mustufa Rangwala (Open ERP) (mra-tinyerp)]

review: Needs Fixing

Unmerged revisions

3645. By pap(openerp) on 2010-06-21

merge

3644. By pap(openerp) on 2010-06-21

[ADD]:log messages for account, hr, project

3643. By pap(openerp) on 2010-06-18

merge

3642. By pap(openerp) on 2010-06-18

merge

3641. By pap(openerp) on 2010-06-18

[IMP]:if there is no invoice line inovice should not be created

3640. By pap(openerp) on 2010-06-18

merge

3639. By pap(openerp) on 2010-06-17

[IMP]:
1) added missing wizards under Financial Management/Periodical Processing
2) added entry_type on move line for different types of invoices

3638. By Mustufa Rangwala <mra@mra-laptop> on 2010-06-17

[MERGE]

3637. By Mustufa Rangwala <mra@mra-laptop> on 2010-06-17

[MERGE]

3636. By Mustufa Rangwala <mra@mra-laptop> on 2010-06-17

[FIX] Purchase: calculation of amount untax was wrong (ref: mtr)

=== modified file 'account/account.py'
--- account/account.py	2010-06-18 13:55:20 +0000
+++ account/account.py	2010-06-21 08:36:32 +0000
@@ -1041,7 +1041,8 @@
     ]
     def post(self, cr, uid, ids, context=None):
         if self.validate(cr, uid, ids, context) and len(ids):
-            for move in self.browse(cr, uid, ids):
+            moves = self.browse(cr, uid, ids)
+            for move in moves:
                 if move.name =='/':
                     new_name = False
                     journal = move.journal_id
@@ -1052,11 +1053,14 @@
                         raise osv.except_osv(_('Error'), _('No sequence defined in the journal !'))
                     if new_name:
                         self.write(cr, uid, [move.id], {'name':new_name})
-
             cr.execute('UPDATE account_move '\
                        'SET state=%s '\
                        'WHERE id IN %s',
                        ('posted', tuple(ids),))
+            moves = self.browse(cr, uid, ids)
+            for move in moves:
+                msg = _("Move %s validated"%(move.name)) 
+                self.log(cr, uid, move.id, msg)
         else:
             raise osv.except_osv(_('Integrity Error !'), _('You can not validate a non-balanced entry !\nMake sure you have configured Payment Term properly !\nIt should contain atleast one Payment Term Line with type "Balance" !'))
         return True
@@ -1995,6 +1999,7 @@
     _order = "code"
     _name = "account.account.template"
     _description ='Templates for Accounts'
+    _log_create=True
 
     _columns = {
         'name': fields.char('Name', size=128, required=True, select=True),
@@ -2150,6 +2155,7 @@
 class account_chart_template(osv.osv):
     _name="account.chart.template"
     _description= "Templates for Account Chart"
+    _log_create=True
 
     _columns={
         'name': fields.char('Name', size=64, required=True),
@@ -2171,6 +2177,7 @@
 
     _name = 'account.tax.template'
     _description = 'Templates for Taxes'
+    _log_create=True
 
     _columns = {
         'chart_template_id': fields.many2one('account.chart.template', 'Chart Template', required=True),
@@ -2248,6 +2255,7 @@
 class account_fiscal_position_template(osv.osv):
     _name = 'account.fiscal.position.template'
     _description = 'Template for Fiscal Position'
+    _log_create=True
 
     _columns = {
         'name': fields.char('Fiscal Position Template', size=64, translate=True, required=True),
@@ -2262,6 +2270,7 @@
     _name = 'account.fiscal.position.tax.template'
     _description = 'Fiscal Position Template Tax Mapping'
     _rec_name = 'position_id'
+    _log_create=True
 
     _columns = {
         'position_id': fields.many2one('account.fiscal.position.template', 'Fiscal Position', required=True, ondelete='cascade'),
@@ -2275,6 +2284,8 @@
     _name = 'account.fiscal.position.account.template'
     _description = 'Fiscal Position Template Account Mapping'
     _rec_name = 'position_id'
+    _log_create=True
+    
     _columns = {
         'position_id': fields.many2one('account.fiscal.position.template', 'Fiscal Position', required=True, ondelete='cascade'),
         'account_src_id': fields.many2one('account.account.template', 'Account Source', domain=[('type','<>','view')], required=True),

=== modified file 'account/account_bank_statement.py'
--- account/account_bank_statement.py	2010-06-16 11:51:39 +0000
+++ account/account_bank_statement.py	2010-06-21 08:36:32 +0000
@@ -122,6 +122,7 @@
     _order = "date desc"
     _name = "account.bank.statement"
     _description = "Bank Statement"
+    _log_create = True
     _columns = {
         'name': fields.char('Name', size=64, required=True, states={'confirm': [('readonly', True)]}),
         'date': fields.date('Date', required=True,
@@ -323,6 +324,9 @@
                     account_move_obj.write(cr, uid, [move_id], {'state':'posted'})
             done.append(st.id)
         self.write(cr, uid, done, {'state':'confirm'}, context=context)
+        for bnk_st in self.browse(cr, uid ,done):
+            msg=_("Bank Statement")+"  '"+bnk_st.name+"'  "+_("confirmed")
+            self.log(cr, uid, bnk_st.id, msg)
         return True
 
     def button_cancel(self, cr, uid, ids, context={}):

=== modified file 'account/account_view.xml'
--- account/account_view.xml	2010-06-18 11:15:59 +0000
+++ account/account_view.xml	2010-06-21 08:36:32 +0000
@@ -1041,7 +1041,7 @@
             <field name="view_mode">tree,form</field>
             <field name="view_id" ref="view_move_tree"/>
             <field name="search_view_id" ref="view_account_move_filter"/>
-            <field name="domain">[('journal_id.type', '=', 'sale')]</field>
+            <field name="domain">[('journal_id.type', 'in', ['sale', 'sale_refund'])]</field>
         </record>
 
         <menuitem action="action_account_moves_sale" id="menu_eaction_account_moves_sale" parent="menu_finance_receivables"/>
@@ -1053,7 +1053,7 @@
             <field name="view_mode">tree,form</field>
             <field name="view_id" ref="view_move_tree"/>
             <field name="search_view_id" ref="view_account_move_filter"/>
-            <field name="domain">[('journal_id.type', '=', 'purchase')]</field>
+            <field name="domain">[('journal_id.type', 'in', ['purchase', 'purchase_refund'])]</field>
         </record>
 
         <menuitem action="action_account_moves_purchase" id="menu_eaction_account_moves_purchase" parent="menu_finance_payables"/>

=== modified file 'account/invoice.py'
--- account/invoice.py	2010-06-18 13:55:20 +0000
+++ account/invoice.py	2010-06-21 08:36:32 +0000
@@ -753,6 +753,8 @@
         cur_obj = self.pool.get('res.currency')
         context = {}
         for inv in self.browse(cr, uid, ids):
+            if not inv.invoice_line:
+                raise osv.except_osv(_('No Invoice Lines !'), _('Please create some invoice lines.'))
             if inv.move_id:
                 continue
 
@@ -774,7 +776,8 @@
 
             # one move line per tax line
             iml += ait_obj.move_line_get(cr, uid, inv.id)
-
+            
+            entry_type=''
             if inv.type in ('in_invoice', 'in_refund'):
                 ref = inv.reference
                 entry_type = 'journal_pur_voucher'
@@ -1090,6 +1093,9 @@
             entry_type = 'cont_voucher'
         if invoice.type in ('in_invoice', 'in_refund'):
             ref = invoice.reference
+            entry_type = 'journal_pur_voucher'
+            if invoice.type in ('in_refund'):
+                entry_type = 'cont_voucher'
         else:
             ref = self._convert_ref(cr, uid, invoice.number)
 

=== modified file 'account/wizard/account_invoice_refund.py'
--- account/wizard/account_invoice_refund.py	2010-06-16 11:51:39 +0000
+++ account/wizard/account_invoice_refund.py	2010-06-21 08:36:32 +0000
@@ -30,6 +30,7 @@
 
     _name = "account.invoice.refund"
     _description = "Invoice Refund"
+    _log_create=True
     _columns = {
        'date': fields.date('Operation date', help='This date will be used as the invoice date for Refund Invoice and Period will be chosen accordingly!'),
        'period': fields.many2one('account.period', 'Force period'),

=== modified file 'account/wizard/account_reconcile_view.xml'
--- account/wizard/account_reconcile_view.xml	2010-06-16 11:41:41 +0000
+++ account/wizard/account_reconcile_view.xml	2010-06-21 08:36:32 +0000
@@ -2,86 +2,130 @@
 <openerp>
     <data>
 
-		<record id="view_account_move_line_reconcile_prompt" model="ir.ui.view">
-            <field name="name">account.move.line.reconcile.prompt.form</field>
-            <field name="model">account.move.line.reconcile.prompt</field>
+<!--		<record id="view_account_move_line_reconcile_prompt" model="ir.ui.view">-->
+<!--            <field name="name">account.move.line.reconcile.prompt.form</field>-->
+<!--            <field name="model">account.move.line.reconcile.prompt</field>-->
+<!--            <field name="type">form</field>-->
+<!--            <field name="arch" type="xml">-->
+<!--                <form string="Reconciliation">-->
+<!--                	<separator string="Are you sure you want to reconcile entries?" colspan="4"/>-->
+<!--				    <group colspan="4" col="6">-->
+<!--                       	<label string ="" colspan="2"/>-->
+<!---->
+<!--                    	<button icon="gtk-cancel" special="cancel" string="Cancel"/>-->
+<!--                    	<button icon="gtk-ok" string="Ok" name="ask_reconcilation" type="object" default_focus="1"/>-->
+<!--                   </group>-->
+<!--               </form>-->
+<!--            </field>-->
+<!--        </record>-->
+
+<!--		<record id="action_view_account_move_line_reconcile_prompt" model="ir.actions.act_window">-->
+<!--        	<field name="name">Reconcile Entries</field>-->
+<!--            <field name="res_model">account.move.line.reconcile.prompt</field>-->
+<!--            <field name="view_type">form</field>-->
+<!--            <field name="view_mode">tree,form</field>-->
+<!--           	<field name="view_id" ref="view_account_move_line_reconcile_prompt"/>-->
+<!--           	<field name="target">new</field>-->
+<!--        </record>-->
+
+<!--        <record model="ir.values" id="action_account_move_line_reconcile_prompt_values">-->
+<!--            <field name="model_id" ref="account.model_account_move_line" />-->
+<!--            <field name="object" eval="1" />-->
+<!--            <field name="name">Reconcile Entries</field>-->
+<!--            <field name="key2">client_action_multi</field>-->
+<!--            <field name="value" eval="'ir.actions.act_window,' +str(ref('action_view_account_move_line_reconcile_prompt'))" />-->
+<!--            <field name="key">action</field>-->
+<!--            <field name="model">account.move.line</field>-->
+<!--		</record>-->
+		
+		<record id="view_account_move_line_reconcile_full" model="ir.ui.view">
+            <field name="name">account.move.line.reconcile.full.form</field>
+            <field name="model">account.move.line.reconcile</field>
             <field name="type">form</field>
             <field name="arch" type="xml">
                 <form string="Reconciliation">
-                	<separator string="Are you sure you want to reconcile entries?" colspan="4"/>
-				    <group colspan="4" col="6">
-                       	<label string ="" colspan="2"/>
-
-                    	<button icon="gtk-cancel" special="cancel" string="Cancel"/>
-                    	<button icon="gtk-ok" string="Ok" name="ask_reconcilation" type="object" default_focus="1"/>
-                   </group>
+                	<group height="210" width="550">
+	                	<separator string="Reconciliation transactions" colspan="4"/>
+					    <field name="trans_nbr"/>
+					    <newline/>
+					    <field name="credit"/>
+					    <field name="debit"/>
+					    <separator string="Write-Off" colspan="4"/>
+					    <field name="writeoff"/>
+					    <group colspan="4" col="6">
+	                       	<label string ="" colspan="2"/>
+	                    	<button icon="gtk-cancel" special="cancel" string="Cancel"/>
+	                    	<button icon="terp-stock_effects-object-colorize" string="Reconcile" name="trans_rec_reconcile_full" type="object" default_focus="1" attrs="{'invisible':[('writeoff','!=',0)]}"/>
+	                    	<button icon="gtk-ok" string="Reconcile With Write-Off" name="trans_rec_addendum_writeoff" type="object" attrs="{'invisible':[('writeoff','==',0)]}"/>
+	                    	<button icon="gtk-ok" string="Partial Reconcile" name="trans_rec_reconcile_partial_reconcile" type="object" attrs="{'invisible':[('writeoff','==',0)]}"/>
+	                   </group>
+	                </group>
                </form>
             </field>
         </record>
-
-		<record id="action_view_account_move_line_reconcile_prompt" model="ir.actions.act_window">
+		
+		<record id="action_view_account_move_line_reconcile" model="ir.actions.act_window">
         	<field name="name">Reconcile Entries</field>
-            <field name="res_model">account.move.line.reconcile.prompt</field>
+            <field name="res_model">account.move.line.reconcile</field>
             <field name="view_type">form</field>
             <field name="view_mode">tree,form</field>
-           	<field name="view_id" ref="view_account_move_line_reconcile_prompt"/>
+           	<field name="view_id" ref="view_account_move_line_reconcile_full"/>
            	<field name="target">new</field>
         </record>
-
+        
         <record model="ir.values" id="action_account_move_line_reconcile_prompt_values">
             <field name="model_id" ref="account.model_account_move_line" />
             <field name="object" eval="1" />
             <field name="name">Reconcile Entries</field>
             <field name="key2">client_action_multi</field>
-            <field name="value" eval="'ir.actions.act_window,' +str(ref('action_view_account_move_line_reconcile_prompt'))" />
+            <field name="value" eval="'ir.actions.act_window,' +str(ref('action_view_account_move_line_reconcile'))" />
             <field name="key">action</field>
             <field name="model">account.move.line</field>
 		</record>
-
-    	<record id="view_account_move_line_reconcile_full" model="ir.ui.view">
-            <field name="name">account.move.line.reconcile.full.form</field>
-            <field name="model">account.move.line.reconcile</field>
-            <field name="type">form</field>
-            <field name="arch" type="xml">
-                <form string="Reconciliation">
-                	<separator string="Reconciliation transactions" colspan="4"/>
-				    <field name="trans_nbr"/>
-				    <newline/>
-				    <field name="credit"/>
-				    <field name="debit"/>
-				    <separator string="Write-Off" colspan="4"/>
-				    <field name="writeoff"/>
-				    <group colspan="4" col="6">
-                       	<label string ="" colspan="2"/>
-                    	<button icon="gtk-cancel" special="cancel" string="Cancel"/>
-                    	<button icon="terp-stock_effects-object-colorize" string="Reconcile" name="trans_rec_reconcile_full" type="object" default_focus="1"/>
-                   </group>
-               </form>
-            </field>
-        </record>
-
-		<record id="view_account_move_line_reconcile_partial" model="ir.ui.view">
-            <field name="name">account.move.line.reconcile.partial.form</field>
-            <field name="model">account.move.line.reconcile</field>
-            <field name="type">form</field>
-            <field name="arch" type="xml">
-                <form string="Reconciliation">
-                	<separator string="Reconciliation transactions" colspan="4"/>
-				    <field name="trans_nbr"/>
-				    <newline/>
-				    <field name="credit"/>
-				    <field name="debit"/>
-				    <separator string="Write-Off" colspan="4"/>
-				    <field name="writeoff"/>
-				    <group colspan="4" col="6">
-                       	<label string ="" colspan="2"/>
-                    	<button icon="gtk-cancel" special="cancel" string="Cancel"/>
-                    	<button icon="gtk-ok" string="Reconcile With Write-Off" name="trans_rec_addendum_writeoff" type="object" default_focus="1"/>
-                    	<button icon="gtk-ok" string="Partial Reconcile" name="trans_rec_reconcile_partial_reconcile" type="object"/>
-                   </group>
-               </form>
-            </field>
-        </record>
+<!--    	<record id="view_account_move_line_reconcile_full" model="ir.ui.view">-->
+<!--            <field name="name">account.move.line.reconcile.full.form</field>-->
+<!--            <field name="model">account.move.line.reconcile</field>-->
+<!--            <field name="type">form</field>-->
+<!--            <field name="arch" type="xml">-->
+<!--                <form string="Reconciliation">-->
+<!--                	<separator string="Reconciliation transactions" colspan="4"/>-->
+<!--				    <field name="trans_nbr"/>-->
+<!--				    <newline/>-->
+<!--				    <field name="credit"/>-->
+<!--				    <field name="debit"/>-->
+<!--				    <separator string="Write-Off" colspan="4"/>-->
+<!--				    <field name="writeoff"/>-->
+<!--				    <group colspan="4" col="6">-->
+<!--                       	<label string ="" colspan="2"/>-->
+<!--                    	<button icon="gtk-cancel" special="cancel" string="Cancel"/>-->
+<!--                    	<button icon="terp-stock_effects-object-colorize" string="Reconcile" name="trans_rec_reconcile_full" type="object" default_focus="1"/>-->
+<!--                   </group>-->
+<!--               </form>-->
+<!--            </field>-->
+<!--        </record>-->
+<!---->
+<!--		<record id="view_account_move_line_reconcile_partial" model="ir.ui.view">-->
+<!--            <field name="name">account.move.line.reconcile.partial.form</field>-->
+<!--            <field name="model">account.move.line.reconcile</field>-->
+<!--            <field name="type">form</field>-->
+<!--            <field name="arch" type="xml">-->
+<!--                <form string="Reconciliation">-->
+<!--                	<separator string="Reconciliation transactions" colspan="4"/>-->
+<!--				    <field name="trans_nbr"/>-->
+<!--				    <newline/>-->
+<!--				    <field name="credit"/>-->
+<!--				    <field name="debit"/>-->
+<!--				    <separator string="Write-Off" colspan="4"/>-->
+<!--				    <field name="writeoff"/>-->
+<!--				    <group colspan="4" col="6">-->
+<!--                       	<label string ="" colspan="2"/>-->
+<!--                    	<button icon="gtk-cancel" special="cancel" string="Cancel"/>-->
+<!--                    	<button icon="gtk-ok" string="Reconcile With Write-Off" name="trans_rec_addendum_writeoff" type="object" default_focus="1"/>-->
+<!--                    	<button icon="gtk-ok" string="Partial Reconcile" name="trans_rec_reconcile_partial_reconcile" type="object"/>-->
+<!--                   </group>-->
+<!--               </form>-->
+<!--            </field>-->
+<!--        </record>-->
 
 		<record id="account_move_line_reconcile_writeoff" model="ir.ui.view">
             <field name="name">account.move.line.reconcile.writeoff.form</field>

=== modified file 'account_analytic_plans/account_analytic_plans.py'
--- account_analytic_plans/account_analytic_plans.py	2010-06-18 08:38:59 +0000
+++ account_analytic_plans/account_analytic_plans.py	2010-06-21 08:36:32 +0000
@@ -55,6 +55,8 @@
 class account_analytic_plan(osv.osv):
     _name = "account.analytic.plan"
     _description = "Analytic Plan"
+    _log_create = True
+    
     _columns = {
         'name': fields.char('Analytic Plan', size=64, required=True, select=True,),
         'plan_ids': fields.one2many('account.analytic.plan.line','plan_id','Analytic Plans'),

=== modified file 'account_budget/crossovered_budget.py'
--- account_budget/crossovered_budget.py	2010-06-16 11:51:39 +0000
+++ account_budget/crossovered_budget.py	2010-06-21 08:36:32 +0000
@@ -110,6 +110,7 @@
 class crossovered_budget(osv.osv):
     _name = "crossovered.budget"
     _description = "Budget"
+    _log_create = True
 
     _columns = {
         'name': fields.char('Name', size=50, required=True,states={'done':[('readonly',True)]}),
@@ -138,6 +139,9 @@
         self.write(cr, uid, ids, {
             'state':'confirm'
         })
+        for bdgt in self.browse(cr, uid, ids):
+            msg = _("Budget")+"  '"+bdgt.name+"'  "+_("confirmed")
+            self.log(cr, uid, bdgt.id, msg)
         return True
 
     def budget_validate(self, cr, uid, ids, *args):
@@ -145,6 +149,9 @@
             'state':'validate',
             'validating_user_id': uid,
         })
+        for bdgt in self.browse(cr, uid, ids):
+            msg = _("Budget")+"  '"+bdgt.name+"'  "+_("validated")
+            self.log(cr, uid, bdgt.id, msg)
         return True
 
     def budget_cancel(self, cr, uid, ids, *args):

=== modified file 'account_payment/account_payment.py'
--- account_payment/account_payment.py	2010-06-18 08:38:59 +0000
+++ account_payment/account_payment.py	2010-06-21 08:36:32 +0000
@@ -68,6 +68,7 @@
     _name = 'payment.order'
     _description = 'Payment Order'
     _rec_name = 'reference'
+    _log_create = True
 
     def get_wizard(self, type):
         logger = netsvc.Logger()
@@ -132,10 +133,14 @@
         return True
 
     def set_done(self, cr, uid, id, *args):
+        from tools.translate import _
         self.write(cr,uid,id,{'date_done': time.strftime('%Y-%m-%d'),
             'state': 'done',})
         wf_service = netsvc.LocalService("workflow")
         wf_service.trg_validate(uid, 'payment.order', id, 'done', cr)
+        for pmnt in self.browse(cr, uid, [id]):
+            msg = _("Payment Order with amount of ")+" '"+str(pmnt.total)+"' "+_("of user")+" '"+pmnt.user_id.name+"' "+_("is done")
+            self.log(cr, uid, pmnt.id, msg)
         return True
 
 payment_order()

=== modified file 'account_voucher/voucher.py'
--- account_voucher/voucher.py	2010-06-16 11:51:39 +0000
+++ account_voucher/voucher.py	2010-06-21 08:36:32 +0000
@@ -100,6 +100,7 @@
     _name = 'account.voucher'
     _description = 'Accounting Voucher'
     _order = "number"
+    _log_create = True
     _columns = {
         'name':fields.char('Name', size=256, required=True, readonly=True, states={'draft':[('readonly',False)]}),
         'type': fields.selection([
@@ -210,6 +211,9 @@
         self.action_number(cr, uid, ids)
         self.action_move_line_create(cr, uid, ids)
         self.write(cr, uid, ids, {'state':'posted'})
+        for voucher in self.browse(cr, uid, ids):
+            msg = _("Voucher")+"  '"+voucher.name+"'  "+_("is posted")
+            self.log(cr, uid, voucher.id, msg)
         return True
 
     def cancel_voucher(self, cr, uid, ids, context={}):

=== modified file 'hr/hr.py'
--- hr/hr.py	2010-06-16 11:51:39 +0000
+++ hr/hr.py	2010-06-21 08:36:32 +0000
@@ -91,6 +91,7 @@
     _name = "hr.employee"
     _description = "Employee"
     _inherits = {'resource.resource':"resource_id"}
+    _log_create = True
     _columns = {
         'country_id' : fields.many2one('res.country', 'Nationality'),
         'birthday' : fields.date("Birthday"),

=== modified file 'project/project.py'
--- project/project.py	2010-06-18 08:38:59 +0000
+++ project/project.py	2010-06-21 08:36:32 +0000
@@ -45,6 +45,7 @@
     _name = "project.project"
     _description = "Project"
     _inherits = {'account.analytic.account':"category_id"}
+    _log_create = True
 
     def search(self, cr, user, args, offset=0, limit=None, order=None, context=None, count=False):
         if context is None:
@@ -164,10 +165,12 @@
     ]
     def set_template(self, cr, uid, ids, context={}):
         res = self.setActive(cr, uid, ids, value=False, context=context)
+        self.log(cr, uid, ids[0], _("Project %s is set as a template"%self.browse(cr,uid,ids[0]).complete_name))
         return res
 
     def set_done(self, cr, uid, ids, context={}):
         self.write(cr, uid, ids, {'state':'close'}, context=context)
+        self.log(cr, uid, ids[0], _("Project %s is done"%self.browse(cr,uid,ids[0]).complete_name))
         return True
 
     def set_cancel(self, cr, uid, ids, context={}):
@@ -419,6 +422,7 @@
                             reopen = False
                     if reopen:
                         self.do_reopen(cr, uid, [parent_id.id])
+                    self.log(cr, uid, task.id, _("Task %s is done"%self.browse(cr,uid,ids[0]).name))
         if mail_send:
             model_data_ids = mod_obj.search(cr,uid,[('model','=','ir.ui.view'),('name','=','view_project_close_task')])
             resource_id = mod_obj.read(cr, uid, model_data_ids, fields=['res_id'])[0]['res_id']
@@ -434,6 +438,7 @@
                 'target': 'new',
                 'nodestroy': True
                     }
+        
         return True
 
     def do_reopen(self, cr, uid, ids, *args):
@@ -453,6 +458,7 @@
                 })
 
             self.write(cr, uid, [task.id], {'state': 'open'})
+            self.log(cr, uid, task.id, _("Task %s is set to open"%self.browse(cr,uid,ids[0]).name))
         return True
 
     def do_cancel(self, cr, uid, ids, *args):