Drupal 8.0.2 on Pantheon
Environmental Indicator 3.x-dev (2015-Dec-14)

Steps to reproduce:

1. Enable module.
2. Go to /admin/config/development/environment-indicator.
3. Save the configuration.

Error message:

Warning: htmlspecialchars() expects parameter 1 to be string, object given in Drupal\Component\Utility\Html::escape() (line 407 of core/lib/Drupal/Component/Utility/Html.php).

CommentFileSizeAuthor
#4 d8-environment_indicator-warning_htmlspecialchars-2661742-4.patch4.09 KBtibezh
#2 environment_indicator-htmlspecialchars_expects_string-2661742.patch3.76 KBakeemw
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ptmkenny created an issue. See original summary.

akeemw’s picture

akeemw CreditAttribution: akeemw at OHO Interactive commented
Status: Active » Needs review
FileSize
environment_indicator-htmlspecialchars_expects_string-2661742.patch3.76 KB

It looks like the returned URLs needed to be converted into strings. Attaching patch that worked for us.

ericduran’s picture

ericduran CreditAttribution: ericduran at NBCUniversal commented
Status: Needs review » Needs work

@akeemw there is also another issue here. The variable`@permissions` should be changed to `:permissions` same goes for @settings.

tibezh’s picture

tibezh CreditAttribution: tibezh at OPTASY, Drupal Ukraine Community commented
Status: Needs work » Needs review
FileSize
d8-environment_indicator-warning_htmlspecialchars-2661742-4.patch4.09 KB

Here is a patch for the issue

e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
CreditAttribution: e0ipso as a volunteer and at Lullabot commented
Status: Needs review » Fixed

Thanks for contributing to move this module forward.

  • e0ipso committed ff3d29e on 8.x-3.x authored by akeemw 
    Issue #2661742 by tibezh, akeemw, ptmkenny, ericduran, e0ipso: Warning:...

Status: Fixed » Closed (fixed)

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