Modernize ApplicationConfiguration
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Dec 10, 08:36

	Unknown Object (File)
	Tue, Dec 9, 22:49

	Unknown Object (File)
	Tue, Dec 9, 18:40

	Unknown Object (File)
	Sun, Dec 7, 19:36

	Unknown Object (File)
	Sun, Dec 7, 11:23

	Unknown Object (File)
	Sat, Dec 6, 11:27

	Unknown Object (File)
	Thu, Dec 4, 16:16

	Unknown Object (File)
	Tue, Dec 2, 13:00

View All 67 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