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

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

Tags

None

Referenced Files

	F3282979: D25907.1742734019.diff
	Sat, Mar 22, 12:46

	F3280493: D25907.1742653143.diff
	Fri, Mar 21, 14:19

	F3280152: D25907.1742643557.diff
	Fri, Mar 21, 11:39

	F3278819: D25907.1742627761.diff
	Fri, Mar 21, 07:16

	F3278716: D25907.1742626946.diff
	Fri, Mar 21, 07:02

	F3278616: D25907.1742625697.diff
	Fri, Mar 21, 06:41

	F3277856: D25907.1742605203.diff
	Fri, Mar 21, 01:00

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

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

Branch

DivinerArticleAtomizer (branched from master)

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 1777
Build 1777: arc lint + arc unit

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

Coverage (All)

Coverage (Touched)

Packages

src/

applications/

diviner/

atomizer/

DivinerArticleAtomizer.php

2 lines

10%

O1: Blessed Committers

Diff 2740

View Options

src/applications/diviner/atomizer/DivinerArticleAtomizer.php

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