ViewExecutable should not be responsible for parsing exposed input

This was discovered via 📌 Search must be indexed after recipe is applied Active where search indexing via automated cron started a session for no reason, resulting in headers already sent warnings. Bumping to major and tagging as a Drupal CMS release target.

There's already a bug open for not checking the configuration settings, so switching to a task. 🐛 Views checks the session for exposed filters when it shouldn't Active .

I added a proposed code to check the remember setting for the current user role before fetching the exposed_input from the session.

Created an MR from your branch for easier review.

One comment on there but the logic looks good to me otherwise.

I'm not sure how easy this will be to test, or if we even have any test coverage of the remember feature at all - maybe if we could we could extend that.

I think it might be worth trying to move this to the filter base class, but we can do that in a separate issue and concentrate on the bug fix here.

OK looks like we do have some existing test coverage because it's failing - didn't check whether it's a bug in the test or the MR yet.