Summary: | Bookmarks not remembered when document is closed and re-opened | ||
---|---|---|---|
Product: | [Applications] kate | Reporter: | Nate Graham <nate> |
Component: | general | Assignee: | KWrite Developers <kwrite-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | Darren.Lissimore |
Priority: | NOR | Keywords: | usability |
Version: | Git | ||
Target Milestone: | --- | ||
Platform: | Neon | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/kate/f0f301be4b29e1e1db429173d3c39f99d97d15c4 | Version Fixed In: |
Description
Nate Graham
2017-08-27 21:02:08 UTC
Confirmed - kate from up to date neon - 17.08.1 . Git commit 5781412002e8cbea3ffcbe73adf65d185fd8f59e by Christoph Cullmann, on behalf of Silas Lenz. Committed on 17/10/2018 at 16:57. Pushed by cullmann into branch 'master'. Open document before using it's checksum to load metainfos Summary: loadMetaInfos tried to use the documents checksum before loading. The empty checksum results in it always failing to load metainfo and returning false, thus not using any of the stored metainfo. Test Plan: 1. Enable "Keep meta-information past sessions". 2. Open file (with or without previous bookmarks). 3. Add or remove a bookmark. 4. Close and reopen the file (and/or close and reopen Kate). 5. Previous bookmarks should be reloaded. Reviewers: #kate, cullmann Reviewed By: #kate, cullmann Subscribers: cullmann, ngraham, kwrite-devel Tags: #kate Differential Revision: https://phabricator.kde.org/D16259 M +4 -4 kate/katedocmanager.cpp https://commits.kde.org/kate/5781412002e8cbea3ffcbe73adf65d185fd8f59e Git commit f0f301be4b29e1e1db429173d3c39f99d97d15c4 by Christoph Cullmann, on behalf of Silas Lenz. Committed on 17/10/2018 at 16:58. Pushed by cullmann into branch 'Applications/18.08'. Open document before using it's checksum to load metainfos Summary: loadMetaInfos tried to use the documents checksum before loading. The empty checksum results in it always failing to load metainfo and returning false, thus not using any of the stored metainfo. Test Plan: 1. Enable "Keep meta-information past sessions". 2. Open file (with or without previous bookmarks). 3. Add or remove a bookmark. 4. Close and reopen the file (and/or close and reopen Kate). 5. Previous bookmarks should be reloaded. Reviewers: #kate, cullmann Reviewed By: #kate, cullmann Subscribers: cullmann, ngraham, kwrite-devel Tags: #kate Differential Revision: https://phabricator.kde.org/D16259 M +4 -4 kate/katedocmanager.cpp https://commits.kde.org/kate/f0f301be4b29e1e1db429173d3c39f99d97d15c4 |