Re: [Kmymoney-devel] Review Request 121803: Changed label for serach bar in ledger from "Search" to "Filter"

2015-01-04 Thread Christian David
E-CREATION kmymoney/widgets/ktreewidgetfilterlinewidget.cpp PRE-CREATION Diff: https://git.reviewboard.kde.org/r/121803/diff/ Testing --- Opened with english local and tested shortcut. Works fine. Thanks, Christian David ___ KMyMoney-devel maili

[Kmymoney-devel] [kmymoney4] [Bug 338296] In Ledger: Keyboard shortcut for Search (Alt-E) does not work

2015-01-04 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=338296 Christian David changed: What|Removed |Added Latest Commit||http://commits.kde.org/kmym

Re: [Kmymoney-devel] Review Request 121803: Changed label for serach bar in ledger from "Search" to "Filter"

2015-01-04 Thread Christian David
b8926939c9 kmymoney/widgets/ktreewidgetfilterlinewidget.h PRE-CREATION kmymoney/widgets/ktreewidgetfilterlinewidget.cpp PRE-CREATION Diff: https://git.reviewboard.kde.org/r/121803/diff/ Testing --- Opened with english local and tested shortcut. Works fi

[Kmymoney-devel] [kmymoney4] [Bug 338296] In Ledger: Keyboard shortcut for Search (Alt-E) does not work

2015-01-04 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=338296 Christian David changed: What|Removed |Added Latest Commit|http://commits.kde.org/kmym |http://commits.kde.org/kmym

[Kmymoney-devel] [kmymoney4] [Bug 341552] When you set a memo for a transfer, it actually gets duplicated for each split, giving place to inconsistences

2015-01-04 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=341552 Christian David changed: What|Removed |Added Status|UNCONFIRMED |CONFIRMED CC

Re: [Kmymoney-devel] Review Request 121288: Database version is set after creating tables

2015-01-10 Thread Christian David
> On Nov. 29, 2014, 10:33 vorm., Christian David wrote: > > kmymoney/mymoney/storage/mymoneystoragesql.cpp, line 740 > > <https://git.reviewboard.kde.org/r/121288/diff/1/?file=331058#file331058line740> > > > > This is still an open issue: ```m_storage->fi

[Kmymoney-devel] [kmymoney4] [Bug 341304] upgrade database to version 1 does not work with sqlite

2015-01-10 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=341304 --- Comment #1 from Christian David --- Git commit d30be72a46be9450a0f205d7c615b96f34a8bf0a by Christian Dávid. Committed on 10/01/2015 at 16:20. Pushed by christiand into branch 'master'. Database version is set after creating tables So t

Re: [Kmymoney-devel] Review Request 121288: Database version is set after creating tables

2015-01-10 Thread Christian David
p(106)] Thanks, Christian David ___ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kmymoney4] [Bug 342952] CMake Warnings with cmake 3.0.2

2015-01-17 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=342952 Christian David changed: What|Removed |Added Ever confirmed|0 |1 Severity|normal

[Kmymoney-devel] [kmymoney4] [Bug 342926] "Next" button may get disabled on Banking page

2015-01-18 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=342926 Christian David changed: What|Removed |Added Version Fixed In||4.7.2 CC

Re: [Kmymoney-devel] Review Request 122138: [GCI2014] Find the maximum in a set of numeric strings.

2015-01-19 Thread Christian David
ust me appended to the string directly :(. Well done, this task was difficult! Your code — C++ and SQL — looks very fine. For all developers: Currently I am writing a test. Then we can replace nearly all ```getRecCount()``` calls with this function and many issues are gone :) - Christian David On J

Re: [Kmymoney-devel] Review Request 122138: [GCI2014] Find the maximum in a set of numeric strings.

2015-01-19 Thread Christian David
ote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122138/ > --- > > (Updated Jan. 19, 2015, 6:02 vorm.)

Re: [Kmymoney-devel] Review Request 122138: [GCI2014] Find the maximum in a set of numeric strings.

