Account created on 5 September 2013, almost 11 years ago
#

Merge Requests

Recent comments

πŸ‡©πŸ‡ͺGermany Duwid

These two tickets are related but not duplicates. In #3271758, the file is validated, and a call to social_profile_get_default_image() is prevented. However, social_profile_get_default_image() is called in multiple places, e.g., by EmailTokenServices::getUserPreview(). As wassper correctly assumed, a check is needed to see if the file exists in social_profile_get_default_image().

Here is a new approach for open_social 12.

πŸ‡©πŸ‡ͺGermany Duwid

Notice: Patch is fixing problem in https://www.drupal.org/project/layout_builder_st/issues/3391512 πŸ› Adjustments to the source language are not included in the translation Closed: duplicate

πŸ‡©πŸ‡ͺGermany Duwid

Thank you for pointing this out! I can confirm that the patch from https://www.drupal.org/project/layout_builder_st/issues/3222423 πŸ› Layout translation form reports unsaved changes and becomes stale Needs review is fixing this issue. I guess because 3222423 is created three years ago - this issue is a duplicate.

@LRoels thank for your time and effort, too.

πŸ‡©πŸ‡ͺGermany Duwid

I also have spam submissions again.
@gaurav.kapoor, @mstef, @danrod can you reopen this issue and set status to "Needs work", please?

πŸ‡©πŸ‡ͺGermany Duwid

This patch robustifys implementation if htmlAsText is not available because select2 widget is programmatically implemented via form api. For example this is the case social group invite module.

πŸ‡©πŸ‡ͺGermany Duwid

You can use this module to get seperated asc and desc labels in a dropdown select widget: https://www.drupal.org/project/search_api_sorts_widget β†’

πŸ‡©πŸ‡ͺGermany Duwid

I updated ticket priority because this breaks the symmetrical behavior.

πŸ‡©πŸ‡ͺGermany Duwid

@handkerchief I can confirm, that this is an issue with core ^10.1.

As a temporary fix:
e.g. after deleting the block in source language (DE), switch to Layout Builder in the translated language (EN) and save the Layout Builder before you start editing the blocks. After saving the EN Layout Builder you should see your adjustments.

πŸ‡©πŸ‡ͺGermany Duwid

I'm working on a case where I need to keep and render the html from select2 results, instead of removing it as described in @GiorgosK workaround. Therefore I will open this issue again, because I think the implemented solution from @Tuuuukka is a great improvement. I just found a little issue in select2.js:
if (drupalSettings.select2.htmlAsText === false) {
htmlAsText is int 0, so it should be:
if (drupalSettings.select2.htmlAsText === 0) {

πŸ‡©πŸ‡ͺGermany Duwid

Module alters theme registry to override field-template, instead of implementing custom theme via theme hook. Module should implement its own theme hook.

Temporary workaround: Override theme registry alter hook with custom path:

/**
 * Implements hook_theme_registry_alter().
 */
function my_module_theme_registry_alter(&$theme_registry) {
  $theme_registry['field__flipbook__flipbook__flipbook']['path'] = \Drupal::theme()->getActiveTheme()->getPath() . '/templates';
}
πŸ‡©πŸ‡ͺGermany Duwid

I came across this question, too. As I read in a support issue for the 3D Flipbook, they recommand just downloading the commercial code from there website (after purchasing) and use this version instead of the free version.

After I inspeced this module, I noticed that this module does not have integration for the purchased library. Simplest solution would be, to copy this module to custom directory and replace the files in /js with the files from the 3D library and pray that it works.

My proposed soltution is to have an backend form where the user can store a custom path of the libraries location. Or at least settings variables to override the path in settings.php.

What do you think?

πŸ‡©πŸ‡ͺGermany Duwid

https://www.drupal.org/project/search_api_location/issues/3351900 πŸ› Trying to access array offset on value of type bool in Drupal\views\Plugin\views\PluginBase->setOptionDefaults() Closed: duplicate

Sorry - already fixed in dev

πŸ‡©πŸ‡ͺGermany Duwid

@mikemooney can you update the issue description and provide more information, please?

After upgrading to Drupal 10 I got the following error: Unsupported operand types: array + null in Drupal\ui_patterns_settings\Plugin\EnumerationSettingTypeBase.php on line 74.

I applied your merge request but it leads into other issues. I created a new merge request with a change that fixed it in my case.

πŸ‡©πŸ‡ͺGermany Duwid

Duwid β†’ changed the visibility of the branch 8.x-2.x to hidden.

πŸ‡©πŸ‡ͺGermany Duwid

Duwid β†’ made their first commit to this issue’s fork.

πŸ‡©πŸ‡ͺGermany Duwid

This issue is a duplicate of https://www.drupal.org/project/business_rules/issues/3286389 πŸ“Œ Automated Drupal 10 compatibility fixes Needs review

πŸ‡©πŸ‡ͺGermany Duwid

Duwid β†’ changed the visibility of the branch 3254223-drimage-not-refreshing-images-inside-layout-builder to hidden.

πŸ‡©πŸ‡ͺGermany Duwid

Duwid β†’ made their first commit to this issue’s fork.

πŸ‡©πŸ‡ͺGermany Duwid

Duwid β†’ changed the visibility of the branch 8.x-3.x to hidden.

πŸ‡©πŸ‡ͺGermany Duwid

Duwid β†’ made their first commit to this issue’s fork.

πŸ‡©πŸ‡ͺGermany Duwid

Duwid β†’ made their first commit to this issue’s fork.

πŸ‡©πŸ‡ͺGermany Duwid

I created patches for 3.0-beta and 3.0-dev to support list text fields. If the plan is to support additional field types in the future, then more effort should definitely be invested here.

πŸ‡©πŸ‡ͺGermany Duwid

Reroll for #2457999 - 309 and #3007424 - 146
Super specific, but hey...
If you have applied #2457999 - 309 and you want to use it in combination with open social 11.9.3 which applies #3007424 - 146.

Add this to your composer.json:

        "patches-ignore": {
            "goalgorilla/open_social": {
                "drupal/core": {
                    "Multiple usages of FieldPluginBase::getEntity do not check for NULL, leading to WSOD": "https://www.drupal.org/files/issues/2023-01-05/3007424-146-9.5.x.patch"
                }
            }
        },
πŸ‡©πŸ‡ͺGermany Duwid

Added the tmgmt job as context to the alter hook.

πŸ‡©πŸ‡ͺGermany Duwid

Sorry, messed up dependency injection in the first patch. I reuploaded fixed patch.

πŸ‡©πŸ‡ͺGermany Duwid

Thank you for the patch. I rerolled it to work with version 1.3.1

Production build 0.69.0 2024