search

htl-leo-itp-2325-4-5BHITM / cam-connect

Verleihsystem für das Equipment der HTL-Leonding
The Unlicense
2 stars 0 forks source link

Fix the update system #44

Closed elYanuki closed 9 months ago

elYanuki commented 9 months ago

No idea why but quarkus screams at you when trying to update litteraly any property. Something to do with to many arguments in the query, could be because the deviceTypes got super complexe and because their inheritance is multi table quarkus has to do a metric fuckton of joins but i dont reaaally belive thats the issue

elYanuki commented 9 months ago

updated the function to use the CCError system and fixed half of the error (could not get non existent json keys: were null because the newlly created rent has almost all values null)

new endboss error is cause by a generic em.find(Rent.class, id) meaning (lets hope im wrong on this) we cannot query for rents because it literally crashed quarkus.. which is a problem since we need to query for rents a lot. Potential solution: would be to change how quarkus handles the extends to be single table which might be a bit uglier but might also be just the same in usage but faster

elYanuki commented 9 months ago

changed devicetype to be single table, that got rid of the join error @Michiii11 please take a look at this to confirm its fixed