2015-01-20 Thread Christian David
; as integer)``` -> 0) as well. Since GCI is over I can take over this task if you want. - Christian David On Jan. 20, 2015, 8:59 vorm., Daniel Ring wrote: > > --- > This is an automatically

Re: [Kmymoney-devel] Review Request 122138: [GCI2014] Find the maximum in a set of numeric strings.

2015-01-20 Thread Christian David
> On Jan. 20, 2015, 5:12 nachm., Christian David wrote: > > On my system the test fails as well, seems to be the same issue as before. > > This solved it: > > > > q.prepare(QString("SELECT MAX(CAST(SUBSTR(%1, :prefixLength) AS INTEGER)) > >

Re: [Kmymoney-devel] Review Request 122138: [GCI2014] Find the maximum in a set of numeric strings.

2015-01-22 Thread Christian David
l see it has a child for each supported RDMBS. During the start of the connection the right version is stored in ```MyMoneyStorageSql::m_driver```. - Christian David On Jan. 21, 2015, 9:35 vorm., Daniel Ring wrote: > > --- &

Re: [Kmymoney-devel] Review Request 122138: [GCI2014] Find the maximum in a set of numeric strings.

2015-01-29 Thread Christian David
we should init the higest id vars (```m_{institution,account,…}s```) *once* with the new function. Or simply the first time a new id is requested - then we can drop nearly all columns of the table kmmFileInfo - any comments? - Christian David On Jan. 24, 2015, 5:59 nachm., Daniel Ring wrote

Re: [Kmymoney-devel] Review Request 122150: Added netherlands to ibanbic folder

2015-01-31 Thread Christian David
Jan. 19, 2015, 4:42 nachm.) > > > Review request for KMymoney and Christian David. > > > Repository: kmymoney > > > Description > --- > > Feeds data to an SQLite Database from an excel file > > > Diffs > - > > kmymoney/

[Kmymoney-devel] [kmymoney4] [Bug 339103] reports cannot be created using the DB backend

2015-01-31 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=339103 Christian David changed: What|Removed |Added Version Fixed In||4.8 Resolution

[Kmymoney-devel] [kmymoney4] [Bug 339103] reports cannot be created using the DB backend

2015-02-01 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=339103 --- Comment #14 from Christian David --- The changes are more than just a bug fix. Back-porting them might be not so easy (definitely more than some cherry-picks). I think we should streamline our power to release 4.8 in near future instead. -- You

Re: [Kmymoney-devel] Review Request 122364: CSV Importer - Fix display on high-definition displays. Fix interaction between QTable and QWizard in same window.

2015-02-01 Thread Christian David
which commit is it based? - Christian David On Feb. 1, 2015, 3:04 nachm., Allan Anderson wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: [Kmymoney-devel] Review Request 122364: CSV Importer - Fix display on high-definition displays. Fix interaction between QTable and QWizard in same window.

2015-02-01 Thread Christian David
> On Feb. 1, 2015, 6:13 nachm., Christian David wrote: > > I cannot apply you patch because of this error: > > > > error: patch failed: kmymoney/plugins/csvimport/csvdialog.h:501 > > error: kmymoney/plugins/csvimport/csvdialog.h: patch does not apply

[Kmymoney-devel] [kmymoney4] [Bug 133364] split and merge money files

2015-02-01 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=133364 Christian David changed: What|Removed |Added CC||s...@gmx.at --- Comment #5 from Christian

[Kmymoney-devel] [kmymoney4] [Bug 185571] missing an archiving function

2015-02-01 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=185571 Christian David changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution

Re: [Kmymoney-devel] Review Request 122441: Fix build with gcc-4.9 and gmp <= 5.1.3

2015-02-07 Thread Christian David
13.2. libalkimia/alkvalue.h <https://git.reviewboard.kde.org/r/122441/#comment52270> Can you add /** @todo When gmp version is higer than 5.1.3, remove cstddef include */ for me. - Christian David On Feb. 5, 2015, 3:28 nachm., Heiko Becker

Re: [Kmymoney-devel] Review Request 122441: Fix build with gcc-4.9 and gmp <= 5.1.3

2015-02-07 Thread Christian David
> On Feb. 7, 2015, 1:28 nachm., Christian David wrote: > > libalkimia/alkvalue.h, line 23 > > <https://git.reviewboard.kde.org/r/122441/diff/1/?file=347156#file347156line23> > > > > Can you add > > > > /** @todo When gmp versio

[Kmymoney-devel] [kmymoney] [Bug 343881] Credit card charges act like payments

2015-02-07 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=343881 Christian David changed: What|Removed |Added Resolution|--- |INVALID Status|UNCONFIRMED

[Kmymoney-devel] [kmymoney4] [Bug 344013] New: Icons on home screen are not shown

2015-02-10 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344013 Bug ID: 344013 Summary: Icons on home screen are not shown Product: kmymoney4 Version: git master Platform: Compiled Sources OS: Linux Status: UNCONFIRMED Severi

[Kmymoney-devel] [kmymoney4] [Bug 344013] Icons on home screen are not shown

2015-02-10 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344013 --- Comment #1 from Christian David --- Created attachment 91012 --> https://bugs.kde.org/attachment.cgi?id=91012&action=edit Screenshot of error -- You are receiving this mail because: You are the assignee for

[Kmymoney-devel] Review Request 122545: Added context menu to outbox

2015-02-12 Thread Christian David
ney/views/konlinejoboutbox.cpp a05821a924d0b4e75864cdd2981859c88284c89a Diff: https://git.reviewboard.kde.org/r/122545/diff/ Testing --- Manual test of new Ui, ```make test``` Thanks, Christian David ___ KMyMoney-devel mailing lis

Re: [Kmymoney-devel] Review Request 122364: CSV Importer - Fix display on high-definition displays. Fix interaction between QTable and QWizard in same window.

2015-02-12 Thread Christian David
340656.patch <https://git.reviewboard.kde.org//r/122364/#fcomment314> This method looks very similar to the following ```…ColumnSelected()``` methods. They could be compacted with some template magic. File Attachment: Updated patch - 0002-BUG-340656.patch <https://git.reviewboard.kde

Re: [Kmymoney-devel] Review Request 122364: CSV Importer - Fix display on high-definition displays. Fix interaction between QTable and QWizard in same window.

2015-02-12 Thread Christian David
> On Feb. 12, 2015, 10:24 nachm., Christian David wrote: > > That is a really huge diff. Here are some things I noticed while skimed > > through it. > > > > It seems you reduced the code; very good! It compiles

Re: [Kmymoney-devel] Review Request 122545: Added context menu to outbox

2015-02-18 Thread Christian David
4e8c1bdc01af kmymoney/views/konlinejoboutbox.h 6dc52685f30ad7b6deb8f8db95ff7ff2906f6921 kmymoney/views/konlinejoboutbox.cpp a05821a924d0b4e75864cdd2981859c88284c89a Diff: https://git.reviewboard.kde.org/r/122545/diff/ Testing --- Manual test of new Ui, ```make test

[Kmymoney-devel] [kmymoney4] [Bug 344324] Can't save any transactions. Enter icon is always disabled.

2015-02-18 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344324 Christian David changed: What|Removed |Added Status|UNCONFIRMED |CONFIRMED CC

[Kmymoney-devel] [kmymoney4] [Bug 344330] New: Password is send several times even if it is wrong

2015-02-18 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344330 Bug ID: 344330 Summary: Password is send several times even if it is wrong Product: kmymoney4 Version: git master Platform: unspecified OS: Linux Status: UNCONFIRMED

[Kmymoney-devel] [kmymoney4] [Bug 342952] CMake Warnings with cmake 3.0.2

2015-02-19 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=342952 Christian David changed: What|Removed |Added Resolution|--- |FIXED Version Fixed In

[Kmymoney-devel] [kmymoney4] [Bug 342952] CMake Warnings with cmake 3.0.2

2015-02-19 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=342952 Christian David changed: What|Removed |Added Resolution|--- |FIXED Version Fixed In

[Kmymoney-devel] [kmymoney4] [Bug 344324] Mark reason for disabled OK button in transaction editor more prominent

2015-02-20 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344324 Christian David changed: What|Removed |Added Version|4.7.1 |git master Platform|MS Windows

[Kmymoney-devel] [kmymoney4] [Bug 344324] Mark reason for disabled OK button in transaction editor more prominent

