Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Filteren op velden van het type enum #61

Open
FilipBruins opened this issue Jan 8, 2024 · 2 comments
Open

Filteren op velden van het type enum #61

FilipBruins opened this issue Jan 8, 2024 · 2 comments

Comments

@FilipBruins
Copy link

Het is niet mogelijk om te filteren op velden van het type enum, de API geeft dan geen resultaten. Voorbeeld: wanneer je EducationSpecifications filtert op educationSpecificationType dan krijg je geen resultaten. Het probleem lijkt te zitten in de OrderedQueryable class, waar de FilterBy methode niet kan omgaan met filters van het type enum.

@FilipBruinsHAN
Copy link
Collaborator

Wij gaan dit vanuit de HAN voorlopig niet oppakken, tenzij dit door RIO als bug wordt teruggemeld. Ik had al wel een klein stukje code geschreven om enum's af te handelen:

if (Property.PropertyType.IsEnum || (Property.PropertyType.IsGenericType && Property.PropertyType.GetGenericTypeDefinition().IsEnum))
 {
     if (Value is not null && Value is string stringValue && !string.IsNullOrEmpty(stringValue))
     {
         // TODO: Equals for enum value
     }
 }

@bartvriesema
Copy link
Collaborator

Voor RIO lijkt het mij ook niet nodig om dit op te lossen. Volgens de documentatie dienen alleen de volgende calls afgehandeld te worden:

  • GET /education-specifications/{educationSpecificationId}?returnTimelineOverrides=true
  • GET /programs/{programId}?returnTimelineOverrides=true
  • GET /programs/{programId}/offerings?pageSize=250&consumer=rio
  • GET /courses/{courseId}?returnTimelineOverrides=true
  • GET /courses/{courseId}/offerings?pageSize=250&consumer=rio

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants