Skip to content

[stable30] fix(maintenance): Show a success message on data-fingerprint command#49591

Merged
skjnldsv merged 1 commit intostable30from
backport/49587/stable30
Dec 12, 2024
Merged

[stable30] fix(maintenance): Show a success message on data-fingerprint command#49591
skjnldsv merged 1 commit intostable30from
backport/49587/stable30

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Dec 2, 2024

Backport of PR #49587

Signed-off-by: Joas Schilling <coding@schilljs.com>

protected function execute(InputInterface $input, OutputInterface $output): int {
$this->config->setSystemValue('data-fingerprint', md5($this->timeFactory->getTime()));
$fingerPrint = md5($this->timeFactory->getTime());

Check notice

Code scanning / Psalm

InvalidScalarArgument

Argument 1 of md5 expects string, but int provided
@blizzz blizzz mentioned this pull request Dec 5, 2024
This was referenced Dec 9, 2024
@skjnldsv skjnldsv merged commit 3ea30b4 into stable30 Dec 12, 2024
@skjnldsv skjnldsv deleted the backport/49587/stable30 branch December 12, 2024 12:24
@blizzz blizzz mentioned this pull request Jan 8, 2025
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants