Remove unecessary unlink in create deriv uri; prevent warnings in unlink in delete (!12) · Merge requests · project / media_contextual_crop

Related to #3514526

Are there any downsides to removing the unlink in createContextualizedDerivativePath()? The method name suggests that only a URI should be generated, with no side effects like deleting files. The unlink is already in the deleteDerivative() method, and that seems like the proper place for it.

Edited Mar 21, 2025 by Amit Asaravala

Remove unecessary unlink in create deriv uri; prevent warnings in unlink in delete

Merge request reports