| Summary: | 'To Account' of an existing money transfer is displayed incorrectly in edit area | ||
|---|---|---|---|
| Product: | [Applications] skrooge | Reporter: | Johannes Goller <jogojapan> |
| Component: | general | Assignee: | Stephane MANKOWSKI <stephane> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | stephane |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Fedora RPMs | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/skrooge/cddee438fb7a8bf5fa4887292620535d193394f2 | Version Fixed In: | |
| Sentry Crash Report: | |||
| Attachments: |
It works on this case
Screenshot of the bug, using previously attached test case |
||
Git commit cddee438fb7a8bf5fa4887292620535d193394f2 by Stephane Mankowski. Committed on 04/06/2012 at 22:24. Pushed by smankowski into branch 'Feature'. M +1 -0 CHANGELOG M +3 -1 plugins/skrooge/skrooge_operation/skgoperationpluginwidget.cpp http://commits.kde.org/skrooge/cddee438fb7a8bf5fa4887292620535d193394f2 I built the latest version from the Feature branch (and confirmed that 'skrooge -v' on the command line returns 1.4.0BETA), but the bug is still there. Are you sure the fix is correct? Created attachment 71600 [details] It works on this case Hi, It works for me on this test case. Could you check in sources if you have the correction? https://projects.kde.org/projects/extragear/office/skrooge/repository/revisions/cddee438fb7a8bf5fa4887292620535d193394f2/diff/plugins/skrooge/skrooge_operation/skgoperationpluginwidget.cpp Thanks for checking. Here is what I did: Confirmed branch: 09:59 [jogojapan@diogenes skrooge]$ git branch * Feature master Confirmed bug fix included in code: 10:00 [jogojapan@diogenes skrooge]$ git log | head commit cddee438fb7a8bf5fa4887292620535d193394f2 Author: Stephane Mankowski <stephane@mankowski.fr> Date: Mon Jun 4 22:24:20 2012 +0200 BUG: 301137: 'To Account' of an existing money transfer is displayed incorrectly in edit area commit 9a55fa2b7cbe27147392d6ed6e23af0951b9cc5c Author: Stephane Mankowski <stephane@mankowski.fr> Date: Sat Jun 2 22:41:27 2012 +0200 To build it I did this: mkdir build cd build cmake .. -DCMAKE_INSTALL_PREFIX=`kde4-config --prefix` -DCMAKE_BUILD_TYPE=release make Note that I did NOT install it. Instead I ran it from within build/bin: cd bin LD_LIBRARY_PATH=../lib/ ./skrooge & (I tried both with and without changing LD_LIBRARY_PATH -- it did not seem to make a difference.) I have attached a screenshot of the view, using the test case you attached (301137.skg). I opened it, went to the Operations view, and clicked on the second line. That is supposed to be a transfer of $10 from B to C, but in the edit area below it gives A as To-Account. Created attachment 71611 [details]
Screenshot of the bug, using previously attached test case
Hi, You said "Note that I did NOT install it. Instead I ran it from within build/bin". The correction has been done in a plugin. Plugins can not be loaded by skrooge if not installed. So, in your case, the previous version of the plugin is loaded, without the correction. You must install you last build of Skrooge to be able to test the correction. Ahh. I see. Sorry for the confusion. I installed it locally now and added the install directory to the KDEDIRS environment variable before running Skrooge. That works, and I can confirm the bug is fixed. Thank you very much. |
Skroog version 1.3.0 When highlighting an existing 'Transfer' in the 'Operations' view, the target account ('To Acocunt') in the edit area (below the main window) is generally displayed incorrectly. Therefore, when I click on an existing transfer operation and then hit the 'Modify' button without actually making any edit, the 'To Account' is changed to whatever was wronly displayed as 'To Account' in the edit area. Reproducible: Always Steps to Reproduce: 1. Create at least three accounts A, B, C 2. Switch to Operations 3. Add a transfer operation from B to C 4. Add / Modify some other operations 5. Click on the transfer operation previously added 6. Look at the edit area below the operations main list Actual Results: The 'To Account' in the edit area will be displayed as account A Expected Results: The 'To Account' should be displayed as C. If you can't reproduce it, try adding a transfer A->B and confirm that the 'To Account' is wrongly displayed as C, or perhaps a few more variations. Depending on how you name the accounts and in what order you added them, the 'To Account' displayed in the edit area may sometimes be the correct one by accident.