Merge pull request #30 from Flowpack/bugfix-no-account-found-in-security-context-causes-crash

JamesAlias · web-flow · commit f75bdb81edf8 · 2023-07-03T11:54:04.000+02:00
BUGFIX: Prevent crash when SecurityContext has no Account
diff --git a/Classes/ContentReleaseManager.php b/Classes/ContentReleaseManager.php
@@ -144,7 +144,7 @@ private function resolveCurrentContentReleaseId(?string $currentContentReleaseId
 
     private function getAccountId(): ?string
     {
-        if ($this->securityContext->isInitialized()) {
+        if ($this->securityContext->isInitialized() && !is_null($this->securityContext->getAccount())) {
             return $this->securityContext->getAccount()->getAccountIdentifier();
         }
 

Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,7 @@ private function resolveCurrentContentReleaseId(?string $currentContentReleaseId`
`144`	`144`
`145`	`145`	`private function getAccountId(): ?string`
`146`	`146`	`{`
`147`		`- if ($this->securityContext->isInitialized()) {`
	`147`	`+ if ($this->securityContext->isInitialized() && !is_null($this->securityContext->getAccount())) {`
`148`	`148`	`return $this->securityContext->getAccount()->getAccountIdentifier();`
`149`	`149`	`}`
`150`	`150`