[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Review Request 120038: Plasmate-Kdev: Port plasmate to KDevPlatform's shell part 2
From: "Giorgos Tsiapaliokas" <giorgos.tsiapaliokas () kde ! org>
Date: 2014-09-02 17:56:16
Message-ID: 20140902175616.11808.83668 () probe ! kde ! org
[Download RAW message or body]
--===============6779990601505648438==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120038/
-----------------------------------------------------------
Review request for Plasma.
Repository: plasmate
Description
-------
This is the second part of the review 'Port plasmate to KDevPlatform's shell'.
This review contains various removes from plasmate's source code.
The removed code is either depracated due to the port to KDevPlatform
or it hasn't been used by anyone AFAIK since 2011.
Also this review changes the directory structure of plasmate's repository.
This change makes sense with the new plugins that will be added and it reflects
the KDevelop's one.
So the new directory structure is something like
plasmate
-plasmate
--app (this is the place from which the plasmate binary will be created)
--plugins (It's not there yet because we don't have any yet :)
-plasmoidviewer
-engineexplorer
Here is the list of commits for this review
Commits:
The big cleanup.
Most of those stuff are deprecated due to the port to KDevPlatform
and the rest of them are leftover from years ago
----
Change the directory structure of plasmate/plasmate
* move all the Startpage related files to plasmate/plasmate/app/startpage
* move the Startpage related tests to plasmate/plasmate/app/tests
* move ProjectManager::exportProject to Publisher::exportProject.
the ProjectManager wasn't making any use of it and by moving the
method, the Publisher doesn't depend anymore to the ProjectManager
-----
Cleanup the packagehandler, plasmoidhandler and themehandler
We will use KDevelop::PackageModel which is a QAIM with its own
info so we don't need our nodes. What we need is a a way
(PackageModel::description) which will associate for example
the filename "main.qml" to "Mainscript".
Diffs
-----
plasmate/tests/modeltest.cpp d356b26
plasmate/tests/packagemodeltest.h 806d8f1
plasmate/tests/packagemodeltest.cpp 27907d6
plasmate/tests/plasmoidhandlertest.h e888c53
plasmate/tests/plasmoidhandlertest.cpp a6abc98
plasmate/tests/projecthandlertest.h 8f6b903
plasmate/tests/projecthandlertest.cpp afeca9c
plasmate/sidebartablewidget.cpp c9980e1
plasmate/startpage.h a81176c
plasmate/startpage.cpp ab94392
plasmate/startpage.ui
plasmate/test-kpart/CMakeLists.txt 761afcd
plasmate/test-kpart/kparttut1ui.rc a76749e
plasmate/test-kpart/main.cpp 9d67674
plasmate/test-kpart/mainwindow.h ea517b3
plasmate/test-kpart/mainwindow.cpp a18a66a
plasmate/tests/CMakeLists.txt 1b5632a
plasmate/tests/modeltest.h 45c68f4
plasmate/sidebardelegate.cpp 07992ac
plasmate/sidebaritem.cpp 40ffc99
plasmate/sidebartablewidget.h 73856e7
plasmate/projectmanager/projectmanager.cpp d691b44
plasmate/publisher/publisher.h 1faa149
plasmate/publisher/publisher.cpp 52173a9
plasmate/sidebar.h 08f26eb
plasmate/sidebar.cpp 7b7aff1
plasmate/sidebardelegate.h 09fc506
plasmate/projecthandler.cpp
plasmate/projectmanager/projectmanager.h 7a0f875
plasmate/plasmate.desktop
plasmate/plasmate.knsrc
plasmate/projecthandler.h
plasmate/packagemodel.h 2ac1cb2
plasmate/packagemodel.cpp 93f158a
plasmate/packagehandler/packagehandler.h 7dbc3bb
plasmate/packagehandler/packagehandler.cpp ead1f37
plasmate/packagehandler/plasmoidhandler.h 11ca4ae
plasmate/packagehandler/plasmoidhandler.cpp 60009c9
plasmate/packagehandler/themehandler.h 6c95d83
plasmate/packagehandler/themehandler.cpp 4d88361
plasmate/editors/svg/tests/battery-oxygen-inkscape.svgz ce87277
plasmate/editors/svg/tests/battery-oxygen.svgz ce87277
plasmate/editors/svg/tests/test_svg.cpp bc725ab
plasmate/editors/text/texteditor.h 4ab8eed
plasmate/editors/text/texteditor.cpp 2e346ca
plasmate/mainwindow.h a609750
plasmate/mainwindow.cpp c7c95a9
plasmate/modeltest/dynamictreemodel.h 6fbd02a
plasmate/modeltest/dynamictreemodel.cpp 2428ebe
plasmate/modeltest/modeltest.h a5a6777
plasmate/modeltest/modeltest.cpp a5f1fe1
plasmate/editors/svg/tests/background.svg ca130f2
plasmate/editors/svg/tests/background.svgz f9e4b1d
plasmate/editors/editpage.h 98a09ca
plasmate/editors/editpage.cpp f21ec25
plasmate/editors/metadata/metadataeditor.cpp c07fafd
plasmate/editors/svg/CMakeLists.txt 0214310
plasmate/editors/svg/svgviewer.h 01d4856
plasmate/editors/svg/svgviewer.cpp 469b3ff
plasmate/dockwidgetshandler.h 5a68554
plasmate/dockwidgetshandler.cpp 274bec8
plasmate/editors/CMakeLists.txt e8a11d0
plasmate/CMakeLists.txt 5415db7
plasmate/app/CMakeLists.txt PRE-CREATION
plasmate/app/main.cpp 3790ac2
plasmate/app/plasmateapp.cpp 46dda60
plasmate/app/tests/CMakeLists.txt PRE-CREATION
plasmate/tests/themehandlertest.h a080d14
plasmate/tests/themehandlertest.cpp 42dab7b
Diff: https://git.reviewboard.kde.org/r/120038/diff/
Testing
-------
Thanks,
Giorgos Tsiapaliokas
--===============6779990601505648438==
MIME-Version: 1.0
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: 7bit
<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/120038/">https://git.reviewboard.kde.org/r/120038/</a>
</td>
</tr>
</table>
<br />
<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 Plasma.</div>
<div>By Giorgos Tsiapaliokas.</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasmate
</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;">This is the second part of the review 'Port plasmate \
to KDevPlatform's shell'.</p> <p style="padding: 0;text-rendering: inherit;margin: \
0;line-height: inherit;white-space: inherit;">This review contains various removes \
from plasmate's source code.<br style="padding: 0;text-rendering: inherit;margin: \
0;line-height: inherit;white-space: normal;" /> The removed code is either depracated \
due to the port to KDevPlatform<br style="padding: 0;text-rendering: inherit;margin: \
0;line-height: inherit;white-space: normal;" /> or it hasn't been used by anyone \
AFAIK since 2011.</p> <p style="padding: 0;text-rendering: inherit;margin: \
0;line-height: inherit;white-space: inherit;">Also this review changes the directory \
structure of plasmate's repository.<br style="padding: 0;text-rendering: \
inherit;margin: 0;line-height: inherit;white-space: normal;" /> This change makes \
sense with the new plugins that will be added and it reflects<br style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" /> the \
KDevelop's one.</p> <p style="padding: 0;text-rendering: inherit;margin: \
0;line-height: inherit;white-space: inherit;">So the new directory structure is \
something like</p> <p style="padding: 0;text-rendering: inherit;margin: \
0;line-height: inherit;white-space: inherit;">plasmate<br style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
normal;" />
-plasmate<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: normal;" />
--app (this is the place from which the plasmate binary will be created)<br \
style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
normal;" />
--plugins (It's not there yet because we don't have any yet :)<br style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
normal;" />
-plasmoidviewer<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: normal;" />
-engineexplorer</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">Here is the list of commits for this review </p> <p \
style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
inherit;">Commits:</p> <p style="padding: 0;text-rendering: inherit;margin: \
0;line-height: inherit;white-space: inherit;"><div class="codehilite" \
style="background: #f8f8f8"><pre style="line-height: 125%">The big cleanup. Most of \
those stuff are deprecated due to the port to KDevPlatform and the rest of them are \
leftover from years ago <span style="color: #666666">----</span>
Change the directory structure of plasmate<span style="color: \
#666666">/</span>plasmate <span style="color: #666666">*</span> move all the \
Startpage related files to plasmate<span style="color: #666666">/</span>plasmate<span \
style="color: #666666">/</span>app<span style="color: #666666">/</span>startpage \
<span style="color: #666666">*</span> move the Startpage related tests to \
plasmate<span style="color: #666666">/</span>plasmate<span style="color: \
#666666">/</span>app<span style="color: #666666">/</span>tests <span style="color: \
#666666">*</span> move ProjectManager<span style="color: \
#666666">::</span>exportProject to Publisher<span style="color: \
#666666">::</span>exportProject. the ProjectManager wasn<span style="border: 1px \
solid #FF0000">'</span>t making any use of it and by moving the method, the \
Publisher doesn<span style="border: 1px solid #FF0000">'</span>t depend anymore \
to the ProjectManager <span style="color: #666666">-----</span>
Cleanup the packagehandler, plasmoidhandler and themehandler
We will use KDevelop<span style="color: #666666">::</span>PackageModel which is a \
QAIM with its own info so we don<span style="border: 1px solid #FF0000">'</span>t \
need our nodes. What we need is a a way (PackageModel<span style="color: \
#666666">::</span>description) which will associate <span style="color: #008000; \
font-weight: bold">for</span> example the filename <span style="color: \
#BA2121">"main.qml"</span> to <span style="color: \
#BA2121">"Mainscript"</span>. </pre></div>
</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>plasmate/tests/modeltest.cpp <span style="color: grey">(d356b26)</span></li>
<li>plasmate/tests/packagemodeltest.h <span style="color: \
grey">(806d8f1)</span></li>
<li>plasmate/tests/packagemodeltest.cpp <span style="color: \
grey">(27907d6)</span></li>
<li>plasmate/tests/plasmoidhandlertest.h <span style="color: \
grey">(e888c53)</span></li>
<li>plasmate/tests/plasmoidhandlertest.cpp <span style="color: \
grey">(a6abc98)</span></li>
<li>plasmate/tests/projecthandlertest.h <span style="color: \
grey">(8f6b903)</span></li>
<li>plasmate/tests/projecthandlertest.cpp <span style="color: \
grey">(afeca9c)</span></li>
<li>plasmate/sidebartablewidget.cpp <span style="color: grey">(c9980e1)</span></li>
<li>plasmate/startpage.h <span style="color: grey">(a81176c)</span></li>
<li>plasmate/startpage.cpp <span style="color: grey">(ab94392)</span></li>
<li>plasmate/startpage.ui <span style="color: grey">()</span></li>
<li>plasmate/test-kpart/CMakeLists.txt <span style="color: \
grey">(761afcd)</span></li>
<li>plasmate/test-kpart/kparttut1ui.rc <span style="color: \
grey">(a76749e)</span></li>
<li>plasmate/test-kpart/main.cpp <span style="color: grey">(9d67674)</span></li>
<li>plasmate/test-kpart/mainwindow.h <span style="color: grey">(ea517b3)</span></li>
<li>plasmate/test-kpart/mainwindow.cpp <span style="color: \
grey">(a18a66a)</span></li>
<li>plasmate/tests/CMakeLists.txt <span style="color: grey">(1b5632a)</span></li>
<li>plasmate/tests/modeltest.h <span style="color: grey">(45c68f4)</span></li>
<li>plasmate/sidebardelegate.cpp <span style="color: grey">(07992ac)</span></li>
<li>plasmate/sidebaritem.cpp <span style="color: grey">(40ffc99)</span></li>
<li>plasmate/sidebartablewidget.h <span style="color: grey">(73856e7)</span></li>
<li>plasmate/projectmanager/projectmanager.cpp <span style="color: \
grey">(d691b44)</span></li>
<li>plasmate/publisher/publisher.h <span style="color: grey">(1faa149)</span></li>
<li>plasmate/publisher/publisher.cpp <span style="color: grey">(52173a9)</span></li>
<li>plasmate/sidebar.h <span style="color: grey">(08f26eb)</span></li>
<li>plasmate/sidebar.cpp <span style="color: grey">(7b7aff1)</span></li>
<li>plasmate/sidebardelegate.h <span style="color: grey">(09fc506)</span></li>
<li>plasmate/projecthandler.cpp <span style="color: grey">()</span></li>
<li>plasmate/projectmanager/projectmanager.h <span style="color: \
grey">(7a0f875)</span></li>
<li>plasmate/plasmate.desktop <span style="color: grey">()</span></li>
<li>plasmate/plasmate.knsrc <span style="color: grey">()</span></li>
<li>plasmate/projecthandler.h <span style="color: grey">()</span></li>
<li>plasmate/packagemodel.h <span style="color: grey">(2ac1cb2)</span></li>
<li>plasmate/packagemodel.cpp <span style="color: grey">(93f158a)</span></li>
<li>plasmate/packagehandler/packagehandler.h <span style="color: \
grey">(7dbc3bb)</span></li>
<li>plasmate/packagehandler/packagehandler.cpp <span style="color: \
grey">(ead1f37)</span></li>
<li>plasmate/packagehandler/plasmoidhandler.h <span style="color: \
grey">(11ca4ae)</span></li>
<li>plasmate/packagehandler/plasmoidhandler.cpp <span style="color: \
grey">(60009c9)</span></li>
<li>plasmate/packagehandler/themehandler.h <span style="color: \
grey">(6c95d83)</span></li>
<li>plasmate/packagehandler/themehandler.cpp <span style="color: \
grey">(4d88361)</span></li>
<li>plasmate/editors/svg/tests/battery-oxygen-inkscape.svgz <span style="color: \
grey">(ce87277)</span></li>
<li>plasmate/editors/svg/tests/battery-oxygen.svgz <span style="color: \
grey">(ce87277)</span></li>
<li>plasmate/editors/svg/tests/test_svg.cpp <span style="color: \
grey">(bc725ab)</span></li>
<li>plasmate/editors/text/texteditor.h <span style="color: \
grey">(4ab8eed)</span></li>
<li>plasmate/editors/text/texteditor.cpp <span style="color: \
grey">(2e346ca)</span></li>
<li>plasmate/mainwindow.h <span style="color: grey">(a609750)</span></li>
<li>plasmate/mainwindow.cpp <span style="color: grey">(c7c95a9)</span></li>
<li>plasmate/modeltest/dynamictreemodel.h <span style="color: \
grey">(6fbd02a)</span></li>
<li>plasmate/modeltest/dynamictreemodel.cpp <span style="color: \
grey">(2428ebe)</span></li>
<li>plasmate/modeltest/modeltest.h <span style="color: grey">(a5a6777)</span></li>
<li>plasmate/modeltest/modeltest.cpp <span style="color: grey">(a5f1fe1)</span></li>
<li>plasmate/editors/svg/tests/background.svg <span style="color: \
grey">(ca130f2)</span></li>
<li>plasmate/editors/svg/tests/background.svgz <span style="color: \
grey">(f9e4b1d)</span></li>
<li>plasmate/editors/editpage.h <span style="color: grey">(98a09ca)</span></li>
<li>plasmate/editors/editpage.cpp <span style="color: grey">(f21ec25)</span></li>
<li>plasmate/editors/metadata/metadataeditor.cpp <span style="color: \
grey">(c07fafd)</span></li>
<li>plasmate/editors/svg/CMakeLists.txt <span style="color: \
grey">(0214310)</span></li>
<li>plasmate/editors/svg/svgviewer.h <span style="color: grey">(01d4856)</span></li>
<li>plasmate/editors/svg/svgviewer.cpp <span style="color: \
grey">(469b3ff)</span></li>
<li>plasmate/dockwidgetshandler.h <span style="color: grey">(5a68554)</span></li>
<li>plasmate/dockwidgetshandler.cpp <span style="color: grey">(274bec8)</span></li>
<li>plasmate/editors/CMakeLists.txt <span style="color: grey">(e8a11d0)</span></li>
<li>plasmate/CMakeLists.txt <span style="color: grey">(5415db7)</span></li>
<li>plasmate/app/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasmate/app/main.cpp <span style="color: grey">(3790ac2)</span></li>
<li>plasmate/app/plasmateapp.cpp <span style="color: grey">(46dda60)</span></li>
<li>plasmate/app/tests/CMakeLists.txt <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plasmate/tests/themehandlertest.h <span style="color: \
grey">(a080d14)</span></li>
<li>plasmate/tests/themehandlertest.cpp <span style="color: \
grey">(42dab7b)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/120038/diff/" style="margin-left: \
3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>
--===============6779990601505648438==--
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic