Skip to content
Snippets Groups Projects
Verified Commit 12097985 authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3441920 by scott_euser, catch, smustgrave, quietone: Support #attached...

Issue #3441920 by scott_euser, catch, smustgrave, quietone: Support #attached settings in ViewAjaxController

(cherry picked from commit 3e486f78)
parent 6bca41b2
No related branches found
No related tags found
19 merge requests!12212Issue #3445525 by alexpott, japerry, catch, mglaman, longwave: Add BC layer...,!10602Issue #3438769 by vinmayiswamy, antonnavi, michelle, amateescu: Sub workspace does not clear,!10301Issue #3469309 by mstrelan, smustgrave, moshe weitzman: Use one-time login...,!10187Issue #3487488 by dakwamine: ExtensionMimeTypeGuesser::guessMimeType must support file names with "0" (zero) like foo.0.zip,!9929Issue #3445469 by pooja_sharma, smustgrave: Add additional test coverage for...,!9787Resolve issue 3479427 - bootstrap barrio issue under Windows,!9742Issue #3463908 by catch, quietone: Split OptionsFieldUiTest into two,!9526Issue #3458177 by mondrake, catch, quietone, godotislate, longwave, larowlan,...,!8949Backport .gitlabci.yml changes.,!8738Issue #3424162 by camilledavis, dineshkumarbollu, smustgrave: Claro...,!8704Make greek characters available in ckeditor5,!8597Draft: Issue #3442259 by catch, quietone, dww: Reduce time of Migrate Upgrade tests...,!8533Issue #3446962 by kim.pepper: Remove incorrectly added...,!8517Issue #3443748 by NexusNovaz, smustgrave: Testcase creates false positive,!7930Resolve #3427374 "Taxonomytid viewsargumentdefault plugin",!7445Issue #3440169: When using drupalGet(), provide an associative array for $headers,!6502Draft: Resolve #2938524 "Plach testing issue",!38582585169-10.1.x,!3226Issue #2987537: Custom menu link entity type should not declare "bundle" entity key
Pipeline #165067 passed with warnings
Pipeline: drupal

#165094

    Pipeline: drupal

    #165086

      Pipeline: drupal

      #165080

        +1
        ......@@ -214,6 +214,10 @@ public function ajaxView(Request $request) {
        $response->addCommand(new PrependCommand(".js-view-dom-id-$dom_id", ['#type' => 'status_messages']));
        $request->query->set('ajax_page_state', $existing_page_state);
        if (!empty($preview['#attached'])) {
        $response->setAttachments($preview['#attached']);
        }
        return $response;
        }
        else {
        ......
        ......@@ -209,6 +209,14 @@ public function testAjaxView() {
        $this->assertSame($response->getView(), $executable);
        $this->assertViewResultCommand($response);
        // Test that the ajax controller for Views contains the
        // Drupal Settings.
        $this->assertEquals([
        'drupalSettings' => [
        'testSetting' => ['Setting'],
        ],
        ], $response->getAttachments());
        }
        /**
        ......@@ -389,7 +397,14 @@ protected function setupValidMocks($use_ajax = self::USE_AJAX) {
        ->willReturn(TRUE);
        $executable->expects($this->atMost(1))
        ->method('preview')
        ->willReturn(['#markup' => 'View result']);
        ->willReturn([
        '#markup' => 'View result',
        '#attached' => [
        'drupalSettings' => [
        'testSetting' => ['Setting'],
        ],
        ],
        ]);
        $this->executableFactory->expects($this->once())
        ->method('get')
        ......
        0% Loading or .
        You are about to add 0 people to the discussion. Proceed with caution.
        Please register or to comment