Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5594] Tekst offertes aanpassen : Geldig tot 30/6
  1. … 3 more files in changeset.
[ICT-4902] aanpassing nodig op offerte/OB/factuur

- reviewopmerking verwerkt

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

- nu de juiste tag gebruikt

  1. … 2 more files in changeset.
[ICT-4902] aanpassing nodig op offerte/OB/factuur
[ICT-4902] aanpassing nodig op offerte/OB/factuur
[ICT-4902] aanpassing nodig op offerte/OB/factuur

- workaround om berichten van BizTalk te 'vertalen'

  1. … 2 more files in changeset.
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!
[ICT-4752] FW: Claims afgezet urgent!

- tijdelijke bypasslogica toegevoegd

  1. … 3 more files in changeset.
[ICT-2489][rvJBA] PM - prijswijziging 1/10/21 - extra tekst op proforma factuur

- VoegToe Melding niet meer uitvoeren. Door de datum-controle ($H < 1okt) werd de melding automatisch uitgeschakeld, maar tis veiliger als de methodcall nu niet meer opgeroepen wordt.

  1. … 2 more files in changeset.
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
[ICT-2489][rvJBA] PM - prijswijziging 1/10/21 - extra tekst op proforma factuur

- VoegToeMeldingLeveringenVanaf01okt() toegevoegd aan AX.DocGen.FactuurService.RenderFactuur

[ICT-1597] Fop Factuur : voor Klant Haefele langteksten verwijderen

- DocGen Utils : method naam aangepast, na terechte review-opmerking --> KlantWenstFactuurZonderProductLangteksten()

- Mock en Fake aangepast

- aanroepende code aangepast

- commentaar debug lijn opgekuist

- overbodige parameter "PasAanIndienOngeldig" (per ongeluk mee gekopieerd) opgekuist

  1. … 3 more files in changeset.
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
[ICT-1597] Fop Factuur : voor Klant Haefele langteksten verwijderen

- DocGen RenderFactuur : Na de controles wordt de method VerwijderProductLangteksten() opgeroepen