CacheAdminA_trunk2010/vhUnitTest/ECONimpl/PM/Maatwerk/LadeUitsparingHelper

GeefUitsparingCode

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 13 Sep 18
Commits this week: 0
Total Lines of Code (LoC): 140
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 08 Dec 2020
12:10 pm

Peter Vercoutter

[ICT-2054] MVX: Convertoren voor uitsparingdata in orde brengen
Options
    • -7
    • +73
    ./Test.cls.xml
  2. … 4 more files in changeset.
Tuesday 18 Sep 2018
5:11 pm

Wim Vermeulen

Friday 14 Sep 2018
12:16 pm

Martijn Vierstraete

12:08 pm

Martijn Vierstraete

Ik snap de redenering maar ik vind dit niet correct. Mss eens in de groep gooien en beslissen wat we hiermee doen?

Ik snap de redenering maar ik vind dit niet correct. Mss eens in de groep gooien en beslissen wat we hiermee doen?

Thursday 13 Sep 2018
1:55 pm

Wim Vermeulen

1:52 pm

Tom Vermeulen

12:16 pm

Wim Vermeulen

De testmethod voor de standaardlade staat nu inderdaad dubbel. Dus ik heb ze in deze klasse verwijderd. Wat de ZijkantLogo-testen van Kurt betreft, daar ga ik wel afblijven, hé http://subversion02...

De testmethod voor de standaardlade staat nu inderdaad dubbel. Dus ik heb ze in deze klasse verwijderd.

Wat de ZijkantLogo-testen van Kurt betreft, daar ga ik wel afblijven, hé

12:07 pm

Wim Vermeulen

Vond het eerst zelf ook een beetje vreemd, waarom da aangepast is :-P Verklaring vrij eenvoudig : Naam moet consistent zijn met de andere vhTest.Utils ... Kenmerken() klassen, omdat de UT's voor de...

Vond het eerst zelf ook een beetje vreemd, waarom da aangepast is :-P
Verklaring vrij eenvoudig :
Naam moet consistent zijn met de andere vhTest.Utils ... Kenmerken() klassen, omdat de UT's voor de convertoren dit oproepen:

Method "Test: TAX Standaardlade"()
{
Do ..AssertConverter("StandaardLade")
}

Method "Test: TAX BinnenLade"()
{
Do ..AssertConverter("BinnenLade")
}

Method "Test: TAX LadeMetVeelExtras"()
{
Do ..AssertConverter("LadeMetVeelExtras")
}

11:52 am

Wim Vermeulen

Anders falen de UT's ... spijtig genoeg. (normaal gezien moet dan de UT-impl aangepast worden en niet de code, maar deze wijziging heeft geen invloed op de werking van de nietmeeleverens, denk.) De...

Anders falen de UT's ... spijtig genoeg.
(normaal gezien moet dan de UT-impl aangepast worden en niet de code, maar deze wijziging heeft geen invloed op de werking van de nietmeeleverens, denk.)
Deze tactiek was ook al toegepast bij LBX door één van m'n voorgangers.

11:45 am

Wim Vermeulen

De "ByRef" is hier om aan te geven dat het EconKenmerken-object zal aangepast worden. Technisch is het niet fout om da als .local door te geven. Objecten worden trouwens altijd als "pointer naar ge...

De "ByRef" is hier om aan te geven dat het EconKenmerken-object zal aangepast worden.
Technisch is het niet fout om da als .local door te geven. Objecten worden trouwens altijd als "pointer naar geheugenplaats" doorgegeven.

Principe is bovendien overgenomen van de 2 lijnen eronder, en dat leek mij niet verkeerd om het zo aan te duiden.

Laat zeker weten als je hier toch aan twijfelt.

11:37 am

Wim Vermeulen

Zal bij volgende opkuis verwijderd worden.

Zal bij volgende opkuis verwijderd worden.

11:12 am

Wim Vermeulen

Vreemd dat deze klasse in de review van UST3954 zit, want is gecommit onder UST3942 :-? Hoe dan ook : nieuwe UT toegevoegd, maar ook gecommit onder UST3942 . De review heb ik wel hier toegevoegd, w...

