Brussels
Account created on 22 March 2018, about 7 years ago
  • Backend Developer at Minsky 
#

Merge Requests

More

Recent comments

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet created an issue.

🇧🇪Belgium dieterholvoet Brussels

This issue has been committed, but never released. Any chance you could create a last 8.x-1.x patch release?

🇧🇪Belgium dieterholvoet Brussels

Not sure if it's a good idea for core, but in the HTTP Cache Control module we added an Expert mode toggle to the Performance config form that toggles maxage inputs between selects with readable presets and number inputs.

🇧🇪Belgium dieterholvoet Brussels

This handler doesn't prevent submissions as far as I know. Are you sure the 'Disable saving of submissions' option is not enabled on your webform? Are submissions being saved when you remove the handler?

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet changed the visibility of the branch 1.0.x to hidden.

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet made their first commit to this issue’s fork.

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet made their first commit to this issue’s fork.

🇧🇪Belgium dieterholvoet Brussels

Dflydev\DotAccessData\Exception\InvalidPathException is not related to the Symfony Dotenv component, so there's no point in catching it. Also, these files do not look like the one that this module provides. I'm guessing there already was a load.environment.php present before you ran the command. You can find the file that this module provides here.

🇧🇪Belgium dieterholvoet Brussels

I tested this with a <h2> and you're right. Without the patch, the id attribute isn't in the filtered HTML and with the patch it is.

🇧🇪Belgium dieterholvoet Brussels

The test failure is about the CKEditor plugin, seems unrelated.

🇧🇪Belgium dieterholvoet Brussels

You're right, I missed that. Created a new issue to fix the namespace: 🐛 The class namespace of the new Search API processor is wrong Active .

🇧🇪Belgium dieterholvoet Brussels

I think you're misunderstanding the purpose of this module, it's not about XSS filtering or stripping HTML. To refer to the project description:

This module provides a Webform handler that allows submission data of certain form elements to not be stored in the database. The submission data can still be used in other handlers preceding the Sanitize submission handler.

🇧🇪Belgium dieterholvoet Brussels

I tested this in Drupal 11.1.6 and Redirect 1.11.0 and I can't reproduce this either.

🇧🇪Belgium dieterholvoet Brussels

This indirectly fixes an issue for me where pages get stuck in an infinite loop when using the flattened site settings loader.

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet made their first commit to this issue’s fork.

🇧🇪Belgium dieterholvoet Brussels

Yes, it looks like negotiating the active domain this early in the request (during the loading of config overrides) is not supported. I started a MR with an approach where the active domain is loaded on demand. Having our own static cache is unnecessary since the active domain is also just a property on the DomainNegotiator service, so there isn't any performance gain.

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet made their first commit to this issue’s fork.

🇧🇪Belgium dieterholvoet Brussels

I also added a custom translation handler to actually make publishing of translations work. The code is mostly based on Drupal\node\NodeTranslationHandler and Drupal\eck\EckTranslationHandler.

🇧🇪Belgium dieterholvoet Brussels

I don't think it's necessary to add a custom setting to control the visibility of the status field. That's what the form display UI is for. I'll change it so the field is hidden by default and can be displayed by moving it in the form display. I also changed the default value of the publishing status to TRUE.

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet changed the visibility of the branch 3090261-setinlineblockdependency-override-might to hidden.

🇧🇪Belgium dieterholvoet Brussels

I started a new MR on an issue branch instead of the main 8.x-2.x branch.

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet changed the visibility of the branch 8.x-2.x to hidden.

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet made their first commit to this issue’s fork.

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet changed the visibility of the branch 1.x to hidden.

🇧🇪Belgium dieterholvoet Brussels

Is the idea of preprocess hook because we don't have a single spot in the plugins/classes to build them?

Yes, just to make it work anywhere #theme => footnote_links is used. I guess it's only used at one place in the module, but it could also be used in another module. And also to support controlling this behaviour by passing #simplify_sequential_citations => TRUE/FALSE.

🇧🇪Belgium dieterholvoet Brussels

Are you sure they are actually still required? The red asterisks still shows, but you shouldn't get any validation errors when saving the entity.

🇧🇪Belgium dieterholvoet Brussels

Maybe also update all existing redirect entities in a update hook to stop using any of the removed status codes?

🇧🇪Belgium dieterholvoet Brussels

This InvalidArgumentException is also being triggered by bot traffic on all our sites using the Lazy module because it uses this RequestPath condition plugin. It's being triggered by visiting the path /index.php.suspected, so not just paths with double slashes. That's why I want to propose to also just catch any InvalidArgumentExceptions thrown in the getAliasByPath() method.

🇧🇪Belgium dieterholvoet Brussels

Never mind, that doesn't fix it. I now have citations that link to #footnote1, while the footnote has an ID of footnote1-0. That's on a page with multiple text areas with footnotes and with all of them aggregated at the bottom of the page.

🇧🇪Belgium dieterholvoet Brussels

For me this fixes the issue where links from citations to grouped references at the bottom of the page weren't working anymore.

🇧🇪Belgium dieterholvoet Brussels

These two .env variables are specific to the Laravel integration of Ignition (source), there are not (yet) supported in the Drupal module. Currently, the only way to change these settings is through the UI. I'm changing this to a feature request.

🇧🇪Belgium dieterholvoet Brussels

Why not postpone adding the argument to the interface until the next major version and until then make it an optional argument in worker classes like this:

public function processItem($data, ?object $metadata = NULL): void

calling the method like this:

$worker->processItem($item->data, $metadata);

That way it works both for workers with and without the method and the necessary changes to queue workers will be a lot more straightforward.

At some future point we could deprecate the processItem() method and just support the new method and interface?

If you're going to eventually do a breaking change, might as well stick to the same interface and method name and make the breaking change the addition of the new argument to the existing method, no?

🇧🇪Belgium dieterholvoet Brussels

Do we really need a new interface and method here? Why not just add the metadata as an optional argument to the processItem function and add it to the interface in a new major?

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet changed the visibility of the branch 3430646-automated-drupal-11 to hidden.

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet changed the visibility of the branch fix-composer-json to hidden.

🇧🇪Belgium dieterholvoet Brussels

Thank you! I opted the project into security coverage, added Gitlab CI, finished and merged all issues and created a first release. I'll stay around for any new issues as well, for the time being.

🇧🇪Belgium dieterholvoet Brussels

Can't seem to reproduce this in the latest version.

Production build 0.71.5 2024