Merge lp:~sylvain-legal/openupgrade-addons/7.0_point_of_sale into lp:openupgrade-addons

[Sylvain LE GAL (GRAP)]

This branch will manage 'point_of_sale' migration.
Work in progress.
Please don't review until the state changes.
Merge Proposal just to inform to community i'm working on the module.

[lp:~sylvain-legal/openupgrade-addons/7.0_point_of_sale]

8137. By Sylvain LE GAL (GRAP) on 2013-10-14

[ADD] First version Beta for 'point_of_sale' migration.

[Sylvain LE GAL (GRAP) (sylvain-legal)]

My apologize for the mistake.

2013/10/15 Sylvain LE GAL (GRAP) <email address hidden>

> The proposal to merge
> lp:~sylvain-legal/openupgrade-addons/7.0_point_of_sale into
> lp:openupgrade-addons has been updated.
>
> Status: Merged => Work in progress
>
> For more details, see:
>
> https://code.launchpad.net/~sylvain-legal/openupgrade-addons/7.0_point_of_sale/+merge/187221
> --
>
> https://code.launchpad.net/~sylvain-legal/openupgrade-addons/7.0_point_of_sale/+merge/187221
> You are the owner of
> lp:~sylvain-legal/openupgrade-addons/7.0_point_of_sale.
>

[Pedro Manuel Baeza (pedro.baeza)]

How is this work going, Sylvain?

[Sylvain LE GAL (GRAP) (sylvain-legal)]

Hi,

I realized a migration 6.1 -> 7.0 in November 2013 with this script. I did'nt marked this MP as 'ready to review' because this script was experimental and I think that other developpers have to work on it to make it more correct maybe.
I kept it in the state 'work in progress' because maybe this script can be usefull for any people.

Regards.

Unmerged revisions

8137. By Sylvain LE GAL (GRAP) on 2013-10-14

[ADD] First version Beta for 'point_of_sale' migration.

