CacheAdminA_trunk2010/WS/AX/DocGen

ProxyDataHelper.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 02 Jun
Commits this week: 0
Total Lines of Code (LoC): 129
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
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