Live feed

Automatically closed - issue fixed for 2 weeks with no activity.

📌 | Drupal CMS | Figure out how to handle the site timezone

Comment 33 minutes ago →

Automatically closed - issue fixed for 2 weeks with no activity.

📌 | Experience Builder | [PP-1] Send page data to Drupal for storage in auto-save store

Comment 41 minutes ago →

🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

larowlan → created an issue.

🐛 | Drush | Theme deletion causes ThemeHandler::addTheme() error

🇺🇸United States othdvlpr

othdvlpr → created an issue.

📌 | Rules | Type hint negate()

🇺🇸United States tr Cascadia

📌 | Rules | Type hint negate()

tr → committed e0116df5 on 4.0.x

Issue #3495750 by tr: Type hint negate()

📌 | Webform | Document webform_bootstrap deprecation

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

I think support for things like Bootstrap should be in separate projects. There is already webform_bootstrap5 → .

🐛 | Drupal core | fatal error on `update` module enable on composer-managed D11.1.0 instance

✨ | Drupal core | Allow attribute-based plugins to discover supplemental attributes from other modules

pgndrupal

as above

drush pm:install update
drush pm:uninstall composer_deploy

*AND*

composer remove composer_deploy
drush cr

composer show | grep -iE "deploy|update"
	(empty)
drush pm:list  | grep -iE "deploy|update"
	Core         Update Manager (update)            Enabled    11.1.0

now, NO WSOD error at /admin/*; poking around the site, can't atm get it to replicate.

if not a fluke, `drush pm:install` does not seem sufficient?

also, now, checking status report @ https://example.com/admin/reports/status#warning ,

1 warning

Drupal core update status
	No update data available
	No update information available. Run cron or check manually.

Cron maintenance tasks
	Last run 3 seconds ago
	(more information)

re-exec'ing cron doesn't cure.

i can't find it in my scribbled notes atm, but iirc, composer_deploy was supposed to solve for the missing update info in a composer-installed Drupal instance

godotislate

Discussed this issue with one of the plugin subsystem maintainers over Slack. He is not completely against having supplemental attributes, but he thinks it might be addressed more easily by allowing arbitrary properties to be set on Plugin attributes: $this->additional[$key] = $value. (Splitting the EntityType attributes into separate attributes is still valid and would not be tied to this.)

I know Drupal core does have a version of this with one plugin type: \Drupal\Core\Layout\Attribute\Layout, which looks like this:

/**
   * Any additional properties and values.
   *
   * @var array
   *
   * @see \Drupal\Core\Layout\LayoutDefinition::$additional
   */
  public readonly array $additional;

  /**
   * Constructs a Layout attribute.
   *
   * @param string $id
   *   The plugin ID.
       <snip ...>
   * @param class-string|null $deriver
   *   (optional) The deriver class.
   * @param mixed $additional
   *   (optional) Additional properties passed in that can be used by a deriver.
   */
  public function __construct(
    public readonly string $id,
    <snip ...>
    public readonly ?string $deriver = NULL,
    ...$additional,
  ) {
    // Layout definitions support arbitrary properties being passed in, which
    // are stored in the 'additional' property in LayoutDefinition. The variadic
    // 'additional' parameter here saves arbitrary parameters passed into the
    // 'additional' property in this attribute class. The 'additional' property
    // gets passed to the LayoutDefinition constructor in ::get().
    // @see \Drupal\Core\Layout\LayoutDefinition::$additional
    // @see \Drupal\Core\Layout\LayoutDefinition::get()
    $this->additional = $additional;
  }

We could possibly add $additional to Drupal\Component\Plugin\Attribute\Plugin, and merge the contents of $this->additional to AttributeBase::get() in Plugin::get(). This would involved updating all the plugin type attribute constructors as well.

📌 | Drupal core | Move jsonapi constants to an enum and deprecate the constants.

🇦🇺Australia mstrelan

I take it back, since these are used as array keys you need to use the scalar value.

The array structure is a bit weird though, maybe it would make sense to have a value object class instead of associative arrays. Maybe enum doesn't make sense after all. Let's see what others think?

📌 | Rules | Type hint negate()

tr → opened merge request !85

📌 | Rules | Type hint negate()

✨ | Acquia DAM | Use ckeditor button to allow the insertion as a anchor tag with link to asset

🇺🇸United States tr Cascadia

tr → created an issue.

🇺🇸United States cpierce42

cpierce42 → created an issue.

🐛 | UI Suite USWDS | Bulk operations don't appear

Automatically closed - issue fixed for 2 weeks with no activity.

🐛 | AI | Headers

🇮🇳India Vivek Panicker Kolkata

I tried the patch provided in #4.
But it doesn't work properly for me.
I get the following response from the chatbot:

I am sorry, but I could not find any relevant information in the archives. Please try to ask the question in a different way or try to ask a different question.

Works fine for me in admin.

📌 | Webform | Document webform_bootstrap deprecation

🇨🇦Canada joseph.olstad

Bootstrap 3.34 is compatible with Drupal 11 and there's no plans to stop at 11.

Drupal 11 is currently planned to be supported until December 2027

📌 | Rules | Parameter xxx has null default value, but is not marked as nullable

🇺🇸United States tr Cascadia

📌 | Rules | Parameter xxx has null default value, but is not marked as nullable

tr → committed 9869e492 on 4.0.x

Issue #3495407 by tr: Parameter xxx has null default value, but is not...

📌 | Experience Builder | Implement auto-save of the page template config entity

🇺🇸United States effulgentsia