Vreemd dat deze klasse in de review van UST3954 zit, want is gecommit onder UST3942 :-?
Hoe dan ook : nieuwe UT toegevoegd, maar ook gecommit onder UST3942 .
De review heb ik wel hier toegevoegd, want gij hebt er achter gevraagd :-D :-D

10:59 am

Wim Vermeulen

[UST3942] TAX-SY: econ-kenmerken (in Caché) toevoegen + Converters aanpassen

- Econ LadeUitsparingHelper : UT toegevoegd voor GeefUitsparingCode()

Options
  1. ./GeefUitsparingCode
    • -0
    • +74
    ./Test.cls.xml
  3. … 1 more file in changeset.
Tuesday 11 Sep 2018
3:36 pm

Wim Vermeulen

Er is een testcase waarbij verpakking probox moet geconverteerd worden. Dit moet uiteraard probox blijven! *vhUnitTest.EXT.Unishop.PurchaseOrder001.ProductConverter.ExtNaarEdi.Tax.Test.cls(Test: ...

Er is een testcase waarbij verpakking probox moet geconverteerd worden. Dit moet uiteraard probox blijven!

  • vhUnitTest.EXT.Unishop.PurchaseOrder001.ProductConverter.ExtNaarEdi.Tax.Test.cls(Test: TAX LadeMetVeelExtras)
    Maar ik maak een nieuwe testmethod bij, die controleert of verpakking "" wordt geconverteerd naar V1.
3:02 pm

Wim Vermeulen

in volgende fase wordt dit opgekuist http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

in volgende fase wordt dit opgekuist

2:56 pm

Wim Vermeulen

Bij volgende update mag dit definitief verdwijnen.

Bij volgende update mag dit definitief verdwijnen.

2:55 pm

Wim Vermeulen

Klopt, maar is in een ander kanban-kaartje, dus ook aparte review

Klopt, maar is in een ander kanban-kaartje, dus ook aparte review

Wednesday 29 Aug 2018
11:10 am

Tom Vermeulen

moet de omgekeerde in APPS.PM.Maatwerk.impl.NaarObjectConverterTAX ook niet nog aangepast worden?

moet de omgekeerde in APPS.PM.Maatwerk.impl.NaarObjectConverterTAX ook niet nog aangepast worden?

11:05 am

Tom Vermeulen

idem

idem

11:05 am

Tom Vermeulen

mag dit niet gewoon weg?

mag dit niet gewoon weg?

10:59 am

Tom Vermeulen

die comments mogen weg zeker?

die comments mogen weg zeker?

10:40 am

Tom Vermeulen

in principe zouden hier nog wat testjes rond moeten komen

in principe zouden hier nog wat testjes rond moeten komen

10:37 am

Tom Vermeulen

oude logica laten staan in dit geval. alpnach stuurt nl die tag niet door en dat zou dus voor verrassingen kunnen zorgen. en in principe is het een sifon-gerelateerd kaartje en zou de werking van v...

oude logica laten staan in dit geval. alpnach stuurt nl die tag niet door en dat zou dus voor verrassingen kunnen zorgen.
en in principe is het een sifon-gerelateerd kaartje en zou de werking van verpakkingtypes niet aangepast moeten worden (ander kaartje nodig, kleine kaartjes enzo)

Wednesday 22 Aug 2018
11:15 am

Martijn Vierstraete

Heeft deze test nog nut na introduceren van ...EdiNaarApps.Tax.Test?

Heeft deze test nog nut na introduceren van ...EdiNaarApps.Tax.Test?

11:10 am

Martijn Vierstraete

Naar analogie met bovenstaande method mag deze ook aangepast worden

Naar analogie met bovenstaande method mag deze ook aangepast worden

11:07 am

Martijn Vierstraete

Waarom InsertOrdered ipv Insert?

Waarom InsertOrdered ipv Insert?

11:05 am

Martijn Vierstraete

Commentaar opkuisen

Commentaar opkuisen

11:03 am

Martijn Vierstraete

Dit is niet byRef

Dit is niet byRef

10:57 am

Martijn Vierstraete

#dim ... As LBXRugHoogte (ipv TAORRugHoogte)

#dim ... As LBXRugHoogte (ipv TAORRugHoogte)

10:56 am

Martijn Vierstraete

Deze property mag verwijderd worden (na onderzoek http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif )

Deze property mag verwijderd worden (na onderzoek )