Problem/Motivation

  • Schema configuration has number of messing schema fields
  • Schema configuration type has a wrong type for number of theme settings

Steps to reproduce

Given that the Bootstrap Barrio 5.5.x or 5.x was installed and sat as the default theme for the site
When changing the "Theme settings" and configs for the Bootstrap Barrio theme
And export configs of the site
Then extra region config items will be exported
When a check with the configuration inspector or with the automated testing tools
Then the error message will be "missing schema" for number of items.
And "variable type is integer but applied schema class is Drupal\Core\TypedData\Plugin\DataType\StringData"

Proposed resolution

  • Have the right list of regions with "clean" and "class".
  • Change clean elements to in "type: integer"

Remaining tasks

  • Fix
  • Test
  • Automated testing
  • Review
  • Release

User interface changes

  • None

API changes

  • None

Data model changes

  • None
CommentFileSizeAuthor
#4 3230626-4--5-5-x-branch.patch5.89 KBrajab natshah
#3 3230626-3.patch5.78 KBrajab natshah

Comments

RajabNatshah created an issue. See original summary.

rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes
rajab natshah’s picture

rajab natshah
he/him
commented
StatusFileSize
new 3230626-3.patch5.78 KB

This is for the 5.x branch to work with ~5.1.0 Bootstrap 4

rajab natshah’s picture

rajab natshah
he/him
commented
StatusFileSize
new 3230626-4--5-5-x-branch.patch5.89 KB

This is for the 5.5.x branch to work with ~5.5.0 Bootstrap 5

rajab natshah’s picture

rajab natshah
he/him
commented
Assigned: rajab natshah » Unassigned
Status: Active » Needs review

Please activate the automated testing for the ~5.5.0 branch for Bootstrap 5.
After committing this fix. the automated test will pass in both 5.x and 5.5.x branches

rajab natshah’s picture

rajab natshah
he/him
commented
Issue summary: View changes

  • RajabNatshah authored 690662f on 5.5.x 
    Issue #3230626 by RajabNatshah: Fix Schema for the configuration files...
hatuhay’s picture

hatuhay commented
Status: Needs review » Fixed
rajab natshah’s picture

rajab natshah
he/him
commented

Thank you, Alberto for having time to follow up on this issue.

This issue was only committed to the 5.5.x branch ( Bootstrap 5 )
Same issue but for the 5.x branch ( Bootstrap 4 )
I attached 2 patches.

After this fix, I managed to get the Automated testing working well for

Vartheme BS4 using Barrio ~5.1.0

14 Sep 2021 at 00:50 EEST

using the following drupalci.yml file

Vartheme BS5 using Barrio ~5.5.0

14 Sep 2021 at 00:52 EEST

using the following drupalci.yml file

Status: Fixed » Closed (fixed)

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