Problem/Motivation

We have an old PHP 5.3 production server and while we await our shiny new PHP 7.x server we'd like to use this module and only one function call seems to break our old clunker.

Proposed resolution

Swap the call to mb_substr with drupal_substr

CommentFileSizeAuthor
#2 3003199-2-mb_substr.patch505 bytesjoelpittet

Comments

joelpittet created an issue. See original summary.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Active » Needs review
StatusFileSize
new 3003199-2-mb_substr.patch505 bytes

Here's a one line patch to solve our woes. Please take pity on our infra 🙏

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Reviewed & tested by the community

RTBC drupal_substr() automatically checks if the unicode library is installed and provides a fallback if the library is missing.

  • joelpittet committed 8291c21 on 7.x-2.x 
    Issue #3003199 by joelpittet, x11: mb_substr() fatal error when mbstring...
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Reviewed & tested by the community » Fixed

Drupal 8 uses mb_substr in core so it's not needed for 8.x branch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.