=== added file 'point_of_sale/migrations/7.0.1.0.1/openupgrade_analysis_work.txt'
--- point_of_sale/migrations/7.0.1.0.1/openupgrade_analysis_work.txt	1970-01-01 00:00:00 +0000
+++ point_of_sale/migrations/7.0.1.0.1/openupgrade_analysis_work.txt	2013-10-14 23:49:02 +0000
@@ -0,0 +1,169 @@
+# - se faire une base de référence de point of sale en 6.1
+# passer une migration V7 avec rien, pour voir comment ça réagit.
+
+
+---Fields in module 'point_of_sale'---
+### Create an session for each bank statement.
+point_of_sale / account.bank.statement   / pos_session_id (many2one)     : NEW relation: pos.session
+
+### Deleting a field related. Nothing to do.
+point_of_sale / account.bank.statement.line / journal_id (char)             : DEL mode: modify
+
+
+### This concept are lost in V7. Nothing to do.
+point_of_sale / account.journal          / auto_cash (boolean)           : DEL 
+point_of_sale / account.journal          / check_dtls (boolean)          : DEL 
+### New concept, use default
+point_of_sale / account.journal          / amount_authorized_diff (float): NEW 
+point_of_sale / account.journal          / self_checkout_payment_method (boolean): NEW 
+
+### New concept. hypothesis : create au new pos_config object for each shop_id / sale_journal / user_id
+point_of_sale / pos.config               / group_by (boolean)            : NEW 
+point_of_sale / pos.config               / iface_cashdrawer (boolean)    : NEW 
+point_of_sale / pos.config               / iface_electronic_scale (boolean): NEW 
+point_of_sale / pos.config               / iface_payment_terminal (boolean): NEW 
+point_of_sale / pos.config               / iface_print_via_proxy (boolean): NEW 
+point_of_sale / pos.config               / iface_self_checkout (boolean) : NEW 
+point_of_sale / pos.config               / iface_vkeyboard (boolean)     : NEW 
+point_of_sale / pos.config               / journal_id (many2one)         : NEW relation: account.journal
+point_of_sale / pos.config               / journal_ids (many2many)       : NEW relation: account.journal
+point_of_sale / pos.config               / name (char)                   : NEW required: required
+point_of_sale / pos.config               / sequence_id (many2one)        : NEW relation: ir.sequence
+point_of_sale / pos.config               / session_ids (one2many)        : NEW relation: pos.session
+point_of_sale / pos.config               / shop_id (many2one)            : NEW relation: sale.shop, required: required, req_default: function
+point_of_sale / pos.config               / state (selection)             : NEW required: required, selection_keys: ['active', 'deprecated', 'inactive'], req_default: active
+
+# Pour les pos_order faire un select shop_id, sale_journal. Créer un pos_config & des pos_sessions associés. !!!! Attention aussi aux users.
+
+# New optional value. Nothing to do.
+point_of_sale / pos.order                / pos_reference (char)          : NEW 
+### The script migration will associate a pos.order to a new 'pos.session'. (associated to sale_journal).
+# No Data loss neither, data Change. Nothing to do.
+point_of_sale / pos.order                / sale_journal (many2one)       : now a function
+# The script migration will associate a pos.order to a new 'pos.session'.
+point_of_sale / pos.order                / session_id (many2one)         : NEW relation: pos.session
+### The script migration will associate a pos.order to a new 'pos.session'. (associated to shop_id).
+# No Data loss neither, data Change. Nothing to do.
+point_of_sale / pos.order                / shop_id (many2one)            : now a function
+
+
+point_of_sale / pos.session              / config_id (many2one)          : NEW relation: pos.config, required: required
+point_of_sale / pos.session              / name (char)                   : NEW required: required, req_default: /
+point_of_sale / pos.session              / order_ids (one2many)          : NEW relation: pos.order
+point_of_sale / pos.session              / start_at (datetime)           : NEW 
+point_of_sale / pos.session              / state (selection)             : NEW required: required, selection_keys: ['closed', 'closing_control', 'opened', 'opening_control'], req_default: opening_control
+point_of_sale / pos.session              / statement_ids (one2many)      : NEW relation: account.bank.statement
+point_of_sale / pos.session              / stop_at (datetime)            : NEW 
+point_of_sale / pos.session              / user_id (many2one)            : NEW relation: res.users, required: required, req_default: function
+
+### 'available_in_pos' is a new concept. If checked, the product will be available to sale in the point of sale. 
+# In 6.1, the product was available if the product had a pos_categ_id defined. 
+# The script migration will set to True the 'available_in_pos' value for all product with a pos_categ_id defined.
+point_of_sale / product.product          / available_in_pos (boolean)    : NEW 
+
+### 'to_weight' is a a new concept. We assume that all products that have a unit 'uos_id' (or 'uom_id') which 'uom_categ_id' is 'product_uom_categ_kgm' can be weight. 
+# The script migration will set to True the 'to_weight' value for that products.
+## USE : uom_categ, uom_categ_id = self.pool.get('ir.model.data').get_object_reference(cr, uid, 'product', 'product_uom_categ_kgm')
+point_of_sale / product.product          / to_weight (boolean)           : NEW 
+
+# New Concept for 'pos.category'. Nothing to do.
+point_of_sale / pos.category             / image (binary)                : NEW 
+
+# New Concept for 'res_user'. Nothing to do.
+point_of_sale / res.users                / ean13 (char)                  : NEW 
+
+# Script migration will associate a new 'pos.config' to each sale user.
+point_of_sale / res.users                / pos_config (many2one)         : NEW relation: pos.config
+
+---XML records in module 'point_of_sale'---
+### Ignore interface and access records
+NEW ir.actions.act_window: point_of_sale.act_pos_config_sessions
+NEW ir.actions.act_window: point_of_sale.act_pos_session_orders
+NEW ir.actions.act_window: point_of_sale.action_pos_box_in
+NEW ir.actions.act_window: point_of_sale.action_pos_box_out
+NEW ir.actions.act_window: point_of_sale.action_pos_config_pos
+NEW ir.actions.act_window: point_of_sale.action_pos_session
+NEW ir.actions.act_window: point_of_sale.action_pos_session_opening
+DEL ir.actions.act_window: point_of_sale.action_box_entries
+DEL ir.actions.act_window: point_of_sale.action_box_out
+DEL ir.actions.act_window: point_of_sale.action_pos_close_statement
+DEL ir.actions.act_window: point_of_sale.action_pos_confirm
+DEL ir.actions.act_window: point_of_sale.action_pos_sale_all
+DEL ir.actions.act_window: point_of_sale.action_product_input
+DEL ir.actions.act_window: point_of_sale.action_product_output
+DEL ir.actions.act_window: point_of_sale.action_report_cash_register_all
+NEW ir.actions.client: point_of_sale.action_client_pos_menu
+NEW ir.actions.report.xml: point_of_sale.report_user_label
+NEW ir.actions.todo: base.open_menu
+NEW ir.model.access: point_of_sale.access_account_journal_cashbox_line
+NEW ir.model.access: point_of_sale.access_ir_sequence_manager
+NEW ir.model.access: point_of_sale.access_pos_config_user
+NEW ir.model.access: point_of_sale.access_pos_session_user
+DEL ir.model.access: point_of_sale.access_account_journal_column
+DEL ir.model.access: point_of_sale.access_account_journal_column_manager
+DEL ir.model.access: point_of_sale.access_account_journal_view
+DEL ir.model.access: point_of_sale.access_account_journal_view_manager
+DEL ir.model.access: point_of_sale.access_pos_config_journal
+DEL ir.model.access: point_of_sale.access_pos_config_journal_manager
+DEL ir.model.access: point_of_sale.access_report_check_register
+NEW ir.rule: point_of_sale.rule_pos_multi_company
+NEW ir.ui.menu: point_of_sale.menu_pos_config_pos
+NEW ir.ui.menu: point_of_sale.menu_pos_session_all
+NEW ir.ui.menu: point_of_sale.menu_pos_session_opening
+DEL ir.ui.menu: point_of_sale.menu_all_menu_all_register
+DEL ir.ui.menu: point_of_sale.menu_close_statement
+DEL ir.ui.menu: point_of_sale.menu_open_statement
+DEL ir.ui.menu: point_of_sale.menu_point_ofsale_all
+DEL ir.ui.menu: point_of_sale.menu_point_open_config
+DEL ir.ui.menu: point_of_sale.menu_point_root_touchscreen
+DEL ir.ui.menu: point_of_sale.menu_report_cash_register_all
+DEL ir.ui.menu: point_of_sale.menu_wizard_enter_jrnl
+DEL ir.ui.menu: point_of_sale.menu_wizard_enter_jrnl2
+DEL ir.ui.menu: point_of_sale.menu_wizard_pos_confirm
+DEL ir.ui.menu: point_of_sale.products_for_input_operations
+DEL ir.ui.menu: point_of_sale.products_for_output_operations
+NEW ir.ui.view: point_of_sale.pos_ean13_generator
+NEW ir.ui.view: point_of_sale.pos_session_opening_form_view
+NEW ir.ui.view: point_of_sale.res_users_form_preference_view
+NEW ir.ui.view: point_of_sale.res_users_form_view
+NEW ir.ui.view: point_of_sale.view_partner_property_form
+NEW ir.ui.view: point_of_sale.view_pos_config_form
+NEW ir.ui.view: point_of_sale.view_pos_config_search
+NEW ir.ui.view: point_of_sale.view_pos_config_tree
+NEW ir.ui.view: point_of_sale.view_pos_session_form
+NEW ir.ui.view: point_of_sale.view_pos_session_search
+NEW ir.ui.view: point_of_sale.view_pos_session_tree
+DEL ir.ui.view: point_of_sale.product_normal_form_pos_view
+DEL ir.ui.view: point_of_sale.view_box_entries
+DEL ir.ui.view: point_of_sale.view_pos_box_out
+DEL ir.ui.view: point_of_sale.view_pos_close_statement
+DEL ir.ui.view: point_of_sale.view_report_cash_register_search
+DEL ir.ui.view: point_of_sale.view_report_cash_register_tree
+
+### Ignore new sequence records
+NEW ir.sequence: point_of_sale.seq_pos_session
+NEW ir.sequence.type: point_of_sale.seq_type_pos_session
+
+### Ignore added chatter configuration
+NEW mail.message: point_of_sale.module_install_notification
+
+### New pos_config. script migration will set it in a obsolete state.
+NEW pos.config: point_of_sale.pos_config_main
+
+### New product. Nothing to do.
+NEW product.product: point_of_sale.unreferenced_product
+
+### Completely new workflow for 'pos.dession'. Nothing to do.
+NEW workflow: point_of_sale.wkf_pos_session
+NEW workflow.activity: point_of_sale.act_close
+NEW workflow.activity: point_of_sale.act_closing_control
+NEW workflow.activity: point_of_sale.act_open
+NEW workflow.activity: point_of_sale.act_opening_control
+NEW workflow.activity: point_of_sale.act_start
+NEW workflow.transition: point_of_sale.trans_closing_control_to_close
+NEW workflow.transition: point_of_sale.trans_closing_control_to_close_no_control
+NEW workflow.transition: point_of_sale.trans_open_to_close
+NEW workflow.transition: point_of_sale.trans_open_to_closing_control
+NEW workflow.transition: point_of_sale.trans_opening_control_to_open
+NEW workflow.transition: point_of_sale.trans_start_to_open
+NEW workflow.transition: point_of_sale.trans_start_to_opening_control
 
