Review Request: Merge the branch KLocaleQMLBindings into master

[Marco Martin]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104716/#review13067
-----------------------------------------------------------

Ship it!


yes, i think is to merge at this point.

it can be only partial, but the api can map to qml only so much, so i think is as far it can go.

to me, the only blocker so far(see below) is that doesn't depend from plasma at all, so should be in the kde namespace


plasma/declarativeimports/locale/CMakeLists.txt
<http://git.reviewboard.kde.org/r/104716/#comment10163>

    should be installed under org/kde/locale.
    the name plasma should not appear at all here



plasma/declarativeimports/locale/localebindingsplugin.cpp
<http://git.reviewboard.kde.org/r/104716/#comment10162>

    this should be org.kde.locale.
    
    plasma has nothing to do with those


- Marco Martin


On April 24, 2012, 8:36 p.m., Giorgos Tsiapaliwkas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104716/
> -----------------------------------------------------------
> 
> (Updated April 24, 2012, 8:36 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
> Hello,
> 
> this branch contains QML bindings for KLocale. I believe that is the time to review it here.
> 
> 
> Diffs
> -----
> 
>   plasma/declarativeimports/locale/CMakeLists.txt PRE-CREATION 
>   plasma/declarativeimports/locale/calendarsystem.h PRE-CREATION 
>   plasma/declarativeimports/locale/calendarsystem.cpp PRE-CREATION 
>   plasma/declarativeimports/locale/locale.h PRE-CREATION 
>   plasma/declarativeimports/locale/locale.cpp PRE-CREATION 
>   plasma/declarativeimports/locale/localebindingsplugin.h PRE-CREATION 
>   plasma/declarativeimports/locale/localebindingsplugin.cpp PRE-CREATION 
>   plasma/declarativeimports/locale/qmldir PRE-CREATION 
>   plasma/declarativeimports/locale/tests/contents/code/klocaleqmltest.qml PRE-CREATION 
>   plasma/declarativeimports/locale/tests/metadata.desktop PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/104716/diff/
> 
> 
> Testing
> -------
> 
> there is a plasmoid in kde-runtime/plasma/declarativeimports/locale/tests in which I have exported every property/q_invokable.
> 
> Some issues that I will like to mention
> 
> a. there is tests/contents/code/klocaleqmltest.qml I believe that it should be tests/contents/ui/main.qml
> b. the test plasmoid is exported as org.kde.klocaleqmltest it should be org.kde.example.localetest
> c. Locale::setActiveCatalog and Locale::setMainCatalog since we didn't want the bindings to influence the core functionality.
> 
> what do you think?
> 
> 
> Thanks,
> 
> Giorgos Tsiapaliwkas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120429/09f8b308/attachment.html>