2015-02-20 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344324 --- Comment #5 from Christian David --- I changed this to a wishlist item. Before we start to code we should contact the VDG. But maybe it is enough to set a tool-tip for the OK button. -- You are receiving this mail because: You are the assignee for

[Kmymoney-devel] [kmymoney4] [Bug 344409] New: User is asked to create a new category in splits table if change is aborted

2015-02-20 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344409 Bug ID: 344409 Summary: User is asked to create a new category in splits table if change is aborted Product: kmymoney4 Version: git master Platform: Compiled Sources

[Kmymoney-devel] [kmymoney4] [Bug 344412] Another build fails - different system - "ui_onlinejobmessagesview.h: No such file or directory"

2015-02-21 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344412 Christian David changed: What|Removed |Added Assignee|kmymoney-devel@kde.org |christian-da...@web.de CC

[Kmymoney-devel] [kmymoney4] [Bug 344412] Another build fails - different system - "ui_onlinejobmessagesview.h: No such file or directory"

2015-02-21 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344412 Christian David changed: What|Removed |Added Status|UNCONFIRMED |CONFIRMED Ever confirmed|0

[Kmymoney-devel] [kmymoney4] [Bug 344412] Another build fails - different system - "ui_onlinejobmessagesview.h: No such file or directory"

2015-02-21 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344412 Christian David changed: What|Removed |Added Latest Commit||http://commits.kde.org/kmym

[Kmymoney-devel] [kmymoney4] [Bug 344412] Another build fails - different system - "ui_onlinejobmessagesview.h: No such file or directory"

2015-02-21 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344412 --- Comment #5 from Christian David --- (In reply to allan from comment #3) > Anyway, to cut a long sorry story short, I cannot reproduce these problems > today. > I also wiped my first PC and all now is sweetness and light. > > I&#x

Re: [Kmymoney-devel] Review Request 122364: CSV Importer - Fix display on high-definition displays. Fix interaction between QTable and QWizard in same window.

2015-02-22 Thread Christian David
> On Feb. 12, 2015, 10:24 nachm., Christian David wrote: > > File Attachment: Updated patch - 0002-BUG-340656.patch > > <https://git.reviewboard.kde.org/r/122364/#fcomment346> > > > > This is more a general annotation: > > > > You are

Re: [Kmymoney-devel] Review Request 122364: CSV Importer - Fix display on high-definition displays. Fix interaction between QTable and QWizard in same window.

2015-04-12 Thread Christian David
> On Feb. 12, 2015, 10:24 nachm., Christian David wrote: > > File Attachment: Updated patch - 0002-BUG-340656.patch > > <https://git.reviewboard.kde.org/r/122364/#fcomment397> > > > > I do not understand this: "…if only one value column…" >

[Kmymoney-devel] [kmymoney4] [Bug 347685] New: Transaction contain erroneous *** UNASSIGNED *** in Tags view if no payee is given

2015-05-13 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=347685 Bug ID: 347685 Summary: Transaction contain erroneous *** UNASSIGNED *** in Tags view if no payee is given Product: kmymoney4 Version: git master Platform: unspecified

[Kmymoney-devel] Review Request 123782: Renamed all .desktop files from kmymoney... to kmymoney-...

2015-05-13 Thread Christian David
- Thanks, Christian David ___ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 123782: Renamed all .desktop files from kmymoney... to kmymoney-...

2015-05-14 Thread Christian David
://git.reviewboard.kde.org/r/123782/#review80319 --- On Mai 13, 2015, 10:39 nachm., Christian David wrote: > > --- > This is an automatically generated e-mail. To

Re: [Kmymoney-devel] Review Request 123782: Renamed all .desktop files from kmymoney... to kmymoney-...

2015-05-14 Thread Christian David
op Diff: https://git.reviewboard.kde.org/r/123782/diff/ Testing --- Thanks, Christian David ___ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kmymoney4] [Bug 347685] Transaction contains erroneous *** UNASSIGNED *** in Tags view if no payee is given

2015-05-15 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=347685 Christian David changed: What|Removed |Added Summary|Transaction contain |Transaction contains

[Kmymoney-devel] [kmymoney4] [Bug 347685] Transaction contains erroneous *** UNASSIGNED *** in Tags view if no payee is given

