Skip to content

Creation of dynamic property Cake\Database\Log\LoggedQuery::$connection is deprecated #18680

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
BGehrels opened this issue May 20, 2025 · 1 comment · Fixed by #18684
Closed

Creation of dynamic property Cake\Database\Log\LoggedQuery::$connection is deprecated #18680

BGehrels opened this issue May 20, 2025 · 1 comment · Fixed by #18684
Assignees
@markstory
Labels
defect
Milestone
5.2.5

Comments

@BGehrels
Copy link
Contributor

Description

The ReconnectStrategy:113 adds ['connection' => $this->connection->configName()] as a context to getDriver()->log(). The Driver will then pass this on to $loggedQuery->setContext(). The later will blindly write each context key to a property, but LoggedQuery has no connection property. This then leads to the warning

Creation of dynamic property Cake\Database\Log\LoggedQuery::$connection is deprecated

CakePHP Version

5.2.4

PHP Version

8.4
@markstory markstory added this to the 5.2.5 milestone May 20, 2025
@markstory markstory self-assigned this May 20, 2025
@markstory markstory mentioned this issue May 21, 2025
Merged
@markstory
Copy link
Member

markstory commented May 21, 2025 •
edited
Loading

Thank you for a great bug report 👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@markstory markstory

Labels
defect
Projects
None yet
Milestone
5.2.5
Development

Successfully merging a pull request may close this issue.

Only apply defined properties to LoggedQuery cakephp/cakephp
2 participants
@markstory @BGehrels
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy