Diffusion Phorge 74dfe6f971af

Fix is_absolute test in markup
Audit Required74dfe6f971af
Actions

Description

Fix is_absolute test in markup

Summary:
See Q59. Fixes rP935d7120ee32.

Call sites should be happy to use PhutilURI when possible.

Test Plan: visit any Repository page. Before - exception, now - data.

Reviewers: O1 Blessed Committers!, valerio.bozzolan

Reviewed By: valerio.bozzolan

Subscribers: speck, tobiaswiese, valerio.bozzolan, Matthew, Cigaryno

Differential Revision: https://we.phorge.it/D25139

Details

Group Auditors

O1: Blessed Committers

Provenance

avivey	Authored on Apr 28 2023, 10:41
avivey	Pushed on Apr 28 2023, 11:09

Reviewer

valerio.bozzolan

Differential Revision

D25139: Fix is_absolute test in markup

Parents

rPb56d86e48d0f: Fix numerous PHP 8.1 "strlen(null)" exceptions preventing homepage to display

Branches

Unknown

Tags

Unknown

Event Timeline

avivey committed rP74dfe6f971af: Fix is_absolute test in markup (authored by avivey).Apr 28 2023, 11:09

avivey added an auditor: O1: Blessed Committers.

This commit now requires audit.Apr 28 2023, 11:10

avivey added a revision: D25139: Fix is_absolute test in markup.Apr 28 2023, 11:10

avivey mentioned this in rP935d7120ee32: PHP 8.1 "preg_match(null)" exception in javelin/markup.php when http_action not….Apr 28 2023, 11:10

avivey mentioned this in Q53: Diffusion Repository Landing Pages - Unhandled Exception ("InvalidArgumentException") (Answer 65).

Changes (1)

Path

Size

Packages

src/

infrastructure/

javelin/

markup.php

O1: Blessed Committers

rP74dfe6f971af

View Options

src/infrastructure/javelin/markup.php