Skip to content

Fix overdue query to use dateTimeParameter (#7868) - #8676

Open
xstable wants to merge 1 commit into
nextcloud:mainfrom
xstable:patch-2
Open

xstable wants to merge 1 commit into
nextcloud:mainfrom
xstable:patch-2

Conversation

@xstable

@xstable xstable commented Nov 4, 2026

Copy link
Copy Markdown

Summary

force to check date against UTC, instead of internal DB-Setting

closes nextcloud#7868
force to check date against UTC, instead of internal DB-Setting

Signed-off-by: XStable <info@xstable.com>

@blizzz blizzz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your attempt!

Comment thread lib/Db/CardMapper.php
$qb->select('id', 'title', 'duedate', 'notified')
->from('deck_cards')
->where($qb->expr()->lt('duedate', $qb->createFunction('NOW()')))
->where($qb->expr()->lt('duedate', $this->dateTimeParameter($qb, $now)))

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$now is not defined here

This branch has not been deployed

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Due-date notifications for Deck cards are triggered at the wrong time

2 participants