<table><tr><td style="">aacid created this revision.<br />aacid added a project: Okular.<br />Restricted Application added a subscriber: Okular.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D8642" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Changes are no longer saved as part of the local configuration files, they can be either saved back to file, to an .okular file (if the file format doesn't support annotations) or not saved at all.</p>

<p>You are probably better having a look at each of the changes individually of<br />
git log  origin/master...origin/dont-use-docdata-for-annots-and-forms</p></div></div><br /><div><strong>TEST PLAN</strong><div>

<ul class="remarkup-list">
<li class="remarkup-list-item">Open pdf file, add anotation, close app<ul class="remarkup-list">
<li class="remarkup-list-item">You get dialog about losing changes, check that save, discard, cancel all do what they say</li>
</ul></li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Open png file, add anotation, close app<ul class="remarkup-list">
<li class="remarkup-list-item">You get dialog about losing changes, check that save, discard, cancel all do what they say<ul class="remarkup-list">
<li class="remarkup-list-item">Check for saving suggests saving as okular archive to not lose the annotation<ul class="remarkup-list">
<li class="remarkup-list-item">Check that opening the saved .okular file indeed keeps the annotation</li>
</ul></li>
</ul></li>
</ul></li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">With old version of okular, add annotation to pdf file, then open with new version, see you get the top bar about having to migrate the contents, save it as a different file, then open again the original file with old version of okular and verify the "annotation that was saved in docdata" is gone, open the second file and confirm the annotation is there.</li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Open pdf file with forms for example autotests/data/formSamples.pdf and modify all fields, also add an annotation and move it, resize it, change its color and delete it. Now check that you can undo/redo and save at all of the steps of the stack. This is important because saving indeed is like an internal reload so the structures of the undo/redo stack need to be updated to the new values seamlessly</li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Open png file add an annotation and save it as okular archive. Then move the annotation, resize it, change its color and delete it. Now check that you can undo/redo and save at all of the steps of the stack. This is important because saving indeed is like an internal reload so the structures of the undo/redo stack need to be updated to the new values seamlessly</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8642" rel="noreferrer">https://phabricator.kde.org/D8642</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/data/file1-docdata.xml<br />
autotests/documenttest.cpp<br />
autotests/parttest.cpp<br />
core/document.cpp<br />
core/document.h<br />
core/document_p.h<br />
core/documentcommands.cpp<br />
core/documentcommands_p.h<br />
core/generator.cpp<br />
core/generator.h<br />
core/observer.h<br />
core/page.cpp<br />
core/page.h<br />
core/page_p.h<br />
doc/index.docbook<br />
generators/kimgio/generator_kimgio.cpp<br />
generators/kimgio/generator_kimgio.h<br />
generators/poppler/generator_pdf.cpp<br />
generators/poppler/generator_pdf.h<br />
part.cpp<br />
part.h<br />
part.rc<br />
shell/shell.cpp<br />
shell/shell.h<br />
ui/annotationmodel.cpp<br />
ui/annotwindow.cpp<br />
ui/annotwindow.h<br />
ui/bookmarklist.cpp<br />
ui/formwidgets.cpp<br />
ui/formwidgets.h<br />
ui/pageview.cpp<br />
ui/pageview.h<br />
ui/pageviewmouseannotation.cpp<br />
ui/pageviewmouseannotation.h<br />
ui/pageviewutils.cpp<br />
ui/pageviewutils.h</div></div></div><br /><div><strong>To: </strong>aacid<br /><strong>Cc: </strong>Okular, aacid<br /></div>