Without the param $primary_keys, drupal_write_record() tries to run an INSERT query, which violates integrity constraints when editting an existing pubtype.

CommentFileSizeAuthor
#3 1870446-biblio-drupal_write_record_no_primary_keys-1-tobetested.patch888 bytesdrupalycious
#1 1870446-biblio-drupal_write_record_no_primary_keys-1.patch890 bytesbaisong

Comments

baisong’s picture

baisong
he/they
Primary language English
Location Occupied Kumeyaay Land; San Diego, CA, US–Tijuana, BC, MX
commented
StatusFileSize
new 1870446-biblio-drupal_write_record_no_primary_keys-1.patch890 bytes

Here's a patch that fixes it for me locally, should work.

drupalycious’s picture

drupalycious commented
Status: Active » Needs review

testing the patch

drupalycious’s picture

drupalycious commented
StatusFileSize
new 1870446-biblio-drupal_write_record_no_primary_keys-1-tobetested.patch888 bytes

I think your patch needs to be renamed because the test bot seems to be ignoring it.
See #1162082: Add a warning if a diff/patch file is selected and issue status is not testable
The name of the patch file should avoid ending with a number. #512962-244: Optimize menu_router_build() / _menu_router_save()

I am uploading your file with a different name to see if it makes any difference.

Status: Needs review » Needs work

The last submitted patch, 3: 1870446-biblio-drupal_write_record_no_primary_keys-1-tobetested.patch, failed testing. View results