Review Request 126171: Compare directories finishing by trailing slash

Olivier Churlaud olivier at churlaud.com
Wed Nov 25 18:10:43 UTC 2015


-----------------------------------------------------------
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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20151125/dd91c615/attachment-0001.html>


More information about the Amarok-devel mailing list