`Console\Helper::strlen()` was deprecated in Symfony 5.3. As we
require Symfony >= 5.4, we can safely use the replacement
function instead that 12.4 and `main` also use.

Note: This already got fixed in main and 12.4 with the upgrade to
Symfony 6.1. So this fix here is for 11.5 only.

Another note: This change aligns the code with 12.4 and `main`. Changing this to the more fitting `::width()` should happen in a separate later change across versions, not only for 11.5.

Resolves: #103503
Related: #98158
Releases: 11.5
Change-Id: Iabd9bf2423501c69e9cd805a6d923f779c8ca04b
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83625


Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Bartsch <bo@cedev.de>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Oliver Bartsch <bo@cedev.de>
Reviewed-by: Jörg Bösche <typo3@joergboesche.de>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Andreas Kienast <a.fernandez@scripting-base.de>

In the image editor, the "Cropped area" was configured with a `ceil`ed
width and height. Since all other dimensions are `floor`ed, this might
cause an overlap.
Due to the configured cropping mode, defining that the cropping area
must never exceed the image dimensions, the cropper library places the
cropper to a position that's mathematically correct but not desired.

To solve this issue, the `width` and `height` properties fed to the
cropper are now `floor`ed as well, basically aligning with the rest of
the coordinates.

Resolves: #101520
Releases: main, 12.4, 11.5
Change-Id: I981bd185eaf0db60662a5a6b7b423d84112c2150
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83647


Tested-by: Andreas Kienast <a.fernandez@scripting-base.de>
Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Andreas Kienast <a.fernandez@scripting-base.de>

The page tree in TYPO3 v11 and v12 has a default timeout of 300ms for
click events to be processed (in order to wait for double-click
detection). This caused race conditions in tests since code that
waited for #nprogress to hide after clicking on the pagetree
did not have anything to wait for (yet), because the progress
loaded was only triggered 300ms later, when the click was detected
to not be a double click event and the content frame was started
to be updated.

Also remove wait timeout that were implicitly added for the same
issue, but were spread throughout the test cases.

Resolves: #103473
Releases: 12.4, 11.5
Change-Id: Ie9c9aff8c8a36581555a9ad352fb98b3d446ba42
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83577


Tested-by: core-ci <typo3@b13.com>
Tested-by: Benjamin Franzke <ben@bnf.dev>
Reviewed-by: Benjamin Franzke <ben@bnf.dev>

Fixes the Doctrine error:
Incorrect integer value: '' for column 'editlock' at row 1

Releases: main, 12.4, 11.5
Resolves: #103461
Resolves: #103462
Change-Id: I0e3f0344157f66c07681bd29a47f4f51e32e4225
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83542


Tested-by: core-ci <typo3@b13.com>
Tested-by: Stefan Bürk <stefan@buerk.tech>
Reviewed-by: Stefan Bürk <stefan@buerk.tech>

New core-testing-* images have been build and published in
the TYPO3 testing-infrastructure repository and pulled to
the core ci gitlab runner instances. [1][2]

This change modifies `Build/Scripts/runTests.sh` to set
new image version to ensure the usage for all changes
based on this change.

For information what changed in the image see the
patches in the testing-infrastrucure repository
linked below.

* core-testing-php74 => 2.8
* core-testing-php80 => 2.8
* core-testing-php81 => 2.12
* core-testing-php82 => 1.12
* core-testing-php83 => 1.13

[1] https://git.typo3.org/typo3/CI/testing-infrastructure/-/commit/ebdaefdb54a4fe970abf0cb1d32567bbd6339de5
[2] https://git.typo3.org/typo3/CI/testing-infrastructure/-/commit/21f88537fab0a6c5be866c602f206acdb289a20a

Resolves: #103469
Releases: main, 12.4, 11.5
Change-Id: I35ffd10e7979d4fd59aab5a7fa5d52330e71e0e1
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83538


