Problem/Motivation

File URL is purged twice on update.

Steps to reproduce

When updating a file, this module purges both the original and new URL of the file, which is great! However, if these are the same, we should only purge the URL once. This will be just a tiny micro-optimization but still seems worth it :)

Proposed resolution

Compare the old and new file URLs before issuing two purges.

CommentFileSizeAuthor
#2 3261445-2.patch576 bytesmfb

Comments

mfb created an issue. See original summary.

mfb’s picture

Status: Active » Needs review
StatusFileSize
new576 bytes

Something like this could do it

  • omarlopesino committed 6bc9b2e on 1.0.x authored by mfb
    Issue #3261445 by mfb: Unnecessary double-purge if modified file has the...
omarlopesino’s picture

Status: Needs review » Fixed

I've checked it and it works fine. So know we won't have that duplicate call to the purge system.
This correction will be ready in the 1.0.0-alpha3 version. Thanks for the contribution!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.