I just wanted to create a new comment on my test site and received the following error page:

The website encountered an unexpected error. Please try again later.
Error: Call to undefined function Symfony\Component\Validator\Constraints\iconv_strlen() in Symfony\Component\Validator\Constraints\LengthValidator->validate() (line 47 of vendor/symfony/validator/Constraints/LengthValidator.php).

It seems the "iconv" PHP extension is required for Drupal 8 (via the included symfony/validator package), but I couldn't find any place where this is documented. The site's "Status report" doesn't report any problems, the relevant handbook page also makes no mention of it.

CommentFileSizeAuthor
#6 2678574-4.patch843.45 KBdawehner
#4 2678574-4.patch383 KBdawehner

Comments

drunken monkey created an issue. See original summary.

dawehner’s picture

dawehner
Primary language German
commented

Urgs, this changed with #2611816: Update to symfony 2.8
Wow: https://github.com/symfony/symfony/pull/16856

dawehner’s picture

dawehner
Primary language German
commented

I see, so we should add the new polyfill, see https://github.com/symfony/symfony/pull/16856/files

dawehner’s picture

dawehner
Primary language German
commented
Component: documentation » base system
Status: Active » Needs review
StatusFileSize
new 2678574-4.patch383 KB

There we go.

Status: Needs review » Needs work

The last submitted patch, 4: 2678574-4.patch, failed testing.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs work » Needs review
StatusFileSize
new 2678574-4.patch843.45 KB

This time with --binary

jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Status: Needs review » Reviewed & tested by the community

Looks great. Thanks.

catch’s picture

catch
he/him
Primary language English
commented
Title: Required PHP "iconv" extension is nowhere documented » Symfony 2.8 requirements change: required PHP "iconv" extension is nowhere documented
Priority: Normal » Major

Should we be recommending iconv in hook_requirements() as well? Does not have to happen here.

catch’s picture

catch
he/him
Primary language English
commented
Issue tags: +beta target
dawehner’s picture

dawehner
Primary language German
commented

Yeah, why not

Version: 8.1.x-dev » 8.2.x-dev

Drupal 8.1.0-beta1 was released on March 2, 2016, which means new developments and disruptive changes should now be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

tic2000’s picture

tic2000 commented
Version: 8.2.x-dev » 8.1.x-dev

I think this should be in 8.1.x

  • catch committed 0bbd5d3 on 8.2.x 
    Issue #2678574 by dawehner: Symfony 2.8 requirements change: required...

  • catch committed b6320a6 on 8.1.x 
    Issue #2678574 by dawehner: Symfony 2.8 requirements change: required...
catch’s picture

catch
he/him
Primary language English
commented
Priority: Major » Critical
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.2.x and cherry-picked to 8.1.x. Thanks!

Also retrospectively bumping this to critical since it would have been a fatal error updating between 8.0.x and 8.1.x for sites without the extension.

Status: Fixed » Closed (fixed)

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

mr.baileys’s picture

mr.baileys
Primary language Dutch
Location 🇧🇪 (Ghent)
commented

Added #2703495: Simplify Unicode::convertToUtf8() now that core has iconv polyfill. as a follow-up.