Glasgow
Account created on 1 December 2015, over 9 years ago
#

Merge Requests

Recent comments

🇬🇧United Kingdom Ok4p1 Glasgow

Hello, you can disregard my previous message. I realized that I wasn’t using the latest D11 patch from the embed module. Once I updated it, the error disappeared, and I have been using the patch from this ticket in production since then, with no issues.

🇬🇧United Kingdom Ok4p1 Glasgow

ok4p1 created an issue.

🇬🇧United Kingdom Ok4p1 Glasgow

This patch doesn't work with 10.3.0

🇬🇧United Kingdom Ok4p1 Glasgow

I get the following error after trying to use the latest MR. Drupal 10.3.8.

Error: Call to undefined method Drupal\embed\EmbedType\EmbedTypeBase::create() in Drupal\entity_embed\Plugin\EmbedType\Entity::create() (line 56 of modules/contrib/entity_embed/src/Plugin/EmbedType/Entity.php).

🇬🇧United Kingdom Ok4p1 Glasgow

@grevil any luck contacting @dydave?

🇬🇧United Kingdom Ok4p1 Glasgow

Hello jaswinsingh, I will take a look, thank you!

🇬🇧United Kingdom Ok4p1 Glasgow

My D11 upgrade is blocked because of this module, can the minor version be created with the updated version? Thanks.

🇬🇧United Kingdom Ok4p1 Glasgow

I can't install the module either

🇬🇧United Kingdom Ok4p1 Glasgow

Not sure what I did wrong the first time I was testing this, but this is indeed working with drush deploy, this ticket can be closed, sorry for the confusion.

🇬🇧United Kingdom Ok4p1 Glasgow

ok4p1 created an issue.

🇬🇧United Kingdom Ok4p1 Glasgow

Hello, why was this ticket closed? The module is not compatible with Drupal 11 yet

🇬🇧United Kingdom Ok4p1 Glasgow

I submitted a merge request to update Composer, I am not sure if there's anything else needed to make this work.

This module is the only dependency preventing me from updating to Site Studio 8.0.

🇬🇧United Kingdom Ok4p1 Glasgow

ok4p1 created an issue.

🇬🇧United Kingdom Ok4p1 Glasgow

Hello, the patch mitigates the issue but I am still experiencing it

🇬🇧United Kingdom Ok4p1 Glasgow

If anyone else faces this issue, my problem was the syntax used to disable the field programmatically so users could not change the default value (something that worked with 1.2.11).

Breaks: $form['field_page_type']['widget']['#attributes'] = ['disabled' => 'disabled'];
Works: $form['field_page_type']['widget']['#attributes']['disabled'] = 'disabled';

🇬🇧United Kingdom Ok4p1 Glasgow

Apologies, I wasn't able to replicate it either on a clean install, so it must be something about my codebase :(

🇬🇧United Kingdom Ok4p1 Glasgow

Hello, thank you for getting back to me. The part I don't get is that this is not a custom element, it is just an entity reference referencing a taxonomy term. Shouldn't that work out-of-the-box without having to customize it, like it did on 1.2.11?

I tried with 1.2.18 and the issue is still there.

🇬🇧United Kingdom Ok4p1 Glasgow

Ok4p1 created an issue.

🇬🇧United Kingdom Ok4p1 Glasgow
🇬🇧United Kingdom Ok4p1 Glasgow
🇬🇧United Kingdom Ok4p1 Glasgow

Ok4p1 created an issue.

🇬🇧United Kingdom Ok4p1 Glasgow

Hello, are you planning to make this work with groups 3.x? I tried the patch but got the following error, thanks.

Drupal\Component\Plugin\Exception\PluginNotFoundException: The "group_content" entity type does not exist. in Drupal\Core\Entity\EntityTypeManager->getDefinition() (line 139 of core/lib/Drupal/Core/Entity/EntityTypeManager.php).

🇬🇧United Kingdom Ok4p1 Glasgow

Same thing, #175 works for me but #176 is still showing up untranslated menu items.

🇬🇧United Kingdom Ok4p1 Glasgow

Ok, but at least "Embed Code" should not be displayed for the core media images, since that was introduced by acquia dam, right now it is visible in both.

🇬🇧United Kingdom Ok4p1 Glasgow

Thank you for the update Vadim Hirbu, do you think we should keep the image style dropdown? It is very confusing having both presented to the user, when media types only display the view modes.

Also now all the images from drupal have the "Embed Code" text displaying in ckeditor which shouldn't be the case since it has no options, I getting rid of it will be better so there is alignment with core media.

🇬🇧United Kingdom Ok4p1 Glasgow

I agree with Jura, it is confusing that for media types the user can select the view mode from the CKEditor, but for DAM images they need to select the image style.

I have added a patch that comments out some of the Acquia DAM code to:
- Add the view modes back
- Remove the CKEditor plugin
- Remove the step to select format when adding DAM images

That way media types images and dam images behave the same way. I don't fully understand all the code I have commented out, but it looks like it is working just fine without it.

🇬🇧United Kingdom Ok4p1 Glasgow

Ignore my last message, I was on the mobile phone and missed the link from the description :)

🇬🇧United Kingdom Ok4p1 Glasgow

@justmeplease would you mind sharing your fork? Thank you.

🇬🇧United Kingdom Ok4p1 Glasgow

I am also experiencing this issue

🇬🇧United Kingdom Ok4p1 Glasgow

Never mind, the modal closes now fine with the latest update of the acquia cms modules.

🇬🇧United Kingdom Ok4p1 Glasgow

Using the patch I no longer get the error but the modal window does not close when I select "Insert selected" (even though the image is selected correctly).

Production build 0.71.5 2024