Use the path instead of the display string in PlacesItemModel::closestItem,

fixes the determination of the closest item for me.
This commit is contained in:
Emmanuel Pescosta 2014-10-21 22:54:15 +02:00
parent c9641c96f9
commit c07a97c4be

View file

@ -214,8 +214,12 @@ int PlacesItemModel::closestItem(const QUrl& url) const
for (int i = 0; i < count(); ++i) {
const QUrl itemUrl = placesItem(i)->url();
if (itemUrl.isParentOf(url)) {
const int length = itemUrl.toDisplayString().length();
if (url == itemUrl) {
// We can't find a closer one, so stop here.
foundIndex = i;
break;
} else if (itemUrl.isParentOf(url)) {
const int length = itemUrl.path().length();
if (length > maxLength) {
foundIndex = i;
maxLength = length;