Problem/Motivation

BEF version 5 added dependencies on several jQuery UI related modules because Drupal 9 no longer contains these in core. When updating a site from version 4, these dependencies should be enabled by an update hook.

Steps to reproduce

  • Install version 4 on a site
  • Update to version 5
  • Notice that Drupal will complain about the module requiring jQuery UI, jQuery UI Slider, jQuery UI Touch Punch and jQuery UI Datepicker

Proposed resolution

Create an update hook that enables the required modules

Remaining tasks

  • Create patch
  • Review
  • Commit

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#5 enable_dependencies-3193305-5.patch1.33 KBSupreetam09
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eelkeblok created an issue. See original summary.

brulain’s picture

+1

eelkeblok’s picture

Issue summary: View changes
jcnventura’s picture

Priority: Normal » Major

This is a bit more than 'Normal'.. Upgrading to bef 5 should really enable these modules.

Supreetam09’s picture

Status: Active » Needs review
FileSize
1.33 KB

Adding this patch to enable dependant modules if they aren't enabled already when module is upgraded from version 4 to 5.

jcnventura’s picture

Status: Needs review » Reviewed & tested by the community

Nice, @Supreetam09

Neslee Canil Pinto’s picture

Okay, this needs to go into the 5.x branch, looks like a must need a fix. Reviewed/tested it and works as expected. Thanks

Neslee Canil Pinto’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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