Problem/Motivation

We don't have a FakeDelete class.

Proposed resolution

Copy the FakeUpdate class to FakeDelete, change the array_walk to array_filter and that's approximately it. These fake things don't really have a test so I have attached a meld screenshot to show how little the difference between the two is. I also tested this manually.

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
Selection_151.png79.59 KBchx
fakedelete.patch2.94 KBchx
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

It will be used by migrate at some point I assume.

alexpott’s picture

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

This issue is a normal bug fix, and doesn't include any disruptive changes, so it is allowed per https://www.drupal.org/core/beta-changes. Committed ae6e7af and pushed to 8.0.x. Thanks!

  • alexpott committed ae6e7af on 8.0.x 
    Issue #2372201 by chx: Fixed FakeDelete is missing.

Status: Fixed » Closed (fixed)

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