Event Registration URL Slug field doesn't allow uppercase letters

[leahjennings]

Prerequisites

• Put an X between the brackets on this line if you have done all of the following:
  • Did you perform a search at https://github.com/issues?q=is%3Aissue+user%3ASparkDevNetwork+-repo%3ARock to see if your bug or enhancement is already reported?
  • Can you reproduce the problem on a fresh install or the demo site?
  • Did you include your Rock version number and client culture setting?

A Picture Is worth a Thousand Words

Description

When adding a URL Slug to an event registration instance, the validation on the field doesn't allow uppercase letters. I believe all that would be needed to fix this is to adjust the regular expression in the following two files to include uppercase letters. This occurs in both the Event Registration Wizard block and the Registration Instance Linkage Detail block. The validation error message implies that there are special characters present, but it is actually failing on the uppercase letter.

Changing from ValidationExpression="^[a-z0-9]+(?:-[a-z0-9]+)*$" /> to ValidationExpression="^[a-zA-Z0-9]+(?:-[a-zA-Z0-9]+)*$" /> would allow uppercase letters.

Rock/RockWeb/Blocks/Event/RegistrationInstanceLinkageDetail.ascx

Line 54 in 9364f83

ValidationExpression="^[a-z0-9]+(?:-[a-z0-9]+)*$" />

Rock/RockWeb/Blocks/Event/EventRegistrationWizard.ascx

Line 254 in 9364f83

ValidationExpression="^[a-z0-9]+(?:-[a-z0-9]+)*$" />

Steps to Reproduce

1. On the demo site, go to Tools > Event Wizard
2. Select "Baptism" as the Registration Template, then click Next
3. In the Slug field, type a word with an uppercase letter in it
4. Notice how the validation error implies there is a special character present, but validation is actually failing on the uppercase letter
   

Expected behavior:

The Url Slug field would accept the slug with uppercase letters in it.

Actual behavior:

An entered Url Slug with an uppercase letter and no special characters fails validation.

Versions

• Rock Version: [Rock McKinley 13.0 (1.13.0.30)]
• Client Culture Setting: [en-US]

[crayzd92]

Hi @leahjennings, the intent here is that the slugs are only lowercase. We are going to work on changing that validation message to more clearly indicate our intent.

[crayzd92]

Fixed by updating the validation message to indicate intent