After core commit 11.x-dev#7e27c7f can no longer save field definitions

I cannot find commit 7e27c7f in the repo. Please check the commit hash.

https://github.com/drupal/core/commit/7e27c7fdd465c62004d2355bcc6a38f52d...

Because of the subtree split, Github commit hashes are different than Gitlab commit hashes.

Bisected it down to:

`COMPOSER_MEMORY_LIMIT=-1 composer require drupal/core:11.x-dev#7e27c7fdd465c62004d2355bcc6a38f52d91b176 drupal/core-dev-pinned:${{ matrix.drupal }}`

https://github.com/colemanw/webform_civicrm/compare/KarinG-D11-2-2?expand=1

📌 [pp-1] Mark several more modules as converted Active and this change record → indicate that adding `skip_procedural_hook_scan: false` to the services file is the correct way. Can someone test that and get a merge request started?

Thanks I tried it but it makes no difference. Cleared cache too.

Specifically this change

index 97fce5da1..a234ca1ee 100644
--- a/webform.services.yml
+++ b/webform.services.yml
@@ -1,3 +1,6 @@
+parameters:
+  webform.skip_procedural_hook_scan: false
+
 services:

   # Plugins.

I get the impression that that parameter is false by default. I'm thinking because the hook HAS been converted to new style, there is some other way you are supposed to tell it about the resources it needs loaded.

Or would a simpler fix be to just move the "render_more" function somewhere else that is always loaded? Or, since it's only used in two files, move it directly into the hook file?

Thanks!

This is because we set system to skip, but system has system_hook_info which needs to still execute.

📌 Remove skip procedural for modules implementing hook_hook_info Active

Thanks. I can confirm that core patch fixes it locally and also in the tests mentioned in comment 5.

Ok, I am marking this as a pending duplicate but I am leaving it open so we don't forget it.

Yeah this issue is 100% core.

While I agree that this is a bug in core, webform is still unnecessarily relying on automatic .inc file loading for helper functions called by that hook.

This is deprecated for removal in D12, while legacy hooks are supported until D13 (and not even formally deprecated yet), so you likely want to keep them around longer to support D10.

That means eventually, all functions in tokens.inc must move to .module and there is no reason to not do it now. that would fix this particular error but the token system is going to be extremely broken due to all non-converted hooks that are still in tokens.inc, such as the ones in token module.

So, changing this to a normal task to move that.

In that case, perhaps the existing proposed resolution is the right solution. That is a smaller change than moving all the functions to the module file.

I really would recommend moving the functions, it's the expected resolution we planned for in core.