Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:

- Refactor => De tekstlijnen, met uitzondering van de titellijn, zijn nu herleid tot 1 enkele Opmerking => Bugfix omdat er een Leverbon was met teveel Opmerkingen-tags en dit een error gaf in Docbase

  1. … 1 more file in changeset.
Good one! Merci voor de hint!!

Good one! Merci voor de hint!!

Dat leek mij wat overkill, omdat de relevante change slechts op 1 plek in 1 klasse zit. Ik dacht dat een ProjectSetting gebruikt werd als je met vele changes in vele klassen zit, die tegelijkertijd...

Dat leek mij wat overkill, omdat de relevante change slechts op 1 plek in 1 klasse zit. Ik dacht dat een ProjectSetting gebruikt werd als je met vele changes in vele klassen zit, die tegelijkertijd aan/af moeten kunnen gezet worden.

Is voor het stukje hieronder eigenlijk de ProjectSettingsApi niet bruikbaar?

Is voor het stukje hieronder eigenlijk de ProjectSettingsApi niet bruikbaar?

klein detail (wrschl niet de moeite om aan te passen): In bovenstaand geval is het soms interessanter/properder om het volgende te doen: If $$$AssertEquals(..OpmerkingenLijst.Count(),2) {{ ... de ...

klein detail (wrschl niet de moeite om aan te passen): In bovenstaand geval is het soms interessanter/properder om het volgende te doen:

If $$$AssertEquals(..OpmerkingenLijst.Count(),2) {{
... de andere asserts
}} (dubbele accolades omdat die Jira anders denkt dat het een macro is)

Stel dat het aantal 1 is, krijg je maar 1 test-errormelding, anders gaat de .GetAt(2) ook nog eens een melding geven die je misschien op het verkeerde been zet bij het proberen fixen

[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:
[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:
[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:

- Toevoeging check of KlantId op gevonden order wel zelfde is als KlantId op de bon => range wordt hergebruikt dus check is niet overbodig, doch ook niet helemaal sluitend, maar toch al tot de klant zelf beperkend

- Kleine refactor van logica van opeenvolgende stappen

  1. … 1 more file in changeset.
[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:

- Grondige refactor => TekstLijn-objectjes gaven geen mooi resultaat op de bon => Lijst van Opmerkingen wordt nu gebruikt om Naleveringsdata te tonen => gebeurt voor flbLevBonEnd om als eerste weer te geven in opmerkingenkader

- GeefStandaardVertaler wordt nu gebruikt om geen groep te moeten specifiëren => vertaling van "Order" bestond al, maar in andere groep

- Lichte bijsturing formatering tekst hier en daar

  1. … 2 more files in changeset.
[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:

- Refactor => Juistere formulering van de UnitTest-namen

- Refactor => OrderId-increment gaat nu per 10 => zorgt voor nog meer diversiteit in Ids, aantallen en data

- Extra UnitTest => 3 orders met elk 2 orderlijnen met allemaal verschillende producten

- New'en van ProductApiStub gebeurt in GeefLeverbon => hierdoor wordt slechts met 1 instantie gewerkt die in de property geïnjecteerd wordt => liep fout sinds alle ProductIds uniek zijn geworden => laatste instantie werd in property gestoken, met toevallig zelfde id's voor IndienMethodCalls als gebruikt voor eerdere instanties (GeefProductOrderLijnIterator per order)

[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:

- Wat meer variatie in aantal openstaande orders en aantal openstaande orderlijnen

[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:

- Toevoeging dat het om richtdata gaat => codechange

  1. … 1 more file in changeset.
[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:

- Fix voor veranderlijke leverdatum waardoor testen vanaf volgende dag faalden

- Gebruik van OrderId zijnde increment integer om meer variatie in aantallen en data te krijgen

[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:

- Komma wordt nu indien nodig wel correct verwijderd

- Uitbreiding extra UnitTest => Er zijn 2 orders met elk 2 ProductOrderLijnen met zelfde producten erop

  1. … 1 more file in changeset.
[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:

- Grondige refactor TestCase => afzondering common code

- Uitbreiding extra UnitTest => Er is 1 order met 2 ProductOrderLijnen

  1. … 1 more file in changeset.
[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:

- 1e testje => refactorke

- 2e testje => er is nog een openstaand order => staan geen productorderlijnen op => niets toevoegen aan leverbon

[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:

- Eerste testje => er zijn geen openstaande orders meer voor die bon

- Todo => verdere testjes, Deploy van vertalingen

  1. … 1 more file in changeset.