Add Config based on CompatForm

[TheSyscall]

An implementation of an INI based configuration form for CompatForms.

It allows developers to create a configuration form for an INI file and have it automatically populate form elements and store the results back into the specified file or section.

If writing the configuration file fails, an error is displayed alongside the full contents of the file to copy and paste manually by the admin.

As suggested here

resolves #5479

[Al2Klimov]

Test

1. Install Fedora 42
2. Apply https://github.com/Al2Klimov/ansible-icinga-fedora using snapshot packages
3. yum remove icingaweb2-module-monitoring
4. Upgrade to https://git.icinga.com/packages/icingaweb/-/jobs/905522
5. Create /usr/share/icingaweb2/modules/test5480
6. icingacli mod en test5480
7. Visit /icingaweb2/test5480
8. Input something
9. Press enter
10. cat /etc/icingaweb2/modules/test5480/config.ini (ok)
11. chmod 0440 /etc/icingaweb2/modules/test5480/config.ini
12. Repeat steps 7-9 (fails, prints helpful message)

So far so good...

/usr/share/icingaweb2/modules/test5480

library/Test5480/Form.php

<?php

namespace Icinga\Module\Test5480;

use Icinga\Web\Form\ConfigForm;

class Form extends ConfigForm
{
    protected function assemble()
    {
        $this->addElement('text', 'foo__bar');
    }
}

application/controllers/IndexController.php

<?php

namespace Icinga\Module\Test5480\Controllers;

use GuzzleHttp\Psr7\ServerRequest;
use Icinga\Module\Test5480\Form;
use ipl\Web\Compat\CompatController;

class IndexController extends CompatController
{
    public function indexAction(): void
    {
        $form = new Form($this->Config());
        $form->handleRequest(ServerRequest::fromGlobals());
        $this->addContent($form);
    }
}

[Al2Klimov]

Suggested change

* It deletes the underlying section regardless of whether form validation passed.

Another round of throwing AI at your PR broke this promise:

Bildschirmaufnahme.2026-06-23.um.17.57.10.mov

You can just remove it if it's not super critical.

[TheSyscall]

Mistakes were made during refactor. Fixed it.

[jrauh01]

Sometimes you do one line doc blocks for properties and sometimes multiline, but i cannot tell which pattern you are following. I think it would be better to standardize this.

[jrauh01]

Please look over ShowConfiguration and inline the element creations where possible.

[lippserd]

Changed my mind, sorry. I think the suggestion is better than the current implementation. Make sure to adjust all call sites.

[lippserd]

Suggested change

	}
	* @return array{?string, ?string} TODO: Change docs.
	*/
	protected function getIniKeyFromName(string $name): array
	{
	return Str::symmetricSplit($name, $this->sectionKeyDelimiter, 2);
	}

[TheSyscall]

Changed method signature

[lippserd]

Suggested change

	$parts = $this->getIniKeyFromName($name);
	if (count($parts) !== 2) {
	return $default;
	}
	return $this->config->get($parts[0], $parts[1], $default);
	[$section, $key] = $this->getIniKeyFromName($name)
	if ($key === null) {
	return $default;
	}
	return $this->config->get($section, $key, $default);

[TheSyscall]

Changed call sites

[jrauh01]

Overall this looks good, I would only adjust those two lines to stay consistent.

[jrauh01]

LGTM