Fix #10486 - Fix filtering in Email queue list view

[SinergiaCRM]

Description

This PR adds the name of the organizations in the filter by Recipient Name and modifies the query that is executed when querying by Recipient Email.

How To Test This

1. Create a campaign with contacts, accounts, leads and users.
2. Send one or more marketing emails.
3. Access the message queue view and check:

• The message related to the account recipient is returned when filtering by Recipient Name
• The corresponding messages are returned when using the Recipient Email filter

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Final checklist

• My code follows the code style of this project found here.
• My change requires a change to the documentation.
• I have read the How to Contribute guidelines.

[serhiisamko091184]

Hello @SinergiaCRM,

thanks for your PR!

Regards,
Serhii

[serhiisamko091184]

Hello @SinergiaCRM,

thanks for your contribution,

I'd like to ask is there a reason to apply changes, particularly to a metadata file in this case (subquery on 'to_email' search field)?

I'm asking as in the EmailMan class there's a function 'create_new_list_query', which could be modified to get the result.

Thanks in advance!

Regards,
Serhii

[SinergiaCRM]

Hello @SinergiaCRM,

thanks for your contribution,

I'd like to ask is there a reason to apply changes, particularly to a metadata file in this case (subquery on 'to_email' search field)?

I'm asking as in the EmailMan class there's a function 'create_new_list_query', which could be modified to get the result.

Thanks in advance!

Regards, Serhii

Hi @serhiisamko091184,

No, there is no particular reason. We applied it in the metadata file because it solved the problem.

For now, we understand that you have located the solution in the create_new_list_query() method, so do you apply it? Do we have to do something in the PR?

Regards!