EH-1700: Give zero duration if jakso is not found from eHOKS #289
Conversation
severij
force-pushed
the
severij/kestojen-uudelleenlaskenta
branch
from
cc39f2a
to
ec4b7c0
Compare
| @@ -42,7 +42,7 @@ | |||
| kestot (nh/oppijan-jaksojen-kestot | |||
| concurrent-jaksot opiskeluoikeudet) | |||
| jakso-key (nh/ids jakso)] | |||
| (if-let [new-kesto (get kestot jakso-key)] | |||
| (if-let [new-kesto (get kestot jakso-key 0)] | |||
There was a problem hiding this comment.
Tää on sikäli bugialtista, että olis kuviteltavissa, että kestot-mapista puuttuu kyseinen avain jostain muusta syystä kuin siksi, että kesto yritettiin laskea mutta epäonnistuttiin. Mun puolesta näin voi tehdä, mutta enemmän eksplisiittisiä mahdollisuuksia olis esim. että oppijan-jaksojen-kestot laittais kestot-mappiin jonku :jaksoa-ei-loydy-keywordin numeron sijaan silloin ku laskennassa on joku ongelma.
Vakavuus: ehdotus
severij
force-pushed
the
severij/kestojen-uudelleenlaskenta
branch
from
1f4d2df
to
c0fe972
Compare
| (if-let [new-kesto (get kestot jakso-key)] | ||
| jakso-key (nh/ids jakso) | ||
| new-kesto | ||
| (if (find-first #(= (:hankkimistapa_id %) |
There was a problem hiding this comment.
tähän käy myös some, joka on semanttisesti lähempänä
vakavuus: idea
| jakso-key (nh/ids jakso) | ||
| new-kesto | ||
| (if (find-first #(= (:hankkimistapa_id %) | ||
| (:hankkimistapa_id jakso)) |
There was a problem hiding this comment.
Onko mahdollista, että nämä yhtaikaiset jaksot ovat eri HOKSeista? Silloin pitää verrata koko (nh/ids jakso):a.
vakavuus: jos on mahdollista että ovat eri HOKSeista, pyyntö.
There was a problem hiding this comment.
Kai se teoriassa on 🤔 Korjaan!
severij
force-pushed
the
severij/kestojen-uudelleenlaskenta
branch
8 times, most recently
from
0bd92fc
to
51aedb2
Compare
severij
force-pushed
the
severij/kestojen-uudelleenlaskenta
branch
from
51aedb2
to
67e671b
Compare
https://jira.eduuni.fi/browse/EH-1700