Use try-finally for disable/enable versioning

[blankse]

Followup to #95

[github-actions]

Review Checklist

• Target branch (1.0 for bug fixes, others 1.x)
• Tests (if it's testable code, there should be a test for it - get help)
• Docs (every functionality needs to be documented, see here)
• Migration incl. install.sql (e.g. if the database schema changes, ...)
• Upgrade notes (deprecations, important information, migration hints, ...)
• Label
• Milestone

[blankse]

@solverat Here is the PR :)

[solverat]

@blankse Thanks for the quick response! I'll merge it for now. We may should consider outsourcing this method to a Helper and use it here too:

opendxp/models/Asset/Video.php

Lines 199 to 201 in e3c5961

	Model\Version::disable();
	$this->save(); // auto save
	Model\Version::enable();

opendxp/models/Asset/Video.php

Lines 228 to 230 in e3c5961

	Model\Version::disable();
	$this->save(); // auto save
	Model\Version::enable();

opendxp/models/Element/Recyclebin/Item.php

Lines 108 to 119 in e3c5961

	Model\Version::disable();
	$className = $element::class;
	/** @var Document|Asset|AbstractObject $dummy */
	$dummy = OpenDxp::getContainer()->get('opendxp.model.factory')->build($className);
	$dummy->setId($element->getId());
	$dummy->setParentId($element->getParentId() ?: 1);
	$dummy->setKey($element->getKey());
	if ($dummy instanceof DataObject\Concrete) {
	$dummy->setOmitMandatoryCheck(true);
	}
	$dummy->save(['isRecycleBinRestore' => true]);
	Model\Version::enable();

opendxp/lib/Messenger/Handler/AssetUpdateTasksHandler.php

Lines 66 to 69 in e3c5961

	Version::disable();
	$asset->markFieldDirty('modificationDate'); // prevent modificationDate from being changed
	$asset->save($saveParams);
	Version::enable();

[solverat]

@blankse Very nice! Thank you for your efforts! Thanks @jdreesen for reviewing!