----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109965/#review32350 -----------------------------------------------------------
licenseQuery, partnerQuery, languageQuery need to be changed to licenseId, partnerId and languageId. they return the ID of the license, partner and language. what they do internally is irrelevant. API must reflect what the API does to the user of the API, not how the API is implemented internally. it makes for more readable code and more maintainable code (as the implementation tends to stay in line with the naming) - Aaron J. Seigo On April 20, 2013, 10:52 a.m., Giorgos Tsiapaliokas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/109965/ > ----------------------------------------------------------- > > (Updated April 20, 2013, 10:52 a.m.) > > > Review request for Plasma. > > > Description > ------- > > This patch > > * removes the duplicated code in assetimporters > * adds asset's size into the db > * and fixes a few small issues > > > Diffs > ----- > > assetimporters/CMakeLists.txt 24e76a0 > assetimporters/database-common/channelscatalog.h 5d39c02 > assetimporters/database-common/channelscatalog.cpp 6ca0aef > assetimporters/database-common/database.h 9883216 > assetimporters/database-common/database.cpp e860bdd > assetimporters/kdeartwork-wallpapers/CMakeLists.txt 56d19b9 > assetimporters/kdeartwork-wallpapers/database.h 6991758 > assetimporters/kdeartwork-wallpapers/database.cpp d75cdda > assetimporters/kdeartwork-wallpapers/kdewallpapersdatabase.h PRE-CREATION > assetimporters/kdeartwork-wallpapers/kdewallpapersdatabase.cpp PRE-CREATION > assetimporters/kdeartwork-wallpapers/main.cpp 708a949 > assetimporters/obs/CMakeLists.txt 2dbcd42 > assetimporters/obs/channelscatalog.h PRE-CREATION > assetimporters/obs/channelscatalog.cpp PRE-CREATION > assetimporters/obs/packagedatabase.h 99f4e17 > assetimporters/obs/packagedatabase.cpp ae43b8e > assetimporters/projectgutenberg/CMakeLists.txt b86cc49 > assetimporters/projectgutenberg/src/CMakeLists.txt 2d48e9c > assetimporters/projectgutenberg/src/database.h 8dba0ba > assetimporters/projectgutenberg/src/database.cpp 75cba69 > assetimporters/projectgutenberg/src/gutenbergdatabase.h PRE-CREATION > assetimporters/projectgutenberg/src/gutenbergdatabase.cpp PRE-CREATION > assetimporters/projectgutenberg/src/main.cpp 46f2340 > sql/bodega.sql 44f8641 > > Diff: http://git.reviewboard.kde.org/r/109965/diff/ > > > Testing > ------- > > I haven't noticed regression. > > > Thanks, > > Giorgos Tsiapaliokas > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
