Hi

When using varnish + expira + i18N + the wrong paths are banned

included patch resolves this.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Mschudders’s picture

FileSize
1.59 KB
Mschudders’s picture

FileSize
1.59 KB
mgifford’s picture

Status: Needs review » Needs work

No longer applies.

doitDave’s picture

Status: Needs work » Needs review

What does "no longer applies" mean in connection with "needs work"? The patch no longer works? The error has disappeared (it has not!)? Could you be more verbose on that?

Setting back to "needs review" and hoping for a solution or more communication, since an important frontend cache related module without proper i18n support is really not AAA.

manarth’s picture

Status: Needs review » Needs work

In this context "No longer applies" is a phrase that's often used to mean that the patch does not cleanly apply to the code (usually because commits that have been made since the patch was created have changed the code enough that the patch command cannot figure out how to apply the patch).

It means that the patch needs updating so that it applies cleanly against the current -dev version of the code (in modules with a lot of activity, people often talk about "Chasing head" when a patch repeatedly needs updating because the code moves forward so quickly).

I also noticed that the patch is relative to the Drupal root: ideally, patches for a module are based on the module directory itself (although I appreciate not everyone follows this best practice).

--- a/sites/all/modules/contrib/varnish/varnish.module
+++ b/sites/all/modules/contrib/varnish/varnish.module

There's a good post about creating patches for core and contrib, which also has some good links and other references.

doitDave’s picture

I was really hoping for some update on the actual issue rather than best practice. No offense. I will try to find a workaround then.

dshields’s picture

Any progress?

henk’s picture

Status: Needs work » Closed (duplicate)

This is duplicated issue, correct solution is here: https://www.drupal.org/node/2546218