Compressed ajax_page_state['libraries'] can exist in both $request->request and $request->query simultaneously

N.B. this is mainly a problem because code like HtmlResponseAttachmentsProcessor, CssCollectionOptimizerLazy, and AjaxBasePageNegotiator all use code like

  $this->requestStack->getCurrentRequest()->get('ajax_page_state')

and Request::get() checks attributes, query, and request in that order.

It's quite critical issue. In my case: on any change in facet all assets are loaded second time on the page.

shouldn't the values be merged? or are we sure the GET data is always more up to date?

Looks reasonable to me

Also looks good to me. I think that eventually we could try to handle state when GET AJAX leads to POST AJAX leads to GET AJAX and stuff like this by not passing down every single GET parameter to every request, but that is very hard and far off, so we need to make it work until then.

nod_ → closed merge request !11029

Committed fda19f8 and pushed to 11.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.