Merge lp:~icsergio/openobject-italia/fix_sequential_dates_61 into lp:~openobject-italia-core-devs/openobject-italia/italian-addons-6.1

[Sergio Corato]

[lp:~icsergio/openobject-italia/fix_sequential_dates_61]

240. By Sergio Corato on 2013-09-05

[FIX] view correction

241. By Sergio Corato on 2013-09-05

[FIX] view correction

[Lorenzo Battistini (elbati)]

Il nuovo campo 'supplier_invoice_number' quando e come viene riempito?

review: Needs Information

[Sergio Corato (icsergio)]

Dal contabile in fase di registrazione fattura d'acquisto

2013/9/6 Lorenzo Battistini - Agile BG <email address hidden>

> Review: Needs Information
>
> Il nuovo campo 'supplier_invoice_number' quando e come viene riempito?
> --
>
> https://code.launchpad.net/~icsergio/openobject-italia/fix_sequential_dates_61/+merge/184182
> You are the owner of
> lp:~icsergio/openobject-italia/fix_sequential_dates_61.
>

[Lorenzo Battistini (elbati)]

On 09/06/2013 01:09 PM, Sergio Corato wrote:
> Dal contabile in fase di registrazione fattura d'acquisto

Non lo vedo comparire in alcuna view. Probabilmente manca il file
account/invoice_view.xml

[lp:~icsergio/openobject-italia/fix_sequential_dates_61]

242. By Sergio Corato on 2013-09-06

[FIX] add missing xml

[Sergio Corato (icsergio)]

piccola dimenticanza, non avevo fatto il bzr add * :)

[lp:~icsergio/openobject-italia/fix_sequential_dates_61]

243. By Sergio Corato on 2013-09-08

[FIX] registration of a sale invoice in state draft after cancel, with already a number

[Lorenzo Battistini (elbati)]

Come mai la parte relativa alle 'in_invoice' e 'in_refund' è stata messa in 'action_move_create' e non in 'action_number' insieme alla parte per le 'out_*' ?

review: Needs Information

[Sergio Corato (icsergio)]

Il 09/09/2013 08:20, Lorenzo Battistini - Agile BG ha scritto:
> Review: Needs Information
>
> Come mai la parte relativa alle 'in_invoice' e 'in_refund' è stata messa in 'action_move_create' e non in 'action_number' insieme alla parte per le 'out_*' ?
>
uhm, mi pareva non ci fossero tutte le variabili necessarie...
Ri-verifico.

--
Sergio Corato

IcsTools.it

web : www.icstools.it
skype : sergiocorato
mail : <email address hidden>

[Sergio Corato (icsergio)]

Il 09/09/2013 13:36, Sergio Corato ha scritto:
> Il 09/09/2013 08:20, Lorenzo Battistini - Agile BG ha scritto:
>> Review: Needs Information
>>
>> Come mai la parte relativa alle 'in_invoice' e 'in_refund' è stata messa in 'action_move_create' e non in 'action_number' insieme alla parte per le 'out_*' ?
>>
> uhm, mi pareva non ci fossero tutte le variabili necessarie...
> Ri-verifico.
>
Ok, ho committato, dovrebbe essere meglio così.

--
Sergio Corato

IcsTools.it

web : www.icstools.it
skype : sergiocorato
mail : <email address hidden>

[lp:~icsergio/openobject-italia/fix_sequential_dates_61]

244. By Sergio Corato on 2013-09-09

[FIX] code improvements

[Lorenzo Battistini (elbati)]

line 51 e 39: 'inv_type = obj_inv.type' è presente due volte
line 63: come mai c'è 'xxxx'?

review: Needs Fixing

[lp:~icsergio/openobject-italia/fix_sequential_dates_61]

245. By Sergio Corato on 2013-09-10

[FIX] typo error

[Sergio Corato (icsergio)]

Il 10/09/2013 09:02, Lorenzo Battistini - Agile BG ha scritto:
> Review: Needs Fixing
>
> line 51 e 39: 'inv_type = obj_inv.type' è presente due volte
> line 63: come mai c'è 'xxxx'?
>
ops, sistemato

