Bug 426151 - Remove not implemented settings option "Insert transaction type into No. field for new transactions"
Summary: Remove not implemented settings option "Insert transaction type into No. fiel...
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: general (show other bugs)
Version: 4.8.4
Platform: Other All
: NOR normal
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-09-03 16:23 UTC by Ralf Habacker
Modified: 2020-09-15 10:59 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 4.8.5,5.1.1
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ralf Habacker 2020-09-03 16:23:03 UTC
The mentioned option in the "data entry" tab of the ledger settings menu is not implemented and therefore should be removed.


STEPS TO REPRODUCE
1. git clone kmymoney source
2. search for 'CopyTypeToNr'

OBSERVED RESULT
The only references for the mentioned settings options are: 

grep -rni CopyTypeToNr *

  kmymoney/settings/kmymoney.kcfg:75:  <entry name="CopyTypeToNr" type="Bool">
  kmymoney/dialogs/settings/ksettingsregisterdecl.ui:191:                <widget class="QCheckBox" name="kcfg_CopyTypeToNr">

Setting options really used in the code needs at least three references as shown with the option 'AutoIncCheckNumber'

grep -rni autoIncCheckNumber *

  kmymoney/settings/kmymoney.kcfg:24:  <entry name="AutoIncCheckNumber" type="Bool">
  kmymoney/dialogs/settings/ksettingsregisterdecl.ui:198:                <widget class="QCheckBox" name="kcfg_AutoIncCheckNumber">
  kmymoney/dialogs/transactioneditor.cpp:1079:          && KMyMoneyGlobalSettings::autoIncCheckNumber()           // and auto inc number turned on?

EXPECTED RESULT
The unimplemented setting option should be removed

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: openSUSE Leap 15.1
KDE Frameworks Version: 5.55.0
Qt Version: 5.9.7

ADDITIONAL INFORMATION
All active branches are affected
Comment 1 Bug Janitor Service 2020-09-03 17:35:53 UTC
A possibly relevant merge request was started @ https://invent.kde.org/office/kmymoney/-/merge_requests/30
Comment 2 Ralf Habacker 2020-09-06 18:57:58 UTC
Git commit fb1e20b15691e0855b59f26616c5abb2c14737d7 by Ralf Habacker.
Committed on 06/09/2020 at 18:56.
Pushed by habacker into branch '4.8'.

Remove not implemented settings option "Insert transaction type into No. field for new transactions"
FIXED-IN:4.8.5

M  +0    -9    doc/details-settings.docbook
M  +0    -7    kmymoney/dialogs/settings/ksettingsregisterdecl.ui
M  +0    -4    kmymoney/settings/kmymoney.kcfg

https://invent.kde.org/office/kmymoney/commit/fb1e20b15691e0855b59f26616c5abb2c14737d7
Comment 3 Ralf Habacker 2020-09-15 06:05:17 UTC
Git commit 587f1614d4fc73908a98c05c5c7b9178aadccbae by Ralf Habacker.
Committed on 15/09/2020 at 06:04.
Pushed by habacker into branch '5.1'.

Remove not implemented settings option "Insert transaction type into No. field for new transactions
FIXED-IN:5.1.1

M  +0    -9    doc/details-settings.docbook
M  +0    -7    kmymoney/dialogs/settings/ksettingsregister.ui
M  +0    -4    kmymoney/settings/kmymoney.kcfg

https://invent.kde.org/office/kmymoney/commit/587f1614d4fc73908a98c05c5c7b9178aadccbae
Comment 4 Thomas Baumgart 2020-09-15 06:09:30 UTC
Git commit 2bd3c0927562bd48cbf03844d86b36ec69c71655 by Thomas Baumgart.
Committed on 15/09/2020 at 06:09.
Pushed by tbaumgart into branch 'master'.

Remove not implemented settings option "Insert transaction type into No. field for new transactions
FIXED-IN:5.1.1

(cherry picked from commit 587f1614d4fc73908a98c05c5c7b9178aadccbae)

M  +0    -9    doc/details-settings.docbook
M  +0    -7    kmymoney/dialogs/settings/ksettingsregister.ui
M  +0    -4    kmymoney/settings/kmymoney.kcfg

https://invent.kde.org/office/kmymoney/commit/2bd3c0927562bd48cbf03844d86b36ec69c71655