Tested-by: Oliver Bartsch <bo@cedev.de>
Reviewed-by: Stefan Bürk <stefan@buerk.tech>
Tested-by: Oliver Klee <typo3-coding@oliverklee.de>
Tested-by: Stefan Bürk <stefan@buerk.tech>
Reviewed-by: Oliver Klee <typo3-coding@oliverklee.de>
Reviewed-by: Oliver Bartsch <bo@cedev.de>
Tested-by: core-ci <typo3@b13.com>

This commit adds `_ga` to the `excludedParameters` configuration used
for the cHash calculation.

Resolves: #103435
Releases: main, 12.4, 11.5
Change-Id: I5c79c9264d1d8f09f45db947d714d27b3da4fb00
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83534


Tested-by: Andreas Kienast <a.fernandez@scripting-base.de>
Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Andreas Kienast <a.fernandez@scripting-base.de>

 * Always clean-up containers when Control-C is pressed
 * Enforce removing networks even if some containers have not been
   fully stopped (does not work with older docker versions, therefore
   only enabled for podman)
 * Bail out if `waitFor` failed instead of resuming with subsequent
   commands
 * Avoid waiting unnecessarily for VNC-web port 7900 in headless mode
 * Do not derive `--pull=never` from the `$CI` environment variable, as
   `$CI` is set both in gitlab [1] and github [2] actions.
   runTests.sh might be used in github actions, therefore this flag is
   moved into our gitlab-ci specific configuration.
 * Fix broken `-b podman -s acceptance -d sqlite` UID mapping and sqlite
   mount
   Error was: `unknown mount option "uid=1000": invalid mount option`
   The sqlite tmpfs is removed, as the sqlite tmpfs can not be shared
   between the PHP development server and codecept (which bootstraps
   the database in `bin/codecept` in classic-mode)
 * Avoid `--add-host` for podman for compat with github ubuntu-latest
   podman version
 * Fix typos

[1] https://docs.gitlab.com/ee/ci/variables/predefined_variables.html
[2] https://docs.github.com/en/actions/learn-github-actions/variables#default-environment-variables

Resolves: #103420
Releases: main, 12.4, 11.5
Change-Id: Ie8fcdc5ebc5ba651242bb54faffdfd8de0dc6026
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83514


Tested-by: core-ci <typo3@b13.com>
Tested-by: Benjamin Franzke <ben@bnf.dev>
Reviewed-by: Benjamin Franzke <ben@bnf.dev>

All applicable acceptance tests are now also executed
in composer mode. Tests that check classic-mode specific
functions are excluded and are therefore now tagged
as classic-mode test.

The composer mode instance is generated with our
CLI setup tools, allowing to mimic the realworld
case where a instance is creating via our setup tools
instead of from fixtures (which we still use for classic
mode tests, but try to reducde/avoid for composer mode tests).

The existing existing classic-mode test acceptance test
execution will keep running as-is.

The v12 backport additionally imports pieces from #102440
as t3editor extension is enabled in this composer-mode,
while v12 classic mode is tested (for historic reasons)
without t3editor extension enabled.

Commands executed:

  > composer require --dev "typo3/cms-styleguide":"~11.5.5"

Resolves: #103297
Releases: main, 12.4, 11.5
Change-Id: I64973f110931b51ed2ef7ef8f8cc3411834fcf37
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83453


Reviewed-by: Benjamin Franzke <ben@bnf.dev>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Benjamin Franzke <ben@bnf.dev>

In #88366, the caches have been renamed, e.g., from `cache_core`
to `core`. Some occurrences of the old names in comments were missed at
that time. This commit streamlines those occurrences to also use
the new names.

Resolves: #103409
Related: #88366
Releases: main, 12.4, 11.5
Change-Id: I69039160b3d3cc5da00d4f3d3e57fce755d22a63
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83476


Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>

Resolves: #103385
Releases: main, 12.4, 11.5
Change-Id: Ifd93dfa4049d0d3550d052fa960ae79cc5a27c34
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83466


