Removed deprecated code in views module

Couple of comments on the MR.

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

Clean-up what was able to find, added views_ui

btw there's views.view.*.mapping.display.sequence.mapping.cache_metadata.mapping.cacheable which has comment as it was deprecated but all views has this property, I bet it needs follow-up somehow

I tried removing that before and got schema errors everywhere.

@andypost can you take a look at the phpcs.

Thank you! fixed CS, re #10 - that's why it needs follow-up

#9/10 what's happened is that issues have been committed that update the views schema, with an update path, but which don't fix the shipped views to be compliant. Then when we remove bc layers like there, everything blows up. This is a problem with the original MRs that went in, but in most cases that was months/years ago.

The ideal thing is to either manually fix or re-export those views so that they have the correct schema, but we could also open a separate issue for anything tricky here and just commit the changes that don't require updating test fixtures.

I ran into similar problems on 📌 Remove redudundat hook_ENTITY_TYPE_presave() and ViewsConfigUpdater methods Active

Opened 📌 Update core views with correct config Active

Kernel test was random failure.

Committed/pushed to 11.x, thanks!

smustgrave → closed merge request !7656

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