Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2373][rvPVR] eCon TBX voor BIDI : Rugwandmateriaal hout/staal beschikbaar (deel 1)
[ICT-2373][rvPVR] eCon TBX voor BIDI : Rugwandmateriaal hout/staal beschikbaar (deel 1)
[ICT-2124] Move Prod.Kenmerk.impl.DataDefinitieRepository klasse
[ICT-2124] Move Prod.Kenmerk.impl.DataDefinitieRepository klasse
Is toch gewone Caché-functionaliteit: $get zal bij lege eerste parameter defaulteren naar de tweede

Is toch gewone Caché-functionaliteit: $get zal bij lege eerste parameter defaulteren naar de tweede

Was wel effe moeilijk te begrijpen van $Get en List ... Meteen duidelijk als je zet :  If '$Data(List) Set List = LangtekstPieces.Engels)

Was wel effe moeilijk te begrijpen van $Get en List ...
Meteen duidelijk als je zet :

 If '$Data(List)  Set List = LangtekstPieces.Engels)
Of of of ... Beter StringUtils. Equals() gebruiken

Of of of ... Beter StringUtils. Equals() gebruiken

[UW709] Taalprobleem in Prod.ProductTekst
[UW709] Taalprobleem in Prod.ProductTekst
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:
Er zit een probleem in de implementatie van ..ConverterData.Add() --> we zullen dat maandag samen effe bekijken. Nu geen tijd aan spenderen!

Er zit een probleem in de implementatie van ..ConverterData.Add() --> we zullen dat maandag samen effe bekijken. Nu geen tijd aan spenderen!

Bedenkingen bij naamgeving en impl : Product (Edi.LadeTBX) wordt hier niet gebruikt, dus mag die parameter weg, en is het eigenlijk geen convert, maar gewoon Bereken...() of Bepaal...() en dan kunn...

Bedenkingen bij naamgeving en impl :
Product (Edi.LadeTBX) wordt hier niet gebruikt, dus mag die parameter weg, en is het eigenlijk geen convert, maar gewoon Bereken...() of Bepaal...()
en dan kunnen de 2 private methods samengevoegd worden tot 1.

[UST4130] VH4: TBX conversie geven problemen bij Mengvorm
[UST4130] VH4: TBX conversie geven problemen bij Mengvorm
  • More
  • CR-405
  • finished reviewing
deze lijn is niet nodig, dedie hieronder ook niet. Dat wordt in principe opgelost door de ..ZetVerwachteExceptie van hierboven. De productiecode zelf zal wel een exceptie raisen waarop je moet test...

deze lijn is niet nodig, dedie hieronder ook niet. Dat wordt in principe opgelost door de ..ZetVerwachteExceptie van hierboven. De productiecode zelf zal wel een exceptie raisen waarop je moet testen (dus met dezelfde omschrijving, en niet 'onze exceptie')

Heb degene voorzien die, volgens mijn inschatting, de klasse zelf testen en niet de dependencies

Heb degene voorzien die, volgens mijn inschatting, de klasse zelf testen en niet de dependencies

nog een paar unittestjes voorzien: fout identnummer, goed identnummer, identnummer zonder resultaat, standaardgedrag, enz enz

nog een paar unittestjes voorzien: fout identnummer, goed identnummer, identnummer zonder resultaat, standaardgedrag, enz enz

Bovenstaande lijn best naar de repo die hieronder gebruikt wordt. Proper uit te mocken. Daar ter plekke wat testjes voorzien. De repo moet een 'geefXXX' voorzien en een error raisen in het geval er...

Bovenstaande lijn best naar de repo die hieronder gebruikt wordt. Proper uit te mocken. Daar ter plekke wat testjes voorzien. De repo moet een 'geefXXX' voorzien en een error raisen in het geval er niets of meer dan 1 wordt gevonden