KMyMoneyProject
ActivePublic

Properties

Description

the BEST Personal Finance Manager for FREE Users, full stop.

The ultimate objectives of KMyMoney are...

  • Accuracy. Using time tested double entry accounting principles KMyMoney can help ensure that your finances are kept in correct order.
  • Ease of use. KMyMoney strives to be the easiest open source personal finance manager to use, especially for the non-technical user.
  • Familiar Features. KMyMoney intends to provide all important features found in the commercially-available, personal finance managers. The current release is closer than ever to that goal and more improvements are already planned or being tested.

Recent Activity

Sat, May 20

wojnilowicz updated the diff for D5925: Implement fetching prices for time range in Online Quotes.
Sat, May 20, 12:25 PM · KMyMoney
wojnilowicz updated the diff for D5924: Rewrite CSV Importer.

kmm_import cannot be STATIC and plugin, so build MODULE and STATIC.

Sat, May 20, 12:25 PM · KMyMoney
wojnilowicz created D5925: Implement fetching prices for time range in Online Quotes.
Sat, May 20, 9:15 AM · KMyMoney
wojnilowicz created D5924: Rewrite CSV Importer.
Sat, May 20, 9:12 AM · KMyMoney

Sun, May 14

wojnilowicz closed D5688: Port KMyMoneyApp::slotFileSaveAs() to KF5 by committing R261:41e6c3d24412: Port KMyMoneyApp::slotFileSaveAs() to KF5.
Sun, May 14, 4:42 AM · KMyMoney
wojnilowicz closed D5687: Separate payees with newline instead of semicolon by committing R261:c5dcfd6cca71: Separate payees with newline instead of semicolon.
Sun, May 14, 4:42 AM · KMyMoney
wojnilowicz closed D5678: Add option for choosing input for investment calculations by committing R261:b3fd5b0f41ab: Add option for choosing input for investment calculations.
Sun, May 14, 4:42 AM · KMyMoney

Tue, May 2

tbaumgart accepted D5687: Separate payees with newline instead of semicolon.

Ah, I see. I did not remember that the \n is already encoded. So everything is fine.

Tue, May 2, 5:14 PM · KMyMoney
wojnilowicz added a comment to D5687: Separate payees with newline instead of semicolon.

I don't get your idea. All we need is separator symbol that isn't going to be used in payee's name and not a mean to encode/decode it.
I think my solution is the least work required for it to work.

I understand, but having a \n in the value part of an attribute of an XML element is a PITA if you want to process your file with an external tool. It is perfectly OK in terms of XML, though. So replacing a ';' in the key value with a %3B and using the ';' as separator makes it a lot easier to process the data line-wise.

Tue, May 2, 3:07 PM · KMyMoney
tbaumgart added a comment to D5687: Separate payees with newline instead of semicolon.

I don't get your idea. All we need is separator symbol that isn't going to be used in payee's name and not a mean to encode/decode it.
I think my solution is the least work required for it to work.

Tue, May 2, 2:43 PM · KMyMoney
wojnilowicz added a comment to D5687: Separate payees with newline instead of semicolon.

I would think to use some percent encoding is more appropriate and easier to extend in the future. Can't we use QUrl::toPercentEncoding() here to do some conversion behind the scenes? That would also be 100% backward compatible. Just an idea.

Tue, May 2, 2:34 PM · KMyMoney
tbaumgart added a comment to D5687: Separate payees with newline instead of semicolon.

I would think to use some percent encoding is more appropriate and easier to extend in the future. Can't we use QUrl::toPercentEncoding() here to do some conversion behind the scenes? That would also be 100% backward compatible. Just an idea.

Tue, May 2, 2:01 PM · KMyMoney
wojnilowicz created D5688: Port KMyMoneyApp::slotFileSaveAs() to KF5.
Tue, May 2, 12:54 PM · KMyMoney
wojnilowicz created D5687: Separate payees with newline instead of semicolon.
Tue, May 2, 12:48 PM · KMyMoney

Mon, May 1

wojnilowicz created D5678: Add option for choosing input for investment calculations.
Mon, May 1, 12:28 PM · KMyMoney
wojnilowicz closed D5482: Replace costly QTabWidget::indexOf from KReportsView.
Mon, May 1, 12:24 PM · KMyMoney

