Modernize code by jrauh01 · Pull Request #611 · Icinga/icingaweb2-module-vspheredb

jrauh01 · 2026-02-20T13:25:17Z

This PR is a broad modernization and cleanup sweep across the whole codebase. The goal is to move the project toward more idiomatic modern PHP, improve type safety, reduce deprecated usages, and improve readability.

Because the vast amount of changes PR explicitly makes no claim to completeness. It is a best effort refactor pass: there may still be remaining places that could benefit from the same transformations. So this should be treated as first big step in an incremental modernization rather than a finished “cleanup of everything”.

Typing and signatures

Add/strengthen parameter, return, and property types throughout the codebase.
Align fluent APIs and inheritance signatures (static/self, consistent return types).
Improve PHPDoc accuracy and consistency, including proper @deprecated tags.
Use ?Type shorthand for nullable PHPDoc annotations.

Modern PHP language features

Replace older constructs with modern equivalents where appropriate:

match, nullsafe ?->, null coalescing ??/??=, ternary, arrow functions fn() =>, first-class callables.
Modern string helpers (str_starts_with, str_ends_with, str_contains) instead of substr/strpos/regex for simple checks.
Variadic arguments instead of func_get_args().

Consistency / formatting / readability

Formattings: imports sorted, redundant else branches removed, small temporary variables inlined, unreachable code commented out.
Standardize array formatting in some places (not all places; requires further discussions about a unified standard) and removes trailing commas where used in multi-line arrays.
Rework query builder method chains for consistent indentation.
Refactor table column creation blocks to a uniform style (chaining, spacing, renderer callbacks).
Remove PHPDoc annotation alignment.

Internal structure improvements

Introduces small abstractions to reduce duplication (for example a shared table base for percent/usage patterns).
Converts some "const string" typed concepts into enums ( ´ObjectType, MonitoringStateTrigger, CheckPluginState, and ResultStatus`).

jrauh01 · 2026-03-04T13:18:06Z

The last two force pushes fixed the failed jobs.

BastianLedererIcinga · 2026-04-02T08:26:57Z

I moved return type additions for ipl forward compatibilty to a different PR.

- Add explicit parameter, return, and property types. - Remove runtime type checks where signatures now enforce valid input. - Change `self` return type to `static` in most cases. - Added phpdocs. (not for all because the commit grew to much) - Format existing phpdocs.

- Use `use` statements to import classes instead of using namespaced references. - Drop unnecessary global prefixes for built-in function calls. (for example `\strlen`, `\sprintf` -> ´strlen`, `sprintf`)

Use match espressions where possible.

- Drop a bunch of temporary variables that were only used once, replacing them with direct expressions. - Replace small loops used only for editing each entry of an array with `array_map()`. - Remove unused variables: - `$e`/`$_` Exception variables. - `foreach` keys. - Other

Remove trailing commas from multi-line arrays for consistency.

Rewrite many `$db->select()->from()->join()->where();` calls to a consistent “one method per line” style. Split longer `from()`/`join()`/`where()` argument lists across lines and align indentation so chained queries are easier to scan and edit.

This commit does **not** format all array to the same format. It formats all keys of each array in the same way so they are uniform in themselves. - If array keys are aligned partly now the full array is aligned. - Remove trailing spaces in inline arrays `[ 'a', 'b' ]` -> `['a', 'b']`.

- Use arrow functions for simple renderers and keep full closures only when multi step logic is needed. - Consistent method chaining layout. - Add empty lines between column objects.

Mixed commit that reformats code for readability. - Reflow a few multi-line calls into a clearer style. - Add spaces after `!` and type casts. - Override `protected $method` instead of setting it in the constructor. - Add empty lines.

Comment out... - code below a return statement, - code inside of an `if (false)` block, - variable assignments directly before the variable is overwritten, - empty `if` block ...to improve readability by reducing noise.

Sort imports alphabetically and remove unused imports.

The DB readiness check was incorrect due to operator precedence. The logical NOT operator (`!`) was applied before the strict comparison (`===`), causing the component state to be negated and then compared to `STATE_READY`. This resulted in the condition never evaluating as intended. Replace the negated comparison with an explicit `!== STATE_READY` check to correctly detect when the DB component is not ready.

`updateDb()` returns either an **integer** or **true**, but never **false**. Because the check works with a strict comparison, this means it will never be **false**. Thus the strict `!== false` comparison is replaced by a truthy check to correctly handle integer return values and failures.

For the methods `Form::onSuccess()` and `HtmlDocument::assemble()`.

This reverts commit e77d6eead174169719298ba2c2123c7f07c8516e. This fix caused the daemon to freeze.

Remove string types to satisfy existing `ipl-html` base implementation of `isValidEvent()`. This commit will be squashed into the original commit later.

PHP 8.2 typed properties make these annotations redundant. The declared type already carries everything the annotation restated. Kept every `@var` that adds a description, narrows the type (e.g. `string[]` over `array`), or lives on a constant.

cla-bot Bot added the cla/signed label Feb 20, 2026

jrauh01 self-assigned this Feb 23, 2026

jrauh01 force-pushed the modernize branch from 0255196 to bc26d96 Compare February 23, 2026 07:38

jrauh01 requested a review from lippserd February 24, 2026 14:18

