Review Request 126851: Places data engine: Rename model role name "index" to "id"

Daniel Faust hessijames at mailbox.org
Wed Jan 27 10:34:41 UTC 2016


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

(Updated Jan. 27, 2016, 11:34 a.m.)


Status
------

This change has been marked as submitted.


Review request for kde-workspace and Plasma.


Changes
-------

Submitted with commit 8bedfde2894a60dc72a0bd40589dfb900776dace by Daniel Faust to branch master.


Repository: plasma-workspace


Description
-------

I wrote a qml applet using the places data engine and noticed that I can't access the index variable of ListView items because it gets overwritten by the places model.
This patch renames the "index" role name to "id" in order to avoid this naming conflict.


Diffs
-----

  dataengines/places/org.kde.places.operations 98a951d 
  dataengines/places/placeservice.cpp e0c93c5 
  dataengines/places/placesproxymodel.h 467fe83 
  dataengines/places/placesproxymodel.cpp 5ea807b 

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


Testing
-------

I couldn't find any other model that is using the places data engine so I hope renaming the model role is fine.
The provided model still works and I tested the "Setup Device" and "Teardown Device" operations of the service.
The operations "Show" and "Hide" don't seem to work anyway and the others were not tested.


Thanks,

Daniel Faust

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


More information about the Plasma-devel mailing list