Sat, Apr 29

ostroffjh added a comment to D5655: Remove the backup function.

I agree that this functionality should be maintained It might not be necessary if a user has an overall backup system set up, and I have thought about using git to maintain a full backup history (but would probably be more useful for the uncompressed xml file). While it may take a few mouse clicks, I think it is fewer than launching a separate program (which would need initial configuration) and the addition of the timestamp to the filename is extremely useful. Also, I do not use the "mount" option, but I can see it being useful to keep the backups on an external thumb drive for example. Perhaps it would be worth asking on the user list whether folks find it useful?

Sat, Apr 29, 5:22 PM · KMyMoney
wojnilowicz added a comment to D5655: Remove the backup function.

I don't like this idea. In the moment there is no third part app to backup KMM, so there will be no backup feature at all.
Moreover I don't get why it must be third part app and not KMM.

Any backup software can be used because it is only needed to save a single file. E.g. borg, backInTime or simply rsync. Btw: Then this is not only a solution for KMyMoney but all other files, too.

The current backup system has some drawbacks, e.g. it works only if

  1. KMyMoney is running, and
  2. the user has to actively selects the backup option, and
  3. has to click a couple of times.

    Both is not very user friendly (well we can discuss the first point).

    Also I just think this is beyond the scope of KMyMoney. Does a user want KMyMoney to have such a function? Or is it better to have a single backup solution for all files (not only KMyMoney)? This request removes a couple hundred lines of codes which we do not need to maintain anymore.

    P.S.: It never worked with databases.
Sat, Apr 29, 4:58 PM · KMyMoney
christiand added a comment to D5655: Remove the backup function.

I don't like this idea. In the moment there is no third part app to backup KMM, so there will be no backup feature at all.
Moreover I don't get why it must be third part app and not KMM.

Sat, Apr 29, 4:42 PM · KMyMoney
wojnilowicz added a comment to D5655: Remove the backup function.

I don't like this idea. In the moment there is no third part app to backup KMM, so there will be no backup feature at all.
Moreover I don't get why it must be third part app and not KMM.

Sat, Apr 29, 3:35 PM · KMyMoney
christiand updated the summary of D5655: Remove the backup function.
Sat, Apr 29, 3:24 PM · KMyMoney
christiand created D5655: Remove the backup function.
Sat, Apr 29, 3:23 PM · KMyMoney

Apr 17 2017

tbaumgart accepted D5482: Replace costly QTabWidget::indexOf from KReportsView.
Apr 17 2017, 5:14 PM · KMyMoney
wojnilowicz created D5482: Replace costly QTabWidget::indexOf from KReportsView.
Apr 17 2017, 4:14 PM · KMyMoney
wojnilowicz closed D5371: Set price precision on every security by committing R261:3ae5debc64ef: Set price precision on every security.
Apr 17 2017, 1:24 PM · KMyMoney
wojnilowicz closed D5477: Reset graph size before redraw by committing R261:6bf10ae89f74: Reset graph size before redraw.
Apr 17 2017, 1:00 PM · KMyMoney
wojnilowicz closed D5476: Draw from zero if no opening balance by committing R261:0a9ec9eadff0: Draw from zero if no opening balance.
Apr 17 2017, 1:00 PM · KMyMoney
wojnilowicz closed D5475: Display right price precision in KEquityPriceUpdateDlg by committing R261:db758e32291b: Display right price precision in KEquityPriceUpdateDlg.
Apr 17 2017, 1:00 PM · KMyMoney
wojnilowicz closed D5474: Use security instead of account in KForecastView by committing R261:61f43d5f3e29: Use security instead of account in KForecastView.
Apr 17 2017, 1:00 PM · KMyMoney
wojnilowicz closed D5473: Categorize reports' data according to report types by committing R261:b9cf11e9f84d: Categorize reports' data according to report types.
Apr 17 2017, 1:00 PM · KMyMoney
tbaumgart accepted D5473: Categorize reports' data according to report types.

I think that is good enough and matches my (code reading) findings. I am all for simplification.

