Account created on 18 March 2012, over 12 years ago
#

Merge Requests

Recent comments

πŸ‡·πŸ‡ΈSerbia g_miric

core: 8.x Needs to be removed from info file.

πŸ‡·πŸ‡ΈSerbia g_miric

This should be solved within: https://www.drupal.org/project/themed_fast_404/issues/3450805 ✨ Make the 404 page more configurable Needs work

πŸ‡·πŸ‡ΈSerbia g_miric

@joseph.olstad That is exactly what the new patch is doing. If there is a 404 page set in the "system.site" config and "use_system_404" is set to TRUE, it will take the get the page and make it static.

It doesn't make sense to take only the body field. In fact, the node that is set as the 404 page doesn't need to have a body field.

And if you really need to render only the body field then you should adjust the template for that node or to use the module setting "404 page body"

πŸ‡·πŸ‡ΈSerbia g_miric

The new feature will be implemented within #3450805 ✨ Make the 404 page more configurable Needs work so I'm closing this issue.

πŸ‡·πŸ‡ΈSerbia g_miric

@kensae Amazing. Thanks a lot!

The PR looks good. I requested some changes. Please ping me if you need help or if you have no time to implement them and I will do it.

πŸ‡·πŸ‡ΈSerbia g_miric

Hi @joseph.olstad,

If this patch works for you, you could apply it at the moment, but globally this wouldn't work:
- It overrides the fast 404 message and that should be optional.
- It assumes that 404 config path starts with "node/" if it's a node, but it could be an alias too.
- It renders only the body field, and it should render the whole page (there could be no body field at all).
- It should support any type of url, e.g. it could be custom route

I will work on this feature as soon as I get some time.

πŸ‡·πŸ‡ΈSerbia g_miric

Hi @joseph.olstad

Originally it was designed to override the default 404 page too, but this could be a feature request:

  • There could be a checkbox (e.g. "Use content from 404 config page instead") in the config page below or above the 404 body field
  • themed_fast_404 should get content of that page (it doesn't need to be node path)

I will work on that as soon as I'm available.

With that said, for some reason the sites/default/files/page-not-found-fr.html content that is generated is still using the settings value but the sites/default/files/page-not-found-en.html gets the expected node values even though I haven't changed any code from 1.0.5 .

The translation UI seems broken also.

I'm not able to reproduce any of these issues. Installed a new Drupal 10 site, added EN and FR languages, set FR as the default language, translated 404 config page, run cron, tested on EN and FR path and it looks good.

πŸ‡·πŸ‡ΈSerbia g_miric

@jcninov If you run cron again it should take the value from configuration page for english language too.

It is designed to override/ignore the default 404 page and use the value from configuration page.
If you have an issue with translating 404 page body value this issue πŸ“Œ Support config translation Fixed fixes it.

πŸ‡·πŸ‡ΈSerbia g_miric

Works as advertised!

Should we change version to 10.2.x-dev or have another MR for it?

πŸ‡·πŸ‡ΈSerbia g_miric

The latest patch worked for me too!

πŸ‡·πŸ‡ΈSerbia g_miric

Works fine!
Thanks for the patch.

πŸ‡·πŸ‡ΈSerbia g_miric

hook_help() isn't a required method and generally I don't like it.
It adds an another place where we need to maintain module documentation (beside module page on drupal.org and readme file) and I find it unneeded.

πŸ‡·πŸ‡ΈSerbia g_miric

You should check phpcs agains .md files.

πŸ‡·πŸ‡ΈSerbia g_miric

Any updates here? Did you guys tested it?

πŸ‡·πŸ‡ΈSerbia g_miric

Target branch changed from 2.0.x to 2.1.x. Please review.

πŸ‡·πŸ‡ΈSerbia g_miric

It isn't a core issue. I applied patch that resolves the core issue and the issue was still present.
Also, without this MR you are not able to excluded extensions. There is a config available for it, but not the code that will support it.

I will rebase in on 2.1.x branch.

πŸ‡·πŸ‡ΈSerbia g_miric

Works as expected!

πŸ‡·πŸ‡ΈSerbia g_miric

Stage file proxy is using StageFileProxySubscriber and it doesn't check config related to excluded extensions.
Here is a MR that fixes it.

Testing steps:

  • Apply MR
  • Set excluded extensions in the settings page (/admin/config/system/stage_file_proxy) - "js, css"
  • Clear cache
πŸ‡·πŸ‡ΈSerbia g_miric

g_miric β†’ made their first commit to this issue’s fork.

πŸ‡·πŸ‡ΈSerbia g_miric

g_miric β†’ made their first commit to this issue’s fork.

πŸ‡·πŸ‡ΈSerbia g_miric

Thanks roderik!
That solves my issue.

Production build 0.71.5 2024