Problem/Motivation

There appears to be a regression and a bit of a WTF with the theme system currently. The theme system itself allows you to enable themes with a numeric value in the theme name but then the breakpoint system then throws an exception.

Drupal\breakpoint\InvalidBreakpointSourceException: Invalid value 'choice5k' for breakpoint source property. Breakpoint source property can only contain lowercase letters and underscores. in Drupal\breakpoint\Entity\Breakpoint->isValid() (line 170 of /.../core/modules/breakpoint/lib/Drupal/breakpoint/Entity/Breakpoint.php).

Steps to reproduce:
1) Copy stark into new theme with a numeric value in name. Rename info.yml and config files.
2) Enable theme.

Files: 
CommentFileSizeAuthor
#3 2084401-numbers-in-theme-name-3.patch1.15 KBTharna
PASSED: [[SimpleTest]]: [MySQL] 58,924 pass(es).
[ View ]
#1 2084401-numbers-in-theme-name-1.patch1.19 KBTharna
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 2084401-numbers-in-theme-name-1.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new1.19 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 2084401-numbers-in-theme-name-1.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Here is a patch to add numbers as allowed values for the brekpoint source check.

Status:Needs review» Needs work

The last submitted patch, 2084401-numbers-in-theme-name-1.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new1.15 KB
PASSED: [[SimpleTest]]: [MySQL] 58,924 pass(es).
[ View ]

Resubmitting with correct paths this time.

Status:Needs review» Reviewed & tested by the community

I love when an issue I forgot about suddenly have green patches. Seems reasonable enough to just fix the broken module. ;)

Component:theme system» breakpoint.module
Status:Reviewed & tested by the community» Fixed

Committed 63d0273 and pushed to 8.x. Thanks!

Status:Fixed» Closed (fixed)

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