deregenboog / ecd

Elektronisch Cliëntendossier
MIT License
3 stars 1 forks source link

Andere implementatie van VerhuurderFilter gekoppeld #1789

Open jtborger opened 5 months ago

jtborger commented 5 months ago

Zie PR #1783 en ticket #1760

Op de een of andere manier kreeg ik het niet met doctrine opgelost en heb ik het programmatisch in de Dao opgelost. Wil ik eigenlijk niet.

Als je master uit checkt gaat dit nu goed, omdat ik in de Dao programmatisch filter. Het is dus belangrijk dat eerst even uit te zetten omdat we het eigenlijk in Doctrine willen oplossen (als dat kan).

Je kunt kijken bv in het Verhuurderoverzicht, als je filtert op 'Niet gekoppeld' (=default) dan staan er zowel ja als nee bij in de lijst. Deze, /tw/verhuurders/864, is bv wel gekoppeld, maar komt dus wel naar voren in het overzicht niet-gekoppeld. Dat komt omdat er oude Huurovereenkomsten zijn die al wel afgesloten zijn, en hij in de join dus koppelt op 'een' Huurovereenkomst die al is afgesloten. Ipv dat ie alleen kijkt naar de meest recente. Ik liep vast in een self-join, window, group by of wat dan ook, terwijl dat naw wel mogelijk is. Maar ook daar was ik lang mee bezig en kreeg het niet voor elkaar...ik heb een bepaalde doctrine blindheid geloof ik. Of allergie. Geen idee ;)

barthuttinga commented 5 months ago

De conditie "gekoppeld" is waar als er aan de volgende condities voldaan wordt:

  1. Er is een huurovereenkomst
  2. Die huurovereenkomst is geen reservering
  3. Die huurovereenkomst heeft een startdatum
  4. Die huurovereenkomst heeft een einddatum in de toekomst, of geen einddatum

Ik gebruik nu een sub-query om deze huurders/verhuurders te selecteren en gebruik in de hoofd-query WHERE IN () of WHERE NOT IN ()