Skip to content
Snippets Groups Projects
Commit 67674589 authored by catch's avatar catch
Browse files

Issue #2856598 by apkwilson, NickDickinsonWilde, ranjith_kumar_k_u, adhariwal,...

Issue #2856598 by apkwilson, NickDickinsonWilde, ranjith_kumar_k_u, adhariwal, yogeshmpawar, smustgrave, quietone: Views field rewrite replacement subtoken yields double encoded HTML entities
parent fa1d8ce3
No related branches found
No related tags found
31 merge requests!11131[10.4.x-only-DO-NOT-MERGE]: Issue ##2842525 Ajax attached to Views exposed filter form does not trigger callbacks,!9470[10.3.x-only-DO-NOT-MERGE]: #3331771 Fix file_get_contents(): Passing null to parameter,!8540Issue #3457061: Bootstrap Modal dialog Not closing after 10.3.0 Update,!8528Issue #3456871 by Tim Bozeman: Support NULL services,!8373Issue #3427374 by danflanagan8, Vighneshh: taxonomy_tid ViewsArgumentDefault...,!7526Expose roles in response,!7352Draft: Resolve #3203489 "Set filename as",!6791Issue #3163299: Ajax exposed filters not working for multiple instances of the same Views block placed on one page,!3878Removed unused condition head title for views,!3818Issue #2140179: $entity->original gets stale between updates,!3742Issue #3328429: Create item list field formatter for displaying ordered and unordered lists,!3731Claro: role=button on status report items,!3668Resolve #3347842 "Deprecate the trusted",!3651Issue #3347736: Create new SDC component for Olivero (header-search),!3531Issue #3336994: StringFormatter always displays links to entity even if the user in context does not have access,!3355Issue #3209129: Scrolling problems when adding a block via layout builder,!3154Fixes #2987987 - CSRF token validation broken on routes with optional parameters.,!3133core/modules/system/css/components/hidden.module.css,!2812Issue #3312049: [Followup] Fix Drupal.Commenting.FunctionComment.MissingReturnType returns for NULL,!2794Issue #3100732: Allow specifying `meta` data on JSON:API objects,!2378Issue #2875033: Optimize joins and table selection in SQL entity query implementation,!2334Issue #3228209: Add hasRole() method to AccountInterface,!2062Issue #3246454: Add weekly granularity to views date sort,!1105Issue #3025039: New non translatable field on translatable content throws error,!1073issue #3191727: Focus states on mobile second level navigation items fixed,!10223132456: Fix issue where views instances are emptied before an ajax request is complete,!877Issue #2708101: Default value for link text is not saved,!617Issue #3043725: Provide a Entity Handler for user cancelation,!579Issue #2230909: Simple decimals fail to pass validation,!560Move callback classRemove outside of the loop,!555Issue #3202493
Pipeline #104241 passed with warnings
Pipeline: drupal

#104248

    ......@@ -986,7 +986,7 @@ protected function addSelfTokens(&$tokens, $item) {
    if (is_array($raw)) {
    if (isset($raw[$id]) && is_scalar($raw[$id])) {
    $tokens['{{ ' . $this->options['id'] . '__' . $id . ' }}'] = Xss::filterAdmin($raw[$id]);
    $tokens['{{ ' . $this->options['id'] . '__' . $id . ' }}'] = $raw[$id];
    }
    else {
    // Make sure that empty values are replaced as well.
    ......@@ -999,7 +999,7 @@ protected function addSelfTokens(&$tokens, $item) {
    // Check if TypedDataInterface is implemented so we know how to render
    // the item as a string.
    if (!empty($property) && $property instanceof TypedDataInterface) {
    $tokens['{{ ' . $this->options['id'] . '__' . $id . ' }}'] = Xss::filterAdmin($property->getString());
    $tokens['{{ ' . $this->options['id'] . '__' . $id . ' }}'] = $property->getString();
    }
    else {
    // Make sure that empty values are replaced as well.
    ......
    langcode: en
    status: true
    dependencies:
    module:
    - node
    id: test_field_self_tokens
    label: ''
    module: views
    description: ''
    tag: ''
    base_table: node_field_data
    base_field: nid
    display:
    default:
    display_plugin: default
    id: default
    display_title: Master
    position: 0
    display_options:
    pager:
    type: none
    options:
    offset: 0
    row:
    type: fields
    fields:
    title:
    id: title
    table: node_field_data
    field: title
    entity_type: node
    entity_field: title
    plugin_id: field
    <?php
    namespace Drupal\Tests\views\Kernel\Handler;
    use Drupal\Core\Render\RenderContext;
    use Drupal\node\Entity\Node;
    use Drupal\node\Entity\NodeType;
    use Drupal\Tests\views\Kernel\ViewsKernelTestBase;
    use Drupal\views\Views;
    /**
    * Tests token escaping in the EntityField handler.
    *
    * @group views
    */
    class FieldSelfTokensTest extends ViewsKernelTestBase {
    /**
    * {@inheritdoc}
    */
    protected static $modules = ['node'];
    /**
    * Views used by this test.
    *
    * @var array
    */
    public static $testViews = ['test_field_self_tokens'];
    /**
    * This method is called before each test.
    */
    protected function setUp($import_test_views = TRUE): void {
    parent::setUp();
    $this->installEntitySchema('user');
    $this->installEntitySchema('node');
    NodeType::create(['type' => 'article', 'name' => 'Article'])->save();
    Node::create([
    'title' => 'Questions & Answers',
    'type' => 'article',
    ])->save();
    }
    /**
    * {@inheritdoc}
    */
    public function testSelfTokenEscaping() {
    $view = Views::getView('test_field_self_tokens');
    $view->initHandlers();
    $this->executeView($view);
    $row = $view->result[0];
    $title_field = $view->field['title'];
    $title_field->options['alter']['text'] = '<p>{{ title__value }}</p>';
    $title_field->options['alter']['alter_text'] = TRUE;
    $output = \Drupal::service('renderer')->executeInRenderContext(new RenderContext(), function () use ($title_field, $row) {
    return $title_field->theme($row);
    });
    $this->assertSame('<p>Questions &amp; Answers</p>', (string) $output);
    }
    }
    • catch @catch

      mentioned in commit 615c69be

      ·

      mentioned in commit 615c69be

      Toggle commit list
    0% Loading or .
    You are about to add 0 people to the discussion. Proceed with caution.
    Finish editing this message first!
    Please register or to comment