Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
New check #2570285: Make sure TranslatableMarkup accepts string values only introduced just before RC1 breaks our installation procedure with error:
exception 'InvalidArgumentException' with message '$string ("Your new site !site has been created.") must be a string.' in /var/aegir/platforms/drupal-8.0.0-rc1/core/lib/Drupal/Core/StringTranslation/TranslatableMarkup.php:83
Comments
Comment #2
omega8cc CreditAttribution: omega8cc commentedRelated Drush discussion: https://github.com/drush-ops/drush/issues/1637#issuecomment-143784752
Comment #3
omega8cc CreditAttribution: omega8cc commentedIt started in Drupal here: #2506427: [meta] !placeholder causes strings to be escaped and makes the sanitization API harder to understand
Comment #4
omega8cc CreditAttribution: omega8cc commentedI should add that we are testing this using latest Drush 8 (master), so we are probably affected by https://github.com/drush-ops/drush/pull/1647
Comment #5
omega8cc CreditAttribution: omega8cc commentedWe should get rid of any remaining
st()
andt()
and use Drush wrapperdt()
everywhere. This fixes the problem. The only remaining cosmetic issue is that!site
gets forward slashes on both sides in the mail body. This doesn't affect mail subject, though. No idea, why?Comment #6
omega8cc CreditAttribution: omega8cc commentedComment #7
omega8cc CreditAttribution: omega8cc commentedBetter Title.
Comment #9
omega8cc CreditAttribution: omega8cc commentedProblem fixed.
Comment #11
gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commented