jrauh01 force-pushed the modernize branch from bc26d96 to 150e8d5 Compare February 25, 2026 09:07

jrauh01 force-pushed the support-php-8.5 branch 2 times, most recently from bdf6581 to db4e52f Compare February 26, 2026 12:11

Base automatically changed from support-php-8.5 to main February 26, 2026 12:29

sukhwinder33445 reviewed Mar 4, 2026

View reviewed changes

Comment thread library/Vspheredb/Web/Form/VCenterShipMetricsForm.php

sukhwinder33445 reviewed Mar 4, 2026

View reviewed changes

Comment thread library/Vspheredb/Web/Form/PerfdataConsumerForm.php

jrauh01 force-pushed the modernize branch 6 times, most recently from 03fb727 to 4d6f364 Compare March 4, 2026 13:10

BastianLedererIcinga mentioned this pull request Mar 4, 2026

Adjust modules for compatibility with ipl-stdlib modernization Icinga/ipl-stdlib#68

Open

5 tasks

jrauh01 force-pushed the modernize branch 2 times, most recently from 2e0a84a to 4d74a21 Compare March 20, 2026 07:47

BastianLedererIcinga reviewed Mar 20, 2026

View reviewed changes

Comment thread library/Vspheredb/Web/Form/VCenterShipMetricsForm.php Outdated

lippserd added this to the v1.9.0 milestone Mar 26, 2026

BastianLedererIcinga force-pushed the modernize branch from db98d77 to af27ffd Compare April 2, 2026 08:25

lippserd removed this from the v1.9.0 milestone Apr 29, 2026

jrauh01 added 5 commits May 13, 2026 14:34

Remove dependency stuff

5588562

Add types

f3c3298

- Add explicit parameter, return, and property types. - Remove runtime type checks where signatures now enforce valid input. - Change `self` return type to `static` in most cases. - Added phpdocs. (not for all because the commit grew to much) - Format existing phpdocs.

Add empty line above return and exit

fdb0610

Use imports

d0d7ac4

- Use `use` statements to import classes instead of using namespaced references. - Drop unnecessary global prefixes for built-in function calls. (for example `\strlen`, `\sprintf` -> ´strlen`, `sprintf`)

Use match expressions

dc86dd2

Use match espressions where possible.

jrauh01 added 25 commits May 13, 2026 15:10

Remove trailing commas from arrays

f3a70ef

Remove trailing commas from multi-line arrays for consistency.

Reformat table column definitions for consistency and readability

7e91194

- Use arrow functions for simple renderers and keep full closures only when multi step logic is needed. - Consistent method chaining layout. - Add empty lines between column objects.

Reformat stuff

50a0570

Mixed commit that reformats code for readability. - Reflow a few multi-line calls into a clearer style. - Add spaces after `!` and type casts. - Override `protected $method` instead of setting it in the constructor. - Add empty lines.

Comment out unreachable or unused code

842688d

Comment out... - code below a return statement, - code inside of an `if (false)` block, - variable assignments directly before the variable is overwritten, - empty `if` block ...to improve readability by reducing noise.

Sort imports

32a5a20

Sort imports alphabetically and remove unused imports.

Translate german comments to englisch

6ae68c3

Add enum ResultStatus

63b5cd1

Make CheckPluginState an enum

018ff6f

Make MonitoringStateTrigger an enum

c08bff1

Make ObjectType an enum

0cad039

Change method visibility to protected like the parent method

612ce08

For the methods `Form::onSuccess()` and `HtmlDocument::assemble()`.

Add abstract class PercentObjectsTable to bundle redundant methods

5c420f4

Use ??= operator

5325047

Use ?? operator

2bf8fd4

Use ?-> operator

c6a5147

Use ternary operator

a263dab

Revert "Fix potential operator precedence bug"

8e1a243

This reverts commit e77d6eead174169719298ba2c2123c7f07c8516e. This fix caused the daemon to freeze.

Remove PhpDoc annotation alignment

6cd2c04

Use ?Type shorthand for nullable PHPDoc annotations

9cbb77e

fixup! Add types

9574f5b

Remove string types to satisfy existing `ipl-html` base implementation of `isValidEvent()`. This commit will be squashed into the original commit later.

jrauh01 force-pushed the modernize branch from af27ffd to 9574f5b Compare May 13, 2026 13:26

jrauh01 added 4 commits May 18, 2026 10:51

fixup! Comment out unreachable or unused code

9f0aa3f

FIXUP and REWRITE: Comment out unused and unreachable code

b6c6b7f

fixup! Use ternary operator

15834ab

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Modernize code#611

Modernize code#611
jrauh01 wants to merge 45 commits into
mainfrom
modernize

jrauh01 commented Feb 20, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

jrauh01 commented Mar 4, 2026

Uh oh!

Uh oh!

BastianLedererIcinga commented Apr 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

jrauh01 commented Feb 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Typing and signatures

Modern PHP language features

Consistency / formatting / readability

Internal structure improvements

Uh oh!

Uh oh!

Uh oh!

jrauh01 commented Mar 4, 2026

Uh oh!

Uh oh!

BastianLedererIcinga commented Apr 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

jrauh01 commented Feb 20, 2026 •

edited

Loading