diff --git a/README.rst b/README.rst index b70ba37..f15f7ab 100644 --- a/README.rst +++ b/README.rst @@ -14,6 +14,29 @@ Requires Changes ======= +*6.0.5 - 06.02.2023* + +- fix: values on non-asset-cashbooks + +*6.0.4 - 05.02.2023* + +- add: investment to evaluation-types + +*6.0.3 - 08.11.2022* + +- add: cashbook-categories for evaluation +- updt: optimized update of evaluation with existing dashboard-actions +- updt: limit bookings in evaluation-result to today + +*6.0.2 - 05.11.2022* + +- evaluation-line: permissions optimized +- evaluation: sorting by sequence + +*6.0.1 - 05.11.2022* + +- works + *6.0.0 - 28.10.2022* - init diff --git a/__init__.py b/__init__.py index e4f23b7..4706fe2 100644 --- a/__init__.py +++ b/__init__.py @@ -9,6 +9,8 @@ from .line import EvaluationLine from .currency import Currency from .evaluation_context import EvaluationContext from .evaluation_wizard import OpenChartWizard +from .investment import InvestmentEvaluation, InvestmentLine + def register(): Pool.register( @@ -20,3 +22,7 @@ def register(): Pool.register( OpenChartWizard, module='cashbook_report', type_='wizard') + Pool.register( + InvestmentEvaluation, + InvestmentLine, + module='cashbook_report', type_='model', depends=['cashbook_investment']) diff --git a/evaluation.py b/evaluation.py index 491a313..e7a78ab 100644 --- a/evaluation.py +++ b/evaluation.py @@ -3,21 +3,16 @@ # The COPYRIGHT file at the top level of this repository contains the # full copyright notices and license terms. -from trytond.model import ModelView, ModelSQL, fields +from trytond.model import ModelView, ModelSQL, fields, sequence_ordered from trytond.pyson import Eval from trytond.transaction import Transaction from trytond.pool import Pool +from trytond.i18n import gettext from .colors import sel_color as sel_bgcolor -from .templates import template_view_graph, template_view_line +from .templates import template_view_graph, template_view_line, \ + cashbook_types, category_types -sel_etype = [ - ('cashbooks', 'Cashbooks'), - ('types', 'Types of Cashbooks'), - ('currencies', 'Currencys'), - #('category', 'Category'), - ] - sel_chart = [ ('vbar', 'Vertical Bars'), ('hbar', 'Horizontal Bars'), @@ -36,7 +31,7 @@ sel_maincolor = [ ] -class Evaluation(ModelSQL, ModelView): +class Evaluation(sequence_ordered(), ModelSQL, ModelView): 'Evaluation' __name__ = 'cashbook_report.evaluation' @@ -44,7 +39,7 @@ class Evaluation(ModelSQL, ModelView): required=True, ondelete="RESTRICT") name = fields.Char(string='Name', required=True) dtype = fields.Selection(string='Data type', required=True, - sort=False, selection=sel_etype, + sort=True, selection='get_sel_etype', help='Type of data displayed') dtype_string = dtype.translated('dtype') chart = fields.Selection(string='Chart type', required=True, @@ -57,7 +52,6 @@ class Evaluation(ModelSQL, ModelView): bgcolor = fields.Selection(string='Background Color', required=True, help='Background color of the chart area.', sort=False, selection=sel_bgcolor) - posted = fields.Boolean(string='Posted', help='Posted amounts only.') currency = fields.Many2One(string='Currency', ondelete='RESTRICT', model_name='currency.currency') @@ -65,7 +59,7 @@ class Evaluation(ModelSQL, ModelView): relation_name='cashbook_report.eval_line', origin='evaluation', target='cashbook', states={ - 'invisible': Eval('dtype', '') != 'cashbooks', + 'invisible': ~Eval('dtype', '').in_(cashbook_types), }, depends=['dtype']) types = fields.Many2Many(string='Types', relation_name='cashbook_report.eval_line', @@ -80,6 +74,12 @@ class Evaluation(ModelSQL, ModelView): states={ 'invisible': Eval('dtype', '') != 'currencies', }, depends=['dtype']) + categories = fields.Many2Many(string='Categories', + relation_name='cashbook_report.eval_line', + origin='evaluation', target='category', + states={ + 'invisible': ~Eval('dtype', '').in_(category_types), + }, depends=['dtype']) line_values = fields.One2Many(string='Line Values', field='evaluation', readonly=True, @@ -108,12 +108,6 @@ class Evaluation(ModelSQL, ModelView): def default_company(): return Transaction().context.get('company') or None - @classmethod - def default_posted(cls): - """ default: False - """ - return False - @classmethod def default_bgcolor(cls): """ default: Yellow 5 @@ -144,6 +138,17 @@ class Evaluation(ModelSQL, ModelView): """ return 'pie' + @classmethod + def get_sel_etype(cls): + """ get list of evaluation-types + """ + return [ + ('cashbooks', gettext('cashbook_report.msg_dtype_cashbook')), + ('types', gettext('cashbook_report.msg_dtype_type')), + ('currencies', gettext('cashbook_report.msg_dtype_currency')), + ('categories', gettext('cashbook_report.msg_dtype_category')), + ] + @classmethod def get_create_view_data(cls, evaluation): """ generate dictionary to create view-xml @@ -200,10 +205,13 @@ class Evaluation(ModelSQL, ModelView): ActWin = pool.get('ir.action.act_window') ActView = pool.get('ir.action.act_window.view') Evaluation2 = pool.get('cashbook_report.evaluation') - - cls.uiview_delete(evaluations) + try : + DashboardAction = pool.get('dashboard.action') + except: + DashboardAction = None to_write_eval = [] + to_write_dbaction = [] for evaluation in evaluations: with Transaction().set_context({ '_check_access': False, @@ -233,6 +241,24 @@ class Evaluation(ModelSQL, ModelView): 'dashb_actview': dashb_actview.id, }]) + # prepare update dasboard-action + if DashboardAction is not None: + if evaluation.dashb_actwin: + db_actions = DashboardAction.search([ + ('act_window.id', '=', evaluation.dashb_actwin.id), + ]) + if len(db_actions) > 0: + to_write_dbaction.extend([ + db_actions, + { + 'act_window': dashb_actwin.id, + }]) + + if len(to_write_dbaction) > 0: + DashboardAction.write(*to_write_dbaction) + + cls.uiview_delete(evaluations) + if len(to_write_eval) > 0: Evaluation2.write(*to_write_eval) @@ -264,33 +290,16 @@ class Evaluation(ModelSQL, ModelView): if evaluation.dtype == values['dtype']: continue - if (values['dtype'] != 'cashbooks') and \ - (len(evaluation.cashbooks) > 0): - to_write.extend([ - [evaluation], - { - 'cashbooks': [ - ('remove', [x.id for x in evaluation.cashbooks]) - ], - }]) - - if (values['dtype'] != 'types') and (len(evaluation.types) > 0): - to_write.extend([ - [evaluation], - { - 'types': [ - ('remove', [x.id for x in evaluation.types]) - ], - }]) - - if (values['dtype'] != 'currencies') and (len(evaluation.currencies) > 0): - to_write.extend([ - [evaluation], - { - 'currencies': [ - ('remove', [x.id for x in evaluation.currencies]) - ], - }]) + for dt in ['cashbooks', 'types', 'currencies', 'categories']: + if (not values['dtype'].startswith(dt)) and \ + (len(getattr(evaluation, dt)) > 0): + to_write.extend([ + [evaluation], + { + dt: [ + ('remove', [x.id for x in getattr(evaluation, dt)]) + ], + }]) args = list(args) args.extend(to_write) diff --git a/investment.py b/investment.py new file mode 100644 index 0000000..2bfb384 --- /dev/null +++ b/investment.py @@ -0,0 +1,149 @@ +# -*- coding: utf-8 -*- +# This file is part of the cashbook-module from m-ds for Tryton. +# The COPYRIGHT file at the top level of this repository contains the +# full copyright notices and license terms. + +from trytond.pool import PoolMeta, Pool +from trytond.i18n import gettext +from decimal import Decimal + + +class InvestmentEvaluation(metaclass=PoolMeta): + __name__ = 'cashbook_report.evaluation' + + @classmethod + def get_sel_etype(cls): + """ get list of evaluation-types + """ + result = super(InvestmentEvaluation, cls).get_sel_etype() + result.extend([ + ('cashbooks_gldiff', gettext('cashbook_report.msg_dtype_cashbook_gldiff')), + ('cashbooks_glperc', gettext('cashbook_report.msg_dtype_cashbook_glperc')), + ('cashbooks_glvalue', gettext('cashbook_report.msg_dtype_cashbooks_glvalue')), + ('category_gldiff', gettext('cashbook_report.msg_dtype_category_gldiff')), + ('category_glvalue', gettext('cashbook_report.msg_dtype_category_glvalue')), + ('category_glperc', gettext('cashbook_report.msg_dtype_category_glperc')), + ]) + return result + +# end InvestmentEvaluation + + +class InvestmentLine(metaclass=PoolMeta): + __name__ = 'cashbook_report.eval_line' + + def get_value_category_glperc(self): + """ get percentual difference of bookings in categories + converted to currency of evaluation + """ + Book = Pool().get('cashbook.book') + + if self.category is None: + return None + + books = Book.search([ + ('state', '=', 'open'), + ('categories.id', '=', self.category.id), + ]) + value = Decimal('0.0') + amount = Decimal('0.0') + + if len(books) > 0: + value = sum([x.current_value_ref for x in books + if (x.current_value_ref is not None) and (x.feature == 'asset')]) + amount = sum([x.balance_ref for x in books + if (x.balance_ref is not None) and (x.feature == 'asset')]) + if amount != Decimal('0.0'): + return self.convert_to_evalcurrency( + books[0].company.currency, + Decimal('100.0') * value / amount - Decimal('100.0'), + ) + return Decimal('0.0') + + def get_value_category_glvalue(self): + """ get current value of bookings in categories + converted to currency of evaluation + """ + Book = Pool().get('cashbook.book') + + if self.category is None: + return None + + books = Book.search([ + ('state', '=', 'open'), + ('categories.id', '=', self.category.id), + ]) + result = Decimal('0.0') + if len(books) > 0: + for book in books: + if book.feature == 'asset': + if book.current_value_ref is not None: + result += book.current_value_ref + else : + if book.balance_ref is not None: + result += book.balance_ref + return self.convert_to_evalcurrency( + books[0].company.currency, result) + return result + + def get_value_category_gldiff(self): + """ get difference amount of bookings in categories + converted to currency of evaluation + """ + Book = Pool().get('cashbook.book') + + if self.category is None: + return None + + books = Book.search([ + ('state', '=', 'open'), + ('categories.id', '=', self.category.id), + ]) + result = Decimal('0.0') + if len(books) > 0: + result = sum([x.current_value_ref - x.balance_ref for x in books + if (x.current_value_ref is not None) and \ + (x.balance_ref is not None) and \ + (x.feature == 'asset')]) + result = self.convert_to_evalcurrency( + books[0].company.currency, result) + return result + + def get_value_cashbooks_gldiff(self): + """ amount of profit/loss of cashbooks + """ + Currency = Pool().get('currency.currency') + + if self.cashbook: + if self.cashbook.feature == 'asset': + return self.convert_to_evalcurrency( + self.cashbook.currency, + self.cashbook.diff_amount) + else : + return Decimal('0.0') + + def get_value_cashbooks_glvalue(self): + """ current value of cashbooks + """ + Currency = Pool().get('currency.currency') + + if self.cashbook: + if self.cashbook.feature == 'asset': + return self.convert_to_evalcurrency( + self.cashbook.currency, + self.cashbook.current_value) + else : + return self.convert_to_evalcurrency( + self.cashbook.currency, + self.cashbook.balance_ref) + + def get_value_cashbooks_glperc(self): + """ percent of profit/loss of cashbooks + """ + if self.cashbook: + if self.cashbook.feature == 'asset': + return self.cashbook.diff_percent + else : + return Decimal('0.0') + +# end InvestmentLine diff --git a/line.py b/line.py index cb86ba8..eb9a517 100644 --- a/line.py +++ b/line.py @@ -11,6 +11,7 @@ from trytond.transaction import Transaction from trytond.i18n import gettext from trytond.exceptions import UserError from trytond.pool import Pool +from .templates import cashbook_types, category_types class EvaluationLine(ModelSQL, ModelView): @@ -23,7 +24,7 @@ class EvaluationLine(ModelSQL, ModelView): cashbook = fields.Many2One(string='Cashbook', select=True, ondelete='CASCADE', model_name='cashbook.book', states={ - 'required': Eval('eval_dtype', '') == 'cashbooks', + 'required': Eval('eval_dtype', '').in_(cashbook_types), }, depends=['eval_dtype']) dtype = fields.Many2One(string='Type', select=True, ondelete='CASCADE', model_name='cashbook.type', @@ -35,6 +36,11 @@ class EvaluationLine(ModelSQL, ModelView): states={ 'required': Eval('eval_dtype', '') == 'currencies', }, depends=['eval_dtype']) + category = fields.Many2One(string='Category', select=True, ondelete='CASCADE', + model_name='cashbook.bookcategory', + states={ + 'required': Eval('eval_dtype', '').in_(category_types), + }, depends=['eval_dtype']) # dtype + currency of evaluation eval_dtype = fields.Function(fields.Char(string='Data type', readonly=True), @@ -74,7 +80,7 @@ class EvaluationLine(ModelSQL, ModelView): # get id of origin chart-form form_id = ModelData.get_id('cashbook_report', 'evalline_view_graph') - # active_chart was added by tree_open-action + # active_evaluation was added by tree_open-action active_evaluation = context.get('active_evaluation', None) # check if we are requested for our default form... @@ -111,7 +117,7 @@ class EvaluationLine(ModelSQL, ModelView): else: return 2 - @fields.depends('eval_dtype', 'cashbook', 'dtype', 'currency', 'name_line') + @fields.depends('eval_dtype', 'category', 'cashbook', 'dtype', 'currency', 'name_line') def on_change_with_name(self, name=None): """ get name of Type """ @@ -122,21 +128,33 @@ class EvaluationLine(ModelSQL, ModelView): # otherwise use rec_name of linked record if self.eval_dtype: + dtype_sel = {'types': 'dtype', 'currencies': 'currency'} + dtype_sel.update({x:'cashbook' for x in cashbook_types}) + dtype_sel.update({x:'category' for x in category_types}) + return getattr( - getattr(self, { - 'cashbooks': 'cashbook', - 'types': 'dtype', - 'currencies': 'currency', - }[self.eval_dtype], None), + getattr(self, dtype_sel[self.eval_dtype], None), 'rec_name', None) + def convert_to_evalcurrency(self, from_currency, amount): + """ convert amount to current evaluation-currency + """ + Currency = Pool().get('currency.currency') + + exp = Decimal(Decimal(1) / 10 ** self.currency_digits) + return Currency.compute( + from_currency, + amount, + self.eval_currency, + ).quantize(exp) + @classmethod def validate(cls, records): """ check parent record """ super(EvaluationLine, cls).validate(records) for record in records: - if (record.evaluation.dtype != 'cashbooks') and \ + if (record.evaluation.dtype not in cashbook_types) and \ (record.cashbook is not None): raise UserError(gettext( 'cashbook_report.msg_invalid_dtype', @@ -154,23 +172,16 @@ class EvaluationLine(ModelSQL, ModelView): 'cashbook_report.msg_invalid_dtype', typename = gettext('cashbook_report.msg_dtype_currency'), )) + if (record.evaluation.dtype not in category_types) and \ + (record.category is not None): + raise UserError(gettext( + 'cashbook_report.msg_invalid_dtype', + typename = gettext('cashbook_report.msg_dtype_category'), + )) - def get_value_cashbooks(self): - """ balance of cashbooks - """ - Currency = Pool().get('currency.currency') - - if self.cashbook: - exp = Decimal(Decimal(1) / 10 ** self.currency_digits) - return Currency.compute( - self.cashbook.currency, - self.cashbook.balance, - self.eval_currency, - ).quantize(exp) - - def get_value_types(self): - """ get balance of bookings in cashbooks by 'type', - converted to currency of evaluation + def get_balance_by_query(self, query): + """ run 'query' on Lines, convert used + currencies to evaluation-currency """ pool = Pool() Lines = pool.get('cashbook.line') @@ -180,19 +191,11 @@ class EvaluationLine(ModelSQL, ModelView): tab_book = Cashbook.__table__() cursor = Transaction().connection.cursor() - if (self.evaluation is None) or (self.dtype is None) or \ - (self.eval_currency is None) or (self.currency_digits is None): - return None - total_amount = Decimal('0.0') with Transaction().set_context({ '_check_access': True, }): - lines = Lines.search([ - ('cashbook.btype.id', '=', self.dtype.id), - ('cashbook.state', '=', 'open'), - ('cashbook.owner.id', '=', Transaction().user), - ], query=True) + lines = Lines.search(query, query=True) query = lines.join(tab_line, condition=lines.id==tab_line.id, ).join(tab_book, condition=tab_book.id==tab_line.cashbook, @@ -203,69 +206,80 @@ class EvaluationLine(ModelSQL, ModelView): ) cursor.execute(*query) balances = cursor.fetchall() - for balance in balances: (id_currency, bal1) = balance if bal1 is not None: - total_amount += Currency.compute( - Currency(id_currency), - bal1, - self.eval_currency, - ) - exp = Decimal(Decimal(1) / 10 ** self.currency_digits) - return total_amount.quantize(exp) + total_amount += self.convert_to_evalcurrency( + Currency(id_currency), bal1) + return total_amount + + def get_value_cashbooks(self): + """ balance of cashbooks + """ + Currency = Pool().get('currency.currency') + + if self.cashbook: + return self.convert_to_evalcurrency( + self.cashbook.currency, + self.cashbook.balance, + ) + + def get_value_categories(self): + """ get balance of bookings in categories + converted to currency of evaluation + """ + IrDate = Pool().get('ir.date') + + if self.category is None: + return None + + return self.get_balance_by_query([ + ('cashbook.categories.id', '=', self.category.id), + ('cashbook.state', '=', 'open'), + ('date', '<=', IrDate.today()), + ]) + + def get_value_types(self): + """ get balance of bookings in cashbooks by 'type', + converted to currency of evaluation + """ + IrDate = Pool().get('ir.date') + + if self.dtype is None: + return None + + return self.get_balance_by_query([ + ('cashbook.btype.id', '=', self.dtype.id), + ('cashbook.state', '=', 'open'), + ('date', '<=', IrDate.today()), + ]) def get_value_currencies(self): """ get balance of bookings in cashbooks by 'currency', converted to currency of evaluation """ - pool = Pool() - Lines = pool.get('cashbook.line') - Currency = pool.get('currency.currency') - tab_line = Lines.__table__() - cursor = Transaction().connection.cursor() + IrDate = Pool().get('ir.date') - if (self.evaluation is None) or (self.currency is None) or \ - (self.eval_currency is None) or (self.currency_digits is None): + if self.currency is None: return None - total_amount = Decimal('0.0') - with Transaction().set_context({ - '_check_access': True, - }): - lines = Lines.search([ + return self.get_balance_by_query([ ('cashbook.currency.id', '=', self.currency.id), ('cashbook.state', '=', 'open'), - ('cashbook.owner.id', '=', Transaction().user), - ], query=True) - - query = lines.join(tab_line, condition=lines.id==tab_line.id, - ).select( - Sum(tab_line.credit - tab_line.debit).as_('balance'), - ) - cursor.execute(*query) - balances = cursor.fetchall() - - for balance in balances: - (bal1,) = balance - - if bal1 is not None: - total_amount += Currency.compute( - self.currency, - bal1, - self.eval_currency, - ) - exp = Decimal(Decimal(1) / 10 ** self.currency_digits) - return total_amount.quantize(exp) + ('date', '<=', IrDate.today()), + ]) @fields.depends('eval_dtype', 'eval_currency', 'currency_digits', \ 'cashbook', '_parent_cashbook.currency', '_parent_cashbook.balance',\ + 'category', '_parent_category.id', 'evaluation', '_parent_evaluation.id', 'dtype', 'currency') def on_change_with_balance(self, name=None): """ balance of cashbook """ - if self.eval_dtype: - return getattr(self, 'get_value_%s' % self.eval_dtype)() + if (self.evaluation is None) or (self.eval_currency is None) or \ + (self.currency_digits is None) or (self.eval_dtype is None): + return None + return getattr(self, 'get_value_%s' % self.eval_dtype)() # end EvaluationLine diff --git a/locale/de.po b/locale/de.po index f01597e..ba30cac 100644 --- a/locale/de.po +++ b/locale/de.po @@ -11,8 +11,20 @@ msgid "Type of evaluation must be '%(typename)s'." msgstr "Typ der Auswertung mus '%(typename)s' sein." msgctxt "model:ir.message,text:msg_dtype_cashbook" -msgid "Cashbooks" -msgstr "Kassenbücher" +msgid "Cashbooks [Amount]" +msgstr "Kassenbücher [Betrag]" + +msgctxt "model:ir.message,text:msg_dtype_cashbook_gldiff" +msgid "Cashbooks [Amount of Profit/Loss]" +msgstr "Kassenbücher [Betrag Gewinn/Verlust]" + +msgctxt "model:ir.message,text:msg_dtype_cashbook_glperc" +msgid "Cashbooks [Percent of Profit/Loss]" +msgstr "Kassenbücher [Prozent Gewinn/Verlust]" + +msgctxt "model:ir.message,text:msg_dtype_cashbooks_glvalue" +msgid "Cashbooks [Current Value]" +msgstr "Kassenbücher [aktueller Wert]" msgctxt "model:ir.message,text:msg_dtype_type" msgid "Types of Cashbooks" @@ -22,6 +34,22 @@ msgctxt "model:ir.message,text:msg_dtype_currency" msgid "Currencies" msgstr "Währungen" +msgctxt "model:ir.message,text:msg_dtype_category" +msgid "Categories [Amount]" +msgstr "Kategorien [Betrag]" + +msgctxt "model:ir.message,text:msg_dtype_category_gldiff" +msgid "Categories [Amount of Profit/Loss]" +msgstr "Kategorien [Betrag Gewinn/Verlust]" + +msgctxt "model:ir.message,text:msg_dtype_category_glvalue" +msgid "Categories [Current Value]" +msgstr "Kategorien [aktueller Wert]" + +msgctxt "model:ir.message,text:msg_dtype_category_glperc" +msgid "Categories [Percent of Profit/Loss]" +msgstr "Kategorien [Prozent Gewinn/Verlust]" + msgctxt "model:ir.message,text:msg_name_graph" msgid "Graph: %(gname)s" msgstr "Diagramm: %(gname)s" @@ -134,6 +162,10 @@ msgctxt "selection:cashbook_report.evaluation,dtype:" msgid "Currencys" msgstr "Währungen" +msgctxt "selection:cashbook_report.evaluation,dtype:" +msgid "Categories" +msgstr "Kategorien" + msgctxt "help:cashbook_report.evaluation,dtype:" msgid "Type of data displayed" msgstr "Art der dargestellten Daten" @@ -226,6 +258,10 @@ msgctxt "field:cashbook_report.evaluation,currencies:" msgid "Currencies" msgstr "Währungen" +msgctxt "field:cashbook_report.evaluation,categories:" +msgid "Categories" +msgstr "Kategorien" + msgctxt "field:cashbook_report.evaluation,line_values:" msgid "Line Values" msgstr "Zeilenwerte" diff --git a/locale/en.po b/locale/en.po index 6af9624..cd993f7 100644 --- a/locale/en.po +++ b/locale/en.po @@ -7,8 +7,20 @@ msgid "Type of evaluation must be '%(typename)s'." msgstr "Type of evaluation must be '%(typename)s'." msgctxt "model:ir.message,text:msg_dtype_cashbook" -msgid "Cashbooks" -msgstr "Cashbooks" +msgid "Cashbooks [Amount]" +msgstr "Cashbooks [Amount]" + +msgctxt "model:ir.message,text:msg_dtype_cashbook_gldiff" +msgid "Cashbooks [Amount of Profit/Loss]" +msgstr "Cashbooks [Amount of Profit/Loss]" + +msgctxt "model:ir.message,text:msg_dtype_cashbook_glperc" +msgid "Cashbooks [Percent of Profit/Loss]" +msgstr "Cashbooks [Percent of Profit/Loss]" + +msgctxt "model:ir.message,text:msg_dtype_cashbooks_glvalue" +msgid "Cashbooks [Current Value]" +msgstr "Cashbooks [Current Value]" msgctxt "model:ir.message,text:msg_dtype_type" msgid "Types of Cashbooks" @@ -18,6 +30,22 @@ msgctxt "model:ir.message,text:msg_dtype_currency" msgid "Currencies" msgstr "Currencies" +msgctxt "model:ir.message,text:msg_dtype_category" +msgid "Categories [Amount]" +msgstr "Categories [Amount]" + +msgctxt "model:ir.message,text:msg_dtype_category_gldiff" +msgid "Categories [Amount of Profit/Loss]" +msgstr "Categories [Amount of Profit/Loss]" + +msgctxt "model:ir.message,text:msg_dtype_category_glvalue" +msgid "Categories [Current Value]" +msgstr "Categories [Current Value]" + +msgctxt "model:ir.message,text:msg_dtype_category_glperc" +msgid "Categories [Percent of Profit/Loss]" +msgstr "Categories [Percent of Profit/Loss]" + msgctxt "model:ir.message,text:msg_name_graph" msgid "Graph: %(gname)s" msgstr "Graph: %(gname)s" @@ -114,6 +142,10 @@ msgctxt "selection:cashbook_report.evaluation,dtype:" msgid "Currencys" msgstr "Currencys" +msgctxt "selection:cashbook_report.evaluation,dtype:" +msgid "Categories" +msgstr "Categories" + msgctxt "help:cashbook_report.evaluation,dtype:" msgid "Type of data displayed" msgstr "Type of data displayed" @@ -206,6 +238,10 @@ msgctxt "field:cashbook_report.evaluation,currencies:" msgid "Currencies" msgstr "Currencies" +msgctxt "field:cashbook_report.evaluation,categories:" +msgid "Categories" +msgstr "Categories" + msgctxt "field:cashbook_report.evaluation,line_values:" msgid "Line Values" msgstr "Line Values" diff --git a/message.xml b/message.xml index 2b66901..82d0674 100644 --- a/message.xml +++ b/message.xml @@ -9,7 +9,16 @@ full copyright notices and license terms. --> Type of evaluation must be '%(typename)s'. - Cashbooks + Cashbooks [Amount] + + + Cashbooks [Amount of Profit/Loss] + + + Cashbooks [Percent of Profit/Loss] + + + Cashbooks [Current Value] Types of Cashbooks @@ -17,6 +26,18 @@ full copyright notices and license terms. --> Currencies + + Categories [Amount] + + + Categories [Amount of Profit/Loss] + + + Categories [Current Value] + + + Categories [Percent of Profit/Loss] + Graph: %(gname)s diff --git a/setup.py b/setup.py index 5b7501b..b1a0cf1 100644 --- a/setup.py +++ b/setup.py @@ -22,7 +22,7 @@ with open(path.join(here, 'README.rst'), encoding='utf-8') as f: # tryton.cfg einlesen config = ConfigParser() -config.readfp(open('tryton.cfg')) +config.read_file(open('tryton.cfg')) info = dict(config.items('tryton')) for key in ('depends', 'extras_depend', 'xml'): if key in info: @@ -68,6 +68,7 @@ setup(name='%s_%s' % (PREFIX, MODULE), description='Tryton module to add reports to cashbook.', long_description=long_description, url='https://www.m-ds.de/', + download_url='https://scmdev.m-ds.de/Tryton/Extra/cashbook_report', author='martin-data services', author_email='service@m-ds.de', license='GPL-3', diff --git a/templates.py b/templates.py index d3432f9..9c9e9af 100644 --- a/templates.py +++ b/templates.py @@ -3,6 +3,8 @@ # The COPYRIGHT file at the top level of this repository contains the # full copyright notices and license terms. +cashbook_types = ['cashbooks', 'cashbooks_gldiff', 'cashbooks_glperc', 'cashbooks_glvalue'] +category_types = ['categories', 'category_gldiff', 'category_glvalue', 'category_glperc'] template_view_line = '' diff --git a/tests/test_report.py b/tests/test_report.py index f424f29..cc211bc 100644 --- a/tests/test_report.py +++ b/tests/test_report.py @@ -3,7 +3,7 @@ # The COPYRIGHT file at the top level of this repository contains the # full copyright notices and license terms. -from trytond.tests.test_tryton import ModuleTestCase, with_transaction +from trytond.tests.test_tryton import ModuleTestCase, with_transaction, activate_module from trytond.pool import Pool from trytond.transaction import Transaction from trytond.exceptions import UserError @@ -16,6 +16,13 @@ class ReportTestCase(CashbookTestCase): 'Test cashbook book report module' module = 'cashbook_report' + @classmethod + def setUpClass(cls): + """ add modelues + """ + super(ReportTestCase, cls).setUpClass() + activate_module(['dashboard', 'cashbook_investment']) + def prep_report_3books(self): """ create 3x cashbooks, add bookings, """ @@ -230,6 +237,15 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(books_owner1[1].name, 'Book 2') self.assertEqual(books_owner1[1].balance, Decimal('12.5')) + # add category to cashbook + Cashbook.write(*[ + [books_owner1[0]], + { + 'categories': [('create', [{ + 'name': 'Book 1, User 1' + }])], + }]) + evaluation1, = Evaluation.create([{ 'name': 'Evaluation User 1 - Cashbooks', 'dtype': 'cashbooks', @@ -269,6 +285,19 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(evaluation3.line_values[1].name, 'usd') self.assertEqual(evaluation3.line_values[1].balance, Decimal('35.71')) + evaluation4, = Evaluation.create([{ + 'name': 'Evaluation User 1 - Categories', + 'dtype': 'categories', + 'categories': [('add', [x.id for x in books_owner1[0].categories])], + }]) + self.assertEqual(len(evaluation4.categories), 1) + self.assertEqual(evaluation4.currency.rec_name, 'Euro') + self.assertEqual(len(evaluation4.line_values), 1) + self.assertEqual(evaluation4.line_values[0].name, 'Book 1, User 1') + self.assertEqual(evaluation4.line_values[0].balance, Decimal('23.81')) + + self.assertEqual(Evaluation.search_count([]), 4) + with Transaction().set_user(users[1].id): with Transaction().set_context({ '_check_access': True, @@ -278,6 +307,15 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(len(books_owner2), 1) self.assertEqual(books_owner2[0].name, 'Book 3') + # add category to cashbook + Cashbook.write(*[ + [books_owner2[0]], + { + 'categories': [('create', [{ + 'name': 'Book 3, User 2' + }])], + }]) + evaluation1, = Evaluation.create([{ 'name': 'Evaluation User 2 - Cashbooks', 'dtype': 'cashbooks', @@ -303,7 +341,7 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(evaluation2.line_values[1].balance, Decimal('0.0')) evaluation3, = Evaluation.create([{ - 'name': 'Evaluation User 3 - Currencies', + 'name': 'Evaluation User 2 - Currencies', 'dtype': 'currencies', 'currencies': [('add', [x.id for x in Currency.search([])])], }]) @@ -315,6 +353,90 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(evaluation3.line_values[1].name, 'usd') self.assertEqual(evaluation3.line_values[1].balance, Decimal('0.0')) + evaluation4, = Evaluation.create([{ + 'name': 'Evaluation User 2 - Categories', + 'dtype': 'categories', + 'categories': [('add', [x.id for x in books_owner2[0].categories])], + }]) + self.assertEqual(len(evaluation4.categories), 1) + self.assertEqual(evaluation4.currency.rec_name, 'Euro') + self.assertEqual(len(evaluation4.line_values), 1) + self.assertEqual(evaluation4.line_values[0].name, 'Book 3, User 2') + self.assertEqual(evaluation4.line_values[0].balance, Decimal('23.0')) + + self.assertEqual(Evaluation.search_count([]), 4) + + # outside of context + # we should have access to all 8 evaluations + evaluations = Evaluation.search([], order=[('name', 'ASC')]) + self.assertEqual(len(evaluations), 8) + + self.assertEqual(evaluations[0].name, 'Evaluation User 1 - Cashbooks') + self.assertEqual(len(evaluations[0].cashbooks), 2) + self.assertEqual(evaluations[0].currency.rec_name, 'Euro') + self.assertEqual(len(evaluations[0].line_values), 2) + self.assertEqual(evaluations[0].line_values[0].name, 'Book 1 | 25.00 usd | Open') + self.assertEqual(evaluations[0].line_values[0].balance, Decimal('23.81')) + self.assertEqual(evaluations[0].line_values[1].name, 'Book 2 | 12.50 usd | Open') + self.assertEqual(evaluations[0].line_values[1].balance, Decimal('11.9')) + + self.assertEqual(evaluations[1].name, 'Evaluation User 1 - Categories') + self.assertEqual(len(evaluations[1].categories), 1) + self.assertEqual(evaluations[1].currency.rec_name, 'Euro') + self.assertEqual(len(evaluations[1].line_values), 1) + self.assertEqual(evaluations[1].line_values[0].name, 'Book 1, User 1') + self.assertEqual(evaluations[1].line_values[0].balance, Decimal('23.81')) + + self.assertEqual(evaluations[2].name, 'Evaluation User 1 - Currencies') + self.assertEqual(len(evaluations[2].currencies), 2) + self.assertEqual(evaluations[2].currency.rec_name, 'Euro') + self.assertEqual(len(evaluations[2].line_values), 2) + self.assertEqual(evaluations[2].line_values[0].name, 'Euro') + self.assertEqual(evaluations[2].line_values[0].balance, Decimal('23.0')) + self.assertEqual(evaluations[2].line_values[1].name, 'usd') + self.assertEqual(evaluations[2].line_values[1].balance, Decimal('35.71')) + + self.assertEqual(evaluations[3].name, 'Evaluation User 1 - Types') + self.assertEqual(len(evaluations[3].types), 2) + self.assertEqual(evaluations[3].currency.rec_name, 'Euro') + self.assertEqual(len(evaluations[3].line_values), 2) + self.assertEqual(evaluations[3].line_values[0].name, 'BK - Bank') + self.assertEqual(evaluations[3].line_values[0].balance, Decimal('23.0')) + self.assertEqual(evaluations[3].line_values[1].name, 'CAS - Cash') + self.assertEqual(evaluations[3].line_values[1].balance, Decimal('35.71')) + + self.assertEqual(evaluations[4].name, 'Evaluation User 2 - Cashbooks') + self.assertEqual(len(evaluations[4].cashbooks), 1) + self.assertEqual(evaluations[4].currency.rec_name, 'Euro') + self.assertEqual(len(evaluations[4].line_values), 1) + self.assertEqual(evaluations[4].line_values[0].name, 'Book 3 | 23.00 € | Open') + self.assertEqual(evaluations[4].line_values[0].balance, Decimal('23.0')) + + self.assertEqual(evaluations[5].name, 'Evaluation User 2 - Categories') + self.assertEqual(len(evaluations[5].categories), 1) + self.assertEqual(evaluations[5].currency.rec_name, 'Euro') + self.assertEqual(len(evaluations[5].line_values), 1) + self.assertEqual(evaluations[5].line_values[0].name, 'Book 3, User 2') + self.assertEqual(evaluations[5].line_values[0].balance, Decimal('23.0')) + + self.assertEqual(evaluations[6].name, 'Evaluation User 2 - Currencies') + self.assertEqual(len(evaluations[6].currencies), 2) + self.assertEqual(evaluations[6].currency.rec_name, 'Euro') + self.assertEqual(len(evaluations[6].line_values), 2) + self.assertEqual(evaluations[6].line_values[0].name, 'Euro') + self.assertEqual(evaluations[6].line_values[0].balance, Decimal('23.0')) + self.assertEqual(evaluations[6].line_values[1].name, 'usd') + self.assertEqual(evaluations[6].line_values[1].balance, Decimal('35.71')) + + self.assertEqual(evaluations[7].name, 'Evaluation User 2 - Types') + self.assertEqual(len(evaluations[7].types), 2) + self.assertEqual(evaluations[7].currency.rec_name, 'Euro') + self.assertEqual(len(evaluations[7].line_values), 2) + self.assertEqual(evaluations[7].line_values[0].name, 'BK - Bank') + self.assertEqual(evaluations[7].line_values[0].balance, Decimal('23.0')) + self.assertEqual(evaluations[7].line_values[1].name, 'CAS - Cash') + self.assertEqual(evaluations[7].line_values[1].balance, Decimal('35.71')) + @with_transaction() def test_report_update_name_of_line(self): """ check replace rec_name-value on line with @@ -367,6 +489,7 @@ class ReportTestCase(CashbookTestCase): Evaluation = pool.get('cashbook_report.evaluation') Types = pool.get('cashbook.type') Currency = pool.get('currency.currency') + Category = pool.get('cashbook.bookcategory') books = self.prep_report_3books() @@ -374,6 +497,15 @@ class ReportTestCase(CashbookTestCase): with Transaction().set_context({ 'company': company.id, }): + + categories = Category.create([{ + 'name': 'Cat 1', + }, { + 'name': 'Cat 2', + }, { + 'name': 'Cat 3', + }]) + # valid evaluation, = Evaluation.create([{ 'name': 'Evaluation 1', @@ -383,6 +515,32 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(len(evaluation.cashbooks), 3) self.assertEqual(len(evaluation.types), 0) self.assertEqual(len(evaluation.currencies), 0) + self.assertEqual(len(evaluation.categories), 0) + + # investment - profit/loss amount + if 'cashbooks_gldiff' in [x[0] for x in Evaluation.get_sel_etype()]: + # no change if switch between cashbook-types + Evaluation.write(*[ + [evaluation], + { + 'dtype': 'cashbooks_gldiff', + }]) + self.assertEqual(len(evaluation.cashbooks), 3) + self.assertEqual(len(evaluation.types), 0) + self.assertEqual(len(evaluation.currencies), 0) + self.assertEqual(len(evaluation.categories), 0) + + Evaluation.write(*[ + [evaluation], + { + 'dtype': 'cashbooks_glperc', + }]) + self.assertEqual(len(evaluation.cashbooks), 3) + self.assertEqual(len(evaluation.types), 0) + self.assertEqual(len(evaluation.currencies), 0) + self.assertEqual(len(evaluation.categories), 0) + else : + print('\n--== Module "cashbook_investment" not installed ==--') Evaluation.write(*[ [evaluation], @@ -393,6 +551,7 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(len(evaluation.cashbooks), 0) self.assertEqual(len(evaluation.types), 2) self.assertEqual(len(evaluation.currencies), 0) + self.assertEqual(len(evaluation.categories), 0) # write same dtype again - no change Evaluation.write(*[ @@ -403,6 +562,7 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(len(evaluation.cashbooks), 0) self.assertEqual(len(evaluation.types), 2) self.assertEqual(len(evaluation.currencies), 0) + self.assertEqual(len(evaluation.categories), 0) Evaluation.write(*[ [evaluation], @@ -413,6 +573,18 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(len(evaluation.cashbooks), 0) self.assertEqual(len(evaluation.types), 0) self.assertEqual(len(evaluation.currencies), 2) + self.assertEqual(len(evaluation.categories), 0) + + Evaluation.write(*[ + [evaluation], + { + 'dtype': 'categories', + 'categories': [('add', [x.id for x in Category.search([])])], + }]) + self.assertEqual(len(evaluation.cashbooks), 0) + self.assertEqual(len(evaluation.types), 0) + self.assertEqual(len(evaluation.currencies), 0) + self.assertEqual(len(evaluation.categories), 3) Evaluation.write(*[ [evaluation], @@ -422,6 +594,7 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(len(evaluation.cashbooks), 0) self.assertEqual(len(evaluation.types), 0) self.assertEqual(len(evaluation.currencies), 0) + self.assertEqual(len(evaluation.categories), 0) @with_transaction() def test_report_dtype_validation(self): @@ -470,6 +643,278 @@ class ReportTestCase(CashbookTestCase): 'types': [('add', [x.id for x in Types.search([])])], }]) + @with_transaction() + def test_report_check_update_of_actionviews(self): + """ create 3x cashbooks, add evaluation, check created + form + actionview + """ + pool = Pool() + Evaluation = pool.get('cashbook_report.evaluation') + try : + DashboardAction = pool.get('dashboard.action') + except: + print('\n--== Module "dashboard" not installed ==--') + DashboardAction = None + + books = self.prep_report_3books() + + company = self.prep_company() + with Transaction().set_context({ + 'company': company.id, + }): + evaluation, = Evaluation.create([{ + 'name': 'Evaluation 1', + 'cashbooks': [('add', [x.id for x in books])], + }]) + + # add dashboard-action + if DashboardAction is not None: + dbaction, = DashboardAction.create([{ + 'user': Transaction().user, + 'act_window': evaluation.dashb_actwin.id, + }]) + self.assertEqual(dbaction.user.rec_name, 'Administrator') + self.assertEqual(dbaction.act_window.name, 'Evaluation 1') + + self.assertEqual(evaluation.dtype, 'cashbooks') + self.assertEqual(evaluation.chart, 'pie') + self.assertEqual(evaluation.legend, True) + self.assertEqual(evaluation.maincolor, 'default') + self.assertEqual(evaluation.bgcolor, '#ffffc0') + self.assertEqual(evaluation.currency.code, 'EUR') + + # check uiview + self.assertEqual(evaluation.ui_view_chart.model, 'cashbook_report.eval_line') + self.assertEqual(evaluation.ui_view_chart.module, 'cashbook_report') + self.assertEqual(evaluation.ui_view_chart.priority, 10) + self.assertEqual(evaluation.ui_view_chart.type, 'graph') + # action-window for dashbord + self.assertEqual(evaluation.dashb_actwin.name, 'Evaluation 1') + self.assertEqual(evaluation.dashb_actwin.res_model, + 'cashbook_report.eval_line') + self.assertEqual(evaluation.dashb_actwin.usage, + 'dashboard') + self.assertEqual(evaluation.dashb_actwin.domain, + '[["evaluation", "=", %d]]' % evaluation.id) + # action-view + self.assertEqual(evaluation.dashb_actview.sequence, 10) + self.assertEqual(evaluation.dashb_actview.view.id, + evaluation.ui_view_chart.id) + self.assertEqual(evaluation.dashb_actview.act_window.id, + evaluation.dashb_actwin.id) + + # update evaluation, this wil re-create the view/act-window + # and update the dashboard-view, without removing it + old_win_id = evaluation.dashb_actwin.id + Evaluation.write(*[ + [evaluation], + { + 'name': 'Evaluation 1a', + }]) + self.assertTrue(old_win_id != evaluation.dashb_actwin.id) + + if DashboardAction is not None: + self.assertEqual(DashboardAction.search_count([]), 1) + + @with_transaction() + def test_report_chart_hbar_book_investment(self): + """ create 3x cashbooks, add bookings, rates + create report with cashbooks, check + """ + pool = Pool() + Evaluation = pool.get('cashbook_report.evaluation') + Book = pool.get('cashbook.book') + + company = self.prep_company() + with Transaction().set_context({ + 'company': company.id, + }): + if 'cashbooks_gldiff' not in [x[0] for x in Evaluation.get_sel_etype()]: + print('\n--== Module "cashbook_investment" not installed ==--') + return + + AccType = pool.get('cashbook.type') + Asset = pool.get('investment.asset') + Product = pool.get('product.template') + Uom = pool.get('product.uom') + CbCategory = pool.get('cashbook.bookcategory') + + at, = AccType.create([{ + 'name': 'depot', + 'short': 'D', + 'feature': 'asset', + 'company': company.id, + }]) + + prod_templ, = Product.create([{ + 'name': 'Aurum', + 'type': 'assets', + 'list_price': Decimal('1.0'), + 'default_uom': Uom.search([('symbol', '=', 'u')])[0].id, + 'products': [('create', [{ + 'description': 'Au', + }])], + }]) + + asset, = Asset.create([{ + 'company': company.id, + 'product': prod_templ.products[0].id, + 'currency': company.currency.id, + 'currency_digits': 4, + 'uom': prod_templ.default_uom.id, + 'rates': [('create', [{ + 'date': date(2022, 5, 1), + 'rate': Decimal('1750.0'), + }, ])], + }]) + self.assertEqual(asset.rec_name, + 'Aurum | 1,750.0000 usd/u | 05/01/2022') + + books = self.prep_report_3books() + cb_cat, = CbCategory.create([{'name': 'CB Category'}]) + Book.write(*[ + books, + { + 'btype': at.id, + 'asset': asset.id, + 'categories': [('add', [cb_cat.id])], + 'quantity_uom': asset.uom.id, + 'quantity_digits': 3, + 'lines': [('write', + [books[0].lines[0].id], # usd + {'quantity': Decimal('2.0'), 'amount': Decimal('3000.0')}, + [books[0].lines[1].id], + {'quantity': Decimal('2.0'), 'amount': Decimal('3100.0')}, + [books[1].lines[0].id], # usd + {'quantity': Decimal('2.0'), 'amount': Decimal('3200.0')}, + [books[1].lines[1].id], + {'quantity': Decimal('2.0'), 'amount': Decimal('3300.0')}, + [books[2].lines[0].id], # euro + {'quantity': Decimal('2.0'), 'amount': Decimal('3300.0')}, + [books[2].lines[1].id], + {'quantity': Decimal('2.0'), 'amount': Decimal('3400.0')}, + )], + }]) + self.assertEqual(books[0].rec_name, 'Book 1 | 6,100.00 usd | Open | 4.000 u') + self.assertEqual(books[0].current_value, Decimal('7000.0')) + self.assertEqual(books[0].diff_amount, Decimal('900.0')) + self.assertEqual(books[0].diff_percent, Decimal('14.75')) + + self.assertEqual(books[1].rec_name, 'Book 2 | 6,500.00 usd | Open | 4.000 u') + self.assertEqual(books[1].current_value, Decimal('7000.0')) + self.assertEqual(books[1].diff_amount, Decimal('500.0')) + self.assertEqual(books[1].diff_percent, Decimal('7.69')) + + self.assertEqual(books[2].rec_name, 'Book 3 | 6,700.00 € | Open | 4.000 u') + self.assertEqual(books[2].current_value, Decimal('6666.67')) + self.assertEqual(books[2].diff_amount, Decimal('-33.33')) + self.assertEqual(books[2].diff_percent, Decimal('-0.5')) + + # evaluation: amount-difference + evaluation, = Evaluation.create([{ + 'name': 'Evaluation 1', + 'dtype': 'cashbooks_gldiff', + 'chart': 'hbar', + 'cashbooks': [('add', [x.id for x in books])], + }]) + self.assertEqual(evaluation.dtype, 'cashbooks_gldiff') + self.assertEqual(evaluation.chart, 'hbar') + self.assertEqual(evaluation.legend, True) + self.assertEqual(evaluation.maincolor, 'default') + self.assertEqual(evaluation.bgcolor, '#ffffc0') + self.assertEqual(evaluation.currency.code, 'EUR') + + self.assertEqual(evaluation.line_values[0].balance, Decimal('857.14')) + self.assertEqual(evaluation.line_values[1].balance, Decimal('476.19')) + self.assertEqual(evaluation.line_values[2].balance, Decimal('-33.33')) + + # evaluation: percent-difference + evaluation2, = Evaluation.create([{ + 'name': 'Evaluation 2', + 'dtype': 'cashbooks_glperc', + 'chart': 'hbar', + 'cashbooks': [('add', [x.id for x in books])], + }]) + self.assertEqual(evaluation2.dtype, 'cashbooks_glperc') + self.assertEqual(evaluation2.chart, 'hbar') + self.assertEqual(evaluation2.legend, True) + self.assertEqual(evaluation2.maincolor, 'default') + self.assertEqual(evaluation2.bgcolor, '#ffffc0') + self.assertEqual(evaluation2.currency.code, 'EUR') + + self.assertEqual(evaluation2.line_values[0].balance, Decimal('14.75')) + self.assertEqual(evaluation2.line_values[1].balance, Decimal('7.69')) + self.assertEqual(evaluation2.line_values[2].balance, Decimal('-0.5')) + + # evaluation: percent-difference + evaluation3, = Evaluation.create([{ + 'name': 'Evaluation 3', + 'dtype': 'cashbooks_glvalue', + 'chart': 'hbar', + 'cashbooks': [('add', [x.id for x in books])], + }]) + self.assertEqual(evaluation3.dtype, 'cashbooks_glvalue') + self.assertEqual(evaluation3.chart, 'hbar') + self.assertEqual(evaluation3.legend, True) + self.assertEqual(evaluation3.maincolor, 'default') + self.assertEqual(evaluation3.bgcolor, '#ffffc0') + self.assertEqual(evaluation3.currency.code, 'EUR') + + self.assertEqual(evaluation3.line_values[0].balance, Decimal('6666.67')) + self.assertEqual(evaluation3.line_values[1].balance, Decimal('6666.67')) + self.assertEqual(evaluation3.line_values[2].balance, Decimal('6666.67')) + + # evaluation: category-current value + evaluation4, = Evaluation.create([{ + 'name': 'Evaluation 4', + 'dtype': 'category_glvalue', + 'chart': 'hbar', + 'categories': [('add', [cb_cat.id])], + }]) + self.assertEqual(evaluation4.dtype, 'category_glvalue') + self.assertEqual(evaluation4.chart, 'hbar') + self.assertEqual(evaluation4.legend, True) + self.assertEqual(evaluation4.maincolor, 'default') + self.assertEqual(evaluation4.bgcolor, '#ffffc0') + self.assertEqual(evaluation4.currency.code, 'EUR') + + self.assertEqual(len(evaluation4.line_values), 1) + self.assertEqual(evaluation4.line_values[0].balance, Decimal('20000.01')) + + # evaluation: category- difference amount + evaluation5, = Evaluation.create([{ + 'name': 'Evaluation 5', + 'dtype': 'category_gldiff', + 'chart': 'hbar', + 'categories': [('add', [cb_cat.id])], + }]) + self.assertEqual(evaluation5.dtype, 'category_gldiff') + self.assertEqual(evaluation5.chart, 'hbar') + self.assertEqual(evaluation5.legend, True) + self.assertEqual(evaluation5.maincolor, 'default') + self.assertEqual(evaluation5.bgcolor, '#ffffc0') + self.assertEqual(evaluation5.currency.code, 'EUR') + + self.assertEqual(len(evaluation5.line_values), 1) + self.assertEqual(evaluation5.line_values[0].balance, Decimal('1300.01')) + + # evaluation: category- difference amount + evaluation6, = Evaluation.create([{ + 'name': 'Evaluation 6', + 'dtype': 'category_glperc', + 'chart': 'hbar', + 'categories': [('add', [cb_cat.id])], + }]) + self.assertEqual(evaluation6.dtype, 'category_glperc') + self.assertEqual(evaluation6.chart, 'hbar') + self.assertEqual(evaluation6.legend, True) + self.assertEqual(evaluation6.maincolor, 'default') + self.assertEqual(evaluation6.bgcolor, '#ffffc0') + self.assertEqual(evaluation6.currency.code, 'EUR') + + self.assertEqual(len(evaluation6.line_values), 1) + self.assertEqual(evaluation6.line_values[0].balance, Decimal('6.95')) + @with_transaction() def test_report_chart_pie_book_red(self): """ create 3x cashbooks, add bookings, @@ -490,7 +935,6 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(evaluation.dtype, 'cashbooks') self.assertEqual(evaluation.chart, 'pie') self.assertEqual(evaluation.legend, True) - self.assertEqual(evaluation.posted, False) self.assertEqual(evaluation.maincolor, 'default') self.assertEqual(evaluation.bgcolor, '#ffffc0') self.assertEqual(evaluation.currency.code, 'EUR') @@ -506,7 +950,7 @@ class ReportTestCase(CashbookTestCase): - + """) @@ -556,7 +1000,6 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(evaluation.dtype, 'types') self.assertEqual(evaluation.chart, 'pie') self.assertEqual(evaluation.legend, True) - self.assertEqual(evaluation.posted, False) self.assertEqual(evaluation.maincolor, 'default') self.assertEqual(evaluation.bgcolor, '#ffffc0') self.assertEqual(evaluation.currency.code, 'EUR') @@ -600,7 +1043,6 @@ class ReportTestCase(CashbookTestCase): self.assertEqual(evaluation.dtype, 'currencies') self.assertEqual(evaluation.chart, 'pie') self.assertEqual(evaluation.legend, True) - self.assertEqual(evaluation.posted, False) self.assertEqual(evaluation.maincolor, 'default') self.assertEqual(evaluation.bgcolor, '#ffffc0') self.assertEqual(evaluation.currency.code, 'EUR') diff --git a/tryton.cfg b/tryton.cfg index 7c0ce6c..4794c91 100644 --- a/tryton.cfg +++ b/tryton.cfg @@ -1,8 +1,12 @@ [tryton] -version=6.0.0 +version=6.0.5 depends: res cashbook + cashbook_bookcategory +extras_depend: + dashboard + cashbook_investment xml: icon.xml message.xml diff --git a/versiondep.txt b/versiondep.txt index dd7b3c7..a366670 100644 --- a/versiondep.txt +++ b/versiondep.txt @@ -1 +1,3 @@ -cashbook;6.0.19;6.0.999;mds +cashbook;6.0.24;6.0.999;mds +cashbook_bookcategory;6.0.3;6.0.999;mds +cashbook_investment;6.0.5;6.0.999;mds diff --git a/view/evaluation_form.xml b/view/evaluation_form.xml index 0d413cb..236f65c 100644 --- a/view/evaluation_form.xml +++ b/view/evaluation_form.xml @@ -20,11 +20,10 @@ full copyright notices and license terms. -->