Tested-by: core-ci <typo3@b13.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>

Several tests for URI route generation are stricter and assert
that either the whole URI matches, or that at least the `?cHash=`
suffix is applied for the corresponding test permutations.

Besides that, individual URI assertions are streamlined to use
AbstractEnhancerLinkGeneratorTestCase::assertGeneratedUriEquals.

Resolves: #103396
Releases: main, 12.4, 11.5
Change-Id: I65ae10fa2984021da653bac33579feb6d10a6464
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83465


Tested-by: core-ci <typo3@b13.com>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>

The commit messages line-length is documented as 72 characters. The
commit-msg hook checks for 72 characters now.

Before the hook checked for 74 characters.

Resolves: #103363
Releases: main, 12.4, 11.5
Change-Id: I15138831a73e84f275e85a5cd88f0b33319f9c1c
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83425


Tested-by: Benni Mack <benni@typo3.org>
Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Benni Mack <benni@typo3.org>

The public URL resolution in lazy loaded TCA forms (e.g. inline records)
is aligned to use the same path resolution mechanism as in PageRender.

Releases: main, 12.4, 11.5
Resolves: #103296
Change-Id: I981af29b34393302c49dd16216f0baa580d60767
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83399


Tested-by: Benjamin Franzke <ben@bnf.dev>
Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Benjamin Franzke <ben@bnf.dev>

This prevents warnings during rendering and also looks better.

Resolves: #103355
Releases: main, 12.4, 11.5
Change-Id: I0a677f4f1c5ee8253538dd414fe23384ad49b6e9
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83420


Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Lina Wolf <112@linawolf.de>
Tested-by: Lina Wolf <112@linawolf.de>

* .gitignore files contain files that are not
  created during rendering
* .editorconfig conatains old styling, there is a
  main .editorconfig with the containing the correct
  styling

Resolves: #103352
Releases: main, 12.4, 11.5
Change-Id: Ic3189d5f10b8e6f8aa8903cd0924a4a4b4fab191
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83415


Reviewed-by: Lina Wolf <112@linawolf.de>
Tested-by: Lina Wolf <112@linawolf.de>
Tested-by: core-ci <typo3@b13.com>

Resolves: #103350
Related: #103304
Releases: main, 12.4, 11.5
Change-Id: I228f946b8bca26b5c334b352ecd48edfd72c620b
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83413


Tested-by: core-ci <typo3@b13.com>
Tested-by: Lina Wolf <112@linawolf.de>
Reviewed-by: Lina Wolf <112@linawolf.de>

With #90418 the container cache has been excluded from the
regular cache-flush-pipeline.
Therefore flushing has been explicitly performed when writing a
new cache entry (with the intent to clean up old cache entries).

Issue is: Cleaning up the entire folder by deleting the folder means
a concurrent request – that is creating the same DI cache as well –
will fail to write the cache when the folder is (re)moved in that
moment.

It turns out this cache-flush is not needed at all, since the cache
identifier is stable for an TYPO3 instance, therefore a pre-flight
flush can be dropped.

Releases: main, 12.4, 11.5
Resolves: #103294
Change-Id: I1267d794291899e2b0d551b1725bd4a3354823bd
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83361


Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Benjamin Franzke <ben@bnf.dev>
Tested-by: Benjamin Franzke <ben@bnf.dev>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Remove old Settings.cfg, was only used for Sphinx

Attention: Backport needs manual changes to the guides.xml.

Resolves: #103334
Releases: main, 12.4, 11.5
Change-Id: If2086510e19ca5dc682d9ff0e45c545a458bdfc1
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83349


Tested-by: Lina Wolf <112@linawolf.de>
Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Lina Wolf <112@linawolf.de>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Remove outdated genindex: not supported anymore
* Remove old Settings.cfg, was only used for Sphinx

Attention: Backport needs manual changes to the guides.xml.

