----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126171/ -----------------------------------------------------------
Review request for Amarok, Maximilian Kossick and Ralf Engels. Repository: amarok Description ------- [This goes on the KF5 branch] After porting KUrl to QUrls it appeared that the folders where not saved in the database. I found the reason and did a dirty fix: src/core-impl/collections/db/sql/SqlScqnResultProcessor.cpp:401 - if( !m_foundDirectories.contains( path ) ) + if( !m_foundDirectories.contains( path +'/' ) ) The patch here should take the problem at its source and correct it in a good way. I had other alternatives in a mail I sent [1] (but I didn't get any answer) [1] https://mail.kde.org/pipermail/amarok-devel/2015-November/013852.html Diffs ----- src/core-impl/collections/db/MountPointManager.cpp e14cb18 Diff: https://git.reviewboard.kde.org/r/126171/diff/ Testing ------- Build, and folders saved in the database Thanks, Olivier Churlaud
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
