Fix: Operador And y total_amount_cache != None

__init__.py

@@ -8,9 +8,9 @@ from . import statement
 from . import user
 from . import configuration_statement
 
+
 def register():
     Pool.register(
-        configuration_statement.Configuration,
         statement.Journal,
         statement.Statement,
         statement.Line,
@@ -27,6 +27,7 @@ def register():
         statement.CloseStatementStart,
         statement.CloseStatementDone,
         statement.PayInvoiceSupplierStart,
+        configuration_statement.Configuration,
         module='sale_payment', type_='model')
     Pool.register(
         sale.WizardSalePayment,

configuration_statement.py

@@ -2,41 +2,41 @@
 # this repository contains the full copyright notices and license terms.
 from trytond.model import ModelView, ModelSQL, ModelSingleton, fields
 from trytond.modules.currency.fields import Monetary
-from trytond.pyson import Eval
+# from trytond.pyson import Eval
 from trytond.transaction import Transaction
 from trytond.pool import Pool
 
+
 class Configuration(ModelSingleton, ModelView, ModelSQL):
     "Configuration Chas Closures"
     __name__ = 'sale.cash_closures'
 
-    company = fields.Many2One(
-        'company.company', "Company", required=True, select=True)
+    # company = fields.Many2One(
+    #     'company.company', "Company", required=True)
     currency = fields.Many2One(
             'currency.currency', "Currency")
-
     mismatch_limit = Monetary(
         "Mismatch Limit", currency='currency', digits='currency')
-    account_mismatch_charge = fields.Many2One('account.account', "Account Mismatch Charge",
-                              domain=[
-                                  ('company', '=', Eval('company', 0)),
-                                  ('type', '!=', None),
-                                  ('closed', '!=', True),
-                              ],)
+    account_mismatch_charge = fields.Many2One(
+        'account.account', "Account Mismatch Charge",
+        domain=[
+            ('type', '!=', None),
+            ('closed', '!=', True),
+        ],)
+    account_mismatch_positive = fields.Many2One(
+        'account.account', "Account Mismatch Positivo",
+        domain=[
+            ('type', '!=', None),
+            ('closed', '!=', True),
+        ],)
 
-    account_mismatch_positive = fields.Many2One('account.account', "Account Mismatch Positivo",
-                              domain=[
-                                  ('company', '=', Eval('company', 0)),
-                                  ('type', '!=', None),
-                                  ('closed', '!=', True),
-                              ],)
-    @staticmethod
-    def default_currency():
+    @classmethod
+    def default_currency(cls):
         Company = Pool().get('company.company')
         company = Transaction().context.get('company')
         if company:
             return Company(company).currency.id
 
-    @staticmethod
-    def default_company():
-        return Transaction().context.get('company')    
+    # @classmethod
+    # def default_company(cls):
+    #     return Transaction().context.get('company')

device.py

@@ -12,7 +12,7 @@ __all__ = ['SaleDevice', 'SaleDeviceStatementJournal']
 class SaleDevice(ModelSQL, ModelView):
     'Sale Device Configuration'
     __name__ = 'sale.device'
-    name = fields.Char('Device Name', required=True, select=True)
+    name = fields.Char('Device Name', required=True)
     shop = fields.Many2One('sale.shop', 'Shop', required=True)
     company = fields.Function(fields.Many2One('company.company', 'Company',),
         'get_company', searcher='search_company')
@@ -54,7 +54,7 @@ class SaleDeviceStatementJournal(ModelSQL):
     __name__ = 'sale.device.account.statement.journal'
     _table = 'sale_device_account_statement_journal'
     device = fields.Many2One('sale.device', 'Sale Device',
-            ondelete='CASCADE', select=True, required=True)
+            ondelete='CASCADE', required=True)
     journal = fields.Many2One('account.statement.journal', 'Statement Journal',
             ondelete='RESTRICT', required=True)
 

sale.py

@@ -2,6 +2,7 @@
 # The COPYRIGHT file at the top level of this repository contains the full
 # copyright notices and license terms.
 from decimal import Decimal
+from sql.operators import And
 from sql.aggregate import Sum
 from sql.conditionals import Coalesce
 
@@ -162,16 +163,18 @@ class Sale(metaclass=PoolMeta):
             condition=(sale.id == payline.sale)
             ).select(
                 sale.id,
-                where=((sale.total_amount_cache != None) &
-                       (sale.state.in_([
-                           'draft',
-                           'quotation',
-                           'confirmed',
-                           'processing',
-                           'done']))),
+                where=(And([
+                    sale.total_amount_cache != None,
+                    sale.state.in_([
+                        'draft',
+                        'quotation',
+                        'confirmed',
+                        'processing',
+                        'done'])])),
                 group_by=(sale.id),
-                having=(Operator(sale.total_amount_cache -
-                                 Sum(Coalesce(payline.amount, 0)), value)
+                having=(Operator(
+                    sale.total_amount_cache - Sum(
+                        Coalesce(payline.amount, 0)), value)
                         ))
         return [('id', 'in', query)]
 
@@ -196,17 +199,17 @@ class Sale(metaclass=PoolMeta):
 class ChangePaymentMethodForm(ModelView):
     'Change Payments Method form'
     __name__ = 'sale.change_payment_method.form'
+
     sale = fields.Many2One('sale.sale', "Sale", required=True)
     statement = fields.Many2One('account.statement', 'Statement Journal',
                                 domain=[
                                     ('id', 'in', Eval('statements', [])),
-                                ],
-                                depends=['journals'], required=True)
+                                ], required=True)
     statements = fields.One2Many('account.statement', None,
                                  'Allowed Statement', readonly=True)
     payment = fields.Many2One(
         'account.statement.line', "Payment", required=True,
-        domain=[('sale.id', '=', Eval('sale'))])
+        domain=[('sale', '=', Eval('sale'))])
 
 
 class ChangePaymentMethod(Wizard):
@@ -379,8 +382,8 @@ class WizardSaleReconcile(Wizard):
                 if not payment.move:
                     continue
                 for line in payment.move.lines:
-                    if (not line.reconciliation and
-                            line.account == account):
+                    if (not line.reconciliation and (
+                            line.account == account)):
                         lines.append(line)
                         amount += line.debit - line.credit
             if lines and amount == Decimal('0.0'):

setup.py

@@ -1,112 +1,70 @@
-#!/usr/bin/env python
-# encoding: utf-8
-
-from setuptools import setup
-import re
-import os
+#!/usr/bin/env python3
 import io
+import os
+import re
 from configparser import ConfigParser
 
-MODULE = 'sale_payment'
-PREFIX = 'trytonspain'
-MODULE2PREFIX = {'sale_shop': 'trytonzz'}
-OWNER = {
-    'nantic':'NaN-tic',
-    'trytonzz':'nanticzz',
-}
+from setuptools import find_packages, setup
+
+MODULE2PREFIX = {}
 
 
 def read(fname):
