Problem/Motivation

When placing block on a page, you can set the block machine name. But the configuration of this block depends on the theme you place your block in.

On the same websites, if there are multiple themes, you can't place the same block on another theme with the same machine name because it is already used.

Proposed resolution

Replace blocks machine name, for example:

  • block.block.book_navigation.yml => block.block.olivero_book_navigation.yml
  • block.block.primary_admin_actions.yml => block.block.olivero_primary_admin_actions.yml

Remaining tasks

  • Make a patch
  • Review

CommentFileSizeAuthor
#11 3187899-11.patch2.17 KBameymudras
#3 Capture d’écran de 2020-12-11 15-57-27.png125.93 KBGrimreaper
#3 Capture d’écran de 2020-12-11 15-57-16.png194.6 KBGrimreaper
#3 Capture d’écran de 2020-12-11 15-57-11.png182.16 KBGrimreaper

Issue fork drupal-3187899

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Grimreaper created an issue. See original summary.

Grimreaper opened merge request !134

Grimreaper’s picture

Grimreaper
Primary language French
Location France 🇫🇷
CreditAttribution: Grimreaper at Smile commented
Status: Active » Needs review
FileSize
Capture d’écran de 2020-12-11 15-57-11.png182.16 KB
Capture d’écran de 2020-12-11 15-57-16.png194.6 KB
Capture d’écran de 2020-12-11 15-57-27.png125.93 KB

I have tested and the two concerned blocks are not affected by the machine name renaming.

See attached screenshots.

Version: 9.2.x-dev » 9.3.x-dev

Drupal 9.2.0-alpha1 will be released the week of May 3, 2021, which means new developments and disruptive changes should now be targeted for the 9.3.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Issue summary: View changes
Issue tags: +Needs upgrade path, +Needs upgrade path tests

Edit: Disregard earlier comment; this is default config.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Issue tags: -Needs upgrade path, -Needs upgrade path tests

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.0-rc1 was released on November 26, 2021, which means new developments and disruptive changes should now be targeted for the 9.4.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.4.x-dev » 9.5.x-dev

Drupal 9.4.0-alpha1 was released on May 6, 2022, which means new developments and disruptive changes should now be targeted for the 9.5.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.5.x-dev » 10.1.x-dev

Drupal 9.5.0-beta2 and Drupal 10.0.0-beta2 were released on September 29, 2022, which means new developments and disruptive changes should now be targeted for the 10.1.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

andy-blum’s picture

andy-blum
he/him/his
Primary language English
Location Ohio, USA
CreditAttribution: andy-blum at Lullabot commented
Status: Needs review » Needs work

Needs reroll/rebase.

ameymudras’s picture

ameymudras CreditAttribution: ameymudras at Salsa Digital commented
Status: Needs work » Needs review
FileSize
3187899-11.patch2.17 KB
Gauravvvv’s picture

Gauravvvv CreditAttribution: Gauravvvv at Axelerant for Drupal India Association commented

This has been fixed in #3278696: Rename Oliveros block.block.book_navigation.yml and block.block.primary_admin_actions.yml confirm standard

We can close this issue.

Gauravvvv’s picture

Gauravvvv CreditAttribution: Gauravvvv at Axelerant for Drupal India Association commented
Status: Needs review » Closed (duplicate)