Commit 431134e

committed

Site Logo code - fix coding standards issues

2 parents cfd247e + 4467d78 commit 431134eCopy full SHA for 431134e

File tree

+11

-2

lines changed

+11

-2

lines changed

Lines changed: 11 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -196,7 +196,11 @@ private function checkAndUploadToCloudinary($logoPath, $publicId, $originalUrl)`
`196`	`196`
`197`	`197`	`// Clean up temp file if it was downloaded`
`198`	`198`	`if (strpos($logoFullPath, '/tmp/') !== false) {`
`199`		`- @unlink($logoFullPath);`
	`199`	`+ try {`
	`200`	`+ unlink($logoFullPath);`
	`201`	`+ } catch (\Exception $unlinkException) {`
	`202`	`+ $this->logger->debug('Could not delete temporary logo file: ' . $unlinkException->getMessage());`
	`203`	`+ }`
`200`	`204`	`}`
`201`	`205`
`202`	`206`	`return $uploadResult['secure_url'];`
`@@ -274,7 +278,12 @@ private function verifyImageAccessibility($url)`
`274`	`278`	`]`
`275`	`279`	`]);`
`276`	`280`
`277`		`- $headers = @get_headers($url, 1, $context);`
	`281`	`+ $headers = false;`
	`282`	`+ try {`
	`283`	`+ $headers = get_headers($url, 1, $context);`
	`284`	`+ } catch (\Exception $headerException) {`
	`285`	`+ $this->logger->debug('Could not retrieve headers for URL: ' . $headerException->getMessage());`
	`286`	`+ }`
`278`	`287`
`279`	`288`	`if ($headers && isset($headers[0])) {`
`280`	`289`	`// Check for successful HTTP status codes (200, 201, etc.)`

Comments

(0)