-    return io.open(
+    content = io.open(
         os.path.join(os.path.dirname(__file__), fname),
         'r', encoding='utf-8').read()
+    content = re.sub(
+        r'(?m)^\.\. toctree::\r?\n((^$|^\s.*$)\r?\n)*', '', content)
+    return content
 
 
 def get_require_version(name):
-    if minor_version % 2:
-        require = '%s >= %s.%s.dev0, < %s.%s'
-    else:
-        require = '%s >= %s.%s, < %s.%s'
+    require = '%s >= %s.%s, < %s.%s'
     require %= (name, major_version, minor_version,
         major_version, minor_version + 1)
     return require
 
-def get_requires(depends='depends'):
-  requires = []
-  for dep in info.get(depends, []):
-      if not re.match(r'(ir|res)(\W|$)', dep):
-          prefix = MODULE2PREFIX.get(dep, 'trytond')
-          owner = OWNER.get(prefix, prefix)
-          if prefix == 'trytond':
-              requires.append(get_require_version('%s_%s' % (prefix, dep)))
-          else:
-              requires.append(
-                  '%(prefix)s-%(dep)s@git+https://github.com/%(owner)s/'
-                  'trytond-%(dep)s.git@%(branch)s'
-                  '#egg=%(prefix)s-%(dep)s-%(series)s'%{
-                          'prefix': prefix,
-                          'owner': owner,
-                          'dep':dep,
-                          'branch': branch,
-                          'series': series,})
-
-  return requires
 
 config = ConfigParser()
-config.readfp(open('tryton.cfg'))
+config.read_file(open(os.path.join(os.path.dirname(__file__), 'tryton.cfg')))
 info = dict(config.items('tryton'))
 for key in ('depends', 'extras_depend', 'xml'):
     if key in info:
         info[key] = info[key].strip().splitlines()
-
 version = info.get('version', '0.0.1')
 major_version, minor_version, _ = version.split('.', 2)
 major_version = int(major_version)
 minor_version = int(minor_version)
+name = 'trytondo_sale_payment'
 
 requires = []
+for dep in info.get('depends', []):
+    if not re.match(r'(ir|res)(\W|$)', dep):
+        prefix = MODULE2PREFIX.get(dep, 'trytond')
+        requires.append(get_require_version('%s_%s' % (prefix, dep)))
+requires.append(get_require_version('trytond'))
 
-series = '%s.%s' % (major_version, minor_version)
-if minor_version % 2:
-    branch = 'master'
-else:
-    branch = series
+tests_require = [get_require_version('proteus')]
 
-requires += get_requires('depends')
-
-tests_require = [
-    get_require_version('proteus'),
-
-    ]
-tests_require += get_requires('extras_depend')
-requires += [get_require_version('trytond_account_statement')]
-
-dependency_links = []
-
-if minor_version % 2:
-    # Add development index for testing with proteus
-    dependency_links.append('https://trydevpi.tryton.org/')
-
-setup(name='%s_%s' % (PREFIX, MODULE),
+setup(name=name,
     version=version,
-    description='',
-    long_description=read('README'),
-    author='trytonspain',
-    url='http://www.nan-tic.com/',
-    download_url='https://github.com:trytonspain/trytond-sale_payment',
-    package_dir={'trytond.modules.%s' % MODULE: '.'},
-    packages=[
-        'trytond.modules.%s' % MODULE,
-        'trytond.modules.%s.tests' % MODULE,
-        ],
+    description='Fork NaNtic sale_payment',
+    long_description=read('README.rst'),
+    author='OneCluster',
+    author_email='info@onecluster.org',
+    url='http://www.tryton.org/',
+    keywords='',
+    package_dir={'trytond.modules.sale_payment': '.'},
+    packages=(
+        ['trytond.modules.sale_payment']
+        + ['trytond.modules.sale_payment.%s' % p
+            for p in find_packages()]
+        ),
     package_data={
-        'trytond.modules.%s' % MODULE: (info.get('xml', [])
-            + ['tryton.cfg', 'locale/*.po', 'tests/*.rst', 'view/*.xml',
-            'icons/*.svg']),
+        'trytond.modules.sale_payment': (info.get('xml', [])
+            + ['tryton.cfg', 'view/*.xml', 'locale/*.po', '*.fodt',
+                'icons/*.svg', 'tests/*.rst', 'tests/*.json']),
         },
-    project_urls = {
-       "Source Code": 'https://github.com:trytonspain/trytond-sale_payment'
-    },
     classifiers=[
         'Development Status :: 5 - Production/Stable',
         'Environment :: Plugins',
@@ -114,25 +72,48 @@ setup(name='%s_%s' % (PREFIX, MODULE),
         'Intended Audience :: Developers',
         'Intended Audience :: Financial and Insurance Industry',
         'Intended Audience :: Legal Industry',
-        'License :: OSI Approved :: GNU General Public License (GPL)',
+        'License :: OSI Approved :: '
+        'GNU General Public License v3 or later (GPLv3+)',
+        'Natural Language :: Bulgarian',
         'Natural Language :: Catalan',
+        'Natural Language :: Chinese (Simplified)',
+        'Natural Language :: Czech',
+        'Natural Language :: Dutch',
         'Natural Language :: English',
+        'Natural Language :: Finnish',
+        'Natural Language :: French',
+        'Natural Language :: German',
+        'Natural Language :: Hungarian',
+        'Natural Language :: Indonesian',
+        'Natural Language :: Italian',
+        'Natural Language :: Persian',
+        'Natural Language :: Polish',
+        'Natural Language :: Portuguese (Brazilian)',
+        'Natural Language :: Romanian',
+        'Natural Language :: Russian',
+        'Natural Language :: Slovenian',
         'Natural Language :: Spanish',
+        'Natural Language :: Turkish',
+        'Natural Language :: Ukrainian',
         'Operating System :: OS Independent',
-        'Programming Language :: Python :: 3.6',
-        'Programming Language :: Python :: 3.7',
+        'Programming Language :: Python :: 3',
+        'Programming Language :: Python :: 3.8',
+        'Programming Language :: Python :: 3.9',
+        'Programming Language :: Python :: 3.10',
+        'Programming Language :: Python :: 3.11',
+        'Programming Language :: Python :: 3.12',
+        'Programming Language :: Python :: Implementation :: CPython',
         'Topic :: Office/Business',
         ],
     license='GPL-3',
-    install_requires=requires,
-    dependency_links=dependency_links,
+    python_requires='>=3.8',
+    # install_requires=requires,
+    extras_require={
+        'test': tests_require,
+        },
     zip_safe=False,
     entry_points="""
     [trytond.modules]
-    %s = trytond.modules.%s
-    """ % (MODULE, MODULE),
-    test_suite='tests',
-    test_loader='trytond.test_loader:Loader',
-    tests_require=tests_require,
-
+    sale_payment = trytond.modules.sale_payment
+    """,  # noqa: E501
     )

statement.py

@@ -4,7 +4,8 @@
 from trytond.model import fields, ModelView
 from trytond.pool import Pool, PoolMeta
 from trytond.transaction import Transaction
-from trytond.wizard import Button, StateTransition, StateAction, StateView, Wizard
+from trytond.wizard import (
+    Button, StateTransition, StateAction, StateView, Wizard)
 from decimal import Decimal
 from trytond.i18n import gettext
 from trytond.modules.currency.fields import Monetary
@@ -52,7 +53,7 @@ class Statement(metaclass=PoolMeta):
 
         query = statement.join(
                 journal, condition=statement.journal == journal.id).join(
-                device_journal,
+                    device_journal,
                     condition=journal.id == device_journal.journal).join(
                 device, condition=device_journal.device == device.id).join(
                 user, condition=device.id == user.sale_device).select(
@@ -121,6 +122,7 @@ class OpenStatementStart(ModelView):
     'Open Statement'
     __name__ = 'open.statement.start'
 
+
 class OpenStatementDone(ModelView):
     'Open Statement'
     __name__ = 'open.statement.done'
@@ -172,7 +174,8 @@ class OpenStatement(Wizard):
             for journal in device.journals:
                 if journal not in journals_of_draft_statements:
                     values = {
-                        'name': '%s - %s' % (device.rec_name, journal.rec_name),
+                        'name': '%s - %s' % (
+                            device.rec_name, journal.rec_name),
                         'journal': journal.id,
                         'company': user.company.id,
                         'start_balance': start_balances.get(journal.id,
@@ -185,8 +188,10 @@ class OpenStatement(Wizard):
                     results.append(gettext('sale_payment.open_statement',
                         statement=journal.rec_name))
                 else:
-                    results.append(gettext('sale_payment.statement_already_opened',
-                        statement=journal.rec_name))
+                    results.append(
+                        gettext(
+                            'sale_payment.statement_already_opened',
+                            statement=journal.rec_name))
             statements.extend(Statement.create(vlist))
             self.result = '\n'.join(results)
         else:
@@ -223,7 +228,6 @@ class CloseStatement(Wizard):
             Button('Done', 'end', 'tryton-ok', default=True),
             ])
 
-            
     def default_done(self, fields):
         return {
             'result': self.result,
@@ -232,7 +236,7 @@ class CloseStatement(Wizard):
     def default_start(self, fields):
         pool = Pool()
         User = pool.get('res.user')
-        StatementLine = pool.get('statement.line')
+        # StatementLine = pool.get('statement.line')
         Statement = pool.get('account.statement')
         statementLines = []
         user = Transaction().user
@@ -258,7 +262,8 @@ class CloseStatement(Wizard):
                     'journal': s.journal.id,
                     'start_balance': s.start_balance,
                     'balance': end_balance,
-                    'end_balance':end_balance+s.start_balance
+                    'end_balance': (
+                        end_balance + s.start_balance)
                 }
                 statementLines.append(line)
 
@@ -305,10 +310,10 @@ class CloseStatement(Wizard):
                     if account and transfer:
                         end_balance = abs(end_balance) - abs(transfer)
                         statement.end_balance = end_balance
-                        lines = statement.lines
+                        # lines = statement.lines
                         conciliation = tuple([StatementLine(
                             date=datetime.today().date(),
-                            amount= abs(transfer)*-1,
+                            amount=abs(transfer) * -1,
                             account=account.id)]
                         )
                         statement.lines = statement.lines + conciliation
@@ -316,17 +321,22 @@ class CloseStatement(Wizard):
                     if mismatch and mismatch.real > 0:
                         pass
 
-                    if (config.mismatch_limit and config.account_mismatch_charge):
-                        if mismatch and (abs(mismatch) >= abs(config.mismatch_limit.real)):
-                            lines = statement.lines
-                            if employee_party == None:
-                                raise UserError(str("Debe definir un Empleado para el Usuario."))
+                    if config.mismatch_limit and (
+                            config.account_mismatch_charge):
+                        if mismatch and (abs(mismatch) >= abs(
+                                config.mismatch_limit.real)):
+                            # lines = statement.lines
+                            if employee_party is None:
+                                raise UserError(str(
+                                    "Debe definir un Empleado para el Usuario."
+                                ))
                             conciliation_mismatch = tuple([StatementLine(
                                 date=datetime.today().date(),
                                 amount=mismatch,
                                 account=config.account_mismatch_charge.id)]
                                                           )
-                            statement.lines = statement.lines + conciliation_mismatch
+                            statement.lines =\
+                                statement.lines + conciliation_mismatch
                             end_balance = abs(end_balance) - abs(mismatch)
 
                     statement.end_balance = end_balance
@@ -335,8 +345,10 @@ class CloseStatement(Wizard):
                     results.append(gettext('sale_payment.close_statement',
                             statement=statement.rec_name))
                 elif statement:
-                    results.append(gettext('sale_payment.statement_already_closed',
-                        statement=statement.rec_name))
+                    results.append(
+                        gettext(
+                            'sale_payment.statement_already_closed',
+                            statement=statement.rec_name))
                 else:
                     results.append(gettext('sale_payment.not_statement_found',
                         journal=journal.rec_name))
@@ -355,31 +367,34 @@ class StatementLine(ModelView):
 
     _states = {'readonly': True}
     company = fields.Many2One(
-        'company.company', "Company", required=True, select=True)
+        'company.company', "Company", required=True)
     journal = fields.Many2One('account.statement.journal', 'Journal',
-                              required=True, select=True,
+                              required=True,
                               states=_states)
     currency = fields.Many2One(
             'currency.currency', "Currency")
     start_balance = Monetary(
-        "Start Balance", currency='currency', digits='currency', states=_states)
+        "Start Balance", currency='currency', digits='currency',
+        states=_states)
     balance = Monetary(
         "Balance", currency='currency', digits='currency', states=_states)
     end_balance = Monetary(
-            "End Balance", currency='currency', digits='currency', readonly=True)
+        "End Balance", currency='currency', digits='currency',
+        readonly=True)
     transfer = Monetary(
         "Transfer", currency='currency', digits='currency')
     real_cash = Monetary(
         "Real Cash", currency='currency', digits='currency')
     mismatch = Monetary(
         "Mismatch", currency='currency', digits='currency', readonly=True)
-    account = fields.Many2One('account.account', "Account",
-                              domain=[
-                                  ('company', '=', Eval('company', 0)),
-                                  ('type', '!=', None),
-                                  ('closed', '!=', True),
-                              ],
-                              states={'required': If(Eval('transfer', True), True)})
+    account = fields.Many2One(
+        'account.account', "Account",
+        domain=[
+            ('company', '=', Eval('company', 0)),
+            ('type', '!=', None),
+            ('closed', '!=', True),
+        ],
+        states={'required': Eval('transfer', True)})
 
     @staticmethod
     def default_currency():
@@ -394,25 +409,31 @@ class StatementLine(ModelView):
 
     @fields.depends('end_balance', 'real_cash', 'mismatch')
     def on_change_real_cash(self):
-        self.mismatch = self.real_cash - self.end_balance
+        if self.real_cash and self.end_balance:
+            self.mismatch = self.real_cash - self.end_balance
+
 
 class PayInvoiceSupplierStart(ModelView):
     'Payment Invoice To Supplier'
     __name__ = 'pay_invoice.statement.start'
 
-    invoice_to_pay = fields.One2Many('line_invoice.pay', None, "Invoice To Pay")
+    invoice_to_pay = fields.One2Many(
+        'line_invoice.pay', None, "Invoice To Pay")
+
 
 class PayInvoiceSupplier(Wizard):
     'Payment Invoice To Supplier'
     __name__ = 'pay_invoice.statement'
 
-    start = StateView('pay_invoice.statement.start',
-                      'sale_payment.pay_invoice_statement_start', [
-                          Button('Cancel',  'end', 'tryton-cancel'),
-                          Button('Pay', 'pay', 'tryton-ok', default=True),
-                          ])
+    start = StateView(
+        'pay_invoice.statement.start',
+        'sale_payment.pay_invoice_statement_start', [
+            Button('Cancel', 'end', 'tryton-cancel'),
+            Button('Pay', 'pay', 'tryton-ok', default=True),
+        ])
 
     pay = StateAction('account_statement.act_statement_form')
+
     def do_pay(self, action):
         pool = Pool()
         User = pool.get('res.user')
@@ -437,12 +458,12 @@ class PayInvoiceSupplier(Wizard):
                 party = pay.party
                 amount = pay.amount
                 statement = draft_statements.get(journal)
-                lines = statement.lines
+                # lines = statement.lines
                 pay_to_add = tuple([StatementLine(
                     date=datetime.today().date(),
                     party=party,
                     related_to=invoice,
-                    amount= abs(amount)*-1,
+                    amount=abs(amount) * -1,
                     account=account.id)])
                 statement.lines = statement.lines + pay_to_add
                 statement.save()
@@ -453,10 +474,11 @@ class LinesInvoiceToPay(ModelView):
     __name__ = 'line_invoice.pay'
 
     company = fields.Many2One(
-        'company.company', "Company", required=True, select=True)
+        'company.company', "Company", required=True)
     journal = fields.Many2One('account.statement.journal', 'Journal',
-                              required=True, select=True)
-    amount = Monetary("Amount", currency='currency', digits='currency', required=True)
+                              required=True)
+    amount = Monetary(
+        "Amount", currency='currency', digits='currency', required=True)
     party = fields.Many2One('party.party', "Party", required=True,
             context={'company': Eval('company', -1)},)
     invoice = fields.Reference(
@@ -471,10 +493,10 @@ class LinesInvoiceToPay(ModelView):
                 If(Bool(Eval('account')),
                     ('account', '=', Eval('account')),
                     ()),
-                If(Eval('statement_state') == 'draft',
-                    ('state', '=', 'posted'),
-                    ('state', '!=', '')),
-                ],},
+                # If(Eval('statement_state') == 'draft',
+                #     ('state', '=', 'posted'),
+                #     ('state', '!=', '')),
+            ]},
         context={'with_payment': False})
     account = fields.Many2One('account.account', "Account",
                               domain=[
@@ -483,18 +505,29 @@ class LinesInvoiceToPay(ModelView):
                                   ('closed', '!=', True),
                               ],)
     description = fields.Char("Description")
+    currency = fields.Many2One(
+        'currency.currency', 'Currency', required=True)
 
     @staticmethod
     def default_company():
         return Transaction().context.get('company')
 
+    @classmethod
+    def default_currency(cls, **pattern):
+        pool = Pool()
+        Company = pool.get('company.company')
+        company = pattern.get('company')
+        if not company:
+            company = cls.default_company()
+        if company:
+            return Company(company).currency.id
+
     @classmethod
     def _get_relations(cls):
         "Return a list of Model names for related_to Reference"
 
         return ['account.invoice']
 
-
     @classmethod
     def get_relations(cls):
         Model = Pool().get('ir.model')

statement.xml

@@ -4,25 +4,6 @@ The COPYRIGHT file at the top level of this repository contains the full
 copyright notices and license terms. -->
 <tryton>
     <data>
-	<record model="ir.action.act_window" id="act_configuration_closures_form">
-            <field name="name">Configuration Closures</field> 
-            <field name="res_model">sale.cash_closures</field> 
-        </record>
-        <record model="ir.ui.view" id="configuration_closures_view_form">
-            <field name="model">sale.cash_closures</field> 
-            <field name="type">form</field>
-            <field name="name">configuration_closures_form</field>
-        </record>
-        <record model="ir.action.act_window.view" id="act_prospect_form_view1">
-            <field name="sequence" eval="10"/>
-            <field name="view" ref="configuration_closures_view_form"/>
-            <field name="act_window" ref="act_configuration_closures_form"/>
-        </record>
-        <menuitem
-            parent="sale.menu_configuration"
-            sequence="10"
-            id="menu_configuration_closure"
-            action="act_configuration_closures_form"/>
         <record model="ir.action.act_window" id="act_sale_statement_form">
             <field name="name">Statements</field>
             <field name="res_model">account.statement</field>
@@ -87,11 +68,6 @@ copyright notices and license terms. -->
             <field name="name">Open Statements</field>
             <field name="wiz_name">open.statement</field>
         </record>
-	<record model="ir.ui.view" id="statement_line_view_form">
-            <field name="model">statement.line</field>
-            <field name="type">form</field>
-            <field name="name">statement_line_form</field>
-        </record>
         <record model="ir.ui.view" id="statement_line_view_tree_sequence">
             <field name="model">statement.line</field>
             <field name="type">tree</field>
@@ -148,6 +124,26 @@ copyright notices and license terms. -->
             <field name="group" ref="account_statement.group_statement"/>
         </record>
 
+        <record model="ir.action.act_window" id="act_configuration_closures_form">
+            <field name="name">Configuration Closures</field>
+            <field name="res_model">sale.cash_closures</field>
+        </record>
+        <record model="ir.ui.view" id="configuration_closures_view_form">
+            <field name="model">sale.cash_closures</field>
+            <field name="type">form</field>
+            <field name="name">configuration_closures_form</field>
+        </record>
+        <record model="ir.action.act_window.view" id="act_prospect_form_view1">
+            <field name="sequence" eval="10"/>
+            <field name="view" ref="configuration_closures_view_form"/>
+            <field name="act_window" ref="act_configuration_closures_form"/>
+        </record>
+        <menuitem
+            parent="sale.menu_configuration"
+            sequence="10"
+            id="menu_configuration_closure"
+            action="act_configuration_closures_form"/>
+
         <record model="ir.model.access" id="access_sale_statement">
             <field name="model" search="[('model', '=', 'account.statement')]"/>
             <field name="group" ref="sale.group_sale"/>

tests/scenario_sale_payment.rst

@@ -19,7 +19,6 @@ Imports::
     >>> today = datetime.date.today()
 
 Install sale::
-
     >>> config = activate_modules(['party', 'sale_payment'])
 
 Create company::

tests/test_scenario.py

@@ -20,4 +20,5 @@ def load_tests(loader, tests, pattern):
                     optionflags=doctest.REPORT_ONLY_FIRST_FAILURE))
     finally:
         os.chdir(cwd)
+
     return tests

tryton.cfg

@@ -1,11 +1,11 @@
 [tryton]
-version=6.4.0
+version=6.8.0
 depends:
     account_statement
     sale_shop
 xml:
+    statement.xml
     device.xml
     sale.xml
-    statement.xml
     user.xml
     message.xml

view/close_statement_start_form.xml

@@ -3,8 +3,7 @@
 The COPYRIGHT file at the top level of this repository contains the full
 copyright notices and license terms. -->
 <form>
-    <image name="tryton-info"
-	   xalign="1" yalign="0.0" xexpand="0" xfill="0"/>
+    <image name="tryton-info" xexpand="0" xfill="0"/>
     <newline/>
     <label
         string="You are going to close statements of your device."

view/statement_line_tree.xml

@@ -1,9 +0,0 @@
-<?xml version="1.0"?>
-<!-- This file is part sale_pos module for Tryton. The COPYRIGHT file at the top level of
-     this repository contains the full copyright notices and license terms. -->
-<tree editable="1">
-    <field name="journal"/>
-    <field name="start_balance"/>
-    <field name="current_balance"/>
-    <field name="account"/>
-</tree>