Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ISSUE-183: Patch Facets/Views Blocks and Update composer.json #184

Merged
merged 3 commits into from
May 16, 2022
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 19 additions & 8 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@
"drupal/jquery_ui_datepicker": "^1.1",
"drupal/jquery_ui_slider": "^1.1",
"drupal/jsonapi_earlyrendering_workaround": "^1.0",
"drupal/moderated_content_bulk_publish": "^1.0",
"drupal/moderated_content_bulk_publish": "^2.0",
"drupal/pathauto": "^1.6",
"drupal/queue_ui": "^2.0",
"drupal/quick_node_clone": "^1.12",
Expand All @@ -75,13 +75,12 @@
"drupal/twig_field": "^1.0",
"drupal/twig_tweak": "^2.6",
"drupal/views_bootstrap": "^4.3",
"drupal/views_bulk_operations": "^3.13",
"drupal/views_bulk_operations": "^4.1",
"drupal/webform": "^6",
"drupal/webform_entity_handler": "1.x-dev",
"drupal/webformnavigation": "^1.0@alpha",
"drupal/workflow": "^1.1",
"drush/drush": "10.5",
"egulias/email-validator": "^2.0",
"fileeye/mimemap": "^1.1.1",
"maennchen/zipstream-php": "^2.1",
"mixnode/mixnode-warcreader-php": "^0.0.6",
Expand All @@ -102,12 +101,10 @@
"swaggest/json-schema": "^0.12.10",
"symfony/event-dispatcher": "^4",
"symfony/yaml": "^4",
"typo3/phar-stream-wrapper": "3.1.6",
"vlucas/phpdotenv": "^5.1",
"webflo/drupal-finder": "^1.0.0",
"webmozart/path-util": "^2.3",
"wikibase/data-model": "9.6.1",
"zaporylie/composer-drupal-optimizations": "^1.1.1"
"wikibase/data-model": "9.6.1"
},
"require-dev": {
"drupal/core-dev": "^9",
Expand All @@ -120,7 +117,18 @@
"prefer-stable": true,
"config": {
"discard-changes": true,
"sort-packages": true
"allow-plugins": {
"composer/installers": true,
"cweagans/composer-patches": true,
"drupal/core-composer-scaffold": true,
"drupal/core-project-message": true,
"dealerdirect/phpcodesniffer-composer-installer": true
},
"preferred-install": {
"archipelago/*": "source",
"strawberryfield/*": "source"
"*": "dist"
aksm marked this conversation as resolved.
Show resolved Hide resolved
}
},
"autoload": {
"classmap": [
Expand Down Expand Up @@ -180,7 +188,10 @@
"patches": {
"drupal/facets": {
"Fix deprecated JQUERY UI Slider": "https://www.drupal.org/files/issues/2021-11-12/issue-3153622-nouislider_25.patch"
}
},
"drupal/core": {
"Fix Ajax/Preserve Facets on exposed forms Drupal 9.3.x": "patches/3032353-25-archipelago-05162022.patch"
}
}
}
}
99 changes: 99 additions & 0 deletions patches/3032353-25-archipelago-05162022.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,99 @@
diff --git a/core/modules/views/js/ajax_view.es6.js b/core/modules/views/js/ajax_view.es6.js
index 5367f83cc9..386df7947d 100644
--- a/core/modules/views/js/ajax_view.es6.js
+++ b/core/modules/views/js/ajax_view.es6.js
@@ -137,6 +137,11 @@
Drupal.views.ajaxView.prototype.attachExposedFormAjax = function () {
const that = this;
this.exposedFormAjax = [];
+
+ // Add exposed_form_display option to the request.
+ if (that.element_settings.submit) {
+ that.element_settings.submit.exposed_form_display = 1;
+ }
// Exclude the reset buttons so no AJAX behaviors are bound. Many things
// break during the form reset phase if using AJAX.
$(
diff --git a/core/modules/views/js/ajax_view.js b/core/modules/views/js/ajax_view.js
index f1e2e0121a..9538a14c9d 100644
--- a/core/modules/views/js/ajax_view.js
+++ b/core/modules/views/js/ajax_view.js
@@ -80,6 +80,11 @@
Drupal.views.ajaxView.prototype.attachExposedFormAjax = function () {
var that = this;
this.exposedFormAjax = [];
+
+ if (that.element_settings.submit) {
+ that.element_settings.submit.exposed_form_display = 1;
+ }
+
$('input[type=submit], button[type=submit], input[type=image]', this.$exposed_form).not('[data-drupal-selector=edit-reset]').each(function (index) {
var selfSettings = $.extend({}, that.element_settings, {
base: $(this).attr('id'),
diff --git a/core/modules/views/src/Controller/ViewAjaxController.php b/core/modules/views/src/Controller/ViewAjaxController.php
index 8b7d115fa5..ad4bc1e8e9 100644
--- a/core/modules/views/src/Controller/ViewAjaxController.php
+++ b/core/modules/views/src/Controller/ViewAjaxController.php
@@ -128,6 +128,7 @@ public function ajaxView(Request $request) {
$dom_id = isset($dom_id) ? preg_replace('/[^a-zA-Z0-9_-]+/', '-', $dom_id) : NULL;
$pager_element = $request->request->get('pager_element');
$pager_element = isset($pager_element) ? intval($pager_element) : NULL;
+ $exposed_form_display = $request->request->get('exposed_form_display');

$response = new ViewAjaxResponse();

@@ -136,17 +137,18 @@ public function ajaxView(Request $request) {
// @todo Remove this parsing once these are removed from the request in
// https://www.drupal.org/node/2504709.
foreach ([
- 'view_name',
- 'view_display_id',
- 'view_args',
- 'view_path',
- 'view_dom_id',
- 'pager_element',
- 'view_base_path',
- AjaxResponseSubscriber::AJAX_REQUEST_PARAMETER,
- FormBuilderInterface::AJAX_FORM_REQUEST,
- MainContentViewSubscriber::WRAPPER_FORMAT,
- ] as $key) {
+ 'view_name',
+ 'view_display_id',
+ 'view_args',
+ 'view_path',
+ 'view_dom_id',
+ 'pager_element',
+ 'view_base_path',
+ 'exposed_form_display',
+ AjaxResponseSubscriber::AJAX_REQUEST_PARAMETER,
+ FormBuilderInterface::AJAX_FORM_REQUEST,
+ MainContentViewSubscriber::WRAPPER_FORMAT,
+ ] as $key) {
$request->query->remove($key);
$request->request->remove($key);
}
@@ -201,6 +203,24 @@ public function ajaxView(Request $request) {
}
$response->addCommand(new ReplaceCommand(".js-view-dom-id-$dom_id", $preview));

+ // Views with ajax enabled aren't refreshing filters placed in blocks.
+ // Only <div> containing view is refreshed. ReplaceCommand is fixing
+ // that for view, if it uses ajax and exposed forms in block.
+ if ($view->display_handler->usesExposed() && $view->display_handler->getOption('exposed_block')) {
+ $view_id = preg_replace('/[^a-zA-Z0-9-]+/', '-', $name . '-' . $display_id);
+ $context = new RenderContext();
+ $exposed_form = $this->renderer->executeInRenderContext($context, function () use ($view) {
+ return $view->display_handler->viewExposedFormBlocks();
+ });
+ if (!$context->isEmpty()) {
+ $bubbleable_metadata = $context->pop();
+ BubbleableMetadata::createFromRenderArray($exposed_form)
+ ->merge($bubbleable_metadata)
+ ->applyTo($exposed_form);
+ }
+ $response->addCommand(new ReplaceCommand("#views-exposed-form-" . $view_id, $this->renderer->render($exposed_form)));
+ }
+
return $response;
}
else {