Revert "gwenview: do not emit Document::saved() twice when the undo stack is clean"

This reverts commit 66f4c1cf16.
This commit is contained in:
Ivailo Monev 2024-03-23 14:45:16 +02:00
parent c316adfdbb
commit 0c54882606

View file

@ -296,15 +296,11 @@ void Document::slotSaveResult(KJob* job)
if (job->error()) { if (job->error()) {
setErrorString(job->errorString()); setErrorString(job->errorString());
} else { } else {
const bool undoStackWasClean = d->mUndoStack.isClean();
d->mUndoStack.setClean(); d->mUndoStack.setClean();
SaveJob* saveJob = static_cast<SaveJob*>(job); SaveJob* saveJob = static_cast<SaveJob*>(job);
d->mUrl = saveJob->newUrl(); d->mUrl = saveJob->newUrl();
d->mImageMetaInfoModel.setUrl(d->mUrl); d->mImageMetaInfoModel.setUrl(d->mUrl);
if (undoStackWasClean) { saved(saveJob->oldUrl(), d->mUrl);
// if the undo stack changes saved() will be emitted
emit saved(saveJob->oldUrl(), d->mUrl);
}
} }
} }
@ -465,7 +461,7 @@ void Document::slotUndoIndexChanged()
// If user just undid all his changes this does not really correspond // If user just undid all his changes this does not really correspond
// to a save, but it's similar enough as far as Document users are // to a save, but it's similar enough as far as Document users are
// concerned // concerned
emit saved(d->mUrl, d->mUrl); saved(d->mUrl, d->mUrl);
} else { } else {
modified(d->mUrl); modified(d->mUrl);
} }