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

Comments

Bunty Badgujar created an issue. See original summary.

bunty badgujar’s picture

bunty badgujar commented
StatusFileSize
new integrity_constraint_violation_2877283_1.patch1.16 KB
damienmckenna’s picture

damienmckenna
Location TN, USA
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 TN, USA
commented
Status: Needs work » Needs review
Parent issue: » #2770889: Comment Notify Next release - 8.x-1.0-beta5
StatusFileSize
new comment_notify-n2877283-4.patch1.33 KB
new comment_notify-n2877283-4.interdiff.txt1.2 KB

I fixed the minor issues.

bunty badgujar’s picture

bunty badgujar commented

Thanks @DamienMcKenna :)

prajaankit’s picture

prajaankit commented
StatusFileSize
new 2018-02-26_1236.png48.87 KB

Hi Bunty,

i Tested this patch , its working fine

prajaankit’s picture

prajaankit commented
nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
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
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.