Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Opgelet : DebugInfo mee gecommit : !!! BELANGRIJK : Write-statements kunnen de response van de WebService (in dit geval WS.AX.DocGen. vanuit BizTalk) écht kapot maken. Dus zeker terug weghalen.

Opgelet : DebugInfo mee gecommit :

!!! BELANGRIJK :
Write-statements kunnen de response van de WebService (in dit geval WS.AX.DocGen. vanuit BizTalk) écht kapot maken.
Dus zeker terug weghalen.

[ICT-4902] aanpassing nodig op offerte/OB/factuur
[ICT-4902] aanpassing nodig op offerte/OB/factuur
Drie slashes gebruiken voor "method commentaar" dan heb je geen UDLText brol in de export

Drie slashes gebruiken voor "method commentaar"
dan heb je geen UDLText brol in de export

[ICT-4752] FW: Claims afgezet urgent!
[ICT-4752] FW: Claims afgezet urgent!
Terechte opmerking : de bewoording is niet echt duidelijk. De keuze van "Leeg" i.p.v. "Nieuw" heb ik aangepast. Echter volgens onze VH-coding guidelines moet het werkwoord steeds vooraan in de met...

Terechte opmerking : de bewoording is niet echt duidelijk. De keuze van "Leeg" i.p.v. "Nieuw" heb ik aangepast.

Echter volgens onze VH-coding guidelines moet het werkwoord steeds vooraan in de methodnaam staan, en mag niet gesplitst worden, dus "VoegToe...() "
Toch heb ik de methodnaam iets beter gemaakt. En het "ophalen van het eerste Order van de FactuurData" ook iets meer expliciet gemaakt door af te zonderen in een private method.
Commit zal toegevoegd worden aan deze review.

Het is zeker en vast belangrijk om de juiste haakjes te plaatsen. Dat heb je goed onthouden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif DOCU: Caché ...

Het is zeker en vast belangrijk om de juiste haakjes te plaatsen. Dat heb je goed onthouden
DOCU: Caché evalueert "van links naar rechts", ZONDER voorrang van bewerkingen (zoals bij + en x )
Dus bij het tweede deel van de if zijn de haakjes zeker nodig :

 (FactuurData.LEVERINGEN.Count() > 0) 

Maar bij

IsProformaFactuur

zet ik meestal geen haakjes, want dit is een boolean variabele, dat kan nooit mislopen.

Ik zou deze methodnaam omdraaien naar "VoegNieuwOrderVooraanToeAanFactuur()". Het duurde ook even voor ik doorhad wat hier exact gebeurde, dus misschien kan 'Nieuw' door 'Leeg' vervangen worden?

Ik zou deze methodnaam omdraaien naar "VoegNieuwOrderVooraanToeAanFactuur()".

Het duurde ook even voor ik doorhad wat hier exact gebeurde, dus misschien kan 'Nieuw' door 'Leeg' vervangen worden?

Ik zou hier persoonlijk extra haakjes toevoegen, omdat mij is verteld dat caché hier soms moeilijk mee kan doen (geen idee of het hier in dit geval ook echt nodig is): If ( ... )

Ik zou hier persoonlijk extra haakjes toevoegen, omdat mij is verteld dat caché hier soms moeilijk mee kan doen (geen idee of het hier in dit geval ook echt nodig is): If ( ... )

[ICT-2489][rvJBA] PM - prijswijziging 1/10/21 - extra tekst op proforma factuur
[ICT-2489][rvJBA] PM - prijswijziging 1/10/21 - extra tekst op proforma factuur
Ik heb mijn twijfels over de naamgeving van deze method. We gaan hier in principe enkel checken of de klant HaefeleBE is en we gaan niets gaan aanmaken denk ik dan?

Ik heb mijn twijfels over de naamgeving van deze method. We gaan hier in principe enkel checken of de klant HaefeleBE is en we gaan niets gaan aanmaken denk ik dan?

ICT-1597: [rvPVR] Fop Factuur : voor Klant Haefele langteksten verwijderen
ICT-1597: [rvPVR] Fop Factuur : voor Klant Haefele langteksten verwijderen
[VRB508] AX-DocGen : FactuurService crasht op "& amp;" in MEMO-tag
[VRB508] AX-DocGen : FactuurService crasht op "& amp;" in MEMO-tag
OnAfterAllTests : verwijderen ConfigItem ( , ..#Applicatie)

OnAfterAllTests : verwijderen ConfigItem ( , ..#Applicatie)

deze param mag terug weg, want bij rappel zijn geen product-images.

deze param mag terug weg, want bij rappel zijn geen product-images.

[UST3807] VHIP057: Rappel: webservice bouwen:
[UST3807] VHIP057: Rappel: webservice bouwen:
[UST3797] Omdat in Ex Arte de optie "Ex Arte Printer" zorgt dat Ex Arte zélf een printerdriver...
[UST3797] Omdat in Ex Arte de optie "Ex Arte Printer" zorgt dat Ex Arte zélf een printerdriver...