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

Merge Requests

More

Recent comments

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet created an issue.

🇧🇪Belgium dieterholvoet Brussels

I fixed this in context of Add support for updating submissions Active , could you try out the latest dev release and see if it fixes your issue?

Maybe there is an issue also with the format of the min and max values. In formkit and html the time input format seems to be hh:mm while it's returned hh:mm:ss.

That shouldn't be an issue according to the MDN docs:

The value is always a 24-hour HH:mm or HH:mm:ss formatted time, with leading zeros, regardless of the UI's input format.

🇧🇪Belgium dieterholvoet Brussels

This seems to have been working for more than a month in one of our projects so I'm going to go ahead and merge this. Any bugs or improvements can go in new tickets.

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet created an issue.

🇧🇪Belgium dieterholvoet Brussels

This module still supports Drupal 9.2. The PHPUnit attributes were only added in PHPUnit 10, which has only been supported since Drupal 11.2.0. Won't this cause issues?

🇧🇪Belgium dieterholvoet Brussels

That's a possibility, or ideally we would add a change record summarizing what was said/decided in the issue.

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet created an issue.

🇧🇪Belgium dieterholvoet Brussels

Left some comments in the MR.

🇧🇪Belgium dieterholvoet Brussels

It should also be the time in seconds the Drush command should spend working on this specific queue in one run, but this is not enforced in this module.

Just realised this is not true. That's what the time-limit option is for, the lease-time only determines how long queue items are held.

🇧🇪Belgium dieterholvoet Brussels

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

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet created an issue.

🇧🇪Belgium dieterholvoet Brussels

dieterholvoet created an issue.

🇧🇪Belgium dieterholvoet Brussels

I think so, but not sure. It's been a while.

🇧🇪Belgium dieterholvoet Brussels

Pipeline is passing now, but still some phpstan & cspell warnings.

🇧🇪Belgium dieterholvoet Brussels

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

🇧🇪Belgium dieterholvoet Brussels

Moving to the Drupal.org project ownership issue queue.

🇧🇪Belgium dieterholvoet Brussels

Setting to Needs work because more changes might be necessary, don't have time to do a deep dive right now.

🇧🇪Belgium dieterholvoet Brussels

I already tried a couple times but the new contribution record system has been acting up. I tried again.

🇧🇪Belgium dieterholvoet Brussels

I'm going to use this patch in a project for a week or two and if all is good, I'll merge.

🇧🇪Belgium dieterholvoet Brussels

Changes to this contribution record don't seem to be saving for me, once I refresh the page everything is unchecked again. I am noticing a lot of console errors in the likes of Uncaught TypeError: $(...).once is not a function

🇧🇪Belgium dieterholvoet Brussels

Tests are finally passing again. Thanks for the work everyone!

🇧🇪Belgium dieterholvoet Brussels

Maybe filter by only entries that were created by Tagify? Not sure if it could hurt to delete everything.

🇧🇪Belgium dieterholvoet Brussels

This fixes the error on the settings form for me.

🇧🇪Belgium dieterholvoet Brussels

Never mind, seems like more changes were necessary.

🇧🇪Belgium dieterholvoet Brussels

Seems like this is already possible if you just leave those two settings empty. I created a MR to leave those settings empty by default and to document the auto detection in the field descriptions.

🇧🇪Belgium dieterholvoet Brussels

I checked it myself using the test form, seems like it has to look like this:

travel_options[one]: "one"
travel_options[two]: "two"
travel_options[three]: "three"
🇧🇪Belgium dieterholvoet Brussels

I created a separate issue for the remaining CSpell issue: 📌 Clean up config keys Active . I'm going to apply a patch of these changes to a website that uses the module a lot and keep it running for a couple weeks. If all is good afterwards, I'll merge this.

🇧🇪Belgium dieterholvoet Brussels

@benstallings sorry, but can't reproduce this with the patch from 🐛 Only show red asterisk when the field is actually required Active applied. required is also set to true in my case, but that shouldn't matter.

