Problem/Motivation

There are couple of schema definitions which contains incompatible types. For ex: Schema definitions should be an object while it contains an array. When running tests on Drupal Core 11, it throws an error and fails the tests. Ex errors:

Drupal\Core\Config\Schema\SchemaIncompleteException: Schema errors for imce.profile.admin with the following errors: imce.profile.admin:conf.usertab missing schema, imce.profile.admin:conf.extensions missing schema, imce.profile.admin:conf.maxsize missing schema, imce.profile.admin:conf.quota missing schema, imce.profile.admin:conf.maxwidth missing schema, imce.profile.admin:conf.maxheight missing schema, imce.profile.admin:conf.replace missing schema, imce.profile.admin:conf.thumbnail_style missing schema, imce.profile.admin:conf.thumbnail_grid_style missing schema, imce.profile.admin:conf.ignore_usage missing schema, imce.profile.admin:conf.url_alter missing schema, imce.profile.admin:conf.folders missing schema

We've to update the `imce.schema.yml` and fix the Schema validation: Incompatible types error because it expects an object, while we provided an array.

Before:

      sequence:
        - type: imce.profile_conf.[%key]

After:

      sequence:
        type: imce.profile_conf.[%key]

Issue fork imce-3458471

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:

Comments

chandu7929 created an issue. See original summary.

ankitv18 made their first commit to this issue’s fork.

ankitv18’s picture

Status: Active » Needs review

Please review the MR!29

chandu7929’s picture

Status: Needs review » Reviewed & tested by the community

Verified changes locally as well by running tests on D11 and all test are passing now. Hence RTBC.

chandu7929’s picture

@ufku Can you please merge and release?

  • ufku committed bd5c248f on 3.x
    Issue #3458471 by chandu7929: Fix schema sequence types
    
ufku’s picture

Version: 3.1.0 » 3.x-dev
Status: Reviewed & tested by the community » Fixed

Fixed. Thanks.

Status: Fixed » Closed (fixed)

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