Skip to content

fix substr cutting special character - #54

Open
peetCreative wants to merge 1 commit into
Project60:masterfrom
peetCreative:fix_substr
Open

fix substr cutting special character#54
peetCreative wants to merge 1 commit into
Project60:masterfrom
peetCreative:fix_substr

Conversation

@peetCreative

Copy link
Copy Markdown

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

'source' => substr($contribution['contribution_source'], 0, 64),

and the mechanism breaks. This PR replaces all occurences of substr.

@mattsches

Copy link
Copy Markdown

Thanks for your PR! The change in

'source' => substr($contribution['contribution_source'], 0, 64),

makes sense IMHO, but replacing the occurrences of substr() in the two other files doesn't.

Could you please edit your PR accordingly?

@vurt2

vurt2 commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Good catch! But @mattsches observation is correct: @peetCreative Can you please change the PR only affection the file SDDNGPostProcessor.php.

@peetCreative

Copy link
Copy Markdown
Author

I updated the commit.

@vurt2

vurt2 commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

@peetCreative Thank you.

@mhodzn

mhodzn commented Aug 21, 2026

Copy link
Copy Markdown

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!

@Detsieber

Copy link
Copy Markdown
Member

I am sorry - @vurt2 (Martin Correll) who is in charge of that, will be on vacation until Aug 28.
Will it be sufficient to wait until then?

@mhodzn

mhodzn commented Aug 25, 2026

Copy link
Copy Markdown

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants