altibiz / ozds

OZDS
Other
0 stars 0 forks source link

54 readasync filter changed from string #88

Closed GabrijelAB closed 3 weeks ago

GabrijelAB commented 1 month ago

Task

@HrvojeJuric

Fixes #54.

Description

Changed all instances of Read-in to the db using strings so that it uses Expressions.

Testing

N/A

Documentation

N/A

HrvojeJuric commented 1 month ago

mislim da se nismo razumjeli s ovim - nije poanta promjenit tu funkciju da ne prima string nego maknut sva mjesta di se koristi sa funkcijama specijaliziranim za taj usecase i onda maknut tu funkciju nikad nije dobro imat neki "master" query koji se koristi svugdje - nije optimizirano i puno koda ovisi o toj funkciji sto dovodi do toga da je ta funkcija jako brittle i jedini razlog zasto smo to imali u pocetku je da mozemo brzo napravit hrpu stvari - u smislu da male promjene u toj funkciji mogu izazvati bugove na hrpi nepredvidjenih mjesta to je nekad ok ali se onda na ta mjesta uvede hrpu testova a ove linq funkcije nam je tesko testirati na nacin kako smo testirali math i vrijeme