----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118022/ -----------------------------------------------------------
(Updated May 7, 2014, 4:05 p.m.) Status ------ This change has been marked as submitted. Review request for Plasma. Repository: plasma-mediacenter Description ------- When backends needed to access some PMC objects like the playlist model or register image providers, they used to fetch a reference to the view's declarative engine. This is risky because it exposes the view to the backends which can then do anything with it. This patch adds a PmcRuntime class which serves as an official interface to these accesses cleanly. Diffs ----- browsingbackends/localfiles/localfilesabstractbackend.h 2d47eba browsingbackends/localfiles/localfilesabstractbackend.cpp b86a28a browsingbackends/localfiles/localpictures/localpicturesbackend.cpp f025c04 browsingbackends/localfiles/localpictures/localpicturesmodel.h b1a81b4 browsingbackends/localfiles/localpictures/localpicturesmodel.cpp c6b1816 browsingbackends/localfiles/localvideos/localvideosbackend.cpp 8c20f27 browsingbackends/localfiles/localvideos/localvideosmodel.h e836c14 browsingbackends/localfiles/localvideos/localvideosmodel.cpp ab24013 browsingbackends/metadatabackends/metadatamusicbackend/metadatamusicbackend.h 50af6c2 browsingbackends/metadatabackends/metadatamusicbackend/metadatamusicbackend.cpp 6991c97 libs/mediacenter/CMakeLists.txt f07d3ec libs/mediacenter/abstractbrowsingbackend.h 537d00d libs/mediacenter/abstractbrowsingbackend.cpp eea4ee5 libs/mediacenter/backendsmodel.h f98f60a libs/mediacenter/backendsmodel.cpp 3a5fc30 libs/mediacenter/pmcruntime.h PRE-CREATION libs/mediacenter/pmcruntime.cpp PRE-CREATION shells/newshell/mainwindow.h 0aaa513 shells/newshell/mainwindow.cpp e369055 shells/newshell/package/contents/ui/mediacenter.qml a62becc Diff: https://git.reviewboard.kde.org/r/118022/diff/ Testing ------- Adding songs to playlist, local pictures previews, local videos previews etc work fine. Thanks, Shantanu Tushar
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel