Page MenuHomePhorge
Differential D25539

Fix an instance of `strlen(null)`
ClosedPublic
Actions

Authored by avivey on Feb 23 2024, 11:29.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 22:23
Unknown Object (File)
Wed, Apr 24, 21:42
Unknown Object (File)
Wed, Apr 17, 18:23
Unknown Object (File)
Wed, Apr 17, 07:01
Unknown Object (File)
Mon, Apr 15, 16:58
Unknown Object (File)
Thu, Apr 11, 06:12
Unknown Object (File)
Mar 27 2024, 18:59
Unknown Object (File)
Mar 25 2024, 07:34
View All 27 Files
Subscribers
Cigaryno
Matthew
tobiaswiese
valerio.bozzolan

Details

Reviewers
bekay
valerio.bozzolan
Group Reviewers
O1: Blessed Committers
Maniphest Tasks
T15064: Make Phorge compatible with PHP 8.1/8.2/8.3
Commits
rP8667ed45fada: Fix an instance of `strlen(null)`
Summary

Ref Q112, ref T15064.

Test Plan

Enter the text [[hello]] in a remarkup field - without this change, it causes a deprecation warning in the log.

This doesn't happen if the content looks like a valid URL - this is the "Wiki Link" syntax.

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

avivey created this revision.Feb 23 2024, 11:29
Owners added a reviewer: O1: Blessed Committers.Feb 23 2024, 11:29
Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptFeb 23 2024, 11:29
avivey requested review of this revision.Feb 23 2024, 11:29
Harbormaster completed remote builds in B1070: Diff 1710.Feb 23 2024, 11:29
avivey mentioned this in Q112: Deprecation Notice from PhrictionRemarkupRule.php:194 (Answer 144).Feb 23 2024, 11:30
avivey added inline comments.Feb 23 2024, 11:32
src/applications/phriction/markup/PhrictionRemarkupRule.php
75–80

I'm pretty much guessing that this is the only place explicitName can be created. $name is extracted from a regexp inline 19.

valerio.bozzolan accepted this revision.Feb 23 2024, 13:14
This revision is now accepted and ready to land.Feb 23 2024, 13:14
Closed by commit rP8667ed45fada: Fix an instance of `strlen(null)`. · Explain WhyFeb 23 2024, 15:06
This revision was automatically updated to reflect the committed changes.
avivey added a commit: rP8667ed45fada: Fix an instance of `strlen(null)`.

Revision Contents
Changeset List

PathSizePackages
src/
applications/
phriction/
markup/
2 linesO1: Blessed Committers

Diff 1712

View Options

src/applications/phriction/markup/PhrictionRemarkupRule.php

Loading...
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0