πŸ‡ͺπŸ‡ΈSpain @einarulfhednar

Account created on 19 April 2023, almost 2 years ago
#

Merge Requests

More

Recent comments

πŸ‡ͺπŸ‡ΈSpain einarulfhednar

Hi, I have created a fork to fix the bug.

Waiting for the module maintainer to review and accept it.

Regards.

πŸ‡ͺπŸ‡ΈSpain einarulfhednar

einarulfhednar β†’ made their first commit to this issue’s fork.

πŸ‡ͺπŸ‡ΈSpain einarulfhednar

einarulfhednar β†’ changed the visibility of the branch 3372066-custom-breadcrumb-text to active.

πŸ‡ͺπŸ‡ΈSpain einarulfhednar

einarulfhednar β†’ changed the visibility of the branch 3372066-custom-breadcrumb-text to hidden.

πŸ‡ͺπŸ‡ΈSpain einarulfhednar

Hello to all,

First of all, thank you for the solution.

I tried to apply the patch for version 2.2.12 of the module on the portal I'm working on (Drupal version 9.5.9 and PHP version 8.0.30) and it failed, so I have generated the patch.

Regards.

πŸ‡ͺπŸ‡ΈSpain einarulfhednar

Problem/Motivation

When we have the scheduled newsletter enabled and we want to disable it by unchecking the 'Enable scheduled newsletter' option, visually the option remains checked and in the 'simplenews_scheduler' table of the database the status of the 'activated' column never changes to 0.

This is because, when the data is collected from the database (line 39 of the simplenws_scheduler.module file) a condition is not set to select those rows that have the status of the 'activated' column to 1 and, therefore, it never goes through the else on line 50 of the same file.

Steps to reproduce

  1. Check Activate scheduled newsletter option.
  2. Click Save scheduler settings button.
  3. Uncheck Activate scheduled newsletter option.
  4. Click Save scheduler settings button.

Proposed resolution

In the data fetching (line 39 of the simplenews_scheduler.module file) add the condition to select the rows of the 'activated' column to 1.

With this we get that, if there are no elements, it passes by else of the condition of line 46 and the variable $checked is set to FALSE.

πŸ‡ͺπŸ‡ΈSpain einarulfhednar

Hello again,

Sorry for the duplicity of comments: I am new and this is my first time.

I attach the correct file.

Thank you.

πŸ‡ͺπŸ‡ΈSpain einarulfhednar

Hello,

I re-rolled the #51 patch for the 9.5.7 Drupal core.

Thanks, I hope it helps.

Production build 0.71.5 2024