search

cypht-org / cypht

Cypht: Lightweight Open Source webmail aggregator [PHP, JS]
http://cypht.org
GNU Lesser General Public License v2.1
979 stars 154 forks source link

[FIX]Profiles: Fix duplicated profile for the same email adress #1096

Closed Baraka24 closed 1 month ago

Baraka24 commented 3 months ago

Related issue

Baraka24 commented 2 months ago

@Shadow243 Thanks for a kind review, I did not figure out some points, you make them clear to me and I applied changes.

Shadow243 commented 2 months ago

@Shadow243 Thanks for a kind review, I did not figure out some points, you make them clear to me and I applied changes.

Perfect. What about 'replyto' ? I think we can add OR in that condition. What do you think about that? The idea is to prevent duplication with 'replyto' field.

Baraka24 commented 2 months ago

What about 'replyto' ? I think we can add OR in that condition. What do you think about that? The idea is to prevent duplication with 'replyto' field.

Adding the 'replyto' field to the condition makes sense as we want to ensure that no profiles are duplicated based on the 'replyto' field as well.