Problem/Motivation

#1996130: REST views: not adding dependencies on Serializer providers added an empty update function, it's in 8.2.x

However it's named 800* which doesn't leave any space to add updates before it.

This is an empty update, so it's extremly unlikely to matter in practice, but it makes it hard to add other updates like #2477899: Multiple valued Base fields won't work in Views, so marking major.

Proposed resolution

8200 instead of 800* for updates that only live in 8.2.x

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 2743297.patch457 bytescatch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch created an issue. See original summary.

catch’s picture

Status: Active » Needs review
FileSize
457 bytes
dawehner’s picture

Status: Needs review » Reviewed & tested by the community

The update system, seriously ...

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: 2743297.patch, failed testing.

mayurjadhav’s picture

Status: Needs work » Reviewed & tested by the community
Issue tags: +Quickfix

+1

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 0fc2e8d and pushed to 8.2.x. Thanks!

  • alexpott committed 0fc2e8d on 8.2.x
    Issue #2743297: Mis-named update function in views
    
alexpott’s picture

I committed this but then realised I had a question - if we do add a new update function to 8.1.x for views then the patch will have to have a different update number for 8.2.x otherwise on a site running 8.2.x won;t get the update...

dawehner’s picture

Right, ideally we then would have a registry of already executed update functions, so that one can skip pre-existing tests if needed.

I'm not sure anymore whether we will have more minor versions after an LTS, but that could be fun given a patch might be applied to even 3 branches at the same time.

Status: Fixed » Closed (fixed)

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