CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST2197]VHIP_GLS: GLS-comp: onafh. maken van Admin-data (transactionele en masterdata)

- betere naam voor Communicatieservice

[UST2241]VHIP_GLS: RegisterContainer implementeren

- juiste convertor

[UST2241]VHIP_GLS: RegisterContainer implementeren

- RegistreerContainerRequest naar LeveringInfo convertor

  1. /WSimpl/AX/Transport/GLS/v01/RegistrationService
[UST2197]VHIP_GLS: GLS-comp: onafh. maken van Admin-data (transactionele en masterdata)

- code afgezonderd in factory

    • -0
    • +29
    /APPS/TRANSP/impl/VerpakkingEenheidFactory.cls.xml
    • -10
    • +2
    /APPS/TRANSP/impl/TransportService.cls.xml
[DEF549] LBX: dwarsverdeling bij spoelbaklade: 74mm
[DEF549] LBX: dwarsverdeling bij spoelbaklade: 74mm
[DEF549] LBX: dwarsverdeling bij spoelbaklade: 74mm

- test namen gewijzigd.

[DEF549] LBX: dwarsverdeling bij spoelbaklade: 74mm

- test gewijzigd : methods private, gemeenschappelijke code naar private method

[DEF549] LBX: dwarsverdeling bij spoelbaklade: 74mm

- in standaardlade test aantonen dat de kenmerkaanpasser opgeroepen wordt, maar geen gewijzigde waarden weergeeft.

zie commit msg

zie commit msg

[DEF549] LBX: dwarsverdeling bij spoelbaklade: 74mm

- standaardlade na aanpassing van de kenmerken.

[UW351R] Probleem Opdeelzaagoptimalisatiebestand

- Reviewer vond het beter dat iedereen de code kon lezen zonder te moeten nadenken

[DEF549] LBX: dwarsverdeling bij spoelbaklade: 74mm

- test spoelbaklade + implementatie.

[UST2197]VHIP_GLS: GLS-comp: onafh. maken van Admin-data (transactionele en masterdata)

- aanpassing voor volgnummer van verpakkingeenheid berichten

    • -0
    • +4
    /APPS/TRANSP/dto/VerpakkingEenheid.cls.xml
[BUS599]Vermelding op facturen Bluran
[BUS599]Vermelding op facturen Bluran
[DEF549] LBX: dwarsverdeling bij spoelbaklade: 74mm

- testjes gemaakt voor de bestaande functionaliteit van de dwarsverdeling.

  • More
  • CR-134
  • finished reviewing
Op het eerste zicht lijkt mij dit toch wel een beetje clever code. En proberen om de $extract in te korten om zo de maximale lengte van een lijn niet te overschrijven dat zal ook wel niet pakken. ...

Op het eerste zicht lijkt mij dit toch wel een beetje clever code.

En proberen om de $extract in te korten om zo de maximale lengte van een lijn niet te overschrijven dat zal ook wel niet pakken.

Het is beter indien de UItvoeringInfo.DueOutTijdstip een %TimeStamp om de FormatAPI:FormatTimeStamp te gebruiken kwestie van het allemaal toch een beetje leesbaarder te hebben voor de gewone sterveling

  • More
  • CR-136
  • finished reviewing
Volgens mij gaat deze test identiek hetzelfde testen dan één van de LijnCacheBuilder testen .. DRY

Volgens mij gaat deze test identiek hetzelfde testen dan één van de LijnCacheBuilder testen .. DRY

Geen logica gebruiken in testen .. beter hiervoor een hardcoded waarde voor gebruiken . zal wel te vinden zijn in de vhTest.Utils Als de ToArray faalt dan zal dit negatieve inpakt hebben op de test...

Geen logica gebruiken in testen .. beter hiervoor een hardcoded waarde voor gebruiken . zal wel te vinden zijn in de vhTest.Utils
Als de ToArray faalt dan zal dit negatieve inpakt hebben op de testen terwijl het er los van staat.

  • More
  • CR-129
  • finished reviewing
[UST2159] Vh4 B.E.: TAX conversie e-Con / APPS.dto (enkel standaardlade)

- EconNaarApps conversie aanpassen : Tax Standaardlade op Apps aangepast om beter aan te sluiten bij tbx en lbx. IngaveTotBestelling test aangepast

  1. … 78 more files in changeset.
  • More
  • CR-118
  • finished reviewing
?? Bedoeling van dit veld ? Is een herhaling van de KlantNaam niet ok ???

?? Bedoeling van dit veld ? Is een herhaling van de KlantNaam niet ok ???

Kunnen we het nummer ook niet halen vanuit die andere Utils ???

Kunnen we het nummer ook niet halen vanuit die andere Utils ???

Goed idee om die data vanuit een andere Utils te halen

Goed idee om die data vanuit een andere Utils te halen

Misschien moeten we het iets explicieter maken dat het gaat over "EenBelg" of misschien nog beter "DenBelg" --> natuurlijk zal dat wel het gevolg hebben dat we "DenBelg" als begrip overal moeten ge...

Misschien moeten we het iets explicieter maken dat het gaat over "EenBelg" of misschien nog beter "DenBelg" --> natuurlijk zal dat wel het gevolg hebben dat we "DenBelg" als begrip overal moeten gebruiken.
Dan lijkt het mij nog beter om dat "DeBelgischeOnderneming" te nemen

[UST2159] Vh4 B.E.: TAX conversie e-Con / APPS.dto (enkel standaardlade)

- EconNaarApps conversie aanpassen : Tax Standaardlade op Apps aangepast om beter aan te sluiten bij tbx en lbx. Na deze commit zullen er wel wat dingen falen op het bord.

    • -0
    • +45
    /ECONimpl/PM/Maatwerk/AppsConverter/EconNaarApps/Tax.cls.xml
Admin database-level account voor Tommy Hebb.
    • -0
    • +25
    /vhDeploy/Deploy20170317xPRASQLUserAddTommyHebb.cls.xml