search

uvdesk / mailbox-component

The Mailbox component provides tools that allows your helpdesk to process tickets from multiple email sources.
MIT License
22 stars 38 forks source link

email are not converted into ticket as mailbox is case sensitive #64

Closed vaishaliwebkul closed 3 years ago

vaishaliwebkul commented 3 years ago

Description
email are not converted into ticket as mailbox is case sensitive

How to reproduce
as a customer, mail to abc@gmail.com OR Abc@gmail.com, both mails will be received as email id are not sensitive but on UVDesk only a single ticket will be created.

papnoisanjeev commented 3 years ago

@vaishaliwebkul

Fixed in master here.

locate path in your project: vendor/uvdesk/mailbox-component/Services/MailboxService.php

Open MailboxService.php and replace its code from here.

PeopleInside commented 3 years ago

Hi @papnoisanjeev , I see this issue and applied the patch. After no ticket was opened by email.

SharedScreenshot

Did you tested the patch? Hope you don't made updates that broke things. I restored my backup.

I see you suggested to replaced all file content, comparing the file I have and the file patched, the file has been had other changes.. not only the patch you are talking here and the new file broke my uvdesk maybe because you are using a most recent not stable version. Don't know.

MailboxService.php.txt MailboxServiceedit.php.txt

papnoisanjeev commented 3 years ago

@PeopleInside

Clear your project cache:

php bin/console c:c

PeopleInside commented 3 years ago

Have you tested: tried to open ticket by email? In my test did not work... the update broken totally my system as ticket was no more opened by email. I think I cleared cache because I made two updates, one is the fix here and other one I tried to update jQuery but get errors so cache was clean.

I opened a new issue asking you to see how update jQuery that has security fix. ( https://github.com/uvdesk/core-framework/issues/352 ) I will not madding changes here, hope when you will release a new version of Uvdesk this update doesn't broken ticket creation by email so I hope the issue I facing testing your fix here was mine and not a bug in the file.

Seems uvdesk open source update are very slow or also you may update here the code that will be no longer matches with code of my installation that is latest one stable released.

Your fix here add one row but if I compare my file with your many different row has been changed... so who know where the issue can be? Maybe in other patch somewhere else you did?

I cannot fully test if you update the code but did not release an updated version of the software. I have also pull request that has not been pulled. https://github.com/uvdesk/core-framework/pull/336 https://github.com/uvdesk/support-center-bundle/pull/106 https://github.com/uvdesk/support-center-bundle/pull/115

vaishaliwebkul commented 3 years ago

I have verified this issue and working fine, please see screenshot https://prnt.sc/10il99t https://prnt.sc/10ilaur