2015-05-17 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=347685 --- Comment #5 from Christian David --- Created attachment 92661 --> https://bugs.kde.org/attachment.cgi?id=92661&action=edit Screenshot with the error Here is a screenshot of the issue, build with d4065b9. -- You are receiving this mail

[Kmymoney-devel] [kmymoney4] [Bug 347685] Transaction contains erroneous *** UNASSIGNED *** in Tags view if no payee is given

2015-05-17 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=347685 --- Comment #6 from Christian David --- Created attachment 92662 --> https://bugs.kde.org/attachment.cgi?id=92662&action=edit How I created the transaction "Spenden" is the category. -- You are receiving this mail because: You are

Re: [Kmymoney-devel] Review Request 123782: Renamed all .desktop files from kmymoney... to kmymoney-...

2015-05-31 Thread Christian David
ll to work correctly. - Christian --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123782/#review80453 --- On Mai 14, 2015, 10:49 vorm., C

Re: [Kmymoney-devel] Review Request 123782: Renamed all .desktop files from kmymoney... to kmymoney-...

2015-05-31 Thread Christian David
ey/plugins/CMakeLists.txt 1a43470fc5ef7a365df9d266a5256f5d7638058a kmymoney/plugins/kmymoneyimporterplugin.desktop kmymoney/plugins/kmymoneyplugin.desktop Diff: https://git.reviewboard.kde.org/r/123782/diff/ Testing --- Thanks, Chris

[Kmymoney-devel] [kmymoney4] [Bug 348492] Payment Images not Showing in KDE5

2015-05-31 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=348492 Christian David changed: What|Removed |Added CC||christian-da...@web.de Resolution

[Kmymoney-devel] [kmymoney4] [Bug 344013] Icons on home screen are not shown

2015-05-31 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344013 Christian David changed: What|Removed |Added CC||sigma...@gmail.com --- Comment #7 from

[Kmymoney-devel] [kmymoney4] [Bug 344013] Icons on home screen are not shown

2015-06-01 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344013 --- Comment #9 from Christian David --- (In reply to Dimitrios Glentadakis from comment #6) > It is ok after disabling the adblock filter in konqueror I didn't try to disable adblock but I am sure this is not the issue. Maybe there was anothe

[Kmymoney-devel] [kmymoney4] [Bug 344013] Icons on home screen are not shown

2015-06-01 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344013 Christian David changed: What|Removed |Added Ever confirmed|0 |1 Status|UNCONFIRMED

[Kmymoney-devel] Review Request 124035: Updated address of FSF in COPYING

2015-06-07 Thread Christian David
c7aea1896f776a9dea105ff388d6058a612d4961 Diff: https://git.reviewboard.kde.org/r/124035/diff/ Testing --- Thanks, Christian David ___ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 124035: Updated address of FSF in COPYING

2015-06-27 Thread Christian David
, Christian David ___ KMyMoney-devel mailing list KMyMoney-devel@kde.org https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kmymoney4] [Bug 349670] New: "Database not empty" warning is always shown for SQLite

2015-06-27 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=349670 Bug ID: 349670 Summary: "Database not empty" warning is always shown for SQLite Product: kmymoney4 Version: git master Platform: Compiled Sources OS: Linux

[Kmymoney-devel] [kmymoney4] [Bug 344013] Icons on home screen are not shown

2015-06-28 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=344013 Christian David changed: What|Removed |Added Assignee|kmymoney-devel@kde.org |christian-da...@web.de -- You are receiving

[Kmymoney-devel] [kmymoney4] [Bug 349684] Crash when creating new credit transfer

2015-06-28 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=349684 --- Comment #1 from Christian David --- Git commit 7c73d6f78dea63279abbd50714b58d372e3ec3ea by Christian Dávid. Committed on 28/06/2015 at 15:46. Pushed by christiand into branch 'master'. Prevent potential use after free Maybe there is a

[Kmymoney-devel] [kmymoney4] [Bug 349684] Crash when creating new credit transfer

2015-06-28 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=349684 --- Comment #2 from Christian David --- Do you have an idea why I could not reproduce this issues? -- You are receiving this mail because: You are the assignee for the bug. ___ KMyMoney-devel mailing list

