Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT]

- backport wijzigingen door PV

    • -1
    • +5
    ./BehoeftePlanning/impl/JaarAfnameGenerator.cls.xml
[ICT]

- backport wijzigingen door PV

    • -2
    • +17
    ./SOPRContract/impl/SOPRExport.cls.xml
[ICT-] [rvWV] PM - lijst van artikelnummers uit Admin krijgen

- Method hernoemd naar GenereerAantalArtikelNummers omdat het niets teruggeeft

[ICT-] [rvWV] PM - lijst van artikelnummers uit Admin krijgen

- Refactor: GeefInAantal en GenereerLijstVanArtikelNummers zijn nu ProcedureBlocks, Aantal is een property en andere variabelen geNewed

    • -23
    • +45
    ./Product/impl/ArticleIDCHUI.cls.xml
[ICT-] [rvWV] PM - lijst van artikelnummers uit Admin krijgen

- Typo's gefixed en 'BestandVolledigeNaam' ipv 'BestandNaamMetPad'

- Indien BestandVolledigeNaam leeg is een foutmelding op het bericht laten verschijnen

    • -9
    • +15
    ./Product/impl/ArticleIDCHUI.cls.xml
variabelenaam is zeker geen verkeerde keuze, maar "BestandVolledigeNaam" is best te gebruiken volgens onze conventies (maar ik vind ze zelf ook niet terug op confluence)

variabelenaam is zeker geen verkeerde keuze, maar
"BestandVolledigeNaam" is best te gebruiken volgens onze conventies (maar ik vind ze zelf ook niet terug op confluence)

Ik ben geen voorstander om veel methods in "ProcedureBlock = 0" te schrijven. Enkel daar waar het nodig is. In dit geval zit je in een Admin-scherm, dus is er sowieso al een deel "ProcedureBlock = ...

Ik ben geen voorstander om veel methods in "ProcedureBlock = 0" te schrijven. Enkel daar waar het nodig is.
In dit geval zit je in een Admin-scherm, dus is er sowieso al een deel "ProcedureBlock = 0".

Maar als je deze wil wegwerken, moet je zeker opletten want soms is dat nogal tricky.
Geef me een seintje als je hierover meer uitleg wenst.

Typo : Wacht om (A)f te sluiten http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Typo : Wacht om (A)f te sluiten

Typo op meerdere plaatsen : artikkel met 1 "k"

Typo op meerdere plaatsen : artikkel met 1 "k"

[ICT-1793] [rvWV] PM - lijst van artikelnummers uit Admin krijgen
[ICT-1793] [rvWV] PM - lijst van artikelnummers uit Admin krijgen
[ICT-1793] [rvWV] PM - lijst van artikelnummers uit Admin krijgen

- Kleine wijziging: nu moet er expliciet aantal 1 ingegeven worden om 1 artikelnummer te genereren. Als er geen getal of 0 wordt ingegeven wordt er niets gegenereerd.

[ICT-] [rvWV] PM - lijst van artikelnummers uit Admin krijgen

- Indien expliciet 0 wordt meegegeven als aantal worden er nu geen artikelnummers gegenereerd

[ICT-] [rvWV] PM - lijst van artikelnummers uit Admin krijgen

- Indien aantal artikelnummers groter is dan 1 wordt er een file gemaakt in de shared folder voor de user

    • -17
    • +37
    ./Product/impl/ArticleIDCHUI.cls.xml
[ICT-] [rvWV] PM - lijst van artikelnummers uit Admin krijgen

- Nieuwe input: aantal artikelnummers te genereren

- Er wordt een lijst van artikelnummers gemaakt

    • -2
    • +27
    ./Product/impl/ArticleIDCHUI.cls.xml
[ICT-2189][rvJWI] MAG: Leverbon: VHIP903: PickingCompleet: soms ontbrekende lijn in bericht naar AX...
[ICT-2189][rvJWI] MAG: Leverbon: VHIP903: PickingCompleet: soms ontbrekende lijn in bericht naar AX...
[ICT-2189][rvJWI] MAG: Leverbon: VHIP903: PickingCompleet: soms ontbrekende lijn in bericht naar AX (gemerkt via "Verschillen AX versus Admin")

- Equals ipv Contains

    • -2
    • +2
    ./impl/IsOngewenstProductInMigratieFilter.cls.xml
[ICT-1226] VHIP903 - VoorraadCacheToAx: serviceproduct introductie: 7.001.213.2:

- Product 2465552 wordt uitgesloten door filter

    • -1
    • +1
    ./impl/IsOngewenstProductInMigratieFilter.cls.xml
Reverse merged revision(s) 59500 from trunk2010:

[ICT-1159] Mapping leveranciersnummer VH/HLX:

- Leveranciersnummer 9106 = 06815

- CODE-AANPASSING MOET IN ANDERE KLASSE GEBEUREN !!!

    • -1
    • +0
    ./impl/Convertor/ConvertorLeveranciersnummerVanHoeckeNaarHalux.cls.xml
  1. … 1 more file in changeset.
[ICT-1159] Mapping leveranciersnummer VH/HLX:

- Leveranciersnummer 9106 = 06815

    • -0
    • +1
    ./impl/Convertor/ConvertorLeveranciersnummerVanHoeckeNaarHalux.cls.xml
  1. … 1 more file in changeset.
DOM.PM.enu.LinkType (hieronder nog een paar keer) (en ook in ander testje)

DOM.PM.enu.LinkType (hieronder nog een paar keer) (en ook in ander testje)

my point exactly (zie comment). Afzonderen naar apart klaske dat getest kan worden

my point exactly (zie comment).
Afzonderen naar apart klaske dat getest kan worden

TestCases gemaakt voor beide public methods (en onderweg maken van product afgezonderd)

TestCases gemaakt voor beide public methods (en onderweg maken van product afgezonderd)

[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:

- Na review => Mockable gemaakt om AxProductValidator onder test te brengen

    • -1
    • +1
    ./impl/IsOngewenstProductInMigratieFilter.cls.xml
  1. … 4 more files in changeset.
ClassMethod!

ClassMethod!

Als ge de lijn toch verandert, kunt ge ineens de ' door $$$Not vervangen

Als ge de lijn toch verandert, kunt ge ineens de ' door $$$Not vervangen

Boyscout principe: er bestaan geen testjes van deze klasse. Misschien is het niet veel moeite om een paar dingen toe te voegen?

Boyscout principe: er bestaan geen testjes van deze klasse. Misschien is het niet veel moeite om een paar dingen toe te voegen?

Op den duur zou een mens zich beginnen afvragen of al deze StartVerwerking-code niet beter in een aparte klasse gestoken wordt. Ooit is dat begonnen als een klein methodje dat niet te veel doet (ju...

Op den duur zou een mens zich beginnen afvragen of al deze StartVerwerking-code niet beter in een aparte klasse gestoken wordt. Ooit is dat begonnen als een klein methodje dat niet te veel doet (juist wat parameters zetten enzo) en dan was het geen probleem dat het niet in een niet-testbare classmethod steekt. Met al de extra logica die er ondertussen is bijgekomen is dat eigenlijk niet goed meer.