Problem/Motivation

\Drupal\build_hooks\Controller\FrontendEnvironmentTitleController::frontendDeploymentTitle interacts with the current request and uses explode to get the entity ID.
It should just use the routing system.

Steps to reproduce

Proposed resolution

Add a type-hinted argument for the $frontend_environment to the method and use that instead

Remaining tasks

User interface changes

API changes

Data model changes

Comments

larowlan created an issue. See original summary.

larowlan’s picture

Issue tags: +Novice
larowlan’s picture

Issue tags: +DrupalGov 2020
Pooja Ganjage’s picture

StatusFileSize
new1.21 KB

Hi,

Creating a patch for this issue.

Please review the patch.

Let me know if any suggestions.

Thanks.

Pooja Ganjage’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 4: 3172096-4.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

sarvjeetsingh’s picture

Assigned: Unassigned » sarvjeetsingh
sarvjeetsingh’s picture

Assigned: sarvjeetsingh » Unassigned
Status: Needs work » Needs review
StatusFileSize
new1.21 KB
new700 bytes

Status: Needs review » Needs work

The last submitted patch, 8: 3172096-5.patch, failed testing. View results

Pooja Ganjage’s picture

StatusFileSize
new1.96 KB
Pooja Ganjage’s picture

StatusFileSize
new3.27 KB