🇳🇱Netherlands @nickvanboven

nickvanboven → made their first commit to this issue’s fork.

Need this to, tested patch and worked

Closing isseu, after looking at it it is a fault at our side, we run the site on php7.4 en need to upgrade this one.

nickvanboven → created an issue.

nickvanboven → created an issue.

Created a fork against 2.0.7 to fix the isseu, patch #4 not working could not apply because of class name not correct

nickvanboven → made their first commit to this issue’s fork.

nickvanboven → made their first commit to this issue’s fork.

Already fixed in https://www.drupal.org/project/facets/issues/3360426 📌 PHP 8.2 compatibility Fixed and mr 143 applies

nickvanboven → created an issue.

nickvanboven → created an issue.

I dont think 700 characters is enough, if u want to disable everything by default the header is already 811 charachter long and this is without even settings a url or wildcard:

accelerometer=(), ambient-light-sensor=(), autoplay=(), battery=(), camera=(), cross-origin-isolated=(), display-capture=(), document-domain=(), encrypted-media=(), execution-while-not-rendered=(), execution-while-out-of-viewport=(), fullscreen=(), geolocation=(), gyroscope=(), keyboard-map=(), magnetometer=(), microphone=(), midi=(), navigation-override=(), payment=(), picture-in-picture=(), publickey-credentials-get=(), screen-wake-lock=(), sync-xhr=(), usb=(), web-share=(), xr-spatial-tracking=(), clipboard-read=(), clipboard-write=(), gamepad=(), speaker-selection=(), conversion-measurement=(), focus-without-user-activation=(), hid=(), idle-detection=(), interest-cohort=(), serial=(), sync-script=(), trust-token-redemption=(), unload=(), window-placement=(), vertical-scroll=()

Something is wrong when requiring the 2.0.0 version i get old code that isnt d10 compatible, Not seeing te code in the repo at the tag 2.0.0

Fixed in https://www.drupal.org/project/facets/issues/2986981 🐛 Ajax facet block seems to lose Views context (filters, etc) RTBC

I have created a fork with the changes from #32 and added a check for the route object, this can be null if the request is serialized and used at a other time, we use this to get the view options on cron when processing some jobs.

nickvanboven → made their first commit to this issue’s fork.

nickvanboven → made their first commit to this issue’s fork.

nickvanboven → made their first commit to this issue’s fork.

I have created 2 merge request, to always use the patch supplied here, We also had isseus when using multilanguage with custom domains.

nickvanboven → made their first commit to this issue’s fork.