Add proper typehints to Html::escape and Html::decodeEntities + trigger_error

Created on 26 April 2024, 7 months ago

Updated 27 April 2024, 7 months ago

Problem/Motivation

In core/lib/Drupal/Component/Utility/Html.php we should add typehint of string to escape() and decodeEntities() and throw a trigger_error for error in 12

Steps to reproduce

N/A

Proposed resolution

Same as problem.

Remaining tasks

Do it
Review

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

📌 Task

Status

Closed: outdated

Version

11.0 🔥

Component

Base →

Last updated about 3 hours ago

Maintained by
🇺🇸United States @effulgentsia
🇬🇧United Kingdom @catch
🇬🇧United Kingdom @alexpott
🇦🇺Australia @larowlan
🇺🇸United States @bnjmnm
🇫🇷France @nod_
🇪🇸Spain @ckrina
🇬🇧United Kingdom @justafish

Created by

🇺🇸United States smustgrave

Live updates comments and jobs are added and updated live.

Novice
It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Merge Requests

!7774Add proper typehints to Html::escape and Html::decodeEntities + trigger_error
Closed
Unnamed author
updated 7 months ago

Comments & Activities

Issue created by @smustgrave
Status changed to Closed: outdated 7 months ago2:33pm 26 April 2024
Comment 7 months ago →
🇺🇸United States smustgrave
Comment 7 months ago →
🇮🇳India pradhumanjainOSL
pradhumanjain2311 → made their first commit to this issue’s fork.
Merge request !7774Issue #3443758: Add proper typehints to Html::escape and Html::decodeEntities + trigger_error → (Closed) created by Unnamed author
Pipeline finished with Failed
7 months ago
Total: 504s
#157579
Comment 7 months ago →
🇺🇸United States smustgrave
@pradhumanjain2311 this issue is closed please close your MR. Also it was tagged novice for new users so should avoid those.
Comment 7 months ago →
System Message
pradhumanjain2311 → closed merge request !7774

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024