SUMMARY Sending or receiving money with Paypal allows you to enter a note describing the transaction. This note is not currently imported into kmymoney STEPS TO REPRODUCE 1. download the portable kmymoney snapshot from https://kmymoney.org/snapshots.php. 2. unzip the archive, launch kmymoney, configure a Paypal account (see https://aquamaniac.de/rdm/projects/aqbanking/wiki/Paypal_einrichten or use the settings menu in kmymoney) and assign it to a kmymoney account. 3. make sure that there are transactions with notes in the Paypal account 4. update the assigned account OBSERVED RESULT The imported transactions do not contain the entered notes EXPECTED RESULT The imported transactions should contain the entered notes SOFTWARE/OS VERSIONS Windows: 10 KDE Frameworks Version: 5.65 Qt Version: 5.11
Current versions of aqbanking do not provide the required attribute, therefore bugreport https://www.aquamaniac.de/rdm/issues/246 was submitted.
aqbanking 6.3.1 got support to fetch transactions notes from a paypal account.
Created attachment 145297 [details] Patch from 4.8 branch to fetch transaction note from a paypal account P
Created attachment 145298 [details] Patch from 4.8 branch to update memos on present transactions
I don't understand the patch "to update memos on present transactions". If a transaction is a duplicate, it is already imported once. How could it change when imported a second time? That can't be. If this is meant only for paypal transactions imported with a prior version of AqBanking, I vote against it, because it might also affect other importers. If you want that, remove the transaction in the ledger and re-import.
> If an operation is a duplicate, it has already been imported once. How can it change if it is imported a second time? Before aqbanking 6.3.1, (paypal) transactions were imported without memo, although there might be a memo, i.e. Aqbanking now provides more data than before. Since the duplicate check does not include the memo field, this change is not detected and therefore this commit checks for the detected duplicates if the memo field of the present transaction is empty and imports the memo field only in this case. This is effectively a one-time attribute update of the memo field. > If this is only for paypal transactions imported with a previous version of AqBanking, I vote against it because it could affect other importers as well. This functionality can in principle be extended to all aqbanking sources, but requires that the aqbanking backend in question populates the memo field. If the memo field is not filled by aqbanking, nothing happens. > If you want to do this, remove the transaction in the general ledger and import it again. That's what I did at first, but found out that if you have a large number of transactions, this is very time consuming as you have to manually find and delete all the transactions that don't have a memo in order to re-import those. To give the user control over this process, an option could be added in the online settings of an account to explicitly enable this feature.
Git commit 74ba29e8c0a31a29bb48236f3b01b38968c58485 by Thomas Baumgart. Committed on 11/01/2022 at 17:14. Pushed by tbaumgart into branch 'master'. Include user supplied paypal note in memo field M +15 -3 kmymoney/plugins/kbanking/kbanking.cpp https://invent.kde.org/office/kmymoney/commit/74ba29e8c0a31a29bb48236f3b01b38968c58485
In an ideal world, kmymoney could, at the next launch after an update that brings this feature, tell the user that there is now the possibility to apply memos to Paypal accounts and ask the user if this should be done once for all existing transactions. This would require skipping the transactions where the memo text is already included.
Git commit 5b5aa00cdb4e4376056b2dddf7c4db932b0e091d by Ralf Habacker, on behalf of Thomas Baumgart. Committed on 02/06/2022 at 15:04. Pushed by habacker into branch '5.1'. Include user supplied paypal note in memo field FIXED-IN:5.1.3 (cherry picked from commit 74ba29e8c0a31a29bb48236f3b01b38968c58485) M +15 -3 kmymoney/plugins/kbanking/kbanking.cpp https://invent.kde.org/office/kmymoney/commit/5b5aa00cdb4e4376056b2dddf7c4db932b0e091d