chore: support nette/forms 3.3.0, require PHP 8.3

[dakorpar]

Closes #118.

What broke

Installing nette/forms 3.3.0 made the library unusable outright:

Declaration of Contributte\FormsBootstrap\Traits\BootstrapContainerTrait::addSubmit(string $name, $caption = null): SubmitButton
must be compatible with Nette\Forms\Container::addSubmit(string $name, Stringable|string|null $caption = null, ?Closure $onSubmit = null): SubmitButton

Container::addSubmit() gained a third ?Closure $onSubmit argument in 3.3.0, so every BootstrapForm / BootstrapContainer was a fatal error at class-load time.

Beyond that, 3.3.0 pulls in Nette\SmartObject on BaseControl, Container and Form, and re-tags several magic properties as @property-deprecated. Two of them are ones this library read:

• BaseControl::$caption → getCaption() — read in BootstrapRenderer::renderLabel(), CheckboxInput::getControl(), ButtonInput::getControl()
• Form::$renderer → getRenderer() — read in BootstrapRow::render() and BootstrapCell::render()

Those emitted 14 distinct E_USER_DEPRECATED notices across 53 tests.

Changes

• composer.json: nette/forms 3.2.9 → 3.3.0, keeping the repo's exact-pin convention.
• BootstrapContainerTrait::addSubmit() takes ?Closure $onSubmit and appends it to the button's onClick, matching what the parent does.
• The five deprecated magic-property reads become getCaption() / getRenderer() calls. Both getters have existed since well before 3.2.9, so this is not a 3.3-only construct.

Nothing else in the 3.2.9 → 3.3.0 diff touches what this library overrides. Container::getControls() changed its return type from \Iterator to iterable, but the renderer only ever foreaches over it; the getValues(true) and getOption($key, $default) deprecated forms that were removed are not used here.

Tests

• testAddSubmitTakesAnOnSubmitHandler / testAddSubmitWithoutHandlerRegistersNoClickListener in tests/Traits/BootstrapContainerTraitTest.php cover the new argument.
• testAddSubmitHandlerIsCalledWhenThatButtonSubmitsTheForm in tests/E2E/FormSubmissionTest.php drives a whole request through a real presenter and asserts the handler actually fires with the submitted values.

make tests (158 tests, 242 assertions, no deprecations), make phpstan and make cs all pass.

Note for whoever merges: the same source also passes tests and PHPStan unchanged against 3.2.9, so widening the constraint to ^3.2.9 is available if supporting both lines is ever wanted — the exact pin here is deliberate, per the convention every previous bump followed.

🤖 Generated with Claude Code

---

Update: PHP 8.3 minimum + dependency bumps

nette/forms 3.3.0 — along with nette/application 3.3.0 and nette/component-model 4.0.1, which it pulls in — declares php: 8.3 - 8.5. PHP 8.2 therefore cannot be supported on this line at all, which is what the --ignore-platform-reqs CI job was papering over.

• composer.json: php: ">=8.2" → ">=8.3", and the exact nette/forms pin relaxed to ^3.3.0 so patch releases are picked up.
• phpunit/phpunit ^11.5 → ^12.5. PHPUnit 12 requires php >=8.3, so this was blocked until now. phpunit.xml points at the 12.5 schema, and the createMock(Presenter::class) calls become createStub(...) — PHPUnit 12 emits a notice for a mock with no configured expectations, and these presenters are pure stand-in parents.
• CI: the 8.2 jobs are gone. qa, static analysis, coverage and --prefer-lowest all run on 8.3; the test matrix is 8.3 / 8.4 / 8.5. The --ignore-platform-reqs include is dropped.
• README version table: dev row ^0.9 now reads ^8.3.

Everything else was already at its newest installable release — the remaining outdated transitives (squizlabs/php_codesniffer 4.x, slevomat/coding-standard 8.31) are held back by contributte/qa ^0.4, whose only newer version is 0.5.x-dev.

Verified on PHP 8.3.31: make tests (158 tests, 242 assertions), make phpstan, make cs all pass, and the --prefer-lowest resolution (forms 3.3.0 + application 3.2.0 + component-model 3.2.0) passes the suite too.

This is a breaking change for consumers on PHP 8.2 — intended for the next major.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.05%. Comparing base (c9300d6) to head (e616b87).

Additional details and impacted files

@@            Coverage Diff            @@
##             master     #129   +/-   ##
=========================================
  Coverage     97.05%   97.05%           
- Complexity      308      309    +1     
=========================================
  Files            25       25           
  Lines           984      986    +2     
=========================================
+ Hits            955      957    +2     
  Misses           29       29           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.