- First commit to issue fork.
SafeStringInterface
is badly named for two reasons:
Rename SafeStringInterface to MarkupInterface
and move it to Drupal\Component\Render
and do quite a few other changes to make this consistent:
Drupal\Component\Utility\FormattableString
to Drupal\Component\Render\FormattableMarkup
Drupal\Core\StringTranslation\TranslatableString
to Drupal\Core\StringTranslation\TranslatableMarkup
Drupal\Component\Utility\PlainTextOutput
to Drupal\Component\Render\PlainTextOutput
Drupal\Component\Utility\SafeStringTrait
to Drupal\Component\Render\MarkupTrait
Drupal\Core\Render\SafeString
to Drupal\Core\Render\Markup
Drupal\Core\Field\FieldFilteredString
to Drupal\Core\Field\FieldFilteredMarkup
Drupal\filter\Render\FilteredString
to Drupal\filter\Render\FilteredMarkup
Drupal\views\Render\ViewsRenderPipelineSafeString
to Drupal\views\Render\ViewsRenderPipelineMarkup
The basic reason why this change makes sense is that it clarifies that the purpose of the entire render system is to produce markup. Another reason is that aligns the names with Twig_Markup
a class that Twig provides something similar.
Is the disruption worth it? @alexpott thinks so - the more clarity this system has the better.
Commit
No
Yes, some recently introduced classes will be renamed, as well as some classes that had existed for longer but are confusingly/harmfully named.
MarkupInterface is not @internal anymore
Fixed
8.0 β°οΈ
base system
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.