CacheAdminA_trunk2010/vhUnitTest/DOM/PM/Maatwerk/TAX/impl/KorttekstBepaler/BepaalKorttekst

TestLosseComponenten.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 11 Oct 22
Commits this week: 0
Total Lines of Code (LoC): 286
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 07 Nov 2022
11:06 am

Tommy Hebb

Wednesday 02 Nov 2022
6:51 pm

Wim Vermeulen

6:51 pm

Wim Vermeulen

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken. Alvast bedankt om het aantal testfiles te filteren. Het geheel lijkt me we...

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken.
Alvast bedankt om het aantal testfiles te filteren.
Het geheel lijkt me wel oké te zijn.

Ook in eCon enkele commits gereviewed.

6:38 pm

Wim Vermeulen

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten) In combinatie met svn rev. 69413 is dit wel een correcte overgang. Hierbij is deze cha...

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten)
In combinatie met svn rev. 69413 is dit wel een correcte overgang.

Hierbij is deze change nagekeken en goedgekeurd!

Friday 21 Oct 2022
10:35 am

Tommy Hebb

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om ui...

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om uit te maken a.d.h.v. de changesets of het al dan niet over code gaat of testfiles.
P.s.: Ook kleine change in Econ (zie svn).

10:31 am

Tommy Hebb

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
Tuesday 11 Oct 2022
12:06 pm

Tommy Hebb

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- Nog een UnitTest die uitgebreid moest worden => Korttekst voor MoventoKoppeling igv LosseComponent Bodem

Options
    • -0
    • +1
    ./TestLosseComponenten.cls.xml
Friday 22 Nov 2019
10:08 am

Martijn Vierstraete

Monday 04 Nov 2019
10:48 am

Tom Vermeulen

Thursday 17 Oct 2019
2:53 pm

Martijn Vierstraete

[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug)
[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug)
Friday 11 Oct 2019
1:43 pm

MartijnV

[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug): gevolgen van commit r58565 (ladekleur invullen)

Options
    • -4
    • +0
    ./TestLosseComponenten.cls.xml
  2. … 3 more files in changeset.
Monday 08 Apr 2019
12:04 pm

Tommy Hebb

12:03 pm

Tom Vermeulen

11:19 am

Tommy Hebb

[UST4203] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Na...
[UST4203] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Na...
11:15 am

Tommy Hebb

[UST4203] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Na testing door JB:

- Behalve voor zijkant altijd een X op positie 9

Options
    • -17
    • +5
    ./TestLosseComponenten.cls.xml
  2. … 1 more file in changeset.
Thursday 28 Mar 2019
5:27 pm

Tommy Hebb

5:27 pm

Tommy Hebb

Dat is juist de bedoeling... Indien vanuit Vhintra GEEN ZijkantLogoId is geselecteerd, dan kan dat 0 of leeg teruggeven en dat moet dan in de korttekst "XX" geven. Anderzijds moest er een ZijkantLo...

Dat is juist de bedoeling...
Indien vanuit Vhintra GEEN ZijkantLogoId is geselecteerd, dan kan dat 0 of leeg teruggeven en dat moet dan in de korttekst "XX" geven.
Anderzijds moest er een ZijkantLogoId uit de (nieuw bedachte) range gereserveerd worden, want de logica die volgens de ZijkantLogoId (1258) de code "XX" teruggeeft mag nu niet meer gebruikt worden voor een echte klant. In de tabel van ZijkantLogoId's is id 1258 dus gereserveerd.

Friday 22 Mar 2019
10:41 am

Wim Vermeulen

10:41 am

Wim Vermeulen

PROBLEEM !!! logo ID 1258 geeft ook "XX" --> zelfde als "geen logo" Hoe ga je dat uitleggen ?

PROBLEEM !!!
logo ID 1258 geeft ook "XX" --> zelfde als "geen logo"
Hoe ga je dat uitleggen ?

Thursday 21 Mar 2019
11:29 am

Tommy Hebb

[UST4192] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Default geen ZijkantLogo in korttekst:

- Als ZijkantLogoId leeg of 0 is => XX teruggeven => betekent geen ZijkantLogo in Korttekst => ook op hoger niveau Tommy

Options
    • -3
    • +3
    ./TestLosseComponenten.cls.xml
  2. … 1 more file in changeset.
10:09 am

Tommy Hebb

[UST4192] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Default...
[UST4192] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Default...
Monday 18 Mar 2019
4:22 pm

Tommy Hebb

Friday 15 Mar 2019
3:02 pm

Wim Vermeulen

11:22 am

Tommy Hebb

Done!

Done!

Thursday 14 Mar 2019
5:33 pm

Tommy Hebb

Is overgenomen van de defaulting van de (énige) implementatieklasse. De mock wordt van de interface gemaakt. Dat maakte dat die ander gedrag vertoonde dan de echte.

Is overgenomen van de defaulting van de (énige) implementatieklasse. De mock wordt van de interface gemaakt. Dat maakte dat die ander gedrag vertoonde dan de echte.

5:29 pm

Tommy Hebb

Die worden allebei in de caller van deze method al uitgesloten

Die worden allebei in de caller van deze method al uitgesloten

Friday 08 Mar 2019
4:02 pm

Wim Vermeulen

object LadeOnderdeel doorgeven, i.p.v. Rol As enu dan moet je voor Bodem niet opnieuw de "..MaakLosseComponent(Kenmerken) " opnieuw oproepen --> vermijd dubbel werk.

object LadeOnderdeel doorgeven, i.p.v. Rol As enu
dan moet je voor Bodem niet opnieuw de "..MaakLosseComponent(Kenmerken) " opnieuw oproepen --> vermijd dubbel werk.

3:04 pm

Wim Vermeulen

Dit is de algemene MaatwerkCalcAPI --> straf dat je hier de parameter gaat defaulten naar klant "Halux". Ik zou verwachten dat je ergens een specifieke oproep hebt, waar je dan gecontroleerd de kla...

Dit is de algemene MaatwerkCalcAPI --> straf dat je hier de parameter gaat defaulten naar klant "Halux".
Ik zou verwachten dat je ergens een specifieke oproep hebt, waar je dan gecontroleerd de klant meegeeft.

Laat voorlopig maar staan, maar hou er wel rekening mee, dat dit soort defaultings "gedurfd" is

2:37 pm

Wim Vermeulen

De bijhorende UT geeft aan dat *ID = "" geeft "" *ID = 0 geeft "" Mijn vraag : is dit wenselijk, of zou het beter zijn om een exceptie te smijten? Want voor deze waarden is het eigenlijk zinloos ...

De bijhorende UT geeft aan dat

  • ID = "" geeft ""
  • ID = 0 geeft ""
    Mijn vraag : is dit wenselijk, of zou het beter zijn om een exceptie te smijten?
    Want voor deze waarden is het eigenlijk zinloos om de bepaler op te roepen.
2:34 pm

Wim Vermeulen

Deze lijn zou ik binnen de Else zetten. Nu wordt ze uitgevoerd, zelfs bij ID > 1332 en nadien wordt dan toch exception gesmeten. M.a.w. de AflfaKey() uitvoeren was niet nodig.

Deze lijn zou ik binnen de Else zetten.
Nu wordt ze uitgevoerd, zelfs bij ID > 1332 en nadien wordt dan toch exception gesmeten. M.a.w. de AflfaKey() uitvoeren was niet nodig.