Problem/Motivation

When a custom block has been created, if the user then deletes the custom block using the “delete” contextual link, and then clicks place block again, the block still appears in the list. If the user then tries to add the block the action fails without error message.

Proposed resolution

Remove the block from cache

Remaining tasks

Remove block from cache

User interface changes

None

API changes

None

Data model changes

None

CommentFileSizeAuthor
#20 2786195-delete-block-8.2.20.patch4.33 KBlarowlan
#20 interdiff.txt825 byteslarowlan
#5 2786195-delete-block.pass_.patch4.21 KBlarowlan
#5 2786195-delete-block.fail_.patch2.13 KBlarowlan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tkoleary created an issue. See original summary.

tkoleary’s picture

tkoleary CreditAttribution: tkoleary at Acquia commented
Parent issue: #2762505: Introduce "outside in" quick edit pattern to core »
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext commented
Component: block.module » block_content.module
Issue summary: View changes
Issue tags: +Needs tests
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext commented
Version: 8.2.x-dev » 8.1.x-dev
Assigned: Unassigned » larowlan

Bug fixes go into 8.1.x first - giddyup

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext commented
Assigned: larowlan » Unassigned
Status: Active » Needs review
Issue tags: -Needs tests
FileSize
2786195-delete-block.fail_.patch2.13 KB
2786195-delete-block.pass_.patch4.21 KB

Something like so

The last submitted patch, 5: 2786195-delete-block.fail_.patch, failed testing.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Status: Needs review » Reviewed & tested by the community

Looks great and have tests!

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett at Acquia commented 
+++ b/core/modules/block_content/src/Entity/BlockContent.php
@@ -104,9 +104,15 @@ public function getTheme() {
   public function postSave(EntityStorageInterface $storage, $update = TRUE) {
...
+  public static function postDelete(EntityStorageInterface $storage, array $entities) {

We should probably audit all of the entities that have just a postSave, I bet there are other entity classes with mismatches like this (SearchPage comes to mind).

+1 for RTBC

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed 6bdf44e to 8.3.x and 66fa7be to 8.2.x and 82bece9 to 8.1.x. Thanks!

  • alexpott committed 6bdf44e on 8.3.x 
    Issue #2786195 by larowlan: Deleted block is not removed from the block...

  • alexpott committed 66fa7be on 8.2.x 
    Issue #2786195 by larowlan: Deleted block is not removed from the block...

  • alexpott committed 82bece9 on 8.1.x 
    Issue #2786195 by larowlan: Deleted block is not removed from the block...
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented

other entity classes with mismatches like this (SearchPage comes to mind).

What did you mean? please elaborate

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented

Also looks a loot patches has failured after this with similar error
For example https://www.drupal.org/pift-ci-job/436365

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Fixed » Needs work

Yes I'm seeing the same. Reverted from all three branches for now.

  • catch committed c90b15c on 8.3.x 
    Revert "Issue #2786195 by larowlan: Deleted block is not removed from...

  • catch committed 27ca9e4 on 8.2.x 
    Revert "Issue #2786195 by larowlan: Deleted block is not removed from...

  • catch committed b5de575 on 8.1.x 
    Revert "Issue #2786195 by larowlan: Deleted block is not removed from...
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext commented

Yeah, 8.2.x adds revision user to the entity.

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext commented
Status: Needs work » Needs review
FileSize
interdiff.txt825 bytes
2786195-delete-block-8.2.20.patch4.33 KB

Added user support to test, should pass on all three branches now - will queue test runs for each.

The last submitted patch, 5: 2786195-delete-block.pass_.patch, failed testing.

  • catch committed 4221911 on 8.3.x 
    Issue #2786195 by larowlan: Deleted block is not removed from the block...

  • catch committed 65598a5 on 8.2.x 
    Issue #2786195 by larowlan: Deleted block is not removed from the block...

  • catch committed a46d123 on 8.1.x 
    Issue #2786195 by larowlan: Deleted block is not removed from the block...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Needs review » Fixed

Thanks! Committed to all three branches.

Status: Fixed » Closed (fixed)

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