Account created on 7 May 2012, about 12 years ago
#

Merge Requests

Recent comments

πŸ‡¬πŸ‡§United Kingdom oldspot

I can confirm without this extra patch to domain_conf our image styles were not being generated.

πŸ‡¬πŸ‡§United Kingdom oldspot

Created MR with potential solution. This worked ok in the tests I've made.

I checked with a normal non-image document too and that didn't cause any issues and simply skipped over the deletion since the 'derivative' files don't exist.

The reason I didn't put it in the ImageImport plugin specifically is because you can still import images with the base FileImport plugin (as it was in our case).

πŸ‡¬πŸ‡§United Kingdom oldspot

I had the same error on a Drupal 7 site so I rolled the above patch including the improvements from #4 into this patch.
Let me know if you prefer raising a separate D7 issue.

πŸ‡¬πŸ‡§United Kingdom oldspot

We've been experiencing these intermittent issues with URL generation and after debugging the "tokens" hooks being called I realised the field_tokens hook wasn't registered in order to be able to replace some URL pattern tokens.

I then came upon this issue and the patch in MR #42 in comment #17 fixed it instantly.

πŸ‡¬πŸ‡§United Kingdom oldspot

There seems to be an issue raised by 'Project Update Bot' https://www.drupal.org/project/shopify/issues/3289593 πŸ“Œ Automated Drupal 10 compatibility fixes Needs review so this one can probably be closed as duplicate?

πŸ‡¬πŸ‡§United Kingdom oldspot

Had the same issue on a user with a lot of revisions and the patch works great.

πŸ‡¬πŸ‡§United Kingdom oldspot

After upgrading to PHP 8.1 and latest diff module version I got the exact same warnings.
Can confirm this patch fixes them and can no longer see them.
Thanks

πŸ‡¬πŸ‡§United Kingdom oldspot

The solution really is to either set the value to Deny or Sameorigin depending on your website’s needs. In our case we just set to Deny.
The proposed resolution above is mainly to help people in the future by preventing them from choosing the broken option.
We’d also have to decide what value to set it to for sites that already have the value set to Allowfrom - should they default to Deny or Sameorigin?

Production build 0.69.0 2024