Resolves: #103336
Releases: main, 12.4, 11.5
Change-Id: I31f40444705530a3a9219fde6530cc46f3888df7
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83345


Tested-by: Lina Wolf <112@linawolf.de>
Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Lina Wolf <112@linawolf.de>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Remove outdated genindex: not supported anymore
* Remove old Settings.cfg, was only used for Sphinx

Attention: Backport needs manual changes to the guides.xml.

Resolves: #103340
Releases: main, 12.4, 11.5
Change-Id: Id690e98fb30595e36eb6d573c7a03098668222dd
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83343


Tested-by: core-ci <typo3@b13.com>
Tested-by: Lina Wolf <112@linawolf.de>
Reviewed-by: Lina Wolf <112@linawolf.de>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Remove outdated genindex: not supported anymore
* Remove old Settings.cfg, was only used for Sphinx

Attention: Backport needs manual changes to the guides.xml.

Resolves: #103339
Releases: main, 12.4, 11.5
Change-Id: Ifde54f6d21ae29ba8c3e8543601048cab2469744
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83341


Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Lina Wolf <112@linawolf.de>
Tested-by: Lina Wolf <112@linawolf.de>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Remove outdated genindex: not supported anymore
* Remove old Settings.cfg, was only used for Sphinx

Attention: Backport needs manual changes to the guides.xml.

Resolves: #103342
Releases: main, 12.4, 11.5
Change-Id: I8d29012e8be9e999129d04c5e84d6a531a9a6b28
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83339


Reviewed-by: Lina Wolf <112@linawolf.de>
Tested-by: core-ci <typo3@b13.com>
Tested-by: Lina Wolf <112@linawolf.de>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Remove outdated genindex: not supported anymore
* Remove old Settings.cfg, was only used for Sphinx

Attention: Backport needs manual changes to the guides.xml.

Resolves: #103338
Releases: main, 12.4, 11.5
Change-Id: Ia5eae156685428c525fd6e3f27b1c8583b64dd84
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83374


Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Lina Wolf <112@linawolf.de>
Tested-by: Lina Wolf <112@linawolf.de>

The setting edit-on-github-directory is needed to find the correct
directory in the monorepository.

Resolves: #103325
Releases: main, 12.4, 11.5
Change-Id: I624cc046bfa899a07c25b9db4a6735c0f99775ad
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83333


Tested-by: core-ci <typo3@b13.com>
Tested-by: Lina Wolf <112@linawolf.de>
Reviewed-by: Lina Wolf <112@linawolf.de>

Ensure recuid is always int+ as the underlying
DB field is `unsigned`.

Resolves: #103101
Releases: main, 12.4, 11.5
Change-Id: I8fa06fa40260bffc5b01eedc4207aec150e7c849
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83337


Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: core-ci <typo3@b13.com>

This patch adds the `gbraid` and `wbraid` parameters to the cHash
parameter exclusion list.

Resolves: #103327
Releases: main, 12.4, 11.5
Change-Id: Ife0f7886980812a6082224a6385a4393782f938b
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83335


Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: core-ci <typo3@b13.com>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Remove superfluous genindex, it is not supported anymore
* Remove old Settings.cfg, was only used for Sphinx
* Resolve issues from former backports

Attention: Backports need manual changes to the guides.xml.

Resolves: #103307
Releases: main, 12.4, 11.5
Change-Id: Ib4aa42b410be84f787f3c0c8fa7481830db06407
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83281


Tested-by: Chris Müller <typo3@krue.ml>
Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Chris Müller <typo3@krue.ml>
Tested-by: Lina Wolf <112@linawolf.de>
Reviewed-by: Lina Wolf <112@linawolf.de>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Remove superfluous genindex, it is not supported anymore
* Remove old Settings.cfg, was only used for Sphinx

Attention: Backports need manual changes to the guides.xml.

Resolves: #103304
Releases: main, 12.4, 11.5
Change-Id: I62218ce550acccf38bd7641a13c122f837bd2f22
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83277


