Account created on 17 July 2007, over 17 years ago
#

Merge Requests

More

Recent comments

🇨🇿Czech Republic parisek

I have same issue within Media upload (using also entity_usage) and this patch helped, thank you, making RTBC

Drupal\Core\Entity\EntityStorageException: SQLSTATE[HY000]: General error: 1267 Illegal mix of collations (ascii_general_ci,IMPLICIT) and (utf8mb4_general_ci,COERCIBLE) for operation '=': DELETE FROM "entity_usage" WHERE ("source_id_string" = :db_condition_placeholder_0) AND ("source_type" = :db_condition_placeholder_1); Array ( [:db_condition_placeholder_0] => 1-1154103-1742542846810-Obrázek_test.png [:db_condition_placeholder_1] => file_resup )

🇨🇿Czech Republic parisek

Just moving to current branch and status to Postponed as it's upstream bug

🇨🇿Czech Republic parisek

Created MR and added check

🇨🇿Czech Republic parisek

I created MR from the patch, tests are failing but I think nothing directly connected with our MR

🇨🇿Czech Republic parisek

parisek changed the visibility of the branch 3126310-implementation-for-link to active.

🇨🇿Czech Republic parisek

parisek changed the visibility of the branch 3126310-implementation-for-link to hidden.

🇨🇿Czech Republic parisek

Thank you, but I still have a misplaced button under the body field.

🇨🇿Czech Republic parisek

Thank you I have same issue and it resolved it, I think we can move on to RTBC

🇨🇿Czech Republic parisek

I have similar issue and this MR fixes it for me (I'm using it exclusively for webform with AJAX enabled), thank you

greatmatter: Could you provide more information where it's not working (exact example) or steps to reproduce?

🇨🇿Czech Republic parisek

I can confirm that it fixes my issue, so marking as duplicate, thank you

🇨🇿Czech Republic parisek

Updated #168 🐛 Untranslated menu items are displayed in menus Needs work patch to latest 10.2 - using "menu.language_tree_manipulator:filterLanguage" as I don't know how to migrate to current MR via $context which is locked to only allow SystemMenuBlock use.

🇨🇿Czech Republic parisek

There is module https://www.drupal.org/project/jquery_once  to fix this for legacy stuff

🇨🇿Czech Republic parisek

I have this problem with older websites (missing views.view.media.yml), new installs are fine. Updated patch to latest Drupal.

🇨🇿Czech Republic parisek

Updated the default installation variable. However, we might need to create an update hook for existing module users to streamline update to Drupal 10.1+

🇨🇿Czech Republic parisek

Working for me, thank you

🇨🇿Czech Republic parisek

We have same issue and #3 fixed it for us, thank you

🇨🇿Czech Republic parisek

I found pretty good example how it could be implemented in this module (use EmailAdjuster) https://www.drupal.org/project/maillog/issues/3259616 Support Symfony Mailer RTBC

I think it's unnecessary to create separate module for this feature.

🇨🇿Czech Republic parisek

xeniksp: Thank you for your patch, it fixed this issue, I created MR and hope that maintainer merge it soon.

🇨🇿Czech Republic parisek

I get same error with current Drupal 10.1.5. Adjusted priority to critical because module is unusable.

🇨🇿Czech Republic parisek

Seems really hard to rebase MR for current branch so closed it.

Found one issue - enclosure field is not translated we just get default language, probably we should implement EntityTranslationRenderTrait (like web/core/modules/views/src/Plugin/views/row/EntityRow.php)

🇨🇿Czech Republic parisek

That sounds great if this module support TMGMT translations together. I changed Title and hope to more people will need this.

Thinking about
- checkbox on node edit form to choose if I need asymmetric translations or not
- use separate tab for asymmetric translations and on regular translation page show only notification

🇨🇿Czech Republic parisek

rafmagsou, thank you for patch, I opened MR should be easier for mainainer to fix it.

🇨🇿Czech Republic parisek

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

🇨🇿Czech Republic parisek

wangshy thank you, I got same error, created MR with fix inspired by your patch

ypeError: Drupal\Core\Entity\EntityDisplayBase::setComponent(): Argument #2 ($options) must be of type array, null given, called in /var/www/html/web/core/lib/Drupal/Core/Entity/EntityViewBuilder.php on line 530 in function Drupal\Core\Entity\EntityDisplayBase->setComponent() (line 340 file /var/www/html/web/core/lib/Drupal/Core/Entity/EntityDisplayBase.php)

🇨🇿Czech Republic parisek

Added MR and disabled logging warning "Found multiple files for the", seems it's not wide issue, patch is enough :)

🇨🇿Czech Republic parisek

Updated MR to support Drupal 10, unfortunately couldn't rebase

🇨🇿Czech Republic parisek

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

🇨🇿Czech Republic parisek

Thank you for promoting this to a general solution. It's working great for me. RTBC.

🇨🇿Czech Republic parisek

Thank you for the patch. It's working great for external URLs. I triggered new tests and everything is green.

Ready to be committed (RTBC).

🇨🇿Czech Republic parisek

Thank you for your message. I'm just curious, why not switch all or most of the meta tag fields to text? I don't think it has any drawbacks.

🇨🇿Czech Republic parisek

Created a simple patch to address this issue just for open graph images.

🇨🇿Czech Republic parisek

Working great for me too. Tests are green. RTBC

🇨🇿Czech Republic parisek

For me quickfix was add new permission for route entity.webform_submission.user via Custom Permissions module

🇨🇿Czech Republic parisek

Added back to NR as I just found fatal error when some node translation is not published yet. It all goes down to this line

$node = $entity_repository->getTranslationFromContext($node);

We cannot pass upublished translation so I added check $node->isPublished() before

🇨🇿Czech Republic parisek

For me this fixes disappeared language switcher links which introduced security fix for Drupal 9.5.5
https://www.drupal.org/project/drupal/issues/3348686 🐛 [regression] Inaccessible language switcher links are removed before alternatives can be provided Fixed

Thank you

🇨🇿Czech Republic parisek

Thank you, working for me, but CI test still not happy. Maybe best we can is separate problematic token to another issue?

🇨🇿Czech Republic parisek

Created MR from attached patch and fixed #7

Changed to major as if both CC and BCC recepients are set, only later one is sent.

Production build 0.71.5 2024