fix: cc and bcc fields not set when sending mail due to incorrect condition in SendMail (v1.16.x)

[krishankumar01]

📑 Description

Closes https://github.com/goravel/goravel/issues/

✅ Checks

• Added test cases for my code

[Copilot]

Pull Request Overview

This PR fixes a bug in the SendMail function where CC and BCC fields were not being set correctly due to incorrect conditional checks. The issue was that the code was checking the length of the email struct's CC/BCC fields instead of the input parameters.

• Fixed conditional checks for CC and BCC field assignment to use input parameters instead of email struct fields
• Ensures CC and BCC recipients are properly included when sending emails

[Copilot]

The fix correctly addresses the bug where the condition was checking len(e.Bcc) instead of len(bcc). Since e.Bcc would be empty at this point, the condition would never be true, preventing BCC recipients from being set.

[Copilot]

The fix correctly addresses the bug where the condition was checking len(e.Cc) instead of len(cc). Since e.Cc would be empty at this point, the condition would never be true, preventing CC recipients from being set.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 66.83%. Comparing base (46568ee) to head (368f135).

Files with missing lines	Patch %	Lines
mail/application.go	0.00%	2 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff            @@
##           v1.16.x    #1144   +/-   ##
========================================
  Coverage    66.83%   66.83%           
========================================
  Files          215      215           
  Lines        14071    14071           
========================================
  Hits          9404     9404           
  Misses        4291     4291           
  Partials       376      376           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[github-actions]

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.50.

Benchmark suite	Current: 368f135	Previous: f9670e1	Ratio
Benchmark_DecryptString	6457 ns/op 2032 B/op 16 allocs/op	2123 ns/op 2032 B/op 16 allocs/op	3.04
Benchmark_DecryptString - ns/op	6457 ns/op	2123 ns/op	3.04

This comment was automatically generated by workflow using github-action-benchmark.