- Issue created by @aasarava
- πΊπΈUnited States aasarava San Diego
aasarava β changed the visibility of the branch 3514526-unlink-warnings-after to hidden.
- Merge request !12Remove unecessary unlink in create deriv uri; prevent warnings in unlink in delete β (Merged) created by aasarava
- πΊπΈUnited States aasarava San Diego
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.
- π«π·France DrDam
the unlink in createContextualizedDerivativePath() are present to prevent non replacement of derivativ in 1.x versions.
Nice catch
Automatically closed - issue fixed for 2 weeks with no activity.