Tested-by: Lina Wolf <112@linawolf.de>
Reviewed-by: Lina Wolf <112@linawolf.de>
Tested-by: core-ci <typo3@b13.com>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Remove superfluous genindex, it is not supported anymore
* Remove old Settings.cfg, was only used for Sphinx
* Remove screenshots.json as the screenshot tool
  is not maintained anymore

Attention: Backports need manual changes to the guides.xml.

Resolves: #103302
Releases: main, 12.4, 11.5
Change-Id: Ia79869cdcf9b91c299dfc18dcd1615f5fd3cad3f
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83285


Reviewed-by: Lina Wolf <112@linawolf.de>
Tested-by: core-ci <typo3@b13.com>
Tested-by: Lina Wolf <112@linawolf.de>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Remove superfluous genindex, it is not supported anymore
* Remove old Settings.cfg, was only used for Sphinx

Attention: Backports need manual changes to the guides.xml.

Resolves: #103300
Releases: main, 12.4, 11.5
Change-Id: Ia6c1f714d08ed9c14674a9f92cc67f1d1553cbd1
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83287


Tested-by: core-ci <typo3@b13.com>
Tested-by: Lina Wolf <112@linawolf.de>
Reviewed-by: Lina Wolf <112@linawolf.de>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Remove superfluous genindex, it is not supported anymore
* Remove old Settings.cfg, was only used for Sphinx
* Fix references
* Remove screenshots.json as the screenshot tool
  is not maintained anymore

Attention: Backports need manual changes to the guides.xml.

Resolves: #103301
Releases: main, 12.4, 11.5
Change-Id: I6d912175b59b3bbfc833931a8cbf41dc19fceecf
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83283


Tested-by: Lina Wolf <112@linawolf.de>
Reviewed-by: Lina Wolf <112@linawolf.de>
Tested-by: core-ci <typo3@b13.com>

Resolves: #103305
Related: #103238
Releases: main, 12.4, 11.5
Change-Id: I7fd40ea54fd51663222513b803c64ba2a8471d58
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83279


Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Lina Wolf <112@linawolf.de>
Tested-by: Lina Wolf <112@linawolf.de>

This patch adds necessary hints about regex capturing groups
to the documentation of the field target of EXT:redirects.

Both columns (source_path and _target) referencing the
regular expression functionality from now on.

Resolves: #103272
Releases: main, 12.4, 11.5
Change-Id: I764184c306a2fb7176da17e7d2e5f1e6ffae467b
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83275


Tested-by: core-ci <typo3@b13.com>
Tested-by: Lina Wolf <112@linawolf.de>
Reviewed-by: Lina Wolf <112@linawolf.de>

To avoid passing integer values, all values are now
cast to string.

Releases: main, 12.4, 11.5
Resolves: #103199
Change-Id: Ie204380c69f252db6944ff9ae2e12af66ad7876b
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83273


Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Replace usage of :php:class: textroles with :php:
  textroles. The former where used without having a target
* remove superfluous genindex, it is not supported anymore
* Remove old Settings.cfg, was only used for Sphinx

Attention: Backports need manual changes to the guides.xml.

Resolves: #103298
Releases: main, 12.4, 11.5
Change-Id: I6a4974f5470403f85fba0ce2d774066624ebb21b
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83237


Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Lina Wolf <112@linawolf.de>
Tested-by: Lina Wolf <112@linawolf.de>

A local instance can sometimes be easier to debug and browser issues
can be introspected, when acceptance tests are possible to be run on
the host instead of in containers only.

Allow to run acceptance tests against a local server using a local
chromedriver. Also avoid the usage of precalculated session hashes,
which has no clear advantage, but requires codecept to perform
database connections directly: The tests now login via the backend
login form instead and store the session cookies between test runs.

As a side effect the codeception suites are cleaned up, to use
more efficient module config, instead of redundantly repeating
all WebDriver settings for headless and non-headless environment.