=== added file 'point_of_sale/migrations/7.0.1.0.1/post-migration.py'
--- point_of_sale/migrations/7.0.1.0.1/post-migration.py	1970-01-01 00:00:00 +0000
+++ point_of_sale/migrations/7.0.1.0.1/post-migration.py	2013-10-14 23:49:02 +0000
@@ -0,0 +1,180 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    OpenERP, Open Source Management Solution
+#    This module copyright (C) 2013 Sylvain LE GAL
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+from openerp import pooler, SUPERUSER_ID
+from openerp.openupgrade import openupgrade, openupgrade_70
+
+def migrate_product_available_in_pos(cr, pool):
+    """ 
+    Initialize product that available_in_pos, using old behaviour of 
+    6.1 version. (In 6.1, product with pos_categ_id set are available in pos)
+    """
+    cr.execute("""UPDATE product_product 
+        SET available_in_pos = True
+        WHERE pos_categ_id is not Null""")
+
+def migrate_product_to_weight(cr, pool):
+    """
+    Initialize product_to_weight value in 'product.product', assuming that
+    product that have an uom weightable will be 'product_to_weight'.
+    """
+    product_product_obj = pool.get('product.product')
+    uom_categ, uom_categ_id = pool.get('ir.model.data').get_object_reference(
+        cr, SUPERUSER_ID, 'product', 'product_uom_categ_kgm')
+    uom_ids = pool.get('product.uom').search(cr, SUPERUSER_ID, 
+        [('category_id', '=', uom_categ_id)])
+    product_ids = product_product_obj.search(cr, SUPERUSER_ID, 
+        [('uom_id', 'in', uom_ids)])
+    product_product_obj.write(cr, SUPERUSER_ID, 
+        product_ids, 
+        {'to_weight' : True,})
+
+def create_pos_config(cr, pool):
+    """
+    pos_config is totally new concept. 
+    Script migration will create pos_config based on datas in point_of_sale (6.1)
+    (One pos_config by shop and sale_journal in 6.1.)
+    Script migration will create too on pos_config by sale user because it's not
+    possible to create 2 pos.session active with the same 'pos.config'.
+    """
+    pos_config_obj = pool.get('pos.config')
+    shop_obj = pool.get('sale.shop')
+    journal_obj = pool.get('account.journal')
+    user_obj = pool.get('res.users')
+    pos_session_obj = pool.get('pos.session')
+    pos_order_obj = pool.get('pos.order')
+    # Create a pos config by shop, and sale_journal
+    cr.execute("""
+        SELECT shop_id, sale_journal sale_journal_id, user_id
+        FROM pos_order 
+        GROUP BY shop_id, sale_journal, user_id;""")
+
+    for shop_id, sale_journal_id, user_id in cr.fetchall():
+        # Generate a readable name
+        name = shop_obj.read(cr, SUPERUSER_ID, [shop_id], ['name'])[0]['name']
+        name += " - " + journal_obj.read(cr, SUPERUSER_ID, [sale_journal_id], ['code'])[0]['code']
+        name += " - " + user_obj.read(cr, SUPERUSER_ID, [user_id], ['login'])[0]['login']
+
+        # Get the bank_journal related to new pos_config, using the history of pos_orders.
+        cr.execute("""
+            SELECT abs.journal_id bank_journal_id 
+            FROM account_bank_statement_line absl
+            INNER JOIN account_bank_statement abs ON abs.id = absl.statement_id
+            INNER JOIN pos_order po ON absl.pos_statement_id= po.id
+            WHERE po.shop_id = {0} and po.sale_journal = {1}
+            GROUP BY abs.journal_id; """.format(shop_id, sale_journal_id))
+
+        bank_journal_ids = []
+        for bank_journal_id in cr.fetchall():
+            bank_journal_ids.append(bank_journal_id[0])
+        pos_config_id = pos_config_obj.create(cr, SUPERUSER_ID, {
+            'name' : name, 
+            'journal_id': sale_journal_id, 
+            'shop_id': shop_id, 
+            'journal_ids' : [(6,0,bank_journal_ids)], 
+            })
+        
+        # associate user to new pos_config.
+        user_obj.write(cr, SUPERUSER_ID, [user_id], {'pos_config' : pos_config_id})
+        
+        # Get all statement_id associated to a pos.session.
+        cr.execute("""
+            SELECT absl.statement_id from account_bank_statement_line absl
+            INNER JOIN pos_order po on po.id = absl.pos_statement_id 
+            WHERE po.shop_id={0} AND po.user_id={1} AND po.sale_journal={2}
+            GROUP BY absl.statement_id;
+        """.format(shop_id, sale_journal_id, user_id))
+        statement_ids = []
+        for statement_id in cr.fetchall():
+            statement_ids.append(statement_id[0])
+
+        # create pos session by sql, to avoid automatic creation of account bank statement 
+        # and other behaviours not wished
+        cr.execute("""
+        INSERT INTO pos_session(
+            create_uid, create_date, write_uid, write_date, 
+            config_id, user_id, name, state,
+            start_at, stop_at, cash_journal_id, cash_register_id
+            )
+            VALUES (
+            {0}, {1}, {0}, {1}, 
+            {2}, {0}, '{3}', 'closed', 
+            Null, Null, Null, Null) 
+            RETURNING id;
+        """.format(user_id, 
+                "(now() at time zone 'UTC')", 
+                pos_config_id, 
+                name.encode('utf-8').strip(), 
+            ))
+        pos_session_id = cr.fetchone()[0]
+        if statement_ids !=[]:
+            #import pdb; pdb.set_trace()
+            for sub_ids in cr.split_for_in_conditions(statement_ids):
+                cr.execute("""
+                UPDATE account_bank_statement
+                SET pos_session_id = %s
+                WHERE id in %%s
+                """ % (pos_session_id), [tuple(sub_ids)])
+#        pos_session_id = pos_session_obj.create(cr, SUPERUSER_ID, {
+#            'config_id' : pos_config_id, 
+#            'start_at' : None, 
+#            'state' : 'closed', 
+#            'statement_ids' : [(6,0,statement_ids)],
+#            'stop_at' : None, 
+#            'user_id' : user_id,
+#            })
+
+        # associate pos.order to the new pos.session
+        cr.execute("""
+            UPDATE pos_order 
+            SET session_id={3}
+            WHERE shop_id={0} AND sale_journal={1} AND user_id={2};
+            """.format(shop_id, sale_journal_id, user_id, pos_session_id))
+
+#        cr.execute("""
+#            SELECT id
+#            FROM pos_order 
+#            WHERE shop_id={0} AND sale_journal={1} AND user_id={2};
+#            """.format(shop_id, sale_journal_id, user_id))
+#        pos_order_ids = []
+#        for pos_order_id in cr.fetchall():
+#            pos_order_ids.append(pos_order_id[0])
+#        
+#        pos_order_obj.write(cr, SUPERUSER_ID, pos_order_ids, {'session_id' : pos_session_id})
+
+def disable_default_pos_config(cr, pool):
+    pass
+    pos_config, pos_config_id = pool.get('ir.model.data').get_object_reference(
+        cr, SUPERUSER_ID, 'point_of_sale', 'pos_config_main')
+    pool.get('pos.config').write(cr, SUPERUSER_ID, 
+        [pos_config_id], 
+        {'state' : 'inactive'})
+
+
[e-mail]()
+def migrate(cr, version):
+    pool = pooler.get_pool(cr.dbname)
+    migrate_product_available_in_pos(cr, pool)
+    migrate_product_to_weight(cr, pool)
+    # create new pos_config
+    create_pos_config(cr, pool)
+    
+    # disable old default pos_config
+    disable_default_pos_config(cr, pool)
 
