@Defcon0

Defcon0 → created an issue.

@danflanagan8

The release doesn't contain the fix even though in the git repo it is. Strange. Please check the ZIP on

https://www.drupal.org/project/viewfield/releases/8.x-3.0-beta8 →

Defcon0 → created an issue.

Defcon0 → created an issue.

Don't know whether it would be better to create a new issue for this, but I created a simple cron in the module for deleting messages older than 30 days as a cron.

Sorry, first one didn't have the correct paths.

New patch for 10.2.5

Rerolled #71 for simple_sitemap 4.1.9 fixing the issue

PHP Fatal error:  Cannot redeclare simple_sitemap_update_8408() (previously declared in /var/www/html/app/web/modules/contrib/simple_sitemap/simple_sitemap.install:942) in /var/www/html/app/web/modules/contrib/simple_sitemap/simple_sitemap.install on line 967

Rerolled #72 for simple_sitemap 4.1.9 fixing the issue

PHP Fatal error: Cannot redeclare simple_sitemap_update_8408() (previously declared in /var/www/html/app/web/modules/contrib/simple_sitemap/simple_sitemap.install:942) in /var/www/html/app/web/modules/contrib/simple_sitemap/simple_sitemap.install on line 967

Ah, I just noted that the problem is a patch referenced in https://www.drupal.org/project/simple_sitemap/issues/3080541 ✨ Exclude content with a "noindex" robots meta tag Closed: won't fix . I comment there

Defcon0 → created an issue.

Defcon0 → created an issue.

Defcon0 → created an issue.

I just compared version 2.1.3 with the previous 2.1.2. The line that causes the issue is

https://git.drupalcode.org/project/stage_file_proxy/-/compare/2.1.2...2....

If I revert that to what it was before, it works again. I created a patch for that for the meantime.

Defcon0 → made their first commit to this issue’s fork.

Defcon0 → created an issue.

At first a big thanks to this new feature. Made parts of our custom implementation obsolete :)

Only one thing bothers me: why is the replacement character only trimmed at the right side of the filename and not on the left one, as well?

https://github.com/drupal/core/blob/10.2.x/modules/file/src/EventSubscri...

I think filenames like -some-filename.png are not so nice. What has been the reason to implement it that way? Couldn't find info in this issue log.

OK, now it works. I needed to regenerate the node access using /admin/reports/status/rebuild . Would've been cool if that was stated somewhere :-/ Found it in a very old video.

Defcon0 → created an issue.

I just installed the module and it's quite great. Only thing bothering me is that export config is stored per user and not in yml config. This is not practicable in a multi user setup. A default config on a node type basis would be highly appreaciated.

I rerolled #71 for a setup with metatags v2 replacing the unserialize() calls by json_decode().

Defcon0 → created an issue.

Same here.

Defcon0 → created an issue.

@camvertiz : Why does your match contain much less code than the Merge request (https://git.drupalcode.org/project/simple_sitemap/-/merge_requests/52.patch)?

Defcon0 → created an issue.

One thing maybe to consider is also checking rewrites/redirects. In my current setup I have some URLs in the sitemap leading to nodes having noindex set. Is there reasonable any way to avoid that?