Changeset View
Changeset View
Standalone View
Standalone View
src/modeldataloader.h
Show First 20 Lines • Show All 56 Lines • ▼ Show 20 Line(s) | 55 | Q_SIGNALS: | |||
---|---|---|---|---|---|
57 | void allAlbumsData(const ModelDataLoader::ListAlbumDataType &allData); | 57 | void allAlbumsData(const ModelDataLoader::ListAlbumDataType &allData); | ||
58 | 58 | | |||
59 | void allArtistsData(const ModelDataLoader::ListArtistDataType &allData); | 59 | void allArtistsData(const ModelDataLoader::ListArtistDataType &allData); | ||
60 | 60 | | |||
61 | void allGenresData(const ModelDataLoader::ListGenreDataType &allData); | 61 | void allGenresData(const ModelDataLoader::ListGenreDataType &allData); | ||
62 | 62 | | |||
63 | void allTracksData(const ModelDataLoader::ListTrackDataType &allData); | 63 | void allTracksData(const ModelDataLoader::ListTrackDataType &allData); | ||
64 | 64 | | |||
65 | void allRadiosData(const ModelDataLoader::ListTrackDataType &radiosData); | ||||
66 | | ||||
67 | void allRadiosDataUIUpdate(TrackDataType radiosData, bool isInsertion); | ||||
astippich: are all these updateUI signals (also below) really necessary? they are not needed for the rest… | |||||
Indeed I have removed radioModified signal which was not needed in this class. For the three others, I kept a defined role for each of them:
Should it be better if I try to integrate them or at least give them some better names? jguidon: Indeed I have removed radioModified signal which was not needed in this class.
For the three… | |||||
If you rename it now to radioModified I think it is pretty consistent with the rest astippich: If you rename it now to radioModified I think it is pretty consistent with the rest | |||||
68 | | ||||
65 | void allTrackData(const ModelDataLoader::TrackDataType &allData); | 69 | void allTrackData(const ModelDataLoader::TrackDataType &allData); | ||
66 | 70 | | |||
71 | void allRadioData(const ModelDataLoader::TrackDataType &allData); | ||||
72 | | ||||
67 | void tracksAdded(ModelDataLoader::ListTrackDataType newData); | 73 | void tracksAdded(ModelDataLoader::ListTrackDataType newData); | ||
68 | 74 | | |||
69 | void trackModified(const ModelDataLoader::TrackDataType &modifiedTrack); | 75 | void trackModified(const ModelDataLoader::TrackDataType &modifiedTrack); | ||
70 | 76 | | |||
71 | void trackRemoved(qulonglong removedTrackId); | 77 | void trackRemoved(qulonglong removedTrackId); | ||
72 | 78 | | |||
73 | void genresAdded(ModelDataLoader::ListGenreDataType newData); | 79 | void genresAdded(ModelDataLoader::ListGenreDataType newData); | ||
74 | 80 | | |||
75 | void artistsAdded(ModelDataLoader::ListArtistDataType newData); | 81 | void artistsAdded(ModelDataLoader::ListArtistDataType newData); | ||
76 | 82 | | |||
77 | void artistRemoved(qulonglong removedDatabaseId); | 83 | void artistRemoved(qulonglong removedDatabaseId); | ||
78 | 84 | | |||
79 | void albumsAdded(ModelDataLoader::ListAlbumDataType newData); | 85 | void albumsAdded(ModelDataLoader::ListAlbumDataType newData); | ||
80 | 86 | | |||
81 | void albumRemoved(qulonglong removedDatabaseId); | 87 | void albumRemoved(qulonglong removedDatabaseId); | ||
82 | 88 | | |||
83 | void albumModified(const ModelDataLoader::AlbumDataType &modifiedAlbum); | 89 | void albumModified(const ModelDataLoader::AlbumDataType &modifiedAlbum); | ||
84 | 90 | | |||
91 | void saveRadioModified(TrackDataType trackDataType); | ||||
92 | | ||||
93 | void deleteRadioInDatabase(qulonglong radioId); | ||||
94 | | ||||
95 | void radioRemoved(qulonglong radioId); | ||||
96 | | ||||
85 | public Q_SLOTS: | 97 | public Q_SLOTS: | ||
86 | 98 | | |||
87 | void loadData(ElisaUtils::PlayListEntryType dataType); | 99 | void loadData(ElisaUtils::PlayListEntryType dataType); | ||
88 | 100 | | |||
89 | void loadDataByAlbumId(ElisaUtils::PlayListEntryType dataType, qulonglong databaseId); | 101 | void loadDataByAlbumId(ElisaUtils::PlayListEntryType dataType, qulonglong databaseId); | ||
90 | 102 | | |||
91 | void loadDataByGenre(ElisaUtils::PlayListEntryType dataType, | 103 | void loadDataByGenre(ElisaUtils::PlayListEntryType dataType, | ||
92 | const QString &genre); | 104 | const QString &genre); | ||
Show All 9 Lines | |||||
102 | 114 | | |||
103 | void loadDataByFileName(ElisaUtils::PlayListEntryType dataType, | 115 | void loadDataByFileName(ElisaUtils::PlayListEntryType dataType, | ||
104 | const QUrl &fileName); | 116 | const QUrl &fileName); | ||
105 | 117 | | |||
106 | void loadRecentlyPlayedData(ElisaUtils::PlayListEntryType dataType); | 118 | void loadRecentlyPlayedData(ElisaUtils::PlayListEntryType dataType); | ||
107 | 119 | | |||
108 | void loadFrequentlyPlayedData(ElisaUtils::PlayListEntryType dataType); | 120 | void loadFrequentlyPlayedData(ElisaUtils::PlayListEntryType dataType); | ||
109 | 121 | | |||
122 | void updateRadioData(TrackDataType trackDataType); | ||||
123 | | ||||
124 | void deleteRadioData(qulonglong radioId); | ||||
125 | | ||||
126 | void updateUIAfterRadioInsertOrUpdate(TrackDataType radio, bool isInsertion); | ||||
127 | | ||||
128 | void updateUIAfterRadioDeleted(qulonglong radioId); | ||||
astippich: can these be renamed for consistency with others? | |||||
129 | | ||||
110 | private: | 130 | private: | ||
111 | 131 | | |||
112 | void databaseTracksAdded(const ListTrackDataType &newData); | 132 | void databaseTracksAdded(const ListTrackDataType &newData); | ||
113 | 133 | | |||
114 | void databaseTrackModified(const TrackDataType &modifiedTrack); | 134 | void databaseTrackModified(const TrackDataType &modifiedTrack); | ||
115 | 135 | | |||
116 | void databaseTrackRemoved(qulonglong removedTrackId); | 136 | void databaseTrackRemoved(qulonglong removedTrackId); | ||
117 | 137 | | |||
Show All 19 Lines |
are all these updateUI signals (also below) really necessary? they are not needed for the rest of the views