![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#7692 closed defect (bug) (fixed)
Missing Escaping, Strict Check, and PHPCS Issues in Multiple Files of jobs.wordpress.net
| Reported by: | deepakrohilla | Owned by: | |
|---|---|---|---|
| Priority: | lowest | Milestone: | |
| Component: | Jobs (jobs.wordpress.net) | Keywords: | has-patch |
| Cc: |
Description
In multiple files strict check , escaping and alignment, phpcs issue found.
Needs to be fixed instances where data output lacks proper escaping especially in areas handling user-generated or dynamic content.
Attachments (3)
Change History (10)
This ticket was mentioned in PR #340 on WordPress/wordpress.org by @deepakrohilla.
2 years ago
#1
- Keywords has-patch added; needs-patch removed
This ticket was mentioned in Slack in #meta by dufresnesteven. View the logs.
2 years ago
This ticket was mentioned in PR #341 on WordPress/wordpress.org by @narenin.
2 years ago
#5
#6
@
13 months ago
- Resolution → fixed
- Status assigned → closed
Closing due to inactivity and lack of severity.
@obenland commented on PR #340:
4 weeks ago
#7
There are a vast amount of code quality fixes that can be done across the meta repository. I'm not sure selected patches like this are the best way solve them. These should rather be updates as part of regular work or in a coordinated effort like it was done in Core.
Strict checks are not always required, loose comparisons are often better in non-security scenarios. Marking as low-priority.