[Kmymoney-devel] [kmymoney4] [Bug 351892] online banking passwords not stored/remembered

2015-08-29 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=351892 Christian David changed: What|Removed |Added Resolution|--- |WAITINGFORINFO CC

[Kmymoney-devel] [kmymoney4] [Bug 352029] New: Tag not shown in ledger if only an category is set

2015-08-30 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=352029 Bug ID: 352029 Summary: Tag not shown in ledger if only an category is set Product: kmymoney4 Version: git master Platform: Compiled Sources OS: Linux Status: UNCONFIRMED

[Kmymoney-devel] [kmymoney4] [Bug 353555] Sqlite is much slower than standard kmy format

2015-10-07 Thread Christian David
https://bugs.kde.org/show_bug.cgi?id=353555 Christian David changed: What|Removed |Added Status|UNCONFIRMED |CONFIRMED Ever confirmed|0

Re: [Kmymoney-devel] Review Request 126536: kbanking: Fix build without qtquick1

2015-12-28 Thread Christian David
> On Dez. 28, 2015, 1:44 vorm., Jeremy Whiting wrote: > > Makes sense to me. I'll try it as soon as possible (will probably take until next year). Just a note: the hole QML script will fail in Qt5 - need to port it first. - Christian -

Re: [Kmymoney-devel] Review Request 126536: kbanking: Fix build without qtquick1

2016-01-11 Thread Christian David
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126536/#review90892 --- Ship it! Ship It! - Christian David On Dez. 27, 2015, 4

Re: Review Request 126708: Fix the new account wizard

2016-01-14 Thread Christian David
you use KIOs webdav://, fish:// or ftp://)? I think it will stop working then. - Christian David On Jan. 10, 2016, 7:44 p.m., Valentin Rusu wrote: > > --- > This is an automatically generated e-mail. To reply, visit

KMyMoney’s icons

2016-01-21 Thread christian-david
Hi Everybody,   as Google Code In task I ask a student to fully port our icons to the new scheme. There I noticed that we have no scalable versions of KMyMoney’s icon in the repository. Where is that stored? Is it correct that the icons of different sizes are created manually?   Greetings Chri

Re: Review Request 126875: Combined the questions from TransactionMatcher::match and MyMoneyStatementReader::askUserToEnterScheduleForMatching

2016-01-24 Thread Christian David
d.kde.org/r/126875/#comment62547> Translated strings should not be combined, as it confuses the translators. It is better to create two long strings using i18np() although this duplicates the text (which is usually bad). - Christian David On Jan. 24, 2016, 9:52 nachm., Artur

Re: Review Request 126875: Combined the questions from TransactionMatcher::match and MyMoneyStatementReader::askUserToEnterScheduleForMatching

2016-01-24 Thread Christian David
> On Jan. 24, 2016, 10:02 nachm., Christian David wrote: > > kmymoney/converter/mymoneystatementreader.cpp, line 1458 > > <https://git.reviewboard.kde.org/r/126875/diff/1/?file=439594#file439594line1458> > > > > Translated strings should not be combined, as

Re: Review Request 126882: Changed to asynchronous download in converter/webpricequote using KIO::file_copy

2016-01-25 Thread Christian David
d your patch at the moment. kmymoney/converter/webpricequote.cpp (line 154) <https://git.reviewboard.kde.org/r/126882/#comment62584> That is a nice trick to create a temporary file! - Christian David On Jan. 25, 2016, 3:56 nach

Re: Review Request 126875: Combined the questions from TransactionMatcher::match and MyMoneyStatementReader::askUserToEnterScheduleForMatching

2016-01-25 Thread Christian David
ps://git.reviewboard.kde.org/r/126875/ > --- > > (Updated Jan. 24, 2016, 11:07 nachm.) > > > Review request for KMymoney and Christian David. > > > Repository: kmymoney > > > Description > --- > > Moved the question from `TransactionMatcher:

Re: KF5 release

2016-01-25 Thread christian-david
> Szőts Ákos hat am 24. Januar 2016 um 10:15 > geschrieben: > > If the current state of the KF5 branch of KMyMoney is just „needed for > testing”, I'll gladly help reporting possible bugs. Otherwise you may > find the > following porting scripts useful: > - https://quickgit.kde.org/?p=kde-dev-s

