Problem/Motivation

strlen() is deprecated.
File: commerce_autosku/src/Plugin/CommerceAutoSkuGenerator/CommerceAutoSkuGeneratorBase.php
See: https://www.drupal.org/node/2850048

Proposed resolution

Use mb_strlen() instead.

Remaining tasks

  1. Create patch
  2. Needs Review
  3. Commit

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 commerce_autosku-strlen_fixes-3027832-2.patch1.26 KBi-trokhanenko

Comments

i-trokhanenko created an issue. See original summary.

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new commerce_autosku-strlen_fixes-3027832-2.patch1.26 KB

Please review.

alex.tiupa’s picture

alex.tiupa
Location Lutsk, Ukraine
commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Patch #2 commerce_autosku-strlen_fixes-3027832-2.patch reviewed and works good.
Please commit it.

bojanz’s picture

bojanz commented
Status: Reviewed & tested by the community » Fixed

Committed, thanks.

Status: Fixed » Closed (fixed)

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