=== added file 'point_of_sale/migrations/7.0.1.0.1/pre-migration.py'
--- point_of_sale/migrations/7.0.1.0.1/pre-migration.py	1970-01-01 00:00:00 +0000
+++ point_of_sale/migrations/7.0.1.0.1/pre-migration.py	2013-10-14 23:49:02 +0000
@@ -0,0 +1,34 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    OpenERP, Open Source Management Solution
+#    This module copyright (C) 2013 Sylvain LE GAL
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+
+from openerp.openupgrade import openupgrade
+
+column_renames = {
+}
+
+xmlid_renames = [
+]
+
+
[e-mail]()
+def migrate(cr, version):
+    openupgrade.rename_columns(cr, column_renames)
+    openupgrade.rename_xmlids(cr, xmlid_renames)
 
=== added file 'point_of_sale/migrations/7.0.1.0.1/user_notes.txt'
--- point_of_sale/migrations/7.0.1.0.1/user_notes.txt	1970-01-01 00:00:00 +0000
+++ point_of_sale/migrations/7.0.1.0.1/user_notes.txt	2013-10-14 23:49:02 +0000
@@ -0,0 +1,14 @@
+Point of Sale V7 change the behaviour of the close of bank_statement.
+It's not possible in V7 to close a session if there is some unpaid pos order in the session.
+
+
+Point of sale V7 is very different. So The script migration will assume that
+The users have used the point of sale V6.1 like a point of sale V7. Specially for the following things : 
+- a user can not sale (use a pos.config) if the pos.config is in use by an other user.
+So script migration will create a pos.config for each sale user.
+
+- 
+
+WARNING : standart V7 doesn't allow to close a pos.session (and so associated account.bank.statement) if a pos.order is not paid in the pos.session.
+
+