Modernize ApplicationConfiguration
ClosedPublic
Actions

Authored by dereckson on Oct 29 2025, 22:59.

Tags

None

Referenced Files

	F21295533: D3827.diff
	Wed, Jan 21, 03:52

	Unknown Object (File)
	Mon, Jan 19, 20:14

	Unknown Object (File)
	Sun, Jan 18, 17:58

	Unknown Object (File)
	Fri, Jan 16, 10:57

	Unknown Object (File)
	Thu, Jan 15, 22:06

	Unknown Object (File)
	Tue, Jan 13, 19:02

	Unknown Object (File)
	Sun, Jan 11, 12:45

	Unknown Object (File)
	Sat, Jan 10, 14:36

View All 98 Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rOBSIDIAN330f740251a9: Modernize ApplicationConfiguration

Summary

Add types to properties

Add property $name explicitly

This fixes the following issue:
PHP Deprecated:
Creation of dynamic property ApplicationConfiguration::$name is deprecated
in /var/51-wwwroot/obsidian/workspaces/src/Engines/Apps/ApplicationConfiguration.php on line 66

Following style conventions for class end

Diff Detail

Repository

rOBSIDIAN Obsidian Workspaces

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Oct 29 2025, 22:59

dereckson created this revision.

Harbormaster completed remote builds in B6116: Diff 9911.Oct 29 2025, 22:59

dereckson accepted this revision.Oct 29 2025, 22:59

This revision is now accepted and ready to land.Oct 29 2025, 22:59

Closed by commit rOBSIDIAN330f740251a9: Modernize ApplicationConfiguration (authored by dereckson). · Explain WhyOct 29 2025, 23:00

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rOBSIDIAN330f740251a9: Modernize ApplicationConfiguration.

Revision Contents
Changeset List

Path

Size

workspaces/

src/

Engines/

Apps/

ApplicationConfiguration.php

24 lines

Diff 9912

View Options

workspaces/src/Engines/Apps/ApplicationConfiguration.php

Modernize ApplicationConfigurationClosedPublicActions