FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix task execution status update not saved by dev-newvisibility · Pull Request #47 · php-task/php-task · GitHub

Fix task execution status update not saved - #47

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

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

Conversation

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 merged commit 9411dfc into php-task:master Jun 1, 2023

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

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL