[PLA-2023] Fix error logging

[enjinabner]

PR Type

Bug fix

---

Description

• Improved error logging in SyncMetadata by changing the log entry to include the error message in an array format. This enhances the structure and readability of the log.

---

Changes walkthrough 📝

	Relevant files
Bug fix	SyncMetadata.php Improve error logging structure in SyncMetadata                    src/Jobs/SyncMetadata.php Modified error logging to include error message in an array. Improved the structure of the error log entry. +1/-1

---

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Error Handling Ensure that the error logging captures all necessary details of the exception, including stack trace, for better debugging.

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Score
Enhancement	Enhance error logging by including the exception trace Include the exception trace in the error log to provide more detailed debugging information. src/Jobs/SyncMetadata.php [40] -Log::error("Unable to sync metadata for attribute ID {$this->attributeId}", ['error' => $e->getMessage()]); +Log::error("Unable to sync metadata for attribute ID {$this->attributeId}", ['error' => $e->getMessage(), 'trace' => $e->getTraceAsString()]); Suggestion importance[1-10]: 8 Why: Including the exception trace in the error log provides more detailed debugging information, which can be crucial for diagnosing issues. This enhancement improves the quality of logging and aids in faster troubleshooting.	8