Review Request 122200: Add overload method for Formats::formatDuration taking int

[Martin Klapetek]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122200/
-----------------------------------------------------------

(Updated Jan. 22, 2015, 4:52 p.m.)


Review request for Plasma and Kai Uwe Broulik.


Changes
-------

Fix whitespace


Repository: kdeclarative


Description
-------

It is currently not possible to use Q_INVOKABLE that takes Q_ENUM/Q_FLAGS as a parameter that is defined in a different class. This is mentioned in the docs and there's also a bug report here - https://bugreports.qt.io/browse/QTBUG-20639

If the default parameter is removed from the formatDuration, Plasma will complain with "Error: Unknown method parameter type KFormat::DurationFormatOptions".

Until that one is fixed, we can either take an integer and cast it to the flags or redeclare the enum from KFormats locally, but that's just not good.


Diffs (updated)
-----

  src/qmlcontrols/kcoreaddons/formats.h 59537bf 
  src/qmlcontrols/kcoreaddons/formats.cpp 256778f 

Diff: https://git.reviewboard.kde.org/r/122200/diff/


Testing
-------

KFormat now properly formats the remaining time based on the passed flags.


Thanks,

Martin Klapetek

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