Skip to content

Commit 0155f0c

Browse files
Merge pull request #6366 from christianbeeznest/fixes-updates96
Internal: Fix undefined variable and null access issues in SocialController
2 parents 7f4fc55 + 46c1ba1 commit 0155f0c

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

src/CoreBundle/Controller/SocialController.php

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -479,12 +479,15 @@ public function getUserProfile(
479479
): JsonResponse {
480480
$user = $userRepository->find($userId);
481481
if (!$user) {
482-
return $this->createNotFoundException('User not found');
482+
throw $this->createNotFoundException('User not found');
483483
}
484484

485485
$baseUrl = $requestStack->getCurrentRequest()->getBaseUrl();
486486
$profileFieldsVisibilityJson = $settingsManager->getSetting('profile.profile_fields_visibility');
487-
$profileFieldsVisibility = json_decode($profileFieldsVisibilityJson, true)['options'] ?? [];
487+
$decoded = json_decode($profileFieldsVisibilityJson, true);
488+
$profileFieldsVisibility = (is_array($decoded) && isset($decoded['options']))
489+
? $decoded['options']
490+
: [];
488491

489492
$vCardUserLink = $profileFieldsVisibility['vcard'] ?? true ? $baseUrl.'/main/social/vcard_export.php?userId='.(int) $userId : '';
490493

@@ -529,7 +532,10 @@ private function getExtraFieldBlock(
529532
}
530533

531534
$fieldVisibilityConfig = $settingsManager->getSetting('profile.profile_fields_visibility');
532-
$fieldVisibility = ($fieldVisibilityConfig && 'false' !== $fieldVisibilityConfig) ? json_decode($fieldVisibilityConfig, true)['options'] : [];
535+
$decoded = json_decode($fieldVisibilityConfig, true);
536+
$fieldVisibility = (is_array($decoded) && isset($decoded['options']))
537+
? $decoded['options']
538+
: [];
533539

534540
$extraUserData = $userRepository->getExtraUserData($userId);
535541
$extraFieldsFormatted = [];

0 commit comments

Comments
 (0)