Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 295840

Block page visibility paths are not validated, using path a without leading forward slash does not work

$
0
0

Problem/Motivation

Block page visibility paths are not validated, using path a without leading forward slash does not work.

Steps to Reproduce

  • Install using standard install profile
  • Navigate to /admin/structure/block/manage/bartik_branding
  • Configure pages using 'user/login' path without leading forward slash and 'Show for the listed pages' option selected
  • Save form
  • Observer there is no validation error message.
  • Navigate to home page and observe no branding block rendered as expected
  • Navigate to /user/login path, observe no branding block rendered, unexpected (or expected?)

Proposed resolution

Use RequestPath::validateConfigurationForm to validate for required leading forward slashes.

Remaining tasks

Get consensus, patch and review. Any tests needed?

User interface changes

Provides error message on block config form.

API changes

None

Data model changes

None


Viewing all articles
Browse latest Browse all 295840

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>