torsten/yourpart3
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor child character loading in FalukantService: Update logic to load child relations using separate queries, improving performance and preventing EagerLoadingError. Simplify data retrieval by eliminating unnecessary eager loading and enhancing clarity in the code structure.

Browse Source
This commit is contained in:
Torsten Schulz (local)
2026-02-04 15:16:05 +01:00
parent 057b038fac
commit d23026121e
1 changed files with 37 additions and 42 deletions
Download Patch File Download Diff File Expand all files Collapse all files

69
backend/services/falukantService.js
View File

@@ -2701,48 +2701,43 @@ class FalukantService extends BaseService {
};
});
}
const charsWithChildren = await FalukantCharacter.findAll({
// Load child relations without FalukantCharacter includes to avoid EagerLoadingError
const userCharacterIds = (await FalukantCharacter.findAll({
where: { userId: user.id },
include: [
{
model: ChildRelation,
as: 'childrenFather',
include: [{
model: FalukantCharacter,
as: 'child',
include: [{ model: FalukantPredefineFirstname, as: 'definedFirstName', attributes: ['name'] }]
}]
},
{
model: ChildRelation,
as: 'childrenMother',
include: [{
model: FalukantCharacter,
as: 'child',
include: [{ model: FalukantPredefineFirstname, as: 'definedFirstName', attributes: ['name'] }]
}]
}
attributes: ['id']
})).map(c => c.id);
const childRels = userCharacterIds.length
? await ChildRelation.findAll({
where: {
[Op.or]: [
{ fatherCharacterId: { [Op.in]: userCharacterIds } },
{ motherCharacterId: { [Op.in]: userCharacterIds } }
]
});
const children = [];
for (const parentChar of charsWithChildren) {
const allRels = [
...(parentChar.childrenFather || []),
...(parentChar.childrenMother || [])
];
for (const rel of allRels) {
const kid = rel.child;
children.push({
childCharacterId: kid.id,
name: kid.definedFirstName?.name || 'Unknown',
gender: kid.gender,
age: calcAge(kid.birthdate),
},
attributes: ['childCharacterId', 'nameSet', 'isHeir', 'createdAt']
})
: [];
const childCharIds = [...new Set(childRels.map(r => r.childCharacterId))];
const childChars = childCharIds.length
? await FalukantCharacter.findAll({
where: { id: childCharIds },
attributes: ['id', 'birthdate', 'gender'],
include: [{ model: FalukantPredefineFirstname, as: 'definedFirstName', attributes: ['name'] }]
})
: [];
const childCharMap = Object.fromEntries(childChars.map(c => [c.id, c]));
const children = childRels.map(rel => {
const kid = childCharMap[rel.childCharacterId];
return {
childCharacterId: rel.childCharacterId,
name: kid?.definedFirstName?.name || 'Unknown',
gender: kid?.gender,
age: kid?.birthdate ? calcAge(kid.birthdate) : null,
hasName: rel.nameSet,
isHeir: rel.isHeir || false,
_createdAt: rel.createdAt,
};
});
}
}
// Sort children globally by relation createdAt ascending (older first)
children.sort((a, b) => new Date(a._createdAt) - new Date(b._createdAt));
const inProgress = ['wooing', 'engaged', 'married'];
@@ -2968,7 +2963,7 @@ class FalukantService extends BaseService {
},
attributes: ['character1Id', 'character2Id']
});
if (!rel) throw new Error('Beziehung nicht gefunden');
if (!rel) return [];
const relatedCharId = rel.character1Id === myChar.id ? rel.character2Id : rel.character1Id;
const relatedChar = await FalukantCharacter.findOne({