๐Ÿ‡บ๐Ÿ‡ธUnited States @el1_1el

Account created on 3 February 2012, over 12 years ago
#

Recent comments

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Fixed and released as 2.0.0-alpha2. Thanks everyone!

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Symfony\Component\EventDispatcher\EventDispatcherInterface swapped args on the dispatch() method (object, string) instead of (string, object). That causes a fatal error in 10.

Attached patch fixes that and seems to work for me in canvas

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Looks like i should get the query before adding the access check. doh! been a long month. new patch attached

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el
๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

el1_1el โ†’ created an issue.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

If anyone else finds they have custom code relying on this commit as well as the api from the 1.0.x version, or is just having other issues with the 1-2 update, and needs to update to drupal 10, I've attached a patch that restores the 1x version on 2.0 for drupal 10.

Also note that you may need to apply the patch for field_group that removes its implements_alter function from the 3.4 version

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

I had this issue after an update from core 9.5.8 to 9.5.11 as well. I rerolled the diff patch to remove the empty .DS_Store differ binary.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

fixed and released as alpha7

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Here's a patch for 10.1.3 - yes it has coding style errors (its not going to be committed w/o tests anyway so ยฏ\_(ใƒ„)_/ยฏ ).

Patch is designed to show that running $text through Xss::filterAdmin when the $tokens var is empty is what causes the issue. I may be totally mistaken or misunderstanding something, but the $build runs the xss filter anyway at return Xss::filterAdmin($children);, so I'm not entirely clear why showing raw twig to users when results are empty is necessary but again ยฏ\_(ใƒ„)_/ยฏ

And moreover - patch is to allow people who have this issue to update to 10.1.x without showing twig syntax to users (ie what my PM/UX manager referred to as "cool gobbly gook")

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

el1_1el โ†’ created an issue.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

el1_1el โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

el1_1el โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

el1_1el โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Hey there,

If you just patch it using the patch in this issue - https://www.drupal.org/project/entity_reference_views_select/issues/3287300 ๐Ÿ“Œ Automated Drupal 10 compatibility fixes Fixed - and tell me its working I'll do another release.

Let me know. Thanks!

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

a couple of mistakes in hook_update_N from 3 that are fixed here

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

I ended up needing the flag/boolean for each calendar entity to check if datetime should be validated. this patch adds that change

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

added to 2.x branch and released as 2.0.0-alpha1

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

beta3 tagged and released

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Sure, why not. beta3 tagged and released

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Ive been using this patch successfully on a salary field for about three years now. It doesnt look to me like the gif is calculating on the correct field since the version numbers arent integers. Maybe the id?

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Merged and fixed. Thanks everyone!

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

dont do the redirect when saving event entity from form

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

add an event_id to the method so we can check against existing events

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Sorry for the obnoxiousness of this patch but there should be links between calendars and events so adding to the overall routing issue. Note that patch 5a with change to view will not apply due to upload changing characters โ€นโ€น. patch 5 has other changes

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Add edit and delete links to actions on calendar list and link to calendar. add edit and delete links to calendar events also

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Add other routes so we can get consistency

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

el1_1el โ†’ created an issue.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Thanks Kelsey!

I merged the first request after a visual review and its telling me "This merge request contains no changes." for the second. Let me know if thats wrong.

I'll leave in dev for community to review.

e

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

np. im content to just leave it in dev version if people want to try it. ive gotten too far behind on all mod maintenance in general due to other priorities so trying to at least get some dev versions updated for people

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

el1_1el โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

Hopefully... but I likely won't have time to do it anytime soon. Happy to merge patches or issue forks for those who have time :-)

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

el1_1el โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

el1_1el โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

el1_1el โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

el1_1el โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

alter title so it comes up in upgrade status link

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

el1_1el โ†’ created an issue.

๐Ÿ‡บ๐Ÿ‡ธUnited States el1_1el

update for 2.x-dev

Production build 0.69.0 2024