Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-163] [rvTVE] Verkoopfacturen synchronisatie is overbodig geworden. De code en backgroundjob mag...
[ICT-163] [rvTVE] Verkoopfacturen synchronisatie is overbodig geworden. De code en backgroundjob mag...
Deze routine moet nog verwijderd worden samen met de background job

Deze routine moet nog verwijderd worden samen met de background job

Voorlopig heb ik enkel de code die uitgevoerd moet worden in de AX.CARETAKER.mac in commentaar gezet om zeker te zijn dat ik de juiste backgroundjob heb genomen (de beschrijving in het kaartje leek...

Voorlopig heb ik enkel de code die uitgevoerd moet worden in de AX.CARETAKER.mac in commentaar gezet om zeker te zijn dat ik de juiste backgroundjob heb genomen (de beschrijving in het kaartje leek mij niet juist, dus heb mij gebaseerd op de titel van het kaartje en de bijhorende backgroundjob dat ik hierbij gevonden heb).

[ICT-163] [rvTVE] Verkoopfacturen synchronisatie is overbodig geworden. De code en backgroundjob mag verwijderd worden.

- Klasse 'AX.Uitgaand.BKH.OpenstaandeVerkoopFactuur.impl.TransactionToOpenstaandeFactuurIdConverter' verwijderen die enkel gebruikt werd in verwijderde klasse 'AX.Uitgaand.BKH.OpenstaandeVerkoopFactuur.impl.Synchroniser'

- Bijhorende Fake en Mock klasse verwijderen

- Bijhorende unittest verwijderen

  1. … 6 more files in changeset.
[ICT-163] [rvTVE] Verkoopfacturen synchronisatie is overbodig geworden. De code en backgroundjob mag verwijderd worden.

- Klasse 'AX.Uitgaand.BKH.OpenstaandeVerkoopFactuur.impl.TransactionRepository' verwijderen die enkel gebruikt werd in verwijderde klasse 'AX.Uitgaand.BKH.OpenstaandeVerkoopFactuur.impl.Synchroniser'

- Bijhorende Fake en Mock klasse verwijderen

- Bijhorende unittest verwijderen

  1. … 4 more files in changeset.
[ICT-163] [rvTVE] Verkoopfacturen synchronisatie is overbodig geworden. De code en backgroundjob mag verwijderd worden.

- Klasse 'AX.Uitgaand.BKH.OpenstaandeVerkoopFactuur.impl.TransactionLockingRepository' verwijderen die enkel gebruikt werd in verwijderde klasse 'AX.Uitgaand.BKH.OpenstaandeVerkoopFactuur.impl.Synchroniser'

- Bijhorende Fake en Mock klasse verwijderen

  1. … 2 more files in changeset.
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
    ./Uitgaand/PM/impl/SamenstellingBestelreferentieProductenInEenString.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
    ./Uitgaand/PM/impl/SamenstellingBestelreferentieProductenInEenString.cls.xml
  1. … 64 more files in changeset.
[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...
    • -2
    • +1
    ./Uitgaand/PM/impl/SamenstellingBestelreferentieProductenInEenString.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
    ./Uitgaand/PM/impl/IsOngewenstProductInMigratieFilter.cls.xml
    • -0
    • +19
    ./Uitgaand/PM/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:
[VRB561] Voorraad: InventTransID, ItemID, ConfigID in PRHIST bewaren:

- Fake Translator gemoved => heeft specifieke implementatie => willen we andere naam geven dan als code zelf Mockable gemaakt zou worden => beveiliging tegen eventueel overschrijven

    • -15
    • +0
    ./Uitgaand/EC/DataAX/impl/Translator.cls.xml
  1. … 1 more file in changeset.
Done!

Done!

try-catch toevoegen en lokaal afhandelen (warning ofzo, maar geen error verder throwen)

try-catch toevoegen en lokaal afhandelen (warning ofzo, maar geen error verder throwen)

misschien nog wat testjes voor het geval de bon zélf niet bestaat

misschien nog wat testjes voor het geval de bon zélf niet bestaat

Best niet in de constructor zetten, maar in een aparte method. In dit concreet geval maakt dat natuurlijk niet echt uit, maar 't is "good practice".

Best niet in de constructor zetten, maar in een aparte method. In dit concreet geval maakt dat natuurlijk niet echt uit, maar 't is "good practice".

[VRB561] Voorraad: InventTransID, ItemID, ConfigID in PRHIST bewaren:
[VRB561] Voorraad: InventTransID, ItemID, ConfigID in PRHIST bewaren:
[VRB561] Voorraad: InventTransID, ItemID, ConfigID in PRHIST bewaren:

- Fake Translator nodig voor UnitTests

    • -0
    • +15
    ./Uitgaand/EC/DataAX/impl/Translator.cls.xml