From 3427ef02bc8712e50c1751cc9be697f532bde896 Mon Sep 17 00:00:00 2001
From: AjmalCybro <ajmal@cybrosys.in>
Date: Tue, 18 Apr 2023 15:27:22 +0530
Subject: [PATCH] Apr 18 : [FIX] Bug Fixed 'base_accounting_kit'

---
 base_accounting_kit/__manifest__.py                        | 2 +-
 base_accounting_kit/data/account_financial_report_data.xml | 2 +-
 base_accounting_kit/doc/RELEASE_NOTES.md                   | 7 ++++++-
 base_accounting_kit/report/report_tax.py                   | 2 --
 base_accounting_kit/wizard/financial_report.py             | 5 ++---
 5 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/base_accounting_kit/__manifest__.py b/base_accounting_kit/__manifest__.py
index 04198b078..e865757ac 100644
--- a/base_accounting_kit/__manifest__.py
+++ b/base_accounting_kit/__manifest__.py
@@ -22,7 +22,7 @@
 
 {
     'name': 'Odoo 15 Full Accounting Kit',
-    'version': '15.0.2.2.2',
+    'version': '15.0.2.2.3',
     'category': 'Accounting',
     'live_test_url': 'https://www.youtube.com/watch?v=peAp2Tx_XIs',
     'summary': """ Asset and Budget Management,
diff --git a/base_accounting_kit/data/account_financial_report_data.xml b/base_accounting_kit/data/account_financial_report_data.xml
index 018f242f9..7a8cccf9b 100644
--- a/base_accounting_kit/data/account_financial_report_data.xml
+++ b/base_accounting_kit/data/account_financial_report_data.xml
@@ -78,7 +78,7 @@
             <field name="display_detail">detail_with_hierarchy</field>
             <field name="type">account_type</field>
             <field name="account_type_ids"
-                   eval="[(4,ref('account.data_account_type_expenses')), (4,ref('account.data_account_type_direct_costs')), (4,ref('account.data_account_type_depreciation'))]"/>
+                   eval="[(4,ref('account.data_account_type_expenses')), (4,ref('account.data_account_type_depreciation'))]"/>
         </record>
 
         <record id="account_financial_report_balancesheet0"
diff --git a/base_accounting_kit/doc/RELEASE_NOTES.md b/base_accounting_kit/doc/RELEASE_NOTES.md
index 9eb03be39..699b0396f 100644
--- a/base_accounting_kit/doc/RELEASE_NOTES.md
+++ b/base_accounting_kit/doc/RELEASE_NOTES.md
@@ -14,4 +14,9 @@
 #### 14.01.2022
 #### Version 15.0.2.0.2
 #### FIX
-- Dashboard cash and bank balance issue fix
\ No newline at end of file
+- Dashboard cash and bank balance issue fix
+
+#### 11.04.2023
+#### Version 15.0.2.2.3
+#### FIX
+- Profit and Loss report issue fix
\ No newline at end of file
diff --git a/base_accounting_kit/report/report_tax.py b/base_accounting_kit/report/report_tax.py
index b7774efc1..410b9d96e 100644
--- a/base_accounting_kit/report/report_tax.py
+++ b/base_accounting_kit/report/report_tax.py
@@ -55,7 +55,6 @@ class ReportTax(models.AbstractModel):
         return sql
 
     def _compute_from_amls(self, options, taxes):
-        print("hhhh")
         # compute the tax amount
         sql = self._sql_from_amls_one()
         tables, where_clause, where_params = self.env[
@@ -63,7 +62,6 @@ class ReportTax(models.AbstractModel):
         query = sql % (tables, where_clause)
         self.env.cr.execute(query, where_params)
         results = self.env.cr.fetchall()
-        print("3333",results)
         for result in results:
             if result[0] in taxes:
                 taxes[result[0]]['tax'] = abs(result[1])
diff --git a/base_accounting_kit/wizard/financial_report.py b/base_accounting_kit/wizard/financial_report.py
index 0f656bfb2..33cc918a2 100644
--- a/base_accounting_kit/wizard/financial_report.py
+++ b/base_accounting_kit/wizard/financial_report.py
@@ -189,8 +189,7 @@ class FinancialReport(models.TransientModel):
                 # it's the sum the leaf accounts
                 #  with such an account type
                 accounts = self.env['account.account'].search([
-                    ('user_type_id', 'in', report.account_type_ids.ids)
-                ])
+                ('user_type_id', 'in', report.account_type_ids.ids)])
                 res[report.id]['account'] = self._compute_account_balance(
                     accounts)
                 for value in res[report.id]['account'].values():
@@ -268,11 +267,11 @@ class FinancialReport(models.TransientModel):
                 # used to display the details of the
                 #  financial report, so it's not needed here.
                 continue
-
             if res[report.id].get('account'):
                 sub_lines = []
                 for account_id, value \
                         in res[report.id]['account'].items():
+
                     # if there are accounts to display,
                     #  we add them to the lines with a level equals
                     #  to their level in