Download & Extend

Duplicate entry in project_release_projects

Project:Project
Version:6.x-1.x-dev
Component:Releases
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review

Issue Summary

If the update fixed just the same values like before the db_affected_rows() are 0. In this case you get:

user warning: Duplicate entry '4' for key 1 query: INSERT INTO project_release_projects (nid, releases, version_format) VALUES (4, 0, '') in sites\all\modules\project\release\includes\project_edit_releases.inc on line 266.

In cache.inc I have seen this will be catched with "@" on the INSERT.

Comments

#1

Status:active» needs review

Patch attached

AttachmentSizeStatusTest resultOperations
1394398 Duplicate entry in project_release_projects.patch1.05 KBIdleFAILED: [[SimpleTest]]: [MySQL] Fetch test patch: failed to retrieve [1394398 Duplicate entry in project_release_projects.patch] from [drupal.org].View details | Re-test

#2

Status:needs review» needs work

The last submitted patch, 1394398 Duplicate entry in project_release_projects.patch, failed testing.

#3

Status:needs work» needs review

#1: 1394398 Duplicate entry in project_release_projects.patch queued for re-testing.

#4

Status:needs review» needs work

The last submitted patch, 1394398 Duplicate entry in project_release_projects.patch, failed testing.

#5

Status:needs work» needs review

d.o Testbot seems to have issue with spaces in patch files...

As db_affected_rows() is used very often I fixed a few more that may have the same issues.

AttachmentSizeStatusTest resultOperations
1394398_Duplicate_entry_in_project_release_projects_all.patch3.66 KBIdlePASSED: [[SimpleTest]]: [MySQL] 253 pass(es).View details | Re-test

#6

Status:needs review» needs work

The last submitted patch, 1394398_Duplicate_entry_in_project_release_projects_all.patch, failed testing.

#7

Version:6.x-1.0-alpha5» 6.x-1.x-dev

#8

Status:needs work» needs review

#5: 1394398_Duplicate_entry_in_project_release_projects_all.patch queued for re-testing.

nobody click here