Apr 17 2017, 12:34 PM · KMyMoney
wojnilowicz added a comment to D5473: Categorize reports' data according to report types.

I believe yes, because it reads and writes only necessary data and 4.8 writes and reads all data. I assume that if 4.8 tries to load any unnecessary data, that 5.0 didn't write, nothing bad will happen. Nevertheless I wouldn't recommend returning to 4.8 once 5.0 saved an report.

Well, there is a comment at the beginning of MyMoneyReport::read(const QDomElement& e) which is left unchanged:

// The goal of this reading method is 100% backward AND 100% forward
// compatibility.  Any report ever created with any version of KMyMoney
// should be able to be loaded by this method (as long as it's one of the
// report types supported in this version, of course)

So we shall make sure that this still is the case. There might be other changes in 5.0 which do not allow to go back to 4.8, but we need to make sure the user knows about it. Believing is probably not good enough at this point. Double checking would be very much appreciated.

Apr 17 2017, 12:15 PM · KMyMoney
tbaumgart added a comment to D5473: Categorize reports' data according to report types.

I believe yes, because it reads and writes only necessary data and 4.8 writes and reads all data. I assume that if 4.8 tries to load any unnecessary data, that 5.0 didn't write, nothing bad will happen. Nevertheless I wouldn't recommend returning to 4.8 once 5.0 saved an report.

Apr 17 2017, 11:51 AM · KMyMoney
tbaumgart accepted D5476: Draw from zero if no opening balance.
Apr 17 2017, 11:38 AM · KMyMoney
wojnilowicz added a comment to D5473: Categorize reports' data according to report types.

Is this change backward compatible or better version transparent, i.e. can version 4.8 still read that format and can 5.0 still read the data written by 4.8?

Apr 17 2017, 11:37 AM · KMyMoney
wojnilowicz added a comment to D5476: Draw from zero if no opening balance.

Before patch

Apr 17 2017, 11:29 AM · KMyMoney
tbaumgart accepted D5479: Shorten thick pens lines in legend.

Looks good to me.

Apr 17 2017, 10:19 AM · KMyMoney
tbaumgart accepted D5477: Reset graph size before redraw.
Apr 17 2017, 10:10 AM · KMyMoney
tbaumgart added a comment to D5476: Draw from zero if no opening balance.

Can you describe the change in a bit more detail, please? Maybe provide screen shots that show the difference. Since there is no bug entry referenced, I don't see the problem.

Apr 17 2017, 10:05 AM · KMyMoney
tbaumgart accepted D5475: Display right price precision in KEquityPriceUpdateDlg.
Apr 17 2017, 10:00 AM · KMyMoney
tbaumgart accepted D5474: Use security instead of account in KForecastView.

Never stumbled across that one, but on the other hand I don't use the forecast with investments not at all. Code wise it looks logical to me.

Apr 17 2017, 9:59 AM · KMyMoney
tbaumgart added a comment to D5473: Categorize reports' data according to report types.

Is this change backward compatible or better version transparent, i.e. can version 4.8 still read that format and can 5.0 still read the data written by 4.8?

Apr 17 2017, 9:54 AM · KMyMoney
wojnilowicz created D5479: Shorten thick pens lines in legend.
Apr 17 2017, 8:57 AM · KMyMoney

Apr 16 2017

wojnilowicz created D5477: Reset graph size before redraw.
Apr 16 2017, 1:21 PM · KMyMoney
wojnilowicz created D5476: Draw from zero if no opening balance.
Apr 16 2017, 1:20 PM · KMyMoney
wojnilowicz created D5475: Display right price precision in KEquityPriceUpdateDlg.
Apr 16 2017, 1:18 PM · KMyMoney
wojnilowicz created D5474: Use security instead of account in KForecastView.
Apr 16 2017, 1:15 PM · KMyMoney
wojnilowicz created D5473: Categorize reports' data according to report types.
Apr 16 2017, 1:13 PM · KMyMoney

Apr 15 2017

markjones removed a watcher for KMyMoney: markjones.
Apr 15 2017, 5:54 PM
wojnilowicz abandoned D5371: Set price precision on every security.
Apr 15 2017, 1:17 PM · KMyMoney