Skip to content

Fix task execution status update not saved#47

Merged
wachterjohannes merged 1 commit intophp-task:masterfrom
dev-newvisibility:master
Jun 1, 2023
Merged

Fix task execution status update not saved#47
wachterjohannes merged 1 commit intophp-task:masterfrom
dev-newvisibility:master

Conversation

@dev-newvisibility
Copy link
Copy Markdown
Contributor

This PR fixes passed and failed status not persisted correctly in some circumstances.

This may occur, for example, if you use Doctrine as persistent storage and you globally set the Query::HINT_REFRESH query hint to true. In

$execution = $this->taskExecutionRepository->findByUuid($execution->getUuid());
a fresh entry from the database is fetched and it has the previous 'running' status.

@wachterjohannes wachterjohannes merged commit 9411dfc into php-task:master Jun 1, 2023
@wachterjohannes
Copy link
Copy Markdown
Member

@dev-newvisibility thanks for our contribution - have already tagged a new release https://github.com/php-task/php-task/releases/tag/2.0.1

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