<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>424674</bug_id>
          
          <creation_ts>2020-07-26 07:18:53 +0000</creation_ts>
          <short_desc>Transaction report is missing splits in certain circumstances</short_desc>
          <delta_ts>2024-09-09 20:49:24 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>kmymoney</product>
          <component>reports</component>
          <version>5.1.0</version>
          <rep_platform>unspecified</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Thomas Baumgart">tbaumgart</reporter>
          <assigned_to name="KMyMoney Devel Mailing List">kmymoney-devel</assigned_to>
          <cc>ralf.habacker</cc>
          
          <cf_commitlink>https://invent.kde.org/office/kmymoney/commit/3a769f67c4efb9659eaf5b61e8bd52942533fa39</cf_commitlink>
          <cf_versionfixedin>5.1.1</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1946316</commentid>
    <comment_count>0</comment_count>
    <who name="Thomas Baumgart">tbaumgart</who>
    <bug_when>2020-07-26 07:18:53 +0000</bug_when>
    <thetext>SUMMARY
Creating a transaction report with filter on account and transaction type to get a report of all deposits into this account, I noticed that data of a split transaction was missing in the report. Besides the deposit into this account it contained other splits.

STEPS TO REPRODUCE
1. Create the following accounts: Liability L, Credit Card C, Expense E
2. Create split transaction with the following splits: CC (-100), L(+50), E(+50)
3. Create transaction report and filter on L and type deposit. Organize by Payee

OBSERVED RESULT

Transaction of (+50) in L is not reported

EXPECTED RESULT

Transaction of (+50) in L is reported</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1946319</commentid>
    <comment_count>1</comment_count>
    <who name="Thomas Baumgart">thb</who>
    <bug_when>2020-07-26 07:32:11 +0000</bug_when>
    <thetext>Git commit c05844bb9afdd36c04e9317ec878564154d8ca0e by Thomas Baumgart.
Committed on 26/07/2020 at 07:31.
Pushed by tbaumgart into branch &apos;master&apos;.

Report all matching transactions in transaction report

In case the first processed split of a multi split transactoin matches
the filter criteria and no other one does, then the transaction was not
included in the report at all.

Since other splits may match the filter also, the first one found will
be stacked and only added, if no other data was added for that
transaction.

(cherry picked from commit 3a769f67c4efb9659eaf5b61e8bd52942533fa39)

M  +28   -1    kmymoney/plugins/views/reports/core/querytable.cpp

https://invent.kde.org/office/kmymoney/commit/c05844bb9afdd36c04e9317ec878564154d8ca0e</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1946322</commentid>
    <comment_count>2</comment_count>
    <who name="Thomas Baumgart">thb</who>
    <bug_when>2020-07-26 07:32:28 +0000</bug_when>
    <thetext>Git commit 3a769f67c4efb9659eaf5b61e8bd52942533fa39 by Thomas Baumgart.
Committed on 26/07/2020 at 07:22.
Pushed by tbaumgart into branch &apos;5.1&apos;.

Report all matching transactions in transaction report

In case the first processed split of a multi split transactoin matches
the filter criteria and no other one does, then the transaction was not
included in the report at all.

Since other splits may match the filter also, the first one found will
be stacked and only added, if no other data was added for that
transaction.
FIXED-IN: 5.1.1

M  +28   -1    kmymoney/plugins/views/reports/core/querytable.cpp

https://invent.kde.org/office/kmymoney/commit/3a769f67c4efb9659eaf5b61e8bd52942533fa39</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2352643</commentid>
    <comment_count>3</comment_count>
      <attachid>173500</attachid>
    <who name="Ralf Habacker">ralf.habacker</who>
    <bug_when>2024-09-09 20:49:24 +0000</bug_when>
    <thetext>Created attachment 173500
Test file to reproduce the working fix</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>173500</attachid>
            <date>2024-09-09 20:49:24 +0000</date>
            <delta_ts>2024-09-09 20:49:24 +0000</delta_ts>
            <desc>Test file to reproduce the working fix</desc>
            <filename>bug-424674.xml</filename>
            <type>text/xml</type>
            <size>4309</size>
            <attacher name="Ralf Habacker">ralf.habacker</attacher>
            
              <data encoding="base64">PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0iVVRGLTgiPz4KPCFET0NUWVBFIEtNWU1PTkVZ
