| Summary: | KRecentDocument::add() is broken after commit 48b78ce3 | ||
|---|---|---|---|
| Product: | [Unmaintained] kdelibs | Reporter: | Eugene Shalygin <eugene.shalygin+bugzilla.kde> |
| Component: | general | Assignee: | kdelibs bugs <kdelibs-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | cfeck, guyfawkesmaskguy, wbauer1, wrapash |
| Priority: | NOR | ||
| Version: | Git | ||
| Target Milestone: | --- | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/kdelibs/13f0e65f8b0bbee8fa32d7634743026a518903c1 | Version Fixed In: | 4.11.1 |
| Sentry Crash Report: | |||
line 115, sorry. But it is easy to find anyway Git commit 13f0e65f8b0bbee8fa32d7634743026a518903c1 by Michael Pyne. Committed on 20/08/2013 at 00:27. Pushed by mpyne into branch 'KDE/4.11'. krecentdocument: Ensure full path is saved with entries. This was accidentally broken in commit 48b78ce3e, which resulted in only the filename (not the full path) being saved with KRecentDocument entries. The suggested fix in the reported bug is not adequate as 'dStr' can be used later in the function, so it needs to retain the semantics it had prior to commit 48b78ce3e. We do appreciate the prompt report though!. FIXED-IN:4.11.1 M +1 -1 kio/kfile/krecentdocument.cpp http://commits.kde.org/kdelibs/13f0e65f8b0bbee8fa32d7634743026a518903c1 *** Bug 323702 has been marked as a duplicate of this bug. *** *** Bug 322529 has been marked as a duplicate of this bug. *** *** Bug 324110 has been marked as a duplicate of this bug. *** (In reply to comment #2) > Git commit 13f0e65f8b0bbee8fa32d7634743026a518903c1 by Michael Pyne. > Committed on 20/08/2013 at 00:27. > Pushed by mpyne into branch 'KDE/4.11'. This commit fixes Bug#324110 for me. Thank you! *** Bug 329084 has been marked as a duplicate of this bug. *** |
in line 113 of krecentdocument.cpp: QString ddesktop = dStr + QLatin1String(".desktop") results of recentDocumentDirectory() call is forgotten, therefore the function creates .desktop files in the current dir. The line should look like: QString ddesktop = path + dStr + QLatin1String(".desktop") Reproducible: Always