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

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

Tags

None

Referenced Files

	F4015334: D25907.1746640318.diff
	Tue, May 6, 17:51

	F3674150: D25907.1745489677.diff
	Wed, Apr 23, 10:14

	F3673649: D25907.1745476522.diff
	Wed, Apr 23, 06:35

	F3673616: D25907.1745476134.diff
	Wed, Apr 23, 06:28

	F3657495: D25907.1745448404.diff
	Tue, Apr 22, 22:46

	F3587158: D25907.1745141240.diff
	Sat, Apr 19, 09:27

	F3576639: D25907.1745058587.diff
	Fri, Apr 18, 10:29

	F3576569: D25907.1745057928.diff
	Fri, Apr 18, 10:18

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.Mar 13 2025, 14:45

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

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

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

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

sgtm

Thanks :)

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

Closed by commit rP3e2d5e4bace1: Fix PHP8.1 strlen(null) error in DivinerArticleAtomizer.php. · Explain WhyMar 14 2025, 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