LUZJTEU+CjxLTVlNT05FWS1GSUxFPgogIDxGSUxFSU5GTz4KICAgIDxDUkVBVElPTl9EQVRFIGRh
dGU9IiIvPgogICAgPExBU1RfTU9ESUZJRURfREFURSBkYXRlPSIyMDI0LTA5LTA5VDIyOjQzOjM3
KzAyOjAwIi8+CiAgICA8VkVSU0lPTiBpZD0iMSIvPgogICAgPEZJWFZFUlNJT04gZGF0ZT0iIi8+
CiAgICA8QVBQVkVSU0lPTiBpZD0iNS4xLjgwLWE4MmYyNGI1NyIvPgogIDwvRklMRUlORk8+CiAg
PFVTRVIgbmFtZT0iIiBlbWFpbD0iIj4KICAgIDxBRERSRVNTIHN0cmVldD0iIiBjaXR5PSIiIHN0
YXRlPSIiIHppcD0iIiB0ZWxlcGhvbmU9IiIgY291bnRyeT0iIiBjb3VudHk9IiIgemlwY29kZT0i
IiBwb3N0Y29kZT0iIi8+CiAgPC9VU0VSPgogIDxJTlNUSVRVVElPTlMvPgogIDxQQVlFRVMvPgog
IDxDT1NUQ0VOVEVSUy8+CiAgPFRBR1MvPgogIDxBQ0NPVU5UUz4KICAgIDxBQ0NPVU5UIGlkPSJB
U3RkOjpBc3NldCIgcGFyZW50YWNjb3VudD0iIiBsYXN0cmVjb25jaWxlZD0iIiBsYXN0bW9kaWZp
ZWQ9IiIgaW5zdGl0dXRpb249IiIgb3BlbmVkPSIiIG51bWJlcj0iIiB0eXBlPSI5IiBuYW1lPSJB
c3NldCIgZGVzY3JpcHRpb249IiIgY3VycmVuY3k9IkVVUiIvPgogICAgPEFDQ09VTlQgaWQ9IkFT
dGQ6OkxpYWJpbGl0eSIgcGFyZW50YWNjb3VudD0iIiBsYXN0cmVjb25jaWxlZD0iIiBsYXN0bW9k
aWZpZWQ9IiIgaW5zdGl0dXRpb249IiIgb3BlbmVkPSIiIG51bWJlcj0iIiB0eXBlPSIxMCIgbmFt
ZT0iTGlhYmlsaXR5IiBkZXNjcmlwdGlvbj0iIiBjdXJyZW5jeT0iRVVSIj4KICAgICAgPFNVQkFD
Q09VTlRTPgogICAgICAgIDxTVUJBQ0NPVU5UIGlkPSJBMDAwMDAyIi8+CiAgICAgICAgPFNVQkFD
Q09VTlQgaWQ9IkEwMDAwMDMiLz4KICAgICAgPC9TVUJBQ0NPVU5UUz4KICAgIDwvQUNDT1VOVD4K
ICAgIDxBQ0NPVU5UIGlkPSJBU3RkOjpFeHBlbnNlIiBwYXJlbnRhY2NvdW50PSIiIGxhc3RyZWNv
bmNpbGVkPSIiIGxhc3Rtb2RpZmllZD0iIiBpbnN0aXR1dGlvbj0iIiBvcGVuZWQ9IiIgbnVtYmVy
PSIiIHR5cGU9IjEzIiBuYW1lPSJFeHBlbnNlIiBkZXNjcmlwdGlvbj0iIiBjdXJyZW5jeT0iRVVS
Ij4KICAgICAgPFNVQkFDQ09VTlRTPgogICAgICAgIDxTVUJBQ0NPVU5UIGlkPSJBMDAwMDA0Ii8+
CiAgICAgIDwvU1VCQUNDT1VOVFM+CiAgICA8L0FDQ09VTlQ+CiAgICA8QUNDT1VOVCBpZD0iQVN0
ZDo6SW5jb21lIiBwYXJlbnRhY2NvdW50PSIiIGxhc3RyZWNvbmNpbGVkPSIiIGxhc3Rtb2RpZmll
ZD0iIiBpbnN0aXR1dGlvbj0iIiBvcGVuZWQ9IiIgbnVtYmVyPSIiIHR5cGU9IjEyIiBuYW1lPSJJ
bmNvbWUiIGRlc2NyaXB0aW9uPSIiIGN1cnJlbmN5PSJFVVIiLz4KICAgIDxBQ0NPVU5UIGlkPSJB
U3RkOjpFcXVpdHkiIHBhcmVudGFjY291bnQ9IiIgbGFzdHJlY29uY2lsZWQ9IiIgbGFzdG1vZGlm
aWVkPSIiIGluc3RpdHV0aW9uPSIiIG9wZW5lZD0iIiBudW1iZXI9IiIgdHlwZT0iMTYiIG5hbWU9
IkVxdWl0eSIgZGVzY3JpcHRpb249IiIgY3VycmVuY3k9IkVVUiIvPgogICAgPEFDQ09VTlQgaWQ9
IkEwMDAwMDIiIHBhcmVudGFjY291bnQ9IkFTdGQ6OkxpYWJpbGl0eSIgbGFzdHJlY29uY2lsZWQ9
IiIgbGFzdG1vZGlmaWVkPSIyMDI0LTA5LTA5IiBpbnN0aXR1dGlvbj0iIiBvcGVuZWQ9IjIwMjQt
MDEtMDEiIG51bWJlcj0iIiB0eXBlPSIxMCIgbmFtZT0iTCIgZGVzY3JpcHRpb249IiIgY3VycmVu
Y3k9IkVVUiIvPgogICAgPEFDQ09VTlQgaWQ9IkEwMDAwMDMiIHBhcmVudGFjY291bnQ9IkFTdGQ6
OkxpYWJpbGl0eSIgbGFzdHJlY29uY2lsZWQ9IiIgbGFzdG1vZGlmaWVkPSIyMDI0LTA5LTA5IiBp
bnN0aXR1dGlvbj0iIiBvcGVuZWQ9IjIwMjQtMDEtMDEiIG51bWJlcj0iIiB0eXBlPSI0IiBuYW1l
PSJDIiBkZXNjcmlwdGlvbj0iIiBjdXJyZW5jeT0iRVVSIi8+CiAgICA8QUNDT1VOVCBpZD0iQTAw
MDAwNCIgcGFyZW50YWNjb3VudD0iQVN0ZDo6RXhwZW5zZSIgbGFzdHJlY29uY2lsZWQ9IiIgbGFz
dG1vZGlmaWVkPSIyMDI0LTA5LTA5IiBpbnN0aXR1dGlvbj0iIiBvcGVuZWQ9IjE5MDAtMDEtMDEi
IG51bWJlcj0iIiB0eXBlPSIxMyIgbmFtZT0iRSIgZGVzY3JpcHRpb249IiIgY3VycmVuY3k9IkVV
UiIvPgogIDwvQUNDT1VOVFM+CiAgPFRSQU5TQUNUSU9OUz4KICAgIDxUUkFOU0FDVElPTiBpZD0i
VDAwMDAwMDAwMDAwMDAwMDAwMSIgcG9zdGRhdGU9IjIwMjQtMDEtMDQiIG1lbW89IiIgZW50cnlk
YXRlPSIyMDI0LTA5LTA5IiBjb21tb2RpdHk9IkVVUiI+CiAgICAgIDxTUExJVFM+CiAgICAgICAg
PFNQTElUIGlkPSJTMDAwMSIgcGF5ZWU9IiIgcmVjb25jaWxlZGF0ZT0iIiBhY3Rpb249IiIgcmVj
b25jaWxlZmxhZz0iMCIgdmFsdWU9Ii0xMDAvMSIgc2hhcmVzPSItMTAwLzEiIHByaWNlPSIxLzEi
IG1lbW89IiIgYWNjb3VudD0iQTAwMDAwMyIgbnVtYmVyPSIiIGJhbmtpZD0iIi8+CiAgICAgICAg
PFNQTElUIGlkPSJTMDAwMiIgcGF5ZWU9IiIgcmVjb25jaWxlZGF0ZT0iIiBhY3Rpb249IiIgcmVj
b25jaWxlZmxhZz0iMCIgdmFsdWU9IjUwLzEiIHNoYXJlcz0iNTAvMSIgcHJpY2U9IjEvMSIgbWVt
bz0iIiBhY2NvdW50PSJBMDAwMDAyIiBudW1iZXI9IiIgYmFua2lkPSIiLz4KICAgICAgICA8U1BM
SVQgaWQ9IlMwMDAzIiBwYXllZT0iIiByZWNvbmNpbGVkYXRlPSIiIGFjdGlvbj0iIiByZWNvbmNp
bGVmbGFnPSIwIiB2YWx1ZT0iNTAvMSIgc2hhcmVzPSI1MC8xIiBwcmljZT0iMS8xIiBtZW1vPSIi
IGFjY291bnQ9IkEwMDAwMDQiIG51bWJlcj0iIiBiYW5raWQ9IiIvPgogICAgICA8L1NQTElUUz4K
ICAgIDwvVFJBTlNBQ1RJT04+CiAgPC9UUkFOU0FDVElPTlM+CiAgPEtFWVZBTFVFUEFJUlM+CiAg
ICA8UEFJUiBrZXk9Ikxhc3RNb2RpZmljYXRpb25EYXRlIiB2YWx1ZT0iMjAyNC0wOS0wOVQyMjo0
MzozNyswMjowMCIvPgogICAgPFBBSVIga2V5PSJrbW0tYmFzZUN1cnJlbmN5IiB2YWx1ZT0iRVVS
Ii8+CiAgICA8UEFJUiBrZXk9ImttbS1pZCIgdmFsdWU9InszY2IzMzYxYS00OTI5LTQ0MDktYTc0
My1mMzBiMDVjN2JhYzB9Ii8+CiAgPC9LRVlWQUxVRVBBSVJTPgogIDxTQ0hFRFVMRVMvPgogIDxT
RUNVUklUSUVTLz4KICA8Q1VSUkVOQ0lFUz4KICAgIDxDVVJSRU5DWSBpZD0iRVVSIiBuYW1lPSJF
dXJvIiBzeW1ib2w9IuKCrCIgdHlwZT0iMyIgcm91bmRpbmctbWV0aG9kPSI3IiBzYWY9IjEwMCIg
cHA9IjQiIHNjZj0iMTAwIi8+CiAgPC9DVVJSRU5DSUVTPgogIDxQUklDRVMvPgogIDxSRVBPUlRT
PgogICAgPFJFUE9SVCBpZD0iUjAwMDAwMiIgdHlwZT0icXVlcnl0YWJsZSAxLjE1IiBncm91cD0i
VHJhbnNhY3Rpb25zIiBuYW1lPSJUcmFuc2FjdGlvbnMgYnkgY2F0ZWdvcnkgKEN1c3RvbWlzZWQp
IiBjb21tZW50PSJDdXN0b20gcmVwb3J0IiBjb252ZXJ0Y3VycmVuY3k9IjEiIGZhdm9yaXRlPSIw
IiBza2lwWmVybz0iMCIgZGF0ZWxvY2s9InllYXJ0b2RhdGUiIHJvd3R5cGU9InBheWVlIiBxdWVy
eWNvbHVtbnM9Im51bWJlcixwYXllZSx0YWcsYWNjb3VudCIgdGF4PSIwIiBwcm9wQnVkZ2V0RGlm
Zj0iMCIgaW52ZXN0bWVudHM9IjAiIGxvYW5zPSIwIiBoaWRldHJhbnNhY3Rpb25zPSIwIiBzaG93
Y29sdW1udG90YWxzPSIxIiBkZXRhaWw9ImFsbCIgaW5jbHVkZXN0cmFuc2ZlcnM9IjAiPgogICAg
ICA8VFlQRSB0eXBlPSJkZXBvc2l0cyIvPgogICAgICA8QUNDT1VOVCBpZD0iQTAwMDAwMiIvPgog
ICAgPC9SRVBPUlQ+CiAgICA8UkVQT1JUIGlkPSJSMDAwMDAzIiB0eXBlPSJxdWVyeXRhYmxlIDEu
MTUiIGdyb3VwPSJUcmFuc2FjdGlvbnMiIG5hbWU9IlRyYW5zYWN0aW9ucyBieSBhY2NvdW50IChD
dXN0b21pc2VkKSIgY29tbWVudD0iQ3VzdG9tIHJlcG9ydCIgY29udmVydGN1cnJlbmN5PSIxIiBm
YXZvcml0ZT0iMCIgc2tpcFplcm89IjAiIGRhdGVsb2NrPSJ5ZWFydG9kYXRlIiByb3d0eXBlPSJw
YXllZSIgcXVlcnljb2x1bW5zPSJudW1iZXIscGF5ZWUsY2F0ZWdvcnksdGFnLGJhbGFuY2UiIHRh
eD0iMCIgcHJvcEJ1ZGdldERpZmY9IjAiIGludmVzdG1lbnRzPSIwIiBsb2Fucz0iMCIgaGlkZXRy
YW5zYWN0aW9ucz0iMCIgc2hvd2NvbHVtbnRvdGFscz0iMSIgZGV0YWlsPSJhbGwiIGluY2x1ZGVz
dHJhbnNmZXJzPSIwIj4KICAgICAgPFRZUEUgdHlwZT0iZGVwb3NpdHMiLz4KICAgICAgPEFDQ09V
TlQgaWQ9IkEwMDAwMDIiLz4KICAgIDwvUkVQT1JUPgogIDwvUkVQT1JUUz4KICA8QlVER0VUUy8+
CiAgPE9OTElORUpPQlMvPgo8L0tNWU1PTkVZLUZJTEU+Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>