Skip to content

fix(logging): Fix query logging with DateTimeImmutable parameters #54230

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
Merged
merged 1 commit into from
Aug 5, 2025
Merged

fix(logging): Fix query logging with DateTimeImmutable parameters #54230

merged 1 commit into from
Aug 5, 2025

Conversation

@nickvergessen
Copy link
Member

Checklist

@nickvergessen
fix(logging): Fix query logging with DateTimeImmutable parameters
c722f8c 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added this to the Nextcloud 32 milestone Aug 4, 2025
@nickvergessen nickvergessen self-assigned this Aug 4, 2025
@nickvergessen nickvergessen requested a review from a team as a code owner August 4, 2025 06:49
@nickvergessen nickvergessen requested review from yemkareems and removed request for a team August 4, 2025 06:49
@nickvergessen nickvergessen added 3. to review Waiting for reviews feature: database Database related DB labels Aug 4, 2025
@nickvergessen
Copy link
Member Author

/backport to stable31

@nickvergessen
Copy link
Member Author

/backport to stable30

@nickvergessen nickvergessen mentioned this pull request Aug 4, 2025
Closed
@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 4, 2025
@nickvergessen nickvergessen merged commit c9fa1db into master Aug 5, 2025
200 of 205 checks passed
@nickvergessen nickvergessen deleted the bugfix/spreed-15625/log-query-with-immutable-dates branch August 5, 2025 10:29
This was referenced Aug 5, 2025
Merged
Merged
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
Merged
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
@skjnldsv skjnldsv added this to the Nextcloud 32 milestone Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ChristophWurst ChristophWurst ChristophWurst approved these changes

@kesselb kesselb kesselb approved these changes

@miaulalala miaulalala Awaiting requested review from miaulalala

@come-nc come-nc Awaiting requested review from come-nc

@yemkareems yemkareems Awaiting requested review from yemkareems yemkareems is a code owner automatically assigned from nextcloud/server-backend

@provokateurin provokateurin Awaiting requested review from provokateurin provokateurin is a code owner automatically assigned from nextcloud/server-backend

Assignees

@nickvergessen nickvergessen

Labels

4. to release Ready to be released and/or waiting for tests to finish bug feature: database Database related DB

Projects

None yet

Milestone

Nextcloud 32

Development

Successfully merging this pull request may close these issues.

Object of class DateTimeImmutable could not be converted to string

5 participants

@nickvergessen @ChristophWurst @kesselb @skjnldsv