Problem/Motivation

Deleting content in bulk is not working in Drupal Core > 8.5.x as #2936964: Change usages of user.private_tempstore to tempstore.private needs to be resolved. But resolving that one in a BC way is impossible. So let's remove that operation so users are not confused.

Proposed resolution

Don't allow to delete content in bulk if Drupal core > 8.5.x

Remaining tasks

Patch

User interface changes

Remove the Delete content operation for nodes so users are not confused.

API changes

None

Data model changes

None

CommentFileSizeAuthor
#3 2936994-not-delete-content-bulk-3.patch1.23 KBpenyaskito
#3 2936994-not-delete-content-bulk.interdiff.2-3.txt657 bytespenyaskito
#2 2936994-not-delete-content-bulk-2.patch600 bytespenyaskito

Comments

penyaskito created an issue. See original summary.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Active » Needs review
StatusFileSize
new 2936994-not-delete-content-bulk-2.patch600 bytes

This will fail as the test is not skipped yet.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
StatusFileSize
new 2936994-not-delete-content-bulk.interdiff.2-3.txt657 bytes
new 2936994-not-delete-content-bulk-3.patch1.23 KB

Skipped the failing test.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

Let's commit this.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Reviewed & tested by the community » Fixed

Committed 8f1e1ed and pushed to 8.x-2.x. Thanks!

  • penyaskito committed 8f1e1ed on 8.x-2.x 
    Issue #2936994 by penyaskito: Don't allow to delete content in bulk if...

Status: Fixed » Closed (fixed)

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