I might be wrong, but I think Wim is off work this whole week for the holidays, so I reviewed this in his stead, and it looks great to me, so I merged it. @wim leers: I hope I didn't miss anything important that you would have caught.

📌 | Experience Builder | Implement auto-save of the page template config entity

effulgentsia → committed f166295f on 0.x authored by longwave →

Issue #3494114 by larowlan, longwave, wim leers: Implement auto-save of...

📌 | Webform | Document webform_bootstrap deprecation

🇨🇦Canada joseph.olstad

I'm creating new bootstrap 3 sites including one I just created this week.

🐛 | Drupal core | MediaSourceBase::getSourceFieldName() doesn't check character max

🇮🇳India shalini_jha

✨ | OAuth2 Server | Provide well-known config and fix issuer

Automatically closed - issue fixed for 2 weeks with no activity.

🐛 | Tome | Image with '#' character in name doesn't get exported correctly

🐛 | Drupal core | Cannot access offset of type Drupal\Core\StringTranslation\TranslatableMarkup in isset or empty in Drupal\update\ProjectCoreCompatibility->getPossibleCoreUpdateVersions

🇺🇸United States gene_chi

I am closing this as a duplicate of 🐛 fatal error on `update` module enable on composer-managed D11.1.0 instance Active , which has more and better information.

🐛 | Drupal core | fatal error on `update` module enable on composer-managed D11.1.0 instance

For what it's worth that site doesn't have any particularly rare modules. I am trying to remember whether (and where) update data is cached...

🐛 | Drupal core | It is not possible to react to an entity being duplicated

🇺🇸United States nicxvan

I concur with @berdir.

IS still needs updating.

📌 | Drupal core | Move jsonapi constants to an enum and deprecate the constants.

🇺🇸United States nicxvan

Honestly I'm not super familiar with what you mean, if you can point to an example I'm happy to modify this approach.

🐛 | Drupal core | fatal error on `update` module enable on composer-managed D11.1.0 instance

Is it still the case when the module code is not present? Update manager looks at all present extensions.

📌 | Drupal core | Move jsonapi constants to an enum and deprecate the constants.

🐛 | Drupal core | Fatal error: Uncaught TypeError: Drupal\Core\Extension\ThemeHandler::addTheme()

🇦🇺Australia mstrelan

I didn't look too thoroughly, but ideally we can pass around the enum itself rather than the string backed values. Is that possible or are there too many bits to untangle?

🐛 | Drupal core | Fatal error: Uncaught TypeError: Drupal\Core\Extension\ThemeHandler::addTheme()

🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

@longwave gave a +1 for 10.3/11.0 backport so did that too

🐛 | Drupal core | Fatal error: Uncaught TypeError: Drupal\Core\Extension\ThemeHandler::addTheme()

larowlan → committed 69b56719 on 11.0.x

Issue #3478628 by oily, alexpott, catch, sheshsharma, osopolar, dunx,...

larowlan → committed 4feb744c on 10.3.x

Issue #3478628 by oily, alexpott, catch, sheshsharma, osopolar, dunx,...

✨ | One Time Login Link Admin | Configurable email

🇺🇸United States MegaKeegMan

Hey, thanks for taking a look. The str_replace method you are using looks like it would work, but it is a little strange, since email configuration generally has good support for tokens. Do you think we should not add the token module → as a dependency?

📌 | securitydrupalorg | Create CVEs for SA-CORE-2024-003, 004, 005, 006, 007, 008

Automatically closed - issue fixed for 2 weeks with no activity.

🐛 | Drupal core | fatal error on `update` module enable on composer-managed D11.1.0 instance

pgndrupal

it was already happening prior to `composer_deploy` installation; i'd installed that while attempting to deal with this.

that said, specifically,

drush pm:uninstall composer_deploy
drush pm:install composer_deploy

and, immediately, at nav to `https://dev.pgnetwork.net/admin/config`

The website encountered an unexpected error. Try again later.

TypeError: Cannot access offset of type Drupal\Core\StringTranslation\TranslatableMarkup in isset or empty in Drupal\update\ProjectCoreCompatibility->getPossibleCoreUpdateVersions() (line 83 of core/modules/update/src/ProjectCoreCompatibility.php). 
...

🐛 | Facets | Hierarchy plugin configuration form

🇺🇸United States edwardsay

Added MR that renders Hierarchy plugin config form.

🐛 | Facets | Hierarchy plugin configuration form

edwardsay → opened merge request !267

🐛 | Barcodes | FieldFormatter throwing error with tecnickcom/tc-lib-barcode 2.3

🇺🇸United States tr Cascadia

✨ | Custom Field | Support SHS widget

🇺🇸United States apmsooner

This is first draft if you want to test out the patch. There's a few hardcoded styles just to get something visually appealing that I'll refactor but functionally... I think it's working really well. In the field settings, you'll see the new "Hierarchical select widget" as an option. Just use the default reference method for now as I haven't looked at the views option. There will be a couple extra settings at the bottom of the form:

- Force deepest level
- Show level labels

I think the descriptions are self explanatory. Try it out and let me know how it works for you. It should support multi-lingual and work fine in paragraphs as well.

🐛 | Facets | Hierarchy plugin configuration form

🇺🇸United States edwardsay

edwardsay → created an issue.

🐛 | Drupal core | fatal error on `update` module enable on composer-managed D11.1.0 instance

At a glance it may be composer_deploy, because it modifies update information. I would try uninstalling it first.

✨ | Custom Field | Support SHS widget