Problem/Motivation

Drupal 8 provides a config schema checker during the tests, listens to the config save event and validates schema.
The config schema checker for this module is failing:

Status    Group      Filename          Line Function                            
--------------------------------------------------------------------------------
Fail      Completion PfizerTiTestTest.   25 Drupal\Tests\pfizer_ti_test\Functio
    The test did not complete due to a fatal error.
Exception Uncaught e ConfigSchemaCheck   93 Drupal\Core\Config\Testing\ConfigSc
    Drupal\Core\Config\Schema\SchemaIncompleteException: Schema errors for
    session_limit.settings with the following errors:
    session_limit.settings:session_limit_roles missing schema,
    session_limit.settings:_core missing schema,
    session_limit.settings:langcode missing schema in
    Drupal\Core\Config\Testing\ConfigSchemaChecker->onConfigSave()

Proposed resolution

Change the schema.yml of the module to follow the official specification https://www.drupal.org/node/1905070

CommentFileSizeAuthor
#2 session_limit-fixing-schema-2762279-d8.patch593 bytesjribeiro
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jribeiro created an issue. See original summary.

jribeiro’s picture

jribeiro CreditAttribution: jribeiro at CI&T for Pfizer, Inc. commented
FileSize
session_limit-fixing-schema-2762279-d8.patch593 bytes
badjava’s picture

badjava CreditAttribution: badjava at Metasun for Pfizer, Inc. commented
Status: Needs review » Reviewed & tested by the community

We've been running this patch for quite a while and it fixes the schema issue. Please commit this patch.

johnennew’s picture

johnennew CreditAttribution: johnennew at Deeson commented

Committed to dev branch, will form part of next release.

johnennew’s picture

johnennew CreditAttribution: johnennew at Deeson commented
Status: Reviewed & tested by the community » Fixed

This is fixed

Status: Fixed » Closed (fixed)

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