Re: Review Request 126875: Combined the questions from TransactionMatcher::match and MyMoneyStatementReader::askUserToEnterScheduleForMatching

2016-02-03 Thread Christian David
> On Jan. 24, 2016, 10:02 p.m., Christian David wrote: > > kmymoney/converter/mymoneystatementreader.cpp, line 1458 > > <https://git.reviewboard.kde.org/r/126875/diff/1/?file=439594#file439594line1458> > > > > Translated strings should not be combined, as it

Re: Review Request 126882: [Fixed Bug 275004] Changed to asynchronous download in converter/webpricequote using KIO::file_copy

2016-02-11 Thread Christian David
erated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126882/ > --- > > (Updated Feb. 11, 2016, 10:45 p.m.) > > > Review request for KMymoney and Christian David. > > > Bugs: 275004 >

Re: Review Request 127057: Fix file dialog filters

2016-02-13 Thread Christian David
econd. - Christian David On Feb. 13, 2016, 3:54 p.m., Frederik Schwarzer wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard

Review Request 127108: Changed way of saving files which fixes some bugs

2016-02-18 Thread Christian David
, uncompressed and anonymous format. I could not test the GPG part because none of my keys is currently shown by the save dialog. Thanks, Christian David

Re: Review Request 126888: Fix locale conflict in kMyMoneyMoneyValidator

2016-02-24 Thread Christian David
cause that should not compile with Qt 5 and KF 5 at all. Also I am suprised that setting ```setLocale(QLocale::c());``` works here. - Christian David On Feb. 4, 2016, 3:20 p.m., Alexey Chernov wrote: > > --- > This is an automatical

Re: Review Request 126875: Combined the questions from TransactionMatcher::match and MyMoneyStatementReader::askUserToEnterScheduleForMatching

2016-02-24 Thread Christian David
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126875/#review92712 --- Ship it! Ship It! - Christian David On Jan. 24, 2016

Re: Review Request 126875: Combined the questions from TransactionMatcher::match and MyMoneyStatementReader::askUserToEnterScheduleForMatching

2016-02-24 Thread Christian David
- On Jan. 24, 2016, 11:07 p.m., Artur Puzio wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126875/ > --- > > (Up

Re: Review Request 126875: Combined the questions from TransactionMatcher::match and MyMoneyStatementReader::askUserToEnterScheduleForMatching

2016-02-24 Thread Christian David
> On Jan. 24, 2016, 10:02 p.m., Christian David wrote: > > kmymoney/converter/mymoneystatementreader.cpp, line 1458 > > <https://git.reviewboard.kde.org/r/126875/diff/1/?file=439594#file439594line1458> > > > > Translated strings should not be combined, as it

Re: Review Request 126888: Fix locale conflict in kMyMoneyMoneyValidator

2016-02-24 Thread Christian David
complicated than needed. The request looks fine to me. Do you have the right to push the changes? - Christian David On Feb. 24, 2016, 12:11 p.m., Alexey Chernov wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 126875: Combined the questions from TransactionMatcher::match and MyMoneyStatementReader::askUserToEnterScheduleForMatching

2016-02-24 Thread Christian David
> On Jan. 24, 2016, 10:02 p.m., Christian David wrote: > > kmymoney/converter/mymoneystatementreader.cpp, line 1458 > > <https://git.reviewboard.kde.org/r/126875/diff/1/?file=439594#file439594line1458> > > > > Translated strings should not be combined, as it

Re: Review Request 127108: Changed way of saving files which should fix some bugs

2016-03-01 Thread Christian David
) --- I saved a file several times using the compressed, uncompressed and anonymous format. I could not test the GPG part because none of my keys is currently shown by the save dialog. New: I tested it with GPG encrypted files. Thanks, Christian David

Re: Review Request 127108: Changed way of saving files which should fix some bugs

2016-03-01 Thread Christian David
bkgpgfile/kgpgfile.cpp <https://git.reviewboard.kde.org/r/127108/#comment63453> I removed this test because ```QFile::open()``` will check that anyway and more important: This can be wrong under some circumstances. - Christian David On March 1, 2016, 8:03

Re: Review Request 127108: Changed way of saving files which should fix some bugs

2016-03-05 Thread Christian David
> On March 1, 2016, 8:16 p.m., Christian David wrote: > > libkgpgfile/kgpgfile.cpp, line 162 > > <https://git.reviewboard.kde.org/r/127108/diff/2/?file=447716#file447716line162> > > > > I removed this test because ```QFile::open()``` will check that anyway

Re: [Kmymoney-devel] Draft release notes for 4.8

2016-03-13 Thread Christian David
I just noted, we cannot release 4.8, yet. I broke the .annon.kmy file. If that is used the account numbers of the stored payees will not be removed. > Alvaro Soliverez hat am 29. November 2015 um 23:37 > geschrieben: > > Here are the draft notes for 4.8.0.

Review Request 127376: Improved kgpgfile's cmake file

2016-03-14 Thread Christian David
e/CMakeLists.txt a41a6a408e3da8769308dae75d4f514aa969dc87 Diff: https://git.reviewboard.kde.org/r/127376/diff/ Testing --- Compiled on Linux. Thanks, Christian David

Re: Review Request 127108: Changed way of saving files which should fix some bugs

2016-03-14 Thread Christian David
shown by the save dialog. New: I tested it with GPG encrypted files. Brand new: Tried to saved in locations I have no write premissions for, to non-existing folders, and write protected files. I tried hard to reproduce bug 256750 - everything seems to be alright. Thanks, Christian David

Re: Review Request 127376: Improved kgpgfile's cmake file

2016-03-19 Thread Christian David
/diff/ Testing --- Compiled on Linux. Thanks, Christian David

Re: Review Request 127108: Changed way of saving files which should fix some bugs

2016-03-24 Thread Christian David
premissions for, to non-existing folders, and write protected files. I tried hard to reproduce bug 256750 - everything seems to be alright. Thanks, Christian David

Re: Review Request 127495: kbanking: Link to ConfigGui to fix the build

2016-03-26 Thread Christian David
. However that should not stop you.). - Christian David On März 25, 2016, 11:44 nachm., Heiko Becker wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Review Request 127556: Quit application if all windows were closed, refactor main()

2016-04-02 Thread Christian David
- kmymoney/kmymoney.cpp 0d0a317d753e9c48740180d13235163928687b2c kmymoney/main.cpp cf77030aeb7de0f744bc2cf928514568d564e2e8 Diff: https://git.reviewboard.kde.org/r/127556/diff/ Testing --- Started and closed the app. Thanks, Christian David

Re: Review Request 127559: BUG 360129 Do not fetch from csvimporterrc if it's empty

2016-04-07 Thread Christian David
> On April 3, 2016, 2:37 p.m., Allan Anderson wrote: > > > > Allan Anderson wrote: > We have had discussons about this in the bug report. I think you are > confusing the issue by stating your experience (using the Polish translation) > as a general issue. For me, if I delete the csvimpor

Re: Review Request 127559: BUG 360129 Do not fetch from csvimporterrc if it's empty

2016-04-07 Thread Christian David
The if() is very long and not needed here. However, I still do not know if this is the issue. Also the ```i18nc()s``` from ```init()``` could go here if the readSettings method is always called, which I do not know either. - Christian David On April 3, 2016, 6:45 a.m., Łukasz W

Re: Review Request 127559: BUG 360129 Do not fetch from csvimporterrc if it's empty

2016-04-08 Thread Christian David
> On April 7, 2016, 9:59 p.m., Christian David wrote: > > kmymoney/plugins/csvimport/investprocessing.cpp, line 1967 > > <https://git.reviewboard.kde.org/r/127559/diff/1/?file=455181#file455181line1967> > > > > Should become > > ```m_shrsinList

Re: Problems building kmymoneywidgets.so

2016-04-13 Thread Christian David
Hi Mitch, USE_QT_DESIGNER known to be broken. I recommend to compile KMyMoney without the flag. Maybe it works if you enable it later – but probably not. Greetings Christian > Mitch Frazier hat am 13. April 2016 um 04:59 > geschrieben: > I've cloned kmymoney from the git repository and it bu

<    1   2   3   4   >