Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-808
  • finished reviewing
[UST3099] VHIP Migratie: 1402 aankooporderregels bevat niet-gemigreerde producten:
[UST3099] VHIP Migratie: 1402 aankooporderregels bevat niet-gemigreerde producten:
[UST3099] VHIP Migratie: 1402 aankooporderregels bevat niet-gemigreerde producten:

- Beetje refactoring => kortere klasse en makkelijker alles terug te vinden

- Volgens leverancierreferentie onbestaand IdentNummer wordt vervangen door een bestaand volgens de prefix

- Check of dit een SoprVervanger heeft en indien ja daar het IdentNummer van doorgeven zodat het er eentje is dat AX kent

    • -120
    • +40
    ./AankoopOrderRegelDixfConverter.cls.xml
  • More
  • CR-700
  • finished reviewing
[UST3036] Migratie - naamgeving bestanden:
[UST3036] Migratie - naamgeving bestanden:
[UST3036] Migratie - naamgeving bestanden:

- Vestiging tussen nummer en naam van migratie

- Aanpassing in RunAll2a

    • -2
    • +2
    ./AankoopOrderRegelDixfGenerator.cls.xml
  1. … 1 more file in changeset.
  • More
  • CR-570
  • finished reviewing
Tracht zoveel mogelijk gebruik te maken van vhTest.Utils voor de standaard gevallen , zodat de setup / refactor en andere zaken simpeler kan.

Tracht zoveel mogelijk gebruik te maken van vhTest.Utils voor de standaard gevallen , zodat de setup / refactor en andere zaken simpeler kan.

[UST2905]VHIP481 BOMBOL: Product ophalen adhv identnr indien geen productnr opgegeven
[UST2905]VHIP481 BOMBOL: Product ophalen adhv identnr indien geen productnr opgegeven
[UST2905]VHIP481 BOMBOL: Product ophalen adhv identnr indien geen productnr opgegeven

- prefactor: BlumProductAPI los van ProductAPI

    • -3
    • +3
    ./AankoopOrderRegelDixfConverter.cls.xml
  1. … 3 more files in changeset.
  • More
  • CR-514
  • finished reviewing
Zoals ik in vorige review vroeg... hoe doe je dat met een ClassMethod? Heb het geprobeerd, maar lukte niet. Wellicht deed ik het fout?

Zoals ik in vorige review vroeg... hoe doe je dat met een ClassMethod? Heb het geprobeerd, maar lukte niet. Wellicht deed ik het fout?

Zelfde commentaar als hierboven. Het komt erop neer dat de volgorde van de stub niet overal hetzelfde mag/kan zijn, afhankelijk van wat er getest wordt. Soms draait de dependency om.

Zelfde commentaar als hierboven. Het komt erop neer dat de volgorde van de stub niet overal hetzelfde mag/kan zijn, afhankelijk van wat er getest wordt. Soms draait de dependency om.

Nee, want in sommige testen is een andere opzet nodig, waar EERST een lokale andere setting nodig is van een andere stub, die in die lokale UtilsStub gebruikt wordt. Als ik een default van de Utils...

Nee, want in sommige testen is een andere opzet nodig, waar EERST een lokale andere setting nodig is van een andere stub, die in die lokale UtilsStub gebruikt wordt.
Als ik een default van de UtilsStub in de onbeforeonetest steek, dan loopt dat mis.

Misschien dat ge de beide vestigingen door parameters kunt vervangen, en dan moet ge gewoon met een # die oproepen (weer een lijn minder)

Misschien dat ge de beide vestigingen door parameters kunt vervangen, en dan moet ge gewoon met een # die oproepen (weer een lijn minder)

kunnen deze 2 lijnen ook niet naar de onbeforeonetest? (ook andere testen checken)

kunnen deze 2 lijnen ook niet naar de onbeforeonetest? (ook andere testen checken)

Converter vervangen door ..GeefConverter van 2 lijnen hoger

Converter vervangen door ..GeefConverter van 2 lijnen hoger

kunnen deze 2 lijnen niet naar de onbeforeonetest? (voor alle testen)

kunnen deze 2 lijnen niet naar de onbeforeonetest? (voor alle testen)

[UST2825] VHIP - migratie: 1402 aankooporderregels - ext.art.omschrijving: Aanpassing:
[UST2825] VHIP - migratie: 1402 aankooporderregels - ext.art.omschrijving: Aanpassing:
[UST2825] VHIP - migratie: 1402 aankooporderregels - ext.art.omschrijving: Aanpassing:

- Zonder puntjes indien BlOrdRsp -> Is altijd Blum

- Onderscheid indien DOM.AKP.ProductToeleveringLijn -> Indien Blum -> puntjes weg (aanpassen testen + extra test)

    • -2
    • +4
    ./AankoopOrderRegelDixfConverter.cls.xml
  1. … 1 more file in changeset.
[UST2524] VHIP - SOPR - aanpassingen:

- Kleine refactoring

- Bijwerken UnitTests (stonden in comment omdat ze niet meer werkten door SOPR-implementatie)

    • -71
    • +50
    ./AankoopOrderRegelDixfConverter.cls.xml
  1. … 1 more file in changeset.
[UST2535] VHIP - laatste aanpassingen migratie voor sprint 10

- SOPR: Wordt nu voor alle Blum-producten juist geconverteerd

- Van Leverancier een property gemaakt voor leesbaarheid/werkbaarheid

- Objecttype van Leverancier is nu de interface

    • -51
    • +54
    ./AankoopOrderRegelDixfConverter.cls.xml
[UST2535] VHIP - laatste aanpassingen migratie voor sprint 10

- Voorziening extra property/veld "Origin" ter controle (welk type object is het? Toelevering of BlOrdRsp?)

P.s.: Staat in generator in comment, want mag niet mee in de migratie!

    • -0
    • +1
    ./AankoopOrderRegelDixfConverter.cls.xml
    • -0
    • +1
    ./AankoopOrderRegelDixfGenerator.cls.xml
  1. … 1 more file in changeset.
[UST2535] VHIP - laatste aanpassingen migratie voor sprint 10

- Uniformiteit betreffende Vestiging

- Uniformiteit betreffende timer en bijhorende boodschap

    • -7
    • +4
    ./AankoopOrderRegelDixfConverter.cls.xml
    • -15
    • +13
    ./ProductToeleveringLijnIterator.cls.xml
    • -1
    • +1
    ./AankoopOrderRegelDixfGenerator.cls.xml
  1. … 22 more files in changeset.
[UST2535] VHIP - laatste aanpassingen migratie voor sprint 10

- FirmaInitialen

    • -1
    • +1
    ./AankoopOrderRegelDixfConverter.cls.xml
[UST2524] VHIP - SOPR - aanpassingen:

- Implementatie SOPR-conversie

- TO DO: Testen met BlOrdRsp herstellen ('kapot' door implementatie SOPR)

    • -11
    • +108
    ./AankoopOrderRegelDixfConverter.cls.xml
  1. … 1 more file in changeset.