ks-no / fiks-arkiv

4 stars 0 forks source link

Bør sortering og parameter også være relatert til respons? #105

Closed jarleborsheim closed 2 years ago

jarleborsheim commented 2 years ago

Vi endret inkluder til å bare inneholde gyldige felter ut i fra hvilken respons man søker etter i issue #99 Men burde ikke det være slik for sortering og parameter også? At hvis man har definert respons til å være Saksmappe så får man bare velge sorteringsfelter og parametere som er gyldig for Saksmappe.

Eksempelkode slik det er nå:

var parameter = new Parameter()
{
    Felt = SokFelt.RegistreringTittel,
    Operator = OperatorType.Equal,
    Parameterverdier = new Parameterverdier()
    {
        Stringvalues = { sokeord }
    }
};
sok.Parameter.Add(parameter);
sok.ResponsType = ResponsType.Utvidet;
sok.Respons = new JournalpostRespons()
{
    Inkluder = InkluderIJournalpost.Avskrivning
};
var sortering = new Sortering()
{
    Felt = SorteringFelt.RegistreringOpprettetDato,
    Type = SorteringType.Synkende
} 
sok.Sortering.Add(sortering);
jarleborsheim commented 2 years ago

Inkluder skal også være unbounded. En PR ligger ute for det her: https://github.com/ks-no/fiks-arkiv-specification/pull/45

jarleborsheim commented 2 years ago

Eksempel på kode etter refaktorering og rydding:

           var sok = new Models.V1.Innsyn.Sok.Sok
            {
                Sokdefinisjon = new JournalpostSokdefinisjon()
                {
                    Inkluder = { JournalpostInkluder.Korrespondansepart },
                    Parametere = { 
                        new JournalpostParameter()
                        {
                            Felt = JournalpostSokefelt.RegistreringTittel,
                            Operator = OperatorType.Equal,
                            SokVerdier = new SokVerdier()
                            {
                                Stringvalues = { "En journalpost tittel med wildcard*" }
                            }
                        } 
                    },
                    Sortering =
                    {
                        new JournalpostSortering()
                        {
                            Felt = JournalpostSorteringsfelt.RegistreringOpprettetDato
                        }
                    }
                },
                Take = 10,
                System = "Integrasjonstester",
                Tidspunkt = DateTime.Now,
                MeldingId = Guid.NewGuid().ToString()
            };
jarleborsheim commented 2 years ago

Stenger da denne er merget og releaset