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
Monday 02 Jun
8:39 am

Peter Vercoutter

[ICT-5594] Tekst offertes aanpassen : Geldig tot 30/6
Options
    • -1
    • +1
    ./ProxyDataHelper.cls.xml
Monday 12 May
2:53 pm

Peter Vercoutter

[ICT-5594] Tekst offertes aanpassen : Geldig tot 30/6

Options
    • -2
    • +2
    ./ProxyDataHelper.cls.xml
  2. … 1 more file in changeset.
12:51 pm

Peter Vercoutter

[ICT-5594] Tekst offertes aanpassen : Geldig tot 30/6

Options
    • -2
    • +2
    ./ProxyDataHelper.cls.xml
  2. … 1 more file in changeset.
Friday 09 May
1:07 pm

Peter Vercoutter

[ICT-5594] Tekst offertes aanpassen : Geldig tot 30/6

- Tekst nog aanpassen op aanvraag van PM

Options
    • -8
    • +8
    ./ProxyDataHelper.cls.xml
Tuesday 06 May
1:21 pm

Peter Vercoutter

[ICT-5594] Tekst offertes aanpassen : Geldig tot 30/6
Options
    • -4
    • +4
    ./ProxyDataHelper.cls.xml
Wednesday 30 Apr
11:08 am

Peter Vercoutter

[ICT-5594] Tekst offertes aanpassen : Geldig tot 30/6
Options
    • -23
    • +23
    ./ProxyDataHelper.cls.xml
  2. … 3 more files in changeset.
Tuesday 02 Nov 2021
4:08 pm

Wim Vermeulen

Friday 08 Oct 2021
11:18 am

Wim Vermeulen

[ICT-2489][rvJBA] PM - prijswijziging 1/10/21 - extra tekst op proforma factuur

- klasse gemarkeerd om te wijzigen van package WS naar WSimpl.AX.DocGen.ProxyDataHelper

Options
    • -0
    • +2
    ./ProxyDataHelper.cls.xml
Thursday 07 Oct 2021
8:36 am

Jelle Baele

Wednesday 29 Sep 2021
6:36 pm

Wim Vermeulen

[ICT-2489][rvJBA] PM - prijswijziging 1/10/21 - extra tekst op proforma factuur - aanpassingen na review opm. :

- method naam gewijzigd naar VoegToeLeegOrderVooraanBijFactuur()

- code afgezonderd in private method GeefEersteOrderVanFactuur() om de context te verduidelijken

Options
    • -6
    • +19
    ./ProxyDataHelper.cls.xml
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
2:51 pm

Wim Vermeulen

[ICT-2489][rvJBA] PM - prijswijziging 1/10/21 - extra tekst op proforma factuur

- TekstVoorOrderref in function argument gestoken (default= "INFO : ")

- comment for testing, per ongeluk mee gecommit. Nu ongedaan gemaakt ... nog eens! :-P

Options
    • -3
    • +2
    ./ProxyDataHelper.cls.xml
2:31 pm

Wim Vermeulen

[ICT-2489][rvJBA] PM - prijswijziging 1/10/21 - extra tekst op proforma factuur

- voorwaarde voor IsProforma is verschillend wanneer de data uit Ax/Biztalk komt. Dan via FACTUURNR = Proforma

Options
    • -1
    • +2
    ./ProxyDataHelper.cls.xml
12:32 pm

Wim Vermeulen

[ICT-2489][rvJBA] PM - prijswijziging 1/10/21 - extra tekst op proforma factuur

- comment for testing, per ongeluk mee gecommit. Nu ongedaan gemaakt :-P

Options
    • -1
    • +0
    ./ProxyDataHelper.cls.xml
12:30 pm

Wim Vermeulen

[ICT-2489][rvJBA] PM - prijswijziging 1/10/21 - extra tekst op proforma factuur

- method VoegToeAanFactuurNieuwOrderVooraan() : eerst een nieuw (leeg) order vooraan de factuur toevoegen. Daarom wordt de de tekstlijn toegevoegd i.p.v. binnen de eerste (reeds aanwezige) order van de factuur/proforma.

- Door de oproep-lijn in commentaar te zetten, valt de implementatie terug op de vorige versie.

Options
    • -0
    • +18
    ./ProxyDataHelper.cls.xml
12:27 pm

Wim Vermeulen

[ICT-2489][rvJBA] PM - prijswijziging 1/10/21 - extra tekst op proforma factuur

- method VoegToeFactuurMeldingLeveringenVanaf01okt() kleine aanpassing

- Open lijnen (CRLF) toegevoegd vooraan en achteraan van de TekstLeveringenVanaf01okt.

Options
    • -2
    • +2
    ./ProxyDataHelper.cls.xml
9:43 am

Wim Vermeulen

[ICT-2489][rvJBA] PM - prijswijziging 1/10/21 - extra tekst op proforma factuur

- method VoegToeFactuurMeldingLeveringenVanaf01okt() toegevoegd, om op de eerste lijn van het eerste (fact)Order van de eerste (fact)Levering een tekstlijn te inserten.

- alleen op voorwaarde de facttype een Proforma is

Options
    • -0
    • +14
    ./ProxyDataHelper.cls.xml
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
2:45 pm

Wim Vermeulen

[ICT-2450][rvNVT] prijswijziging 01/10/21 - extra tekst op offerte en orderbevestiging

- nieuwe klasse ProxyDataHelper.cls toegevoegd met method VoegToeMeldingLeveringenVanaf01okt : tekstlijn vooraan toevoegen voor prijswijziging : tussen 15/08 en 30/09/2021 wordt deze melding toegevoegd, vanwege de geplande prijswijzigingen vanaf 01/10/2021. Na deze datum zal de tekst niet meer toegevoegd worden (wordt automatisch uitgeschakeld).

- public methods voor OfferteData en OrderData.

Options
    • -0
    • +83
    ./ProxyDataHelper.cls.xml