Skip to content

Commit eeba74b

Browse files
committed
Ensure address is added to both digital post pdf and fjernprint pdf
1 parent a99952c commit eeba74b

2 files changed

Lines changed: 20 additions & 7 deletions

File tree

modules/os2forms_digital_post/src/EventSubscriber/Os2formsDigitalPostSubscriber.php

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -101,14 +101,16 @@ public function setDigitalPostContext(WebformSubmissionInterface $submission, Co
101101
public function getDigitalPostContext(WebformSubmissionInterface $submission): CompanyLookupResult|CprLookupResult|null {
102102
$key = $this->createSessionKeyFromSubmission($submission);
103103

104-
$digitalPostContext = $this->session->get($key);
104+
return $this->session->get($key);
105+
}
105106

106-
// We only need/use it once, so just remove it after fetching it.
107-
if ($digitalPostContext) {
108-
$this->session->remove($key);
109-
}
107+
/**
108+
* Delete Digital Post context from the current request.
109+
*/
110+
public function deleteDigitalPostContext(WebformSubmissionInterface $submission): bool {
111+
$key = $this->createSessionKeyFromSubmission($submission);
110112

111-
return $digitalPostContext;
113+
return (bool) $this->session->remove($key);
112114
}
113115

114116
/**

modules/os2forms_digital_post/src/Helper/WebformHelperSF1601.php

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ public function sendDigitalPost(WebformSubmissionInterface $submission, array $h
156156
$recipientIdentifierType = 'CPR';
157157
}
158158

159-
$this->digitalPostSubscriber->setDigitalPostContext($submission, $lookupResult);
159+
160160

161161
$senderSettings = $this->settings->getSender();
162162
$messageOptions = [
@@ -169,7 +169,13 @@ public function sendDigitalPost(WebformSubmissionInterface $submission, array $h
169169
WebformHandlerSF1601::SENDER_LABEL => $handlerMessageSettings[WebformHandlerSF1601::SENDER_LABEL],
170170
WebformHandlerSF1601::MESSAGE_HEADER_LABEL => $handlerMessageSettings[WebformHandlerSF1601::MESSAGE_HEADER_LABEL],
171171
];
172+
173+
// Set flag indicating digital post context.
174+
$this->digitalPostSubscriber->setDigitalPostContext($submission, $lookupResult);
172175
$message = $this->meMoHelper->buildWebformSubmissionMessage($submission, $messageOptions, $handlerSettings, $lookupResult);
176+
// Remove flag.
177+
$this->digitalPostSubscriber->deleteDigitalPostContext($submission);
178+
173179
$forsendelseOptions = [
174180
self::RECIPIENT_IDENTIFIER_TYPE => $recipientIdentifierType,
175181
self::RECIPIENT_IDENTIFIER => $recipientIdentifier,
@@ -181,7 +187,12 @@ public function sendDigitalPost(WebformSubmissionInterface $submission, array $h
181187
WebformHandlerSF1601::SENDER_LABEL => $handlerMessageSettings[WebformHandlerSF1601::SENDER_LABEL],
182188
WebformHandlerSF1601::MESSAGE_HEADER_LABEL => $handlerMessageSettings[WebformHandlerSF1601::MESSAGE_HEADER_LABEL],
183189
];
190+
191+
// Set flag indicating digital post context.
192+
$this->digitalPostSubscriber->setDigitalPostContext($submission, $lookupResult);
184193
$forsendelse = $this->forsendelseHelper->buildSubmissionForsendelse($submission, $forsendelseOptions, $handlerSettings, $lookupResult);
194+
// Remove flag.
195+
$this->digitalPostSubscriber->deleteDigitalPostContext($submission);
185196

186197
$type = $handlerMessageSettings[WebformHandlerSF1601::TYPE] ?? SF1601::TYPE_DIGITAL_POST;
187198

0 commit comments

Comments
 (0)