<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/129393/">https://git.reviewboard.kde.org/r/129393/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On Grudzień 25th, 2016, 1:37 po południu CET, <b>Łukasz Wojniłowicz</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">After this change, CSV Importer plugin isn't displayed in File->Import menu, even after steps suggested in following post
http://kmymoney-devel.kde.narkive.com/hSaamIlT/plugins-path-in-master-branch
So please add making this working again to your TODO list.</p></pre>
</blockquote>
<p>On Grudzień 29th, 2016, 7:02 po południu CET, <b>Christian David</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hi ?ukasz,
are the other plugins shown? Unfortunatly it will take a week (or two) before I can have a look on this issue. However, I am quite sure I checked the import/export plugins before I send this patch. Let me revaluate that…
Greetings
Christian</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Hi Christian,
I've got all plugins selected in KMM settings but in File->Import menu I see only QIF and GnuCash. Earlier I didn’t use anything besides CSV, so I don’t know if there should be e.g. OFX or Weboob in the import menu.
I cannot check it with Aqbanking plugin, that you've tested, because I do not meet required dependencies during compilation time.
Greetings
?ukasz</p></pre>
<br />
<p>- Łukasz</p>
<br />
<p>On Listopad 26th, 2016, 4:12 po południu CET, Christian David wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for KMymoney.</div>
<div>By Christian David.</div>
<p style="color: grey;"><i>Updated Lis 26, 2016, 4:12 po południu</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kmymoney
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Hi,</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">this is my long overdue change on the plugin system. It actually started in Randa.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Sorry for such a long review request! I think it is ready for publishing, I made this review request only to give everybody a chance to ask questions :)</p>
<h1 style="font-size: 100%;text-rendering: inherit;padding: 0;white-space: normal;margin: 0;line-height: inherit;">Commit 1</h1>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This is a major change. Instead of a KPluginFactory based plugin
loading, Qt's plugin system is used. The docu was adopeted to reflect
this.</p>
<h1 style="font-size: 100%;text-rendering: inherit;padding: 0;white-space: normal;margin: 0;line-height: inherit;">Commit 2</h1>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The plugins for onlineTasks are now loaded with QPluginLoader on demand.
The necessary information is encoded in Qt's json plugin header.</p>
<h1 style="font-size: 100%;text-rendering: inherit;padding: 0;white-space: normal;margin: 0;line-height: inherit;">Commit 3</h1>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Moved plugin loader out of kmm_plugin and put it into the executable</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The plugin shared library is for plugins now and does not include the
loading of plugins (which was kind of wired). All loading operations of
standard plugins is done in KMyMoneyApp now.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Therefor the interface system for plugins needed do be reworked to
great degree. Some overly complex operations were replaced by simpler
ones. All plugins were adopted to that. Also it is more type safe now.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Now the user configuration which plugins should be loaded or not is
respected (standard plugins only).</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">An old bug was fixed: If the configuration in the plugin settings
changed the plugins are loaded/unloaded after the user accepts the
dialog (perviously this was done when the checkbox was clicked).</p>
<h1 style="font-size: 100%;text-rendering: inherit;padding: 0;white-space: normal;margin: 0;line-height: inherit;">Todos</h1>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The new system is still not finished. However, it is a great step in
the right direction. An uncomplete list of FIXMEs (or bugs?):</p>
<ol style="padding: 0;text-rendering: inherit;margin: 0 0 0 2em;line-height: inherit;white-space: normal;">
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">pluginloader.{h,cpp} are gui only now and should be merged into the
settings dialog.</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">The UI for plugins still has some issues, e.g. the online banking
plugins which are loaded on demand (and are not deactivatable) have a
checkbox for loading and unloading which has no function.</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">Unloading plugins may not work correctly (I think it never did).</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">The payeeIdentifiers are not listet in the plugin settings page.</li>
</ol></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Opened KMyMoney with aqbanking enabled and disabled. Checked if the Aqbanking Settings menu entry exists.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>CMakeLists.txt <span style="color: grey">(09970dd5626e308c443ea20ee20ecf8503b21209)</span></li>
<li>kmymoney/CMakeLists.txt <span style="color: grey">(acbe6a3c91a62a15871a0061486a7d0771b3a5e6)</span></li>
<li>kmymoney/dialogs/konlinetransferform.h <span style="color: grey">(e4761904d68a92471c0b05cc9e5b2bcd1461511d)</span></li>
<li>kmymoney/dialogs/konlinetransferform.cpp <span style="color: grey">(07e73ab17f93cd8fd804b9d11ede2b049f76eefe)</span></li>
<li>kmymoney/dialogs/settings/ksettingsplugins.cpp <span style="color: grey">(007a59d4be08fe779245dd5d52f664ebfcee447d)</span></li>
<li>kmymoney/kmymoney.h <span style="color: grey">(8d10023b5e35be8f4449973ea16e0ec0cc0acca2)</span></li>
<li>kmymoney/kmymoney.cpp <span style="color: grey">(043342dd87cbc03557c5e2b29eb3b9dfcd1083dd)</span></li>
<li>kmymoney/mymoney/onlinejobadministration.h <span style="color: grey">(e72d1a6e46c7c8bcc7cf05193e04bdc8bd9103f5)</span></li>
<li>kmymoney/mymoney/onlinejobadministration.cpp <span style="color: grey">(52658747a44bc5477ccd75a531aef471b4553b19)</span></li>
<li>kmymoney/pluginloader.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/pluginloader.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/CMakeLists.txt <span style="color: grey">(fd7cad75256c0a43d7d1fd3a23b37cef7fab8bf9)</span></li>
<li>kmymoney/plugins/csvexport/CMakeLists.txt <span style="color: grey">(9f2e29fd43e32cd3c6acc7f597c45d522eac4eaf)</span></li>
<li>kmymoney/plugins/csvexport/csvexport.json.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/csvexport/csvexporterplugin.h <span style="color: grey">(f163ade2d59ae1b6eb32958a317384e77a658523)</span></li>
<li>kmymoney/plugins/csvexport/csvexporterplugin.cpp <span style="color: grey">(31027c65111edbad8ca78b00f5b7df572056bb37)</span></li>
<li>kmymoney/plugins/csvexport/kmm_csvexport.desktop <span style="color: grey">(957a9edc4101504e0db08378392e29358327b1c6)</span></li>
<li>kmymoney/plugins/csvimport/CMakeLists.txt <span style="color: grey">(c3e13285944f3624d69f7cc7cde0d4d9014a2a2c)</span></li>
<li>kmymoney/plugins/csvimport/csvimport.json.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/csvimport/csvimporterplugin.h <span style="color: grey">(e66f9cf8e7bfdf9a431d7c57db2e905f5e89ec66)</span></li>
<li>kmymoney/plugins/csvimport/csvimporterplugin.cpp <span style="color: grey">(9a28dca3efd54b82882ef71595439d34b09a5177)</span></li>
<li>kmymoney/plugins/csvimport/kmm_csvimport.desktop <span style="color: grey">(52dfc6ef5a425f70f4caba42e92378665926c774)</span></li>
<li>kmymoney/plugins/icalendarexport/CMakeLists.txt <span style="color: grey">(b85f29282114b5c8eb4d680308da59177992a7e6)</span></li>
<li>kmymoney/plugins/icalendarexport/icalendarexport.h <span style="color: grey">(ae02cb35041c7bb1423fe2939da1252d1003065a)</span></li>
<li>kmymoney/plugins/icalendarexport/icalendarexport.cpp <span style="color: grey">(d1028338c8e3644fc280395c5c853d369f19f4eb)</span></li>
<li>kmymoney/plugins/icalendarexport/kmm_icalendarexport.desktop <span style="color: grey">(4313fdf277f82e6759e48931b3b2aba059b2ece3)</span></li>
<li>kmymoney/plugins/icalendarexport/kmm_icalendarexport.json.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/interfaceloader.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/interfaceloader.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/interfaces/kmmviewinterface.h <span style="color: grey">(d86e5efafd7ceac57b5a107696ffbc57ef64896b)</span></li>
<li>kmymoney/plugins/kbanking/CMakeLists.txt <span style="color: grey">(7a6b656eaa198b0556dc9329c37d9fe34009ad68)</span></li>
<li>kmymoney/plugins/kbanking/kbanking.json.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/kbanking/kmm_kbanking.desktop <span style="color: grey">(414964f6c9fe3d34f03189783aa915d3ba37ff18)</span></li>
<li>kmymoney/plugins/kbanking/mymoneybanking.h <span style="color: grey">(fe0cb5c5feeeac9ffb4657e49da318cc429d2ac4)</span></li>
<li>kmymoney/plugins/kbanking/mymoneybanking.cpp <span style="color: grey">(ad53d4ba608e221a920a4899ebbfab7466c0293f)</span></li>
<li>kmymoney/plugins/kmymoney-onlinetaskui.desktop <span style="color: grey">(23cfd182a0e12c0be9a97f0490ceedc3456f1b84)</span></li>
<li>kmymoney/plugins/kmymoney-payeeidentifierdelegate.desktop <span style="color: grey">(cc2689f5c1f36ac4d75d1d6bd7ce0c0877b7ca18)</span></li>
<li>kmymoney/plugins/kmymoney-plugin.desktop <span style="color: grey">(28673a8ed8ff8578f046c02b67da4e305f610ecf)</span></li>
<li>kmymoney/plugins/kmymoneyplugin.h <span style="color: grey">(12748dc80cca025212d53db13f0e7ccce71cc465)</span></li>
<li>kmymoney/plugins/kmymoneyplugin.cpp <span style="color: grey">(b55b497607089c4cf036696695b79fd341ec9a3a)</span></li>
<li>kmymoney/plugins/ofximport/CMakeLists.txt <span style="color: grey">(e97b1e87bf1b2141dca623903480691f1f5d0455)</span></li>
<li>kmymoney/plugins/ofximport/kmm_ofximport.desktop <span style="color: grey">(393a0887abbecbf4efb931b8d0ef1da474572025)</span></li>
<li>kmymoney/plugins/ofximport/ofximport.json.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/ofximport/ofximporterplugin.h <span style="color: grey">(667a0996654911d58a6d183ba2d0b244a9457172)</span></li>
<li>kmymoney/plugins/ofximport/ofximporterplugin.cpp <span style="color: grey">(46842eee0870c8f2c21678dde112f497b7c8e2a0)</span></li>
<li>kmymoney/plugins/onlinejobpluginmockup/CMakeLists.txt <span style="color: grey">(17db89d743b4f5871aa720654f9fb1af0591bde1)</span></li>
<li>kmymoney/plugins/onlinejobpluginmockup/kmm_onlinejobpluginmockup.desktop <span style="color: grey">(2b5d33c495b342b1cade6d4cbd2b62165c18e29d)</span></li>
<li>kmymoney/plugins/onlinejobpluginmockup/kmm_onlinejobpluginmockup.json.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/onlinejobpluginmockup/onlinejobpluginmockup.h <span style="color: grey">(f6165d3545d45c1fae6cc72543ffa7750d2a29ee)</span></li>
<li>kmymoney/plugins/onlinejobpluginmockup/onlinejobpluginmockup.cpp <span style="color: grey">(561cf4fa8d925194950e93273810f1d862acf6bf)</span></li>
<li>kmymoney/plugins/onlinepluginextended.h <span style="color: grey">(ec84a2c8867cfbcfb1a54a8d81a484a0c036a656)</span></li>
<li>kmymoney/plugins/onlinetasks/sepa/CMakeLists.txt <span style="color: grey">(30f5e14246a3d63a42e63c886822011fb4496256)</span></li>
<li>kmymoney/plugins/onlinetasks/sepa/kmymoney-sepaorders.desktop.in <span style="color: grey">(c622722047ce8f789f876d4808abd5ee33200bd3)</span></li>
<li>kmymoney/plugins/onlinetasks/sepa/kmymoney-sepaorders.json.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/onlinetasks/sepa/kmymoney-sepaordersui.desktop.in <span style="color: grey">(966eb5e7c25938dbb2d12c6427a2f67ec643ff35)</span></li>
<li>kmymoney/plugins/onlinetasks/sepa/kmymoney-sepastorageplugin.desktop.in <span style="color: grey">(a118770c40bfd2040cc9bfd3f68c1538ec44fb09)</span></li>
<li>kmymoney/plugins/onlinetasks/sepa/sepaonlinetasksloader.h <span style="color: grey">(fa6e54f60fffec913bf3f68256aa370b4e5a07b6)</span></li>
<li>kmymoney/plugins/onlinetasks/sepa/sepaonlinetasksloader.cpp <span style="color: grey">(5bf528e9b3cbece1f37af6101f159793bc0c35a6)</span></li>
<li>kmymoney/plugins/pluginloader.h <span style="color: grey">(3f7f2ec42d0ef60a6ad2d83601a7dfdc397f4fd6)</span></li>
<li>kmymoney/plugins/pluginloader.cpp <span style="color: grey">(20c3b1120c380d3ccec91f412673f18906f5b5f0)</span></li>
<li>kmymoney/plugins/printcheck/CMakeLists.txt <span style="color: grey">(1303f29138a4098824bd5d8209d3a4baadee25da)</span></li>
<li>kmymoney/plugins/printcheck/kmm_printcheck.desktop <span style="color: grey">(97bec6edd7af67582340c14355421e0b121de969)</span></li>
<li>kmymoney/plugins/printcheck/kmm_printcheck.json.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/printcheck/printcheck.h <span style="color: grey">(2f16753e35a632593cee5ffb1890b14f6d6a3ff5)</span></li>
<li>kmymoney/plugins/printcheck/printcheck.cpp <span style="color: grey">(8fa469857db50dbb9a3c00f02df38f225ed56017)</span></li>
<li>kmymoney/plugins/reconciliationreport/CMakeLists.txt <span style="color: grey">(19751f5bf13926f60310d1b42be123f5e4ca11ca)</span></li>
<li>kmymoney/plugins/reconciliationreport/kmm_reconciliationreport.desktop <span style="color: grey">(ec3019df28a881dbec5cda3dd1a30df4aa3ec175)</span></li>
<li>kmymoney/plugins/reconciliationreport/kmm_reconciliationreport.json.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/reconciliationreport/reconciliationreport.h <span style="color: grey">(4d33aa349431deb5b5eb8526d07bfa18a1d77e6d)</span></li>
<li>kmymoney/plugins/reconciliationreport/reconciliationreport.cpp <span style="color: grey">(304b032d5b5ea60114447fb72ff7ac1f317b9e99)</span></li>
<li>kmymoney/plugins/weboob/CMakeLists.txt <span style="color: grey">(b99cc0729847d8217fdec483568b42e6069b9974)</span></li>
<li>kmymoney/plugins/weboob/dialogs/mapaccount.h <span style="color: grey">(041620d2585a5e0e289ba53d89ac7a2ab19768c6)</span></li>
<li>kmymoney/plugins/weboob/dialogs/mapaccount.cpp <span style="color: grey">(f5e36e5a515b58d5a53cb51559569d6fba267bdd)</span></li>
<li>kmymoney/plugins/weboob/kmm_weboob.desktop <span style="color: grey">(1b0ab9e22d7bc3397744eae3ed44fbb30943963b)</span></li>
<li>kmymoney/plugins/weboob/kmm_weboob.json.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>kmymoney/plugins/weboob/plugin.h <span style="color: grey">(c7a8ab13c92988b5ea59e4a06035c399e1e2a76d)</span></li>
<li>kmymoney/plugins/weboob/plugin.cpp <span style="color: grey">(49dee5e1b3c6190f05ec478e2dac8bb9c3dac5ce)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/129393/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>