search

Lesterhuis-Training-en-Consultancy / moodle-block_configurable_reports

This block is a Moodle custom reports builder. You can create custom reports without SQL knowledge. It's a tool suitable for admins or teachers.
http://moodle.org/plugins/view.php?plugin=block_configurable_reports
1 stars 0 forks source link

Filter vs Export versie 4.1 #1

Closed gemguardian closed 4 months ago

gemguardian commented 5 months ago

@luukverhoeven Ik loop nu tegen een probleem aan in de 4.1 versie van het blok COonfigurable report. Bijgaand heb je een database query met diverse filters.

Wanneer we de filter gebruiken wordt dit goed getoond op de pagina zelf. Maar bij de export worden ALLE records meegenomen. image

gemguardian commented 5 months ago
Comment from Freshservice:
Freshservice: 21460 linked successfully.
Freshservice: #21460
By: Gemma Lesterhuis <gemma@ltnc.nl>
luukverhoeven commented 5 months ago

@gemguardian Bij de oude versie van de plug-in kwam alleen de geselecteerde data terug die binnen de filters aanwezig was?

gemguardian commented 5 months ago

@luukverhoeven ik denk dat we elkaar goed begrijpen en het antwoord ja is.

In de oude versie van de Plugin maakte ik een filtering, dit werd dan op het scherm getoond zoals in de screenshot. En dan zou ik bij de Excel/CSV import alleen deze regel terug zien en niet alle regels uit de database query.

luukverhoeven commented 5 months ago

Zie nu ook je mail, daarin staat het ook dat het eerder correct werkt. Ik ga het onderzoeken, staat op de lijst.

gemguardian commented 4 months ago

@luukverhoeven kun jij aangeven wanneer je hier mee aan de slag gaat?

@servicedeskltnc

luukverhoeven commented 4 months ago

@gemguardian Deze stond op de todo, maar had geen duedata :(

Het issue heb ik gevonden, kun je testen of naar wens werkt?

gemguardian commented 4 months ago

@luukverhoeven dit werkt - net nu hij doorgezet heeft naar 4.1 :) Kun je deze fix pushen naar de orginele en meldde wat het issue was?

luukverhoeven commented 4 months ago

@gemguardian Pull request aangemaakt, https://github.com/jleyva/moodle-block_configurablereports/pull/248