third_party_settings:
  required_api:
    required_plugin: required_by_role
    required_plugin_options:
      - editor
required: true
🇧🇪Belgium dieterholvoet Brussels

Thanks for the quick response! I sent the same message to @eme through their contact form.

🇧🇪Belgium dieterholvoet Brussels

I sent the following message to @mandclu, the only maintainer who has been working on the module in the last 4 years:

Hi,

Could you post a comment on my offer to co-maintain Form Mode Control ( https://www.drupal.org/project/form_mode_control/issues/3542611 💬 Offering to co-maintain Form Mode Control Active )? I would like to become a co-maintainer of the module to help resolve any pending and future issues.

Thanks in advance,
Dieter Holvoet

🇧🇪Belgium dieterholvoet Brussels

I tested the MR and apart from a small issue I fixed in the MR, nothing broke. I didn't encounter the issue as described here though, so I can't really say anything about that.

🇧🇪Belgium dieterholvoet Brussels

I did some profiling. I loaded the /admin/structure/menu/manage/admin page under different conditions:

Conclusion: good improvement, but not as good as the solution with the core patch, so I would leave this issue open for the time being. The new solution introduces a lot of custom code, but if you're willing to maintain all that I'd say go for it!

🇧🇪Belgium dieterholvoet Brussels

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

🇧🇪Belgium dieterholvoet Brussels

If you're unsure on how exactly to submit certain types of values, you should submit the Webform test form with the Network tab of your browser developer tools open and inspect the outgoing request.

🇧🇪Belgium dieterholvoet Brussels

The fix here is to use replaceWith instead of replace as AJAX method ( change record ).

🇧🇪Belgium dieterholvoet Brussels

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

🇧🇪Belgium dieterholvoet Brussels

I added a new 'Only use merge variables' option as described in the issue description. Ideally this new feature would get some tests.

🇧🇪Belgium dieterholvoet Brussels

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

🇧🇪Belgium dieterholvoet Brussels

The current MR fixes the issue for me! The disabled state is now propagated from the address element to its country element.

🇧🇪Belgium dieterholvoet Brussels

@benstallings what's the field type you tested with?

🇧🇪Belgium dieterholvoet Brussels

Okay, I was able to reproduce the issue, but only on simple fields that have HTML5 required validation through the required attribute. Before I was testing on more complex fields like entity reference and paragraphs fields. The good news is that the patch from 🐛 Only show red asterisk when the field is actually required Active seems to fix the issue. Please test this and let me know if it works for you, so I can merge the fix and make a release. Don't forget to clear caches after applying the patch.

🇧🇪Belgium dieterholvoet Brussels

You're misunderstanding the intention of the issue. This is about making sure that the 'Break lock' link does not appear in the dropdown of the sticky actions next to the title on the edit form, and instead it displays as an actual button. You're talking about the entity operations on the content overview.

🇧🇪Belgium dieterholvoet Brussels

Good news, we're going to be working on a more complete JSON Forms schema integration for a client soon.

🇧🇪Belgium dieterholvoet Brussels

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

🇧🇪Belgium dieterholvoet Brussels

Found the culprit. The issue was triggered by security updates last night. There's an associated core issue: 🐛 APCu requirement for 32MB always displays since APCu 5.1.25 Active .

The apc.shm_segments ini option has been removed. Multiple SHM segments are no longer supported. (They were already not supported when using mmap, which is the default mode of operation)

Source

I suggest we add a fallback to 1 if the ini directive is missing.

🇧🇪Belgium dieterholvoet Brussels

Not sure if the other check is actually necessary, I'll leave that to @primsi to determine.

🇧🇪Belgium dieterholvoet Brussels

Well I checked phpinfo() and the directive is missing. Haven't figured out how yet, I'm checking with our hosting partner, but it couldn't hurt checking for this situation. It's interesting that we're not the only one encountering this issue in the span of a couple days.

Production build 0.71.5 2024