fix substr cutting special character - #54
Conversation
|
Thanks for your PR! The change in makes sense IMHO, but replacing the occurrences of Could you please edit your PR accordingly? |
|
Good catch! But @mattsches observation is correct: @peetCreative Can you please change the PR only affection the file |
|
I updated the commit. |
|
@peetCreative Thank you. |
|
I'm taking over for @peetCreative at the moment. Can we expect the pull request to be merged to master soon? That would be very helpful for our project! |
|
I am sorry - @vurt2 (Martin Correll) who is in charge of that, will be on vacation until Aug 28. |
|
Yes, that is fine. We have implemented the PR fix in our project for now, I just wanted to follow up. Thanks for letting me know! |
substr cuts special character in half. So one should replace this function with mb_substring.
Example: A event has the name and the event title contains ß so that it becomes the 64 th character of the contribution source. Then it gets cut in
org.project60.sepapp/CRM/Core/Payment/SDDNGPostProcessor.php
Line 111 in a676114
and the mechanism breaks. This PR replaces all occurences of
substr.