Problem/Motivation

The update hook simplenews_update_840002 fails with an SQL error, if the current project does not contain a subscriber without a subscription.
Query condition 'id NOT IN ()' cannot be empty.

Steps to reproduce

* Setup Drupal 9 project
* Install simplenews:^3
* Configure simplenews
* Do not add a subscriber without a subscription
* Update to simplenews:^4
* Run update hooks

Proposed resolution

* Use provided patch, whichs adds a condition in the affected update hook to not execute the causing SQL query if no results are available. I will add the patch in the first comment.

CommentFileSizeAuthor
#2 Issue_#3396668_Adds_safety_condition_to_database_query_for_update_hook_840002.patch1.05 KBjocowood

Issue fork simplenews-3396668

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

JoCowood created an issue. See original summary.

jocowood’s picture

jocowood
Primary language German
Location Kamp-Lintfort
commented
StatusFileSize
new Issue_#3396668_Adds_safety_condition_to_database_query_for_update_hook_840002.patch1.05 KB
adamps’s picture

adamps commented
Status: Needs review » Needs work

Thanks it looks good. Please create a MR, this project now uses gitlab testing.

Unfortunately commit would need to wait until someone can figure out how to fix the tests, see #3395901: Use Gitlab CI in Simplenews.

JoCowood opened merge request !41

jocowood’s picture

jocowood
Primary language German
Location Kamp-Lintfort
commented
Status: Needs work » Needs review

MR created

adamps’s picture

adamps commented
Status: Needs review » Reviewed & tested by the community

Thanks. I'll commit this once the branch tests are working so we can get a pass here.

  • AdamPS committed b0a7f4a1 on 4.x authored by JoCowood 
    Issue #3396668 by JoCowood: Update hook 840002 fails with SQL error
adamps’s picture

adamps commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

adamps’s picture

adamps commented

FYI there's still a bug: #3405456: Update fails if there are no newsletters with subscriptions

jocowood’s picture

jocowood
Primary language German
Location Kamp-Lintfort
commented

Thank you for the hint AdamPS