Skip to content

[TASK] Move code from system extension integrity to extension lowlevel#7

Closed
andygrunwald wants to merge 1 commit intoTYPO3:masterfrom
andygrunwald:changeset_14954_1
Closed

[TASK] Move code from system extension integrity to extension lowlevel#7
andygrunwald wants to merge 1 commit intoTYPO3:masterfrom
andygrunwald:changeset_14954_1

Conversation

@andygrunwald
Copy link
Copy Markdown
Contributor

Project: TYPO3v4/Core
Branch: master
Patchset: 1
ID: I22020736e91d7d0079ecf47e9b7d30a758f98e4c
Author: Markus Klein

URL: http://review.typo3.org/14954

This pull request was created automatically to use the awesome TravisCI-Service.
The TYPO3 review process is maintained by Gerrit Code Review.
For further information to this pull request please have a look at the original one: http://review.typo3.org/14954

Please be so nice to does not leave a comment here :)
Thanks.

PS: We love you! ❤️

Resolves: #40950
Releases: 6.0

Change-Id: I22020736e91d7d0079ecf47e9b7d30a758f98e4c
@andygrunwald
Copy link
Copy Markdown
Contributor Author

Just a little bit testing.
Close :)

reviewtypo3org pushed a commit that referenced this pull request Mar 22, 2022
This patch raises doctrine/dbal to 2.13.8 as minimum version,
which contains a bugfix to avoid a native php error is emitted,
stating "mysqli::real_connect(): Passing null to parameter #7".

See: doctrine/dbal#5296

One phpstan ignore pattern slightly changed because of a
changed return type declarion of doctrine/dbal. Tackling
the corresponding error should be done in a dedicated test
after proper investigation.

Used commands:

> composer req doctrine/dbal:^2.13.8
> composer req doctrine/dbal:^2.13.8 \
  -d typo3/sysext/core --no-update
> composer req doctrine/dbal:^2.13.8 \
  -d typo3/sysext/install --no-update
> composer req doctrine/dbal:^2.13.8 \
  -d typo3/sysext/redirects --no-update
> composer req doctrine/dbal:^2.13.8 \
  -d typo3/sysext/core --no-update
> Build/Scripts/runTests.sh -s phpstanGenerateBaseline

Resolves: #97222
Releases: 11.5
Change-Id: I6c1712a792780bd2966b3977d43f767e59304bd5
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74013
Tested-by: core-ci <typo3@b13.com>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Benni Mack <benni@typo3.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants