Version: 3.96.1 (using KDE 4.4.0) Compiler: gcc 4.4.1 OS: Linux Installed from: Fedora RPMs Steps to reproduce: 1) Open price editor 2) Select an existing entry 3) Click on edit button or select edit from context menu 4) Exchange Rate/Price Editor dialog opens Expected result 1) Date from selected entry should be showing Actual result 1) Current date is shown
Created attachment 41368 [details] Remove setting of current date patch for kmm4 All this patch does is remove the line that sets the date to be the current date from setupPriceEditor.
Created attachment 41369 [details] Backport of remove setting of current date patch for kmm2 Equivalent patch for kmymoney2
The currency calculator is used all over the place, not only the price editor dialog, from schedules to the ledger. A solution is to make sure a date is set upon construction, with the calling party setting the corresponding date (date of transaction, schedule, investment transaction, etc.)
(In reply to comment #3) > The currency calculator is used all over the place, not only the price editor > dialog, from schedules to the ledger. > > A solution is to make sure a date is set upon construction, with the calling > party setting the corresponding date (date of transaction, schedule, investment > transaction, etc.) Sorry, I should have said, I'd already checked that all the callers do set a date.
Commit it to SVN, then.
SVN commit 1100896 by iann: BUG: 229638 Editing price in Price Editor always sets date to current date M +0 -1 kcurrencycalculator.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1100896
SVN commit 1102870 by iann: BUG: 229638 Check for invalid date too M +4 -1 kcurrencycalculator.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1102870
Created attachment 41603 [details] Updated backport to kmm2 which includes valid date check patch v2 Rolls up the two commits to kmm4 into one patch backported for kmm2.