#2168011: Remove all 7.x to 8.x update hooks and disallow updates from the previous major version left some functions in that don't look necessary:

  • update_add_uuids
  • update_add_cache_columns

Both of these functions would only be useful during a major version upgrade afaics

CommentFileSizeAuthor
d8.remove-update-functions.patch1.95 KBalexpott

Comments

jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix, +Novice

RTBC if green.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Since only update functions would call these, and since we already have a change notice about those being removed, I think we're fine to just mark this fixed.

xjm’s picture

xjm
she/her
Primary language English
commented
Related issues: +#2168011: Remove all 7.x to 8.x update hooks and disallow updates from the previous major version
xjm’s picture

xjm
she/her
Primary language English
commented

Added the reference to this to the change record and updated it (it has a list of the removed helper functions).

Status: Fixed » Closed (fixed)

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