CacheAdminA_trunk2010/APPS/Halux/Docs/impl

LadeKlantEtiketBuilder

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 8
Last Commit: 20 Aug 15
Commits this week: 0
Total Lines of Code (LoC): 297
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 28 May 2024
3:00 pm

Joannes Lemmens

Tuesday 06 Jun 2023
11:22 am

Peter Vercoutter

Thursday 27 Apr 2023
8:39 am

Joannes Lemmens

Wednesday 26 Apr 2023
3:17 pm

Peter Vercoutter

[ICT-3572] Klantenetiket - Voor alle ladetypes corpusbreedte vermelden + aanpassing langtekst
[ICT-3572] Klantenetiket - Voor alle ladetypes corpusbreedte vermelden + aanpassing langtekst
Wednesday 24 Aug 2022
2:04 pm

Jelle Baele

1:58 pm

Jelle Baele

8:54 am

Peter Vercoutter

Tuesday 23 Aug 2022
8:41 am

Jelle Baele

8:39 am

Jelle Baele

[ICT-3054] [rvPVR] RVG aansturingactie: Klantetiketten
[ICT-3054] [rvPVR] RVG aansturingactie: Klantetiketten
Tuesday 07 Sep 2021
2:06 pm

Tommy Hebb

2:05 pm

Tommy Hebb

Ik vermoed dat vhUnitTest.APPS.Halux.Docs.TestDataEtiketten nergens anders gebruikt wordt en dus mag opgekuist worden hé http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emo...

Ik vermoed dat vhUnitTest.APPS.Halux.Docs.TestDataEtiketten nergens anders gebruikt wordt en dus mag opgekuist worden hé

1:49 pm

Tommy Hebb

Info2 wordt wel geassert, maar doet niet echt ter zake in deze UT. Toch niet volgens de naam ervan.

Info2 wordt wel geassert, maar doet niet echt ter zake in deze UT. Toch niet volgens de naam ervan.

1:48 pm

Tommy Hebb

Ik zou de UT-naam iets duidelijker maken, want "Klant OrderNummer" houdt eigenlijk pas steek als voorgaande UT in acht wordt genomen en mettertijd staan die 2 UT misschien niet meer vlak bij elkaar...

Ik zou de UT-naam iets duidelijker maken, want "Klant OrderNummer" houdt eigenlijk pas steek als voorgaande UT in acht wordt genomen en mettertijd staan die 2 UT misschien niet meer vlak bij elkaar. Ik zou ook, net zoals hierboven, vermelden dat het over het veld Info1 gaat.

1:46 pm

Tommy Hebb

Misschien nog bijzetten "in Info1";

Misschien nog bijzetten "in Info1";

1:40 pm

Tommy Hebb

Ofwel klopt de naam van de UT niet, ofwel de implementatie. Er is geen assert op ProductImg en als ik die toevoeg (met een verwachte waarde anders dan leeg), dan slaagt de test, terwijl ProductImg ...

Ofwel klopt de naam van de UT niet, ofwel de implementatie. Er is geen assert op ProductImg en als ik die toevoeg (met een verwachte waarde anders dan leeg), dan slaagt de test, terwijl ProductImg volgens de UT-naam leeg zou moeten zijn!
In de "oude" UT ontbrak die assert blijkbaar ook en de UT-naam is ongewijzigd volgens deze review, dus zou er eens nagevraagd/nagekeken moeten worden wat eigenlijk de bedoeling is, zodat het juiste kan aangepast worden.

12:06 pm

Tommy Hebb

Ik zou de projectreferentie hier en in de volgende Unit Test vooraf in een variabele steken met een duidende naam, zodat het wat meer opvalt. Duurde even voordat ik doorhad dat in de volgende UnitT...

Ik zou de projectreferentie hier en in de volgende Unit Test vooraf in een variabele steken met een duidende naam, zodat het wat meer opvalt. Duurde even voordat ik doorhad dat in de volgende UnitTest het 2-tje weg was en aangezien dàt het enige verschil is mag het wel wat opvallen

10:32 am

Tommy Hebb

Is het niet een beetje raar dat deze properties nu leeg zijn voor de assert? Worden blijkbaar niet mee uitgemockt?

Is het niet een beetje raar dat deze properties nu leeg zijn voor de assert? Worden blijkbaar niet mee uitgemockt?

Thursday 12 Aug 2021
4:59 pm

Jelle Baele

4:57 pm

Jelle Baele

[ICT-449] UST3816 - [rvTHB] Deel 2 van Tests die klanten maken: schijfruimte-probl: tests onafh van...
[ICT-449] UST3816 - [rvTHB] Deel 2 van Tests die klanten maken: schijfruimte-probl: tests onafh van...
Tuesday 16 Feb 2021
10:00 am

Tommy Hebb

8:24 am

Tom Vermeulen

En eigenlijk moet die in de constructor genewed worden. Ik weet het, aan de ene kant "deze klasse gebruikt die manier van werken nog niet" en aan de andere kant het "boyscout principe". Your call

En eigenlijk moet die in de constructor genewed worden. Ik weet het, aan de ene kant "deze klasse gebruikt die manier van werken nog niet" en aan de andere kant het "boyscout principe". Your call

Monday 15 Feb 2021
12:57 pm

Tom Vermeulen

12:50 pm

Tommy Hebb

Done!

Done!

12:23 pm

Tommy Hebb

= Code van Wim. Wellicht zijn er teveel verschillende mogelijkheden om een al te specifieke te gebruiken hé.

= Code van Wim. Wellicht zijn er teveel verschillende mogelijkheden om een al te specifieke te gebruiken hé.

12:20 pm

Tommy Hebb

Goed gezien :-s Bij deze aangepast!

Goed gezien :-s Bij deze aangepast!

Friday 12 Feb 2021
2:01 pm

Tom Vermeulen

ik veronderstel dat dit een copy paste van lbx is, maar misschien toch de ' door $$$Not vervangen

ik veronderstel dat dit een copy paste van lbx is, maar misschien toch de ' door $$$Not vervangen

2:00 pm

Tom Vermeulen

Als er ooit vhossen bij mvx komen, gaat ge het probleem hebben dat er geen vhossetiketten worden afgedrukt. Dat is een uitbreiding die dan nog moet toegevoegd worden op de KlantEtiket.PrinterFactory

Als er ooit vhossen bij mvx komen, gaat ge het probleem hebben dat er geen vhossetiketten worden afgedrukt. Dat is een uitbreiding die dan nog moet toegevoegd worden op de KlantEtiket.PrinterFactory

1:27 pm

Tom Vermeulen

is het niet beter om een specifiekere exception te gebruiken? CommunicationErrorException ofzo

is het niet beter om een specifiekere exception te gebruiken? CommunicationErrorException ofzo

Thursday 11 Feb 2021
12:09 pm

Tom Vermeulen

moet hier ook geen ...Merivobox voorzien worden (zelfs als die gelijk is aan de Legrabox)? Kwestie van consequent namen te gebruiken

moet hier ook geen ...Merivobox voorzien worden (zelfs als die gelijk is aan de Legrabox)? Kwestie van consequent namen te gebruiken

12:05 pm

Tom Vermeulen

groter dan wat? (550, dat zie ik wel door naar de code te kijken, maar als je een comment toevoegt doe je dat om niet naar de code te moeten kijken http://subversion02.vanhoecke.be/static/ogdo0b/2s...

groter dan wat? (550, dat zie ik wel door naar de code te kijken, maar als je een comment toevoegt doe je dat om niet naar de code te moeten kijken ) (meerdere keren)