--
Sergio Corato

IcsTools.it

web : www.icstools.it
skype : sergiocorato
mail : <email address hidden>

[Lorenzo Battistini (elbati)]

Grazie!

Per non lasciare nuove funzionalità solo su versioni vecchie, riesci a proporre le stesse modifiche su 'account_invoice_sequential_dates' per la 7? (considerando che il campo 'supplier_invoice_number' sulla 7 c'è già)

review: Approve

[Sergio Corato (icsergio)]

Il 11/09/2013 11:56, Lorenzo Battistini - Agile BG ha scritto:
> Review: Approve
>
> Grazie!
>
e grazie a te!
> Per non lasciare nuove funzionalità solo su versioni vecchie, riesci a proporre le stesse modifiche su 'account_invoice_sequential_dates' per la 7? (considerando che il campo 'supplier_invoice_number' sulla 7 c'è già)
>
penso di sì

--
Sergio Corato

IcsTools.it

web : www.icstools.it
skype : sergiocorato
mail : <email address hidden>

[Leonardo Pistone (lepistone)]

Ciao Sergio, Lorenzo

scusate se scrivo dopo il merge, mi sono accorto ora.
È corretto aggiungere una dipendenza?

Grazie!

review: Needs Information

[Sergio Corato (icsergio)]

Il 18/09/2013 16:27, Leonardo Pistone - Agile BG ha scritto:
> Review: Needs Information
>
> Ciao Sergio, Lorenzo
>
> scusate se scrivo dopo il merge, mi sono accorto ora.
> È corretto aggiungere una dipendenza?
>
> Grazie!
>
Ciao Leonardo,
serve perchè questo modulo controlla la correttezza anche tramite il
campo registration_date che c'è nel modulo in dipendenza.
Si è appunto scelto di tenere separato questi controlli dal modulo
account_invoice_entry_date, ma la dipendenza è necessaria.
Comunque se hai altri idee, siamo qua :D

--
Sergio Corato

IcsTools.it

web : www.icstools.it
skype : sergiocorato
mail : <email address hidden>

[Leonardo Pistone (lepistone)]

Sergio,
Mi chiedevo solo se era una dimenticanza o ora una cosa pensata, se è pensata, OK!
grazie! leo

=== modified file 'l10n_it_account/__openerp__.py'
--- l10n_it_account/__openerp__.py	2013-05-07 12:26:04 +0000
+++ l10n_it_account/__openerp__.py	2013-09-10 11:40:17 +0000
@@ -33,9 +33,10 @@
     'author': 'OpenERP Italian Community',
     'website': 'http://www.openerp-italia.org',
     'license': 'AGPL-3',
-    "depends" : ['account','base_vat','account_chart','base_iban', 'l10n_it_base'],
+    "depends" : ['account','base_vat','account_chart','base_iban', 'l10n_it_base','account_invoice_entry_date'],
     "init_xml": [
         'account/partner_view.xml',
+        'account/invoice_view.xml',
         'wizard/fiscalcode_to_data_view.xml',
     ],
     "update_xml" : [],

=== modified file 'l10n_it_account/account/invoice.py'
--- l10n_it_account/account/invoice.py	2013-03-13 10:12:16 +0000
+++ l10n_it_account/account/invoice.py	2013-09-10 11:40:17 +0000
@@ -22,31 +22,52 @@
 import netsvc
 import pooler, tools
 
-from osv import fields, osv
+from openerp.osv import fields, orm
 from tools.translate import _
 
-class account_invoice(osv.osv):
+class account_invoice(orm.Model):
     
     _inherit = 'account.invoice'
+    _columns = {
+        'supplier_invoice_number': fields.char('Supplier invoice nr', size=16),
+        }
 
     def action_number(self, cr, uid, ids, context=None):
         super(account_invoice, self).action_number(cr, uid, ids, context)
         for obj_inv in self.browse(cr, uid, ids):
             inv_type = obj_inv.type
-            if inv_type == 'in_invoice' or inv_type == 'in_refund':
-                return True
             number = obj_inv.number
             date_invoice = obj_inv.date_invoice
+            reg_date = obj_inv.registration_date
             journal = obj_inv.journal_id.id
-            res = self.search(cr, uid, [('type','=',inv_type),('date_invoice','>',date_invoice), 
-                ('number', '<', number), ('journal_id','=',journal)])
-            if res:
-                raise osv.except_osv(_('Date Inconsistency'),
+            date_start = obj_inv.registration_date or obj_inv.date_invoice or time.strftime('%Y-%m-%d')
+            date_stop = obj_inv.registration_date or obj_inv.date_invoice or time.strftime('%Y-%m-%d')
+            period_ids = self.pool.get('account.period').search(
+                cr, uid, [('date_start','<=',date_start),('date_stop','>=',date_stop), 
+                ('company_id', '=', obj_inv.company_id.id)])
+            if inv_type == 'out_invoice' or inv_type == 'out_refund':
+                #check if another invoice with a minor number and a superior date_invoice is posted
+                res = self.search(cr, uid, [('type','=',inv_type),('date_invoice','>',date_invoice), 
+                    ('number', '<', number), ('journal_id','=',journal),('period_id','in',period_ids)])
+                if res:
+                    raise orm.except_orm(_('Date Inconsistency'),
+                            _('Cannot create invoice! Post the invoice with a greater date'))
+            if inv_type == 'in_invoice' or inv_type == 'in_refund':
+                #check if an invoice with a superior registration_date is posted
+                res = self.search(cr, uid, [('type','=',inv_type),('registration_date','>',reg_date), 
+                    ('journal_id','=',journal),('period_id','in',period_ids)], context=context)
+                if res:
+                    raise orm.except_orm(_('Date Inconsistency'),
                         _('Cannot create invoice! Post the invoice with a greater date'))
+                #check duplication (only supplier's invoices)
+                supplier_invoice_number = obj_inv.supplier_invoice_number
+                partner_id = obj_inv.partner_id.id
+                res = self.search(cr, uid, [('type','=',inv_type),('date_invoice','=',date_invoice), 
+                    ('journal_id','=',journal),('supplier_invoice_number','=',supplier_invoice_number),
+                    ('partner_id','=',partner_id),('state','not in',('draft','cancel'))], context=context)
+                if res:
+                    raise orm.except_orm(_('Invoice Duplication'),
+                        _('Invoice already posted!'))
         return True
-    
-account_invoice()
-
-
 
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added file 'l10n_it_account/account/invoice_view.xml'
--- l10n_it_account/account/invoice_view.xml	1970-01-01 00:00:00 +0000
+++ l10n_it_account/account/invoice_view.xml	2013-09-10 11:40:17 +0000
@@ -0,0 +1,28 @@
+<?xml version="1.0" encoding="utf-8"?>
+<openerp>
+    <data>
+
+        <record id="invoice_tree_reg_date" model="ir.ui.view">
+            <field name="name">account.invoice.tree.reg_date</field>
+            <field name="model">account.invoice</field>
+            <field name="inherit_id" ref="account.invoice_tree"/>
+            <field name="arch" type="xml">
+                <field name="date_invoice" position="after">
+                    <field name="supplier_invoice_number" invisible="context.get('type') == 'out_invoice'"/>
+                </field>
+            </field>
+        </record>
+        
+        <record id="invoice_supplier_form_reg_date" model="ir.ui.view">
+            <field name="name">account.invoice.supplier.form.reg_date</field>
+            <field name="model">account.invoice</field>
+            <field name="inherit_id" ref="account.invoice_supplier_form"/>
+            <field name="arch" type="xml">
+                <field name="date_invoice" position="after">
+                    <field name="supplier_invoice_number" attrs="{'readonly':[('state','!=','draft')]}"/>
+                </field>
+            </field>
+        </record>
+        
+     </data>
+</openerp>