When i try to update anonymous' s user comment from /admin/content/comment it will give below exception.

Drupal\Core\Database\IntegrityConstraintViolationException: SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '54' for key 'PRIMARY': INSERT INTO {comment_notify} (cid, notify, notify_hash) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2); Array ( [:db_insert_placeholder_0] => 54 [:db_insert_placeholder_1] => 0 [:db_insert_placeholder_2] => vi1-ladBlR7VJB3aJG4oAUlJ7lrl5eP0qWwsqCQYX8c ) in Drupal\Core\Database\Connection->handleQueryException()

CommentFileSizeAuthor
#6 2018-02-26_1236.png48.87 KBprajaankit
#4 comment_notify-n2877283-4.interdiff.txt1.2 KBDamienMcKenna
#4 comment_notify-n2877283-4.patch1.33 KBDamienMcKenna
#2 integrity_constraint_violation_2877283_1.patch1.16 KBBunty Badgujar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bunty Badgujar created an issue. See original summary.

Bunty Badgujar’s picture

Bunty Badgujar CreditAttribution: Bunty Badgujar commented
FileSize
integrity_constraint_violation_2877283_1.patch1.16 KB
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Version: 8.x-1.0-beta1 » 8.x-1.x-dev
Status: Active » Needs work

This needs a little polish to match the Drupal coding standards.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Needs review
Parent issue: » #2770889: Comment Notify Next release - 8.x-1.0-beta5
FileSize
comment_notify-n2877283-4.patch1.33 KB
comment_notify-n2877283-4.interdiff.txt1.2 KB

I fixed the minor issues.

Bunty Badgujar’s picture

Bunty Badgujar CreditAttribution: Bunty Badgujar commented

Thanks @DamienMcKenna :)

prajaankit’s picture

prajaankit CreditAttribution: prajaankit at ]init[ AG commented
FileSize
2018-02-26_1236.png48.87 KB

Hi Bunty,

i Tested this patch , its working fine

prajaankit’s picture

prajaankit CreditAttribution: prajaankit at ]init[ AG commented
nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
CreditAttribution: nkoporec as a volunteer and at Agiledrop - Your Trusted Drupal Teammates commented
Status: Needs review » Reviewed & tested by the community

Tested the patch and it removes the bug.Moving to RTBC.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles at CARD.com commented
Status: Reviewed & tested by the community » Fixed

OK, thanks for your work on this, everyone!

I've committed this and created a new release https://www.drupal.org/project/comment_notify/releases/8.x-1.0-beta2

Status: Fixed » Closed (fixed)

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