Merge lp:~vauxoo/addons-vauxoo/7.0-account_move_folio-dev-hbto into lp:addons-vauxoo/7.0

[hbto [Vauxoo] http://www.vauxoo.com]

[IMP][account_move_folio] Added new module to keep a record of Journal Entry Folios

[Moisés López - http://www.vauxoo.com (moylop260)]

Aprobado!!!
Puedes por favor, hace el merge.
Junto con el depends en el data del cliente, para verlo funcionando en runbot, por favor.

review: Approve

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

done
On Oct 12, 2013 4:30 PM, "Moisés López - http://www.vauxoo.com" <
<email address hidden>> wrote:

> Review: Approve
>
> Aprobado!!!
> Puedes por favor, hace el merge.
> Junto con el depends en el data del cliente, para verlo funcionando en
> runbot, por favor.
> --
>
> https://code.launchpad.net/~vauxoo/addons-vauxoo/7.0-account_move_folio-dev-hbto/+merge/190784
> You proposed lp:~vauxoo/addons-vauxoo/7.0-account_move_folio-dev-hbto for
> merging.
>

[lp:~vauxoo/addons-vauxoo/7.0-account_move_folio-dev-hbto]

863. By hbto [Vauxoo] http://www.vauxoo.com on 2013-10-12

[MERGE][account_move_folio] Added new module to keep a record of Journal Entry Folios

[Nhomar - Vauxoo (nhomar)]

Moi.

Específicamente está bien ésto, pero no te parece mejor un objeto que nos permita auditar cualquier "sequence" y/o cualquier folio agnótico al objeto, y configurarlo vía objeto?

DIgo por que es válido para muchos otros modelos, cómo facturas, órdenes de venta, lineas presupuestarias, etc etc, solo lo comento como idea en éste merge, para saber si es discutible hacer lo genérico como una evolución de éste?

Saludo.s

review: Approve

[Moisés López - http://www.vauxoo.com (moylop260)]

@Nhomar,
A mí en lo personal el único objeto que vale la pena auditar es:
account.move

Los demás objetos como purchase, sale, picking... si se brincan folios no es tan crítico legalmente.
Por lo que en este caso, se pone enfasis solo en account.move

Sin embargo, técnicamente sí se puede hacer algo genérico para todos los sequence.

FYI hay un tema al respecto, que si lo dejamos genérico necesitamos invertir tiempo en R&D porque el rollback con el "wait noupdate" que utiliza el ir.sequence, está siendo afectado el sequence, aun cuando hay rollback de por medio. (POC en WIP para determinar el tema)
Por ahora, este MP cumple con lo requerido por el cliente.

[Nhomar - Vauxoo (nhomar)]

Listo"

Agreed!

El 12 de octubre de 2013 19:51, Moisés López - http://www.vauxoo.com <
<email address hidden>> escribió:

> @Nhomar,
> A mí en lo personal el único objeto que vale la pena auditar es:
> account.move
>
> Los demás objetos como purchase, sale, picking... si se brincan folios no
> es tan crítico legalmente.
> Por lo que en este caso, se pone enfasis solo en account.move
>
> Sin embargo, técnicamente sí se puede hacer algo genérico para todos los
> sequence.
>
> FYI hay un tema al respecto, que si lo dejamos genérico necesitamos
> invertir tiempo en R&D porque el rollback con el "wait noupdate" que
> utiliza el ir.sequence, está siendo afectado el sequence, aun cuando hay
> rollback de por medio. (POC en WIP para determinar el tema)
> Por ahora, este MP cumple con lo requerido por el cliente.
> --
>
> https://code.launchpad.net/~vauxoo/addons-vauxoo/7.0-account_move_folio-dev-hbto/+merge/190784
> You are reviewing the proposed merge of
> lp:~vauxoo/addons-vauxoo/7.0-account_move_folio-dev-hbto into
> lp:addons-vauxoo/7.0.
>

--
--------------------
Saludos Cordiales

Nhomar G. Hernandez M.
[phone]
Skype: nhomar00
Web-Blog: http://geronimo.com.ve
Servicios IT: http://vauxoo.com
Linux-Counter: 467724
Correos:
<email address hidden>
<email address hidden>
twitter @nhomar

=== added directory 'account_move_folio'
=== added file 'account_move_folio/__init__.py'
--- account_move_folio/__init__.py	1970-01-01 00:00:00 +0000
+++ account_move_folio/__init__.py	2013-10-12 06:58:58 +0000
@@ -0,0 +1,2 @@
+# -*- encoding: utf-8 -*-
+import model

=== added file 'account_move_folio/__openerp__.py'
--- account_move_folio/__openerp__.py	1970-01-01 00:00:00 +0000
+++ account_move_folio/__openerp__.py	2013-10-12 06:58:58 +0000
@@ -0,0 +1,31 @@
+# -*- encoding: utf-8 -*-
+{
+    "name": "Account Move Folio",
+    "version": "1.0",
+    "author": "Vauxoo",
+    'category' : 'Accounting & Finance',
+    "description" : """
+Account Move Folio
+==================
+
+This Module keeps a record of all the sequences that have been used in the
+Journal Entries in OpenERP, No matter if a Journal Entry is deleted it will
+stand as a way to audit all the sequences that have been used.
+    """,
+    "website": "http://www.vauxoo.com/",
+    "license": "AGPL-3",
+    "depends": [
+            "account",
+                ],
+    "demo": [
+    ],
+    "data": [
+        'security/ir.model.access.csv',
+        'view/view.xml',
+        'view/menues_and_actions.xml',
+    ],
+    "test" : [
+    ],
+    "installable": True,
+    "active": False,
+}

=== added directory 'account_move_folio/i18n'
=== added directory 'account_move_folio/model'
=== added file 'account_move_folio/model/__init__.py'
--- account_move_folio/model/__init__.py	1970-01-01 00:00:00 +0000
+++ account_move_folio/model/__init__.py	2013-10-12 06:58:58 +0000
@@ -0,0 +1,2 @@
+# -*- encoding: utf-8 -*-
+import account_move_folio

=== added file 'account_move_folio/model/account_move_folio.py'
--- account_move_folio/model/account_move_folio.py	1970-01-01 00:00:00 +0000
+++ account_move_folio/model/account_move_folio.py	2013-10-12 06:58:58 +0000
@@ -0,0 +1,63 @@
+# -*- encoding: utf-8 -*-
+from openerp.osv import fields, osv
+
+class account_move_folio(osv.Model):
+    _name = 'account.move.folio'
+    _description = "Records of Folios in Journal Entries"
+    _columns = {
+        'name':fields.char('Folio', 256, help='Folio Number', required = True), 
+        'move_id':fields.many2one('account.move', 'Journal Entry', help='Journal Entry'), 
+        'journal_id':fields.many2one('account.journal', 'Journal', help='Entry Journal'), 
+        'period_id':fields.many2one('account.period', 'Period', help='Entry Period'), 
+        'date':fields.date('Date', help='Entry Date'), 
+        'company_id':fields.many2one('res.company', 'Company', help='Entry Company'), 
+        'move_state':fields.related('move_id','state', type='selection',
+            selection=[('draft','Unposted'), ('posted','Posted')], store=True,
+            string='Entry State', help='Entry State'), 
+        'period_state':fields.related('period_id','state', type='selection',
+            selection=[('draft','Open'), ('done','Closed')], store=True,
+            string='Period State', help='Period State'), 
+    }
+
+    _defaults = {
+        'company_id': lambda self, cr, uid, c: self.pool.get('res.users').browse(cr, uid, uid, c).company_id.id,
+    }
+
+class account_move(osv.Model):
+    _inherit = 'account.move'
+
+    _columns = {
+        'folio_id': fields.many2one('account.move.folio', 'Folio Record'),
+    }
+
+    def foliate(self, cr, uid, ids, context=None):
+        context = context or {}
+        folio_obj = self.pool.get('account.move.folio')
+        invoice = context.get('invoice', False)
+        for move in self.browse(cr, uid, ids, context=context):
+            if not move.folio_id:
+                values = {
+                    'name': move.name,
+                    'move_id': move.id,
+                    'journal_id': move.journal_id.id,
+                    'period_id': move.period_id.id,
+                    'date': move.date,
+                }
+                if invoice:
+                    folio_ids = folio_obj.search(cr, uid, [('name','=',move.name)],context=context)
+                    if folio_ids:
+                        folio_id = folio_ids[0]
+                        folio_obj.write(cr, uid, folio_id, values, context=context)
+                    else:
+                        folio_id = folio_obj.create(cr, uid, values,context=context)
+                else:
+                    folio_id = folio_obj.create(cr, uid, values,context=context)
+                move.write({'folio_id':folio_id},context=context)
+        return True
+    
+    def post(self, cr, uid, ids, context=None):
+        context = context or {}
+        super(account_move, self).post(cr, uid, ids, context=context)
+        self.foliate(cr, uid, ids, context=context)
+        return True
+

=== added directory 'account_move_folio/security'
=== added file 'account_move_folio/security/ir.model.access.csv'
--- account_move_folio/security/ir.model.access.csv	1970-01-01 00:00:00 +0000
+++ account_move_folio/security/ir.model.access.csv	2013-10-12 06:58:58 +0000
@@ -0,0 +1,2 @@
+id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
+access_account_move_folio_manager,account.move.folio,model_account_move_folio,account.group_account_manager,1,1,1,1

=== added directory 'account_move_folio/view'
=== added file 'account_move_folio/view/menues_and_actions.xml'
--- account_move_folio/view/menues_and_actions.xml	1970-01-01 00:00:00 +0000
+++ account_move_folio/view/menues_and_actions.xml	2013-10-12 06:58:58 +0000
@@ -0,0 +1,22 @@
+<?xml version='1.0' encoding='utf-8'?>
+<openerp>
+    <data>
+        <record id="account_move_folio_action" model="ir.actions.act_window">
+            <field name="name">Entry Folios</field>
+            <field name="res_model">account.move.folio</field>
+            <field name="view_type">form</field>
+            <field name="view_mode">tree,form</field>
+        </record>
+        <menuitem 
+            id="move_folio_main_menu" 
+            name="Entry Folios"
+            parent="account.menu_finance" 
+            sequence="5"
+            />
+        <menuitem 
+            id="move_folio_menu" 
+            parent="move_folio_main_menu" 
+            action='account_move_folio_action'
+            />
+    </data>
+</openerp>

=== added file 'account_move_folio/view/view.xml'
--- account_move_folio/view/view.xml	1970-01-01 00:00:00 +0000
+++ account_move_folio/view/view.xml	2013-10-12 06:58:58 +0000
@@ -0,0 +1,47 @@
+<?xml version='1.0' encoding='utf-8'?>
+<openerp>
+    <data>
+        <record id="view_move_form" model="ir.ui.view">
+            <field name="name">account.move.form</field>
+            <field name="model">account.move</field>
+            <field name="inherit_id" ref="account.view_move_form"/>
+            <field name="arch" type="xml">
+                <xpath expr='//field[@name="ref"]' position='after'>
+                    <field name="folio_id" readonly='1' invisible='0'/>
+                </xpath>
+            </field>
+        </record>
+        <record model="ir.ui.view" id="account_move_folio_form_view">
+            <field name="name">account.move.folio.form</field>
+            <field name="model">account.move.folio</field>
+            <field name="arch" type="xml">
+                <form create='0' edit='0' delete='0' string="Account Move Folio Form">
+                    <field name="name" readonly='1'/>
+                    <field name="move_id" readonly='1'/>
+                    <field name="journal_id" readonly='1'/>
+                    <field name="period_id" readonly='1'/>
+                    <field name="date" readonly='1'/>
+                    <field name="move_state" readonly='1'/>
+                    <field name="period_state" readonly='1'/>
+                    <field name="company_id" readonly='1' groups="base.group_multi_company"/>
+                </form>
+            </field>
+        </record>
+        <record model="ir.ui.view" id="account_move_folio_tree_view">
+            <field name="name">account.move.folio.tree</field>
+            <field name="model">account.move.folio</field>
+            <field name="arch" type="xml">
+                <tree create='0' edit='0' delete='0' string="Account Move Folio Tree">
+                    <field name="name" readonly='1'/>
+                    <field name="move_id" readonly='1'/>
+                    <field name="journal_id" readonly='1'/>
+                    <field name="period_id" readonly='1'/>
+                    <field name="date" readonly='1'/>
+                    <field name="move_state" readonly='1'/>
+                    <field name="period_state" readonly='1'/>
+                    <field name="company_id" readonly='1' groups="base.group_multi_company"/>
+                </tree>
+            </field>
+        </record>
+    </data>
+</openerp>