Review Request: Fix sourceFilter to actually filter the sources in QML DataModel

[Commit Hook]

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


This review has been submitted with commit 4542bfbc22e9b20c13d7687239c4392ba16e7d07 by Viranch Mehta to branch master.

- Commit Hook


On June 15, 2012, 10:20 p.m., Viranch Mehta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105267/
> -----------------------------------------------------------
> 
> (Updated June 15, 2012, 10:20 p.m.)
> 
> 
> Review request for Plasma and Marco Martin.
> 
> 
> Description
> -------
> 
> This fixes a bug in PlasmaCore.DataModel. The DataModel does not correctly filter the sources when sourceFilter is provided.
> 
> The dataUpdated function does terminate when sourceName does not match the sourceFilter, but when it does match, all sources from the data engine are added to the said DataModel instead of checking for the sourceFilter again. This patch introduces that check.
> 
> (the patch is clearer than the description)
> 
> 
> Diffs
> -----
> 
>   plasma/declarativeimports/core/datamodel.cpp 9400dbe 
> 
> Diff: http://git.reviewboard.kde.org/r/105267/diff/
> 
> 
> Testing
> -------
> 
> Tested with Battery Monitor applet when there are multiple batteries and a model is required consisting of only the battery sources from powermanagement engine.
> 
> Works as expected with the patch.
> 
> 
> Thanks,
> 
> Viranch Mehta
> 
>

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