Filter displayed geen resultaten #72

Closed
opened 2017-01-05 19:48:30 +00:00 by awaly · 2 comments
awaly commented 2017-01-05 19:48:30 +00:00 (Migrated from gitlab.awaly.nl)

Created by: shyzus

Ophalen van alle gegevens werkt juist maar het ophalen en filteren van de gegevens gaat verkeerd het geeft geen errors maar displayed ook niks. Helemaal niks ook geen console output in debug modus.
Zie de branche FoundLuggageTravellerBack-End.
Regels:
156-193 file: FoundLuggageTravellerController.java
240-292 file: LuggageService.java

*Created by: shyzus* Ophalen van alle gegevens werkt juist maar het ophalen en filteren van de gegevens gaat verkeerd het geeft geen errors maar displayed ook niks. Helemaal niks ook geen console output in debug modus. Zie de branche FoundLuggageTravellerBack-End. Regels: 156-193 file: FoundLuggageTravellerController.java 240-292 file: LuggageService.java
awaly commented 2017-01-06 16:46:22 +00:00 (Migrated from gitlab.awaly.nl)

Je set de text via de controller naar empty, in de search methode valideert hij of de string iets is en dan pas voert hij een filter uit. Dus als je bijvoorbeeld voor name null doorgeeft dan weet hij dat hij niet hoeft te filteren op name. Als je nou de tekst vervangt met empty denkt hij dat hij moet zoeken op "empty" wat geen resultaten weergeeft. Ik raad aan om de tekst niet te vervangen met "empty"

By Luca Camphuisen on 2017-01-06T17:46:22 (imported from GitLab)

Je set de text via de controller naar empty, in de search methode valideert hij of de string iets is en dan pas voert hij een filter uit. Dus als je bijvoorbeeld voor name null doorgeeft dan weet hij dat hij niet hoeft te filteren op name. Als je nou de tekst vervangt met empty denkt hij dat hij moet zoeken op "empty" wat geen resultaten weergeeft. Ik raad aan om de tekst niet te vervangen met "empty" *By Luca Camphuisen on 2017-01-06T17:46:22 (imported from GitLab)*
awaly commented 2017-01-07 04:20:17 +00:00 (Migrated from gitlab.awaly.nl)

Created by: shyzus

Dit is nu opgelost denk ik, zie pull request #74 voor de veranderingen.

By Achmed Waly on 2017-01-07T05:20:17 (imported from GitLab)

*Created by: shyzus* Dit is nu opgelost denk ik, zie pull request #74 voor de veranderingen. *By Achmed Waly on 2017-01-07T05:20:17 (imported from GitLab)*
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
HvA/fysapp#72
No description provided.