Belgium 🇧🇪
Account created on 7 January 2016, over 9 years ago
#

Merge Requests

More

Recent comments

🇧🇪Belgium BramDriesen Belgium 🇧🇪

add Switzerland 

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Works great :-)

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Going to re-roll this for 3.x

🇧🇪Belgium BramDriesen Belgium 🇧🇪

This seems to be a regression from adding D11 support.

https://git.drupalcode.org/issue/minisite-3524424/-/commit/340184efdee81...

So this might need to be solved differently.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Great to see this happening 🙌 my votes go to:
- Marine Gandy (Mupsi)
- Joris Vercammen (borisson_)

🇧🇪Belgium BramDriesen Belgium 🇧🇪

I didn't test nor investigate this yet, but noticed the cache_metadata is set to -1. Doesn't that effect this as well.

Also need to check how the Drupal core node view for example does caching, as this is just an admin overview.

Will leave it at NR for now. Thanks for the report and merge request @alorenc

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Also updated in the event.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Think Jonas also deserves credit no? :-)

🇧🇪Belgium BramDriesen Belgium 🇧🇪

I noticed today that the user is seeking guidance on how to contribute properly on slack.
https://drupal.slack.com/archives/C1BMUQ9U6/p1745611803988939

Alberto Cocchiara
:palm_tree: 10:10 PM
hi all, I'm a Solution Architect working on Global project in the past 12 years, some of this are on Drupal where i moved my first step in Enterprise CMS practice, nowdays i work on Adobe AEM mostly but i want to give back something to Drupal community, that i'm part from almost 7 years, so i'm looking for a mentor to be guided to properly contribute in an effective way, thanks to anyone can point me or become my mentor :good-luck:

🇧🇪Belgium BramDriesen Belgium 🇧🇪

No problem! Thank you for the MR 😇

🇧🇪Belgium BramDriesen Belgium 🇧🇪

@mfrosch Please don't name an issue fork branch the same as the base branch in the future :-P it's a nightmare to work with when you want to rebase set base branch on the issue fork.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

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

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Will tag a new release because of the broken update hook.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

No activity over 2 days, un-assigning. Feel free to pick this up again.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

I would advise against this. Just imagine you're using this to store the database credentials, and somebody manages to do a screw up on production wiping the entire file. Meaning if this was done through the UI, Drupal would crash and you have to scramble to find someone to fix the file on the server. You might also not want everyone to be able to get access to those secrets, having the possibility as a site admin to just enable this via a module, seems a bit as a risk to me.

I guess this can also happen there, but at least a sysadmin shouldn't make such a mistake :-).

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Just needs a rebase, there is another issue open also adding an update hook so let's wait for that one.

Tested, could reproduce and this MR fixed the issue.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Code is also not OK, this is part of the entity already, no need to define a new basefield. No upgrade path provided either.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

We could look into implementing this with a recent core feature. Not that much logic needed for this.

https://www.drupal.org/node/2896596

🇧🇪Belgium BramDriesen Belgium 🇧🇪

If you applied the patch from issue 🐛 Entity mismatched after 1.6.0 to 2.0.0-alpha3 upgrade Active which was not rebased, you're overriding the update hook. poll_update_8007 should be the install of the field. poll_update_8008 should belong to the issue you are referencing.

Please check what is in your install file after applying the patches.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Isn't this more a Drupalorg Infra ticket? If I'm not mistaken that queue is responsible for the GitLab CI pipelines.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

I tested MR !103 from @ericgsmith and this seem to cover our use case as well. Since it's more flexible as the first approach, I would assume this is the better way to go.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Thanks @ivnish

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Patch looks good, no longer shows the generic login error when the user is created in a blocked state.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

I believe there are some other ways as well like ignoring some files.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Great! Thanks

🇧🇪Belgium BramDriesen Belgium 🇧🇪

To add something meaningful to this issue (beside my tests above :-) ), I updated the technical details code snippet.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

This is now implemented for anonymous users. Not sure if this is needed for none anonymous users.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Works as expected now, thanks! Merged :-)

🇧🇪Belgium BramDriesen Belgium 🇧🇪

This introduced a warning

Deprecated function: str_ends_with(): Passing null to parameter #1 ($haystack) of type string is deprecated in Drupal\openid_connect_windows_aad\Plugin\OpenIDConnectClient\WindowsAad->__construct() (line 82 of modules/contrib/openid_connect_windows_aad/src/Plugin/OpenIDConnectClient/WindowsAad.php).

Created a follow up: 🐛 Deprecated function: str_ends_with(): Passing null to parameter #1 ($haystack) of type string is deprecated in Drupal\openid_connect_windows_aad\Plugin\OpenIDConnectClient\WindowsAad->__construct() (line 82 of modules/contrib/openid_connect_windows_aad/sr Active

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Feel free to re-open if still relevant. But I would assume this is indeed fixed.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Hi,

The easiest way for you to achieve this I think is to create a custom entity/node with an entity reference field to "Poll". That way you can link multiple polls to your entity and render them. That way you still have control over each poll separately.

I don't really see/understand the need to have this feature in the Poll module itself.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

This can be taken up again since 📌 Anonymous users - Multiple voting from one IP Needs review landed.

I would go for the checkbox approach as mentioned in #6

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Number 2 is included in the "regular" poll. The others would be nice features although I'm not sure what the concordset exactly is.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Closing this one as a duplicate of Multiple Poll Types Active

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Needs a re-roll after the anonymous vote restrictions got added. Probably removes some duplicate code from that issue as well (remark 1 of #10)

🇧🇪Belgium BramDriesen Belgium 🇧🇪

I added some questions. I'm not sure if this fixes the issue for @fgm. So would appreciate if he could test this.

Also needs a rebase.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

This can be taken up now.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Thanks everyone who participated on this issue for the past 8 years! Please create a new issue if you encounter any issues with this.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

I guess that's not a problem. I haven't checked how other modules do it, but I guess you can show the "browse tokens" text conditionally when the token (contrib) module is installed.

As for replacing tokens in the notification message. I think the only thing we need to do is implement: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Utility%2...

During the rendering of the message.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

1) "Anonymous vote restriction" field doesn't save (or doesn't load saved value) when save/edit poll

I could not replicate. That sounds more like the database schema was not installed/updated correctly?

The required field logic is fixed by using a state.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

I think this is ready for review once more. Preferably also some manual testing as there was one issue the tests didn't catch which I'm still not sure why that test was nog failing (see last commit).

🇧🇪Belgium BramDriesen Belgium 🇧🇪

fixed a few things.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

one comment was not fixed.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

+1 For RTBC. Totally forgot about this issue.

🇧🇪Belgium BramDriesen Belgium 🇧🇪

Thanks for testing!

Production build 0.71.5 2024