Problem/Motivation

In #3400368: Deprecate path.temporary in system.file configuration schema (Drupal 8.8) system.file.path.temporary was deprecated. This was the only key used in system.file.path, but that key in the schema was not removed.

Lets use this issue to deprecate the config key and add an upgrade path.

Steps to reproduce

Proposed resolution

  1. Remove system.file.path from system.schema.yml.
  2. Create an upgrade path to remove the key.
  3. Create an upgrade path test.

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3517070

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

Comments

bbrala created an issue. See original summary.

bbrala opened merge request !11721

bbrala’s picture

bbrala
Location Netherlands
commented
Issue summary: View changes
bbrala’s picture

bbrala
Location Netherlands
commented

Added the changes from the parent. Added a change record. Added a test for the upgrade path.

bbrala’s picture

bbrala
Location Netherlands
commented
Status: Active » Needs review
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented
Status: Needs review » Reviewed & tested by the community

This looks great, thanks for cleaning this up.

alexpott’s picture

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

Committed 849a687 and pushed to 11.x. Thanks!

  • alexpott committed 849a6872 on 11.x 
    Issue #3517070 by bbrala: Remove system.file.path config from system....
quietone’s picture

quietone commented
Assigned: bbrala » Unassigned

bbrala closed merge request !11721

Status: Fixed » Closed (fixed)

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