Problem/Motivation

❯ composer why drupal/update_notifications_disable-update
drupal/upgrade_status 2.0.0-alpha1 requires drupal/update_notifications_disable-update (*)

Having update_notifications_disable module present in the filesystem, throws this on the status page:

Error: Call to undefined function _update_get_cache_multiple() in _upgrade_status_get_cached_available_releases() (line 172 of upgrade_status/upgrade_status.module).

Steps to reproduce

composer require drupal/upgrade_status:^2.0@alpha

Proposed resolution

Figure out where composer is getting drupal/update_notifications_disable

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork upgrade_status-3311591

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

joelpittet created an issue. See original summary.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Version: 7.x-2.0-alpha1 » 7.x-2.x-dev
Status: Active » Needs review

Fewf I found where this was coming from #3117533: Dependency on core update module pulls in update_notifications_disable-update module

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Related issues: +#3117533: Dependency on core update module pulls in update_notifications_disable-update module

joelpittet opened merge request !39

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

Hm, is drupal:update a thing that Drupal 7 already understands?

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Needs review » Reviewed & tested by the community

Ok so this format is supported from Drupal 7.40: https://www.drupal.org/docs/7/creating-custom-modules/writing-module-inf... - 2015 october. I think that is safe to assume.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Title: update_notifications_disable has sneaked in via composer » update_notifications_disable is incorrectly installed by composer due to a dependency

  • Gábor Hojtsy committed cf60cac on 7.x-2.x 
    Issue #3311591 by joelpittet: update_notifications_disable is...
gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed!

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented

Using `dev-2.x` is not installing that `update_notifications_disable`! Thanks @Gábor Hojtsy!
Might be worth releasing an alpha2?

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

I was looking at all 7.x issues whether other things popped up about 7.x-2.x. #3305362: An AJAX HTTP error occurred. HTTP Result Code: 200 happened to one user but others did not reproduce it. So yeah I can make an alpha2 now, sure.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

Done https://www.drupal.org/project/upgrade_status/releases/7.x-2.0-alpha2

Status: Fixed » Closed (fixed)

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