Fix PHP8.1 strlen(null) error in DivinerArticleAtomizer.php
ClosedPublic
Actions

Authored by Sten on Thu, Mar 13, 14:45.

Tags

None

Referenced Files

	F3251051: D25907.1742336599.diff
	Mon, Mar 17, 22:23

	F3250662: D25907.1742310291.diff
	Mon, Mar 17, 15:04

	F3250554: D25907.1742303845.diff
	Mon, Mar 17, 13:17

	F3250396: D25907.1742300598.diff
	Mon, Mar 17, 12:23

	F3248637: D25907.1742264801.diff
	Mon, Mar 17, 02:26

	F3223555: D25907.1741970324.diff
	Thu, Mar 13, 16:38

	F3223543: D25907.1741967648.diff
	Thu, Mar 13, 15:54

Subscribers

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T16012: DivinerArticleAtomizer.php fails with passing null to strlen error under PHP 8.1

Commits

rP3e2d5e4bace1: Fix PHP8.1 strlen(null) error in DivinerArticleAtomizer.php

Summary

Fix the strlen(null) error in DivinerArticleAtomizer.php

Fixes T16012

Test Plan

Create Test.book and test.md as per T16012, then run:

/phorge/bin/diviner generate --book Test.book

Diff Detail

Repository

rP Phorge

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Sten created this revision.Thu, Mar 13, 14:45

Owners added a reviewer: O1: Blessed Committers.Thu, Mar 13, 14:45

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptThu, Mar 13, 14:45

Sten requested review of this revision.Thu, Mar 13, 14:45

Harbormaster completed remote builds in B1777: Diff 2740.Thu, Mar 13, 14:45

sgtm

Thanks :)

This revision is now accepted and ready to land.Thu, Mar 13, 22:35

Closed by commit rP3e2d5e4bace1: Fix PHP8.1 strlen(null) error in DivinerArticleAtomizer.php. · Explain WhyFri, Mar 14, 09:11

This revision was automatically updated to reflect the committed changes.

Sten added a commit: rP3e2d5e4bace1: Fix PHP8.1 strlen(null) error in DivinerArticleAtomizer.php.

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

diviner/

atomizer/

DivinerArticleAtomizer.php

2 lines

O1: Blessed Committers

Diff 2745

View Options

src/applications/diviner/atomizer/DivinerArticleAtomizer.php

Fix PHP8.1 strlen(null) error in DivinerArticleAtomizer.phpClosedPublicActions