To execute codeception with local chromedriver against a custom
server URL run:

  chromedriver --silent & # use `killall chromedriver` when done

  typo3TestingAcceptanceBaseUrl=https://mycore.example.com/ \
  typo3TestingAcceptanceAdminPassword="MyAdminPa$$w0rd" \
  typo3TestingAcceptanceEditorPassword="MyEditorPa$$w0rd" \
  bin/codecept -d -c typo3/sysext/core/Tests/codeception.yml \
    run Application

Resolves: #103086
Releases: main, 12.4, 11.5
Change-Id: Id1210706147e0d66c4a905a53e6cc13b15bf00e8
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83036


Tested-by: core-ci <typo3@b13.com>
Tested-by: Benjamin Franzke <ben@bnf.dev>
Reviewed-by: Benjamin Franzke <ben@bnf.dev>

Google gradually rolls out the `gad_source` parameter as a replacement
for the already excluded `gad` parameter.

This patch modifies the exclude list.

Resolves: #103264
Releases: main, 12.4, 11.5
Change-Id: I4feacd48ed2cd1640e4d7a4ba95470e8702303d0
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83109


Tested-by: Andreas Kienast <a.fernandez@scripting-base.de>
Reviewed-by: Andreas Kienast <a.fernandez@scripting-base.de>
Tested-by: core-ci <typo3@b13.com>

* Introduce a guides.xml, the presence of this file
  triggers the new rendering.
* Replace sphinx-only syntax
* remove superfluous genindex, it is not supported anymore
* Remove old Settings.cfg, was only used for Sphinx

Attention: Backports need manual changes to the guides.xml.

Resolves: #103238
Releases: main, 12.4, 11.5
Change-Id: I5d9239a9217d8d6c2faa08cef58436971ee83113
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83103


Tested-by: Lina Wolf <112@linawolf.de>
Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Lina Wolf <112@linawolf.de>
Tested-by: Chris Müller <typo3@krue.ml>
Reviewed-by: Chris Müller <typo3@krue.ml>

In relation mode "manyToMany" it is okay to have
a limit of one.

Code-wise the FlexForm code is slightly simplified
as there is no support for the "manytoMany" case.

Resolves: #103188
Releases: main, 12.4, 11.5
Change-Id: Ib2bbafadefc39f603a9ec4fda20aebcee85cdd7a
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83112


Tested-by: Oliver Bartsch <bo@cedev.de>
Tested-by: core-ci <typo3@b13.com>
Reviewed-by: Oliver Bartsch <bo@cedev.de>

Using timestamps as array keys in form yaml configuration
can result in sparse [1] JavaScript arrays which contain
"empty slots".

Example yaml configuration:

  templateRootPaths:
    1708504268: 'EXT:example/Resources/Private/Emails/Templates'

This value is properly serialized into an object by PHP json_encode
(as it does not contain sequential integer keys starting at 0), but
the FormEditor data model produces arrays from numerical keys during
clone operations of such data sets and basically creates sparse arrays
like:

> const example = []
> example[1708504268] = 'EXT:example/Resources/Private/Emails/Templates'
> console.log(example)
[
  <1708504268 empty items>,
  'EXT:forms_extended/Resources/Private/Emails/Templates'
]

Some array-looping operations like Array.reduce() will loop through all
empty slots [1] and therefore preduce a lot of processing load
(1708504269 loops for the example above) that ultimately cause freezes.

The array to object copy operation is now replaced by a modern spread
operation that is defined to skip empty slots.

[1] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Indexed_collections#sparse_arrays

Releases: main, 12.4, 11.5
Resolves: #103168
Change-Id: I10b586ff5c5b0fb3698bfbbe28b2f93d932b71a2
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83079


Reviewed-by: Benjamin Franzke <ben@bnf.dev>
Tested-by: Benjamin Franzke <ben@bnf.dev>
Tested-by: core-ci <typo3@b13.com>
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83082