Follow-up to #2302331: Move drupal_valid_path to PathValidator service

Problem/Motivation

We want to remove include files (including path.inc) and move from procedural to OO code.

Proposed resolution

Replace instances of deprecated drupal_valid_path with PathValidator

Remaining tasks

RTBC

User interface changes

None

API changes

drupal_valid_path() becomes deprecated

CommentFileSizeAuthor
#3 2303361-path_validator-3.patch6.68 KBtim.plunkett
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 76,326 pass(es). View
#1 2303361-remove-valid-path-1.patch8.58 KBkim.pepper
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 74,095 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more

Comments

kim.pepper’s picture

Status: Active » Needs review
FileSize
8.58 KB
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 74,095 pass(es). View

This replaces all found usages of drupal_valid_path().

dawehner’s picture

It would be great to skip the changes in menu for a while, given that they will otherwise conflict with the MENU beta deadline issue.

tim.plunkett’s picture

FileSize
6.68 KB
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 76,326 pass(es). View

Rerolled. Much less to do now.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Seems legit.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 332fad7 and pushed to 8.0.x. Thanks!

  • alexpott committed 332fad7 on 8.0.x
    Issue #2303361 by tim.plunkett, kim.pepper: Replace instances of...

Status: Fixed » Closed (fixed)

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