Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Reverse merged revision(s) 56420 from trunk2010:

Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

revert van de revert omdat mijn wijzigingen wel goed waren, maar de rest van de ventjes niet :)

    • -2
    • +1
    ./impl/SamenstellingBestelreferentieProductenInEenString.cls.xml
    • -2
    • +1
    ./impl/Convertor/Sopr/ProductConvertor.cls.xml
  1. … 64 more files in changeset.
Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

    • -1
    • +2
    ./impl/SamenstellingBestelreferentieProductenInEenString.cls.xml
    • -1
    • +2
    ./impl/Convertor/Sopr/ProductConvertor.cls.xml
  1. … 64 more files in changeset.
[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...
    • -2
    • +1
    ./impl/SamenstellingBestelreferentieProductenInEenString.cls.xml
    • -2
    • +1
    ./impl/Convertor/Sopr/ProductConvertor.cls.xml
  1. … 64 more files 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

    • -0
    • +26
    ./impl/IsOngewenstProductInMigratieFilter.cls.xml
    • -0
    • +19
    ./impl/SoprIteratorFilter.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.

[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:
[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:
  • More
  • CR-820
  • finished reviewing
[UST3301] VHIP Migratie: 0602VrijgegProducten: Migratie aliassen - deel 2 - bestaande prod en hun...
[UST3301] VHIP Migratie: 0602VrijgegProducten: Migratie aliassen - deel 2 - bestaande prod en hun...
[UST3301] VHIP Migratie: 0602VrijgegProducten: Migratie aliassen - deel 2 - bestaande prod en hun alternatief:

- Velden AltItemId en UseAltItemId

- DTO VrijgegProduct => properties erbij gezet en alle properties gesorteerd volgens volgorde PakketReferentie VrijgegProducten

    • -0
    • +19
    ./VrijgegProduct/impl/AliasConverter.cls.xml
  1. … 9 more files in changeset.
  • More
  • CR-796
  • finished reviewing
[UST3300] VHIP Migratie: 0609BehoeftePlanning: Afzonderen VeiligheidsVoorraad:
[UST3300] VHIP Migratie: 0609BehoeftePlanning: Afzonderen VeiligheidsVoorraad:
[UST3300] VHIP Migratie: 0609BehoeftePlanning: Afzonderen VeiligheidsVoorraad:

- De VeiligheidsVoorraadCalculator is nu Mockable

- Aanpassing van de Converter => verwijst nu naar VeiligheidsVoorraadCalculator

    • -0
    • +19
    ./BehoeftePlanning/impl/VeiligheidsVoorraadCalculator.cls.xml
  1. … 6 more files in changeset.
[UST3255] VHIP Migr - SoprConverter: Aanpassing PV: BSKey
    • -0
    • +6
    ./impl/Convertor/Sopr/ProductConvertor.cls.xml
  1. … 1 more file in changeset.
Jo, Is het "MIN Extradagen" of "PLUS" extra dagen? want in kanban staat "... dagen bijtellen" . Deze opmerking zonder hier echt verder over na te denken. Als je dit zelf effe kritisch bekijkt, of ...

Jo,
Is het "MIN Extradagen" of "PLUS" extra dagen? want in kanban staat "... dagen bijtellen" .

Deze opmerking zonder hier echt verder over na te denken. Als je dit zelf effe kritisch bekijkt, of afstemt met analyst/opdrachtgever, dan is het voor mij goed genoeg

  • More
  • CR-750
  • finished reviewing
[UST3122] VHIP184 - LeadTimePurch - rekening houden met 'ExtraLeveringsdagen'
[UST3122] VHIP184 - LeadTimePurch - rekening houden met 'ExtraLeveringsdagen'
[UST3122] VHIP184 - LeadTimePurch - rekening houden met 'ExtraLeveringsdagen'

- extraleverdagen in mindering brengen in de berekening van de levertermijn

- opkuis

    • -10
    • +5
    ./VrijgegProduct/impl/VoorraadInstellingCalculator.cls.xml
  1. … 5 more files in changeset.
  • More
  • CR-715
  • finished reviewing