<?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>428434</bug_id>
          
          <creation_ts>2020-10-29 14:44:03 +0000</creation_ts>
          <short_desc>Column not deleted on update of a forecast with different cycle duration</short_desc>
          <delta_ts>2020-11-01 11:59:39 +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>general</component>
          <version>5.0.8</version>
          <rep_platform>Mint (Ubuntu based)</rep_platform>
          <op_sys>Linux</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>minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter>sayanel</reporter>
          <assigned_to name="KMyMoney Devel Mailing List">kmymoney-devel</assigned_to>
          
          
          <cf_commitlink>https://invent.kde.org/office/kmymoney/commit/7323b85a3df8329c6bc1baf3197ee9953afd6833</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>1969810</commentid>
    <comment_count>0</comment_count>
    <who name="">sayanel</who>
    <bug_when>2020-10-29 14:44:03 +0000</bug_when>
    <thetext>SUMMARY
Extra columns, if there are any, are not deleted when changing the &quot;Days of cycle&quot; of a forecast.

STEPS TO REPRODUCE
1. Go to &apos;forecast&apos; tab. Days to forecast: 90. Days of cycle: 15. Click forecast
2. Displayed summary has a total of 9 columns
3. Change days of cycle to 30 (or whatever number above). Hit forecast.
4. Displayed output has now 6 (or less) columns


OBSERVED RESULT
Columns 7 to 9 still have their old headers (&quot;75 days&quot;, &quot;90 days&quot; and &quot;Total variation&quot;) and are empty. They should have been deleted or have their header removed.

EXPECTED RESULT
Their should be only 6 columns, whatever the previous setting is.


SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Linux Mint 20 Cinnamon. Cinnamon 4.6.7.
KDE Plasma Version: ?
KDE Frameworks Version: 5.68.0
Qt Version: 5.12.8 (build against 5.12.8)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1970663</commentid>
    <comment_count>1</comment_count>
    <who name="Thomas Baumgart">thb</who>
    <bug_when>2020-11-01 11:58:32 +0000</bug_when>
    <thetext>Git commit 2bc5474f3043d0814dfddec52ef70ae2f8edd6d9 by Thomas Baumgart.
Committed on 01/11/2020 at 11:52.
Pushed by tbaumgart into branch &apos;master&apos;.

Adjust column count in forecast view

A call to QTreeWidget::clear() does not affect the column count. This
needs to be handled separately.

M  +4    -2    kmymoney/plugins/views/forecast/kforecastview_p.h

https://invent.kde.org/office/kmymoney/commit/2bc5474f3043d0814dfddec52ef70ae2f8edd6d9</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1970664</commentid>
    <comment_count>2</comment_count>
    <who name="Thomas Baumgart">thb</who>
    <bug_when>2020-11-01 11:59:39 +0000</bug_when>
    <thetext>Git commit 7323b85a3df8329c6bc1baf3197ee9953afd6833 by Thomas Baumgart.
Committed on 01/11/2020 at 11:59.
Pushed by tbaumgart into branch &apos;5.1&apos;.

Adjust column count in forecast view

A call to QTreeWidget::clear() does not affect the column count. This
needs to be handled separately.
FIXED-IN: 5.1.1

(cherry picked from commit 2bc5474f3043d0814dfddec52ef70ae2f8edd6d9)

M  +4    -2    kmymoney/plugins/views/forecast/kforecastview_p.h

https://invent.kde.org/office/kmymoney/commit/7323b85a3df8329c6bc1baf3197ee9953afd6833</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>