Contrib.social

Dependencies lost after sensor save for the database_aggregator plugin

Open on Drupal.org →
Created on 9 July 2024, 2 months ago
Updated 30 July 2024, about 1 month ago

Problem/Motivation

We noticed this, when we uninstalled the captcha module and the captcha sensor remained.

Steps to reproduce

  1. Install the captcha module.
  2. `drush cget monitoring.sensor_config.captcha_failed_count` and confirm that monitoring.sensor_config.captcha_failed_count has the dependency on captcha
  3. Edit the captcha_failed_count sensor and save it.
  4. `drush cget monitoring.sensor_config.captcha_failed_count` again and there will be no captcha dependency

Proposed resolution

Discussed this with @Berdir: "it's ugly, but I fear the only option here is a loop over all modules, then load the isntall file, then check if the hook exists and invoke it"

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Needs review

Version

1.0

Component

Sensors

Created by

🇸🇮Slovenia Primsi

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024