Skip to content

Commit abdfa66

Browse files
authored
Merge pull request #3397 from codeeu/dev
Dev
2 parents ed9a0f8 + 7d9fe77 commit abdfa66

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

app/Http/Controllers/CertificateBackendController.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -421,7 +421,10 @@ public function manualCreateSend(Request $request): JsonResponse
421421
$name = $excellence->name_for_certificate ?? trim($user->firstname . ' ' . $user->lastname);
422422
$numberOfActivities = $type === 'SuperOrganiser' ? $user->activities($edition) : 0;
423423

424-
if (! $sendOnly && ! $excellence->certificate_url) {
424+
// Manual "generate only" should always regenerate, even if a URL already exists.
425+
// For combined flow, only generate when certificate is missing.
426+
$shouldGenerate = ! $sendOnly && ($generateOnly || ! $excellence->certificate_url);
427+
if ($shouldGenerate) {
425428
try {
426429
$cert = new \App\CertificateExcellence(
427430
$edition,

0 commit comments

Comments
 (0)