> On Dec. 11, 2014, 3:27 p.m., Mark Gaiser wrote: > > src/core/kfileitem.cpp, line 255 > > <https://git.reviewboard.kde.org/r/121447/diff/1/?file=332652#file332652line255> > > > > --> add here > > > > } else { > > // Fix for IO slaves that don't set UDS_MIME_TYPE for a folder. > > if (m_fileMode & QT_STAT_MASK) == QT_STAT_DIR) { > > m_entry.insert(KIO::UDSEntry::UDS_MIME_TYPE, "inode/directory"); > > m_mimeType = db.mimeTypeForName("inode/directory"); > > m_bMimeTypeKnown = true; > > } > > } > > > > Not tested! Just written in comment box :) > > > > I think that's about all you'd need to fix this. > > But if this is accaptable is probably up to David to decide. > > > > I'm also not 100% sure that you catch all cases when readUDSEntry().
+1 I also think that this is better way to fix it. Avoids code duplication and the correct mime type for folders is set a early as possible, so other code can rely on it. - Emmanuel ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121447/#review71804 ----------------------------------------------------------- On Dec. 11, 2014, 2:22 p.m., Àlex Fiestas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121447/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2014, 2:22 p.m.) > > > Review request for KDE Frameworks. > > > Repository: kio > > > Description > ------- > > If we know that the item is a dir, return directly the correct mimetype for > directories. > > More info of why this is needed at: > https://git.reviewboard.kde.org/r/120909/ > > > Diffs > ----- > > src/core/kfileitem.cpp 6a2cfa5 > > Diff: https://git.reviewboard.kde.org/r/121447/diff/ > > > Testing > ------- > > > Thanks, > > Àlex Fiestas > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
