Remove datetime upgrade path check from its hook_install() [#1919420]

Follow-up to: #501428: Date and time field type in core

Contrib to core upgrade paths are handled in contrib.

This code would be executed permanently whenever datetime module is installed on every single site and installation for the next 6+ years.

Comment	File	Size	Author
	drupal8.datetime-install.0.patch	934 bytes	sun

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

swentel CreditAttribution: swentel commented 22 February 2013 at 18:26

Status:

Needs review

» Reviewed & tested by the community

Comment #2

webchick

she/they

English

Vancouver 🇨🇦

CreditAttribution: webchick commented 28 February 2013 at 02:44

Status:

Reviewed & tested by the community

» Fixed

Yeah, I think this is right. It's one thing to put workaround code in hook_update_N() (which, while uncommon, is not entirely unprecedented; we've done it for Actions module in D6, for example), but we shouldn't be putting this kind of code in places that run on new sites.

In this case, seems like something DateTime module in contrib should be taking care of itself, so...

Committed and pushed to 8.x. Thanks!