FrederikBaerentsen/BrickTracker
8
20
Fork 12
Code Issues 10 Pull Requests Actions Packages Projects 1 Releases 9 Wiki Activity

Wishlist error #152

New Issue
Closed
opened 2026-04-17 09:44:43 +02:00 by derwohat · 9 comments
derwohat commented 2026-04-17 09:44:43 +02:00
Copy Link
Copy Source

Hi,

After update to 1.4 I get the "Error: FOREIGN KEY constraint failed." if I try to remove a set from the wishlist.

Other than that: congrats to that nice little update!

Greetings
Dennis

Hi, After update to 1.4 I get the "Error: FOREIGN KEY constraint failed." if I try to remove a set from the wishlist. Other than that: congrats to that nice little update! Greetings Dennis
FrederikBaerentsen commented 2026-04-17 09:54:41 +02:00
Owner
Copy Link
Copy Source

I cannot reproduce this. Can you try and run the 'Check Data Integrity' under Admin -> Database -> Maintenance?

I cannot reproduce this. Can you try and run the 'Check Data Integrity' under Admin -> Database -> Maintenance?
FrederikBaerentsen added the Kind/Bug label 2026-04-17 09:54:45 +02:00
FrederikBaerentsen added reference release/1.4 2026-04-17 09:54:50 +02:00
derwohat commented 2026-04-17 10:16:15 +02:00
Author
Copy Link
Copy Source

I already did the check yesterday and again today: No integrity issues found! Your database is healthy.

The set I tried to remove is: 75409-1

I already did the check yesterday and again today: No integrity issues found! Your database is healthy. The set I tried to remove is: 75409-1
FrederikBaerentsen commented 2026-04-17 11:25:48 +02:00
Owner
Copy Link
Copy Source

Would it be possible for you to send me a copy of your database?

Would it be possible for you to send me a copy of your database?
derwohat commented 2026-04-17 12:25:51 +02:00
Author
Copy Link
Copy Source

Yes. Its 17 MB. Where can I send it to?

Yes. Its 17 MB. Where can I send it to?
FrederikBaerentsen commented 2026-04-17 12:29:06 +02:00
Owner
Copy Link
Copy Source

If you cant attach it here (forgot what the max upload size is), can you upload it to https://seafile.baerentsen.space/u/d/4b3970edd649433fb12d/

If you cant attach it here (forgot what the max upload size is), can you upload it to https://seafile.baerentsen.space/u/d/4b3970edd649433fb12d/
derwohat commented 2026-04-17 13:39:43 +02:00
Author
Copy Link
Copy Source

Done

Done
FrederikBaerentsen commented 2026-04-17 14:52:26 +02:00
Owner
Copy Link
Copy Source

Thanks for sharing your database, that made it easy to track down the issue. The bug was in the order things got deleted. Wish owners were being removed after the wish itself, but the database requires it to go the other way around. Delete the owner first, then the wish. Since you had an owner assigned to that wish, the FK error occurred. I have no owners in my test data so I never saw it. The fix is done and will be part of 1.4.1. If you want to try it now, pull the :dev Docker image tag and it should be good.

Fixed in 69318e7b0b.

Thanks for sharing your database, that made it easy to track down the issue. The bug was in the order things got deleted. Wish owners were being removed after the wish itself, but the database requires it to go the other way around. Delete the owner first, then the wish. Since you had an owner assigned to that wish, the FK error occurred. I have no owners in my test data so I never saw it. The fix is done and will be part of `1.4.1`. If you want to try it now, pull the `:dev` Docker image tag and it should be good. Fixed in [69318e7b0b](https://gitea.baerentsen.space/FrederikBaerentsen/BrickTracker/commit/69318e7b0b157a969a87e5a498a40a4fb7e08654).
derwohat commented 2026-04-17 15:01:58 +02:00
Author
Copy Link
Copy Source

Ok, now I deselected the wish owner, but I still cant remove the wish.

Ok, now I deselected the wish owner, but I still cant remove the wish.
FrederikBaerentsen commented 2026-04-17 16:34:30 +02:00
Owner
Copy Link
Copy Source

Did you switch and pull the :dev branch?

The issue with just removing the owner is that the set still is in the table bricktracker_wish_owners, it just shows 0.

Did you switch and pull the `:dev` branch? The issue with just removing the owner is that the set still is in the table `bricktracker_wish_owners`, it just shows `0`.
Sign in to join this conversation.
release/1.4
Branches Tags
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
 Out of Scope
Request is out of scope
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
Unraid
Unsupported/Can't help
No Label Kind/Bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
FrederikBaerentsen
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: FrederikBaerentsen/BrickTracker#152
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?