CacheAdminA_trunk2010/WS/AX

DocGen

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 7
Last Commit: 24 Feb 17
Commits this week: 0
Total Lines of Code (LoC): 820
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 13 Sep 2024
10:23 am

Tom Vermeulen

Wednesday 28 Aug 2024
5:20 pm

Wim Vermeulen

5:18 pm

Wim Vermeulen

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.

Wednesday 14 Aug 2024
11:37 am

Tom Vermeulen

[ICT-4902] aanpassing nodig op offerte/OB/factuur
[ICT-4902] aanpassing nodig op offerte/OB/factuur
Monday 01 Jul 2024
2:45 pm

Tom Vermeulen

Wednesday 26 Jun 2024
1:29 pm

Wim Vermeulen

1:27 pm

Wim Vermeulen

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

Tuesday 25 Jun 2024
3:31 pm

Tom Vermeulen

[ICT-4752] FW: Claims afgezet urgent!
[ICT-4752] FW: Claims afgezet urgent!
Thursday 14 Apr 2022
12:39 pm

Joannes Lemmens

Wednesday 30 Mar 2022
4:03 pm

Wim Vermeulen

3:56 pm

Wim Vermeulen

Dat is oké, lijkt me. De method VoegToeMeldingLeveringenVanaf01okt() zou eerder voor "Order" en "Factuur" zijn. Misschien kan je dat wel bij Sofie vragen, want nu wordt de prijswijziging enkel aang...

Dat is oké, lijkt me.
De method VoegToeMeldingLeveringenVanaf01okt() zou eerder voor "Order" en "Factuur" zijn.
Misschien kan je dat wel bij Sofie vragen, want nu wordt de prijswijziging enkel aangekondigd op de offerte.

2:39 pm

Joannes Lemmens

[ICT-2823] DOC - offerte - tijdelijke vermelding 'tot 30/4' ipv '30 dagen':
[ICT-2823] DOC - offerte - tijdelijke vermelding 'tot 30/4' ipv '30 dagen':
2:37 pm

Joannes Lemmens

Opmerking: de lijn "VoegToeMeldingLeveringVanaf01okt" is niet ge-uncomment ( ... in de zin dat hier geen aanpassingen gedaan zijn). Indien gewenst kan ik deze functionaliteit inschakelen (met de ov...

Opmerking: de lijn "VoegToeMeldingLeveringVanaf01okt" is niet ge-uncomment ( ... in de zin dat hier geen aanpassingen gedaan zijn).
Indien gewenst kan ik deze functionaliteit inschakelen (met de overige aanpassingen die hiervoor nodig zijn).

Thursday 16 Dec 2021
9:41 am

Wim Vermeulen

Wednesday 15 Dec 2021
4:06 pm

Jelle Baele

Tuesday 07 Dec 2021
4:11 pm

Wim Vermeulen

[ICT-2593][rvJBA] Vermelding stock product op offerte verduidelijken/weglaten
[ICT-2593][rvJBA] Vermelding stock product op offerte verduidelijken/weglaten
Tuesday 02 Nov 2021
4:08 pm

Wim Vermeulen

Thursday 07 Oct 2021
8:36 am

Jelle Baele

Wednesday 29 Sep 2021
5:18 pm

Wim Vermeulen

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.

4:56 pm

Wim Vermeulen

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.

Monday 27 Sep 2021
9:37 am

Jelle Baele

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?

9:33 am

Jelle Baele

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 ( ... )

Thursday 09 Sep 2021
3:55 pm

Wim Vermeulen

[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
Monday 02 Aug 2021
11:45 am

Wim Vermeulen

10:31 am

Jelle Baele

Friday 30 Jul 2021
3:09 pm

Wim Vermeulen

[ICT-2450][rvJBA] prijswijziging 01/10/21 - extra tekst op offerte en orderbevestiging
[ICT-2450][rvJBA] prijswijziging 01/10/21 - extra tekst op offerte en orderbevestiging
Monday 04 Jun 2018
2:25 pm

Tommy Hebb

2:17 pm

Wim Vermeulen

2:16 pm

Wim Vermeulen

OnAfterAllTests : verwijderen ConfigItem ( , ..#Applicatie)

OnAfterAllTests : verwijderen ConfigItem ( , ..#Applicatie)

2:06 pm

Wim Vermeulen

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

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