CacheAdminA_trunk2010/APPS/Halux/common/impl

AantallenInToeleveringTeller.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 18 Feb
Commits this week: 0
Total Lines of Code (LoC): 71
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 18 Feb
10:20 am

Peter Vercoutter

[PRJ051-297] Productiegroep van toeleveringen met enkel laden van BWC
Options
    • -1
    • +1
    ./AantallenInToeleveringTeller.cls.xml
  2. … 1 more file in changeset.
8:59 am

Peter Vercoutter

[PRJ051-297] Productiegroep van toeleveringen met enkel laden van BWC
Options
    • -0
    • +6
    ./AantallenInToeleveringTeller.cls.xml
  2. … 11 more files in changeset.
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)

12:05 pm

Tom Vermeulen

'> ?? gebruik beter <= (meerdere keren)

'> ?? gebruik beter <= (meerdere keren)

12:01 pm

Tom Vermeulen

detail, maar in principe gebruiken we die domeincontext niet meer (zie WV en MVI voor details) en kan die gewoon genewed worden. (staat op meerdere plaatsen, ga dat niet overal schrijven http://sub...

detail, maar in principe gebruiken we die domeincontext niet meer (zie WV en MVI voor details) en kan die gewoon genewed worden. (staat op meerdere plaatsen, ga dat niet overal schrijven )

Monday 01 Feb 2021
3:06 pm

Tommy Hebb

[ICT-2026] HX: Planning: MVX: Probox overdoosetiketten:
[ICT-2026] HX: Planning: MVX: Probox overdoosetiketten:
Wednesday 13 Jan 2021
9:58 am

Tommy Hebb

[ICT-2026] HX: Planning: MVX: Probox overdoosetiketten:

- Uitbreiding voor MVX

Options
    • -2
    • +3
    ./AantallenInToeleveringTeller.cls.xml
  2. … 1 more file in changeset.
Thursday 05 Mar 2020
8:52 am

Laura Landuyt

Monday 27 Jan 2020
11:09 am

Kurt Callebaut

Thursday 16 Jan 2020
9:57 am

Wim Vermeulen

Monday 13 Jan 2020
9:19 am

Wim Vermeulen

Bij Probox en Pallet zet je dan achteraan : ... .DanReturn($$$True).ExactAantalKeer(0)

Bij Probox en Pallet zet je dan achteraan : ... .DanReturn($$$True).ExactAantalKeer(0)

Friday 10 Jan 2020
11:00 am

Wim Vermeulen

Ik heb nog een klein verzoek voor verbetering : De "IndienMethodCall()" vervangen door "VerwachtMethodCall() + .Verifieer() Bij Probox en Pallet zet je dan achteraan : ... .DanReturn($$$True).Exac...

Ik heb nog een klein verzoek voor verbetering :

De "IndienMethodCall()" vervangen door "VerwachtMethodCall() + .Verifieer()
Bij Probox en Pallet zet je dan achteraan : ... .DanReturn($$$True).ExactAantalKeer(0)
Op deze manier krijg je ook meteen duidelijke feedback indien de test faalt.

10:38 am

Wim Vermeulen

"TBXSpoelbak" is ondertussen gewijzigd naar TBXSifon ( in de methodnaam ook http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif ) en hieronder bij "VHoss" oo...

"TBXSpoelbak" is ondertussen gewijzigd naar TBXSifon ( in de methodnaam ook )
en hieronder bij "VHoss" ook aub.

Monday 06 Jan 2020
9:36 am

LauraL

[ICT-878] [+KCA] HX: TBX sifonlades in aparte toelevering/batch indien aantal>10

Refactor code elseif ipv tweede if in /AantallenInToeleveringTeller1

Options
    • -3
    • +2
    ./AantallenInToeleveringTeller.cls.xml
Friday 20 Dec 2019
12:21 pm

Wim Vermeulen

Code voor ToeleveringsplitserMock niet in elke method apart toevoegen.

Code voor ToeleveringsplitserMock niet in elke method apart toevoegen.

11:48 am

Wim Vermeulen

Eigenlijk moet je hier " ... HeeftProductSpecificatie").DanReturn($$$True) " zetten, want anders heeft de ToeleveringSplitser sowieso geen invloed op deze Test. En als ik dan nog een stapje verder ...

Eigenlijk moet je hier " ... HeeftProductSpecificatie").DanReturn($$$True) " zetten, want anders heeft de ToeleveringSplitser sowieso geen invloed op deze Test.
En als ik dan nog een stapje verder probeer te denken, dan kan je deze testmethod herleiden tot ca. 5 lijnen code, zoals een aantal andere tests in deze klasse.
Bij twijfel, geef maar een seintje.

FYI : het is normaal dat code in de testen soms onoverzichtelijk groot wordt (na verschillende evoluties in ontwikkeling). Dan heb je soms iemand anders nodig om zulke vereenvoudigingen aan te geven

11:33 am

Wim Vermeulen

MaatwerkProductStub mag nu terug weg hé.

MaatwerkProductStub mag nu terug weg hé.

11:20 am

Wim Vermeulen

Deze method mag terug verwijderd worden, want geen nieuwe testen meer maken voor TBX Spoelbak.

Deze method mag terug verwijderd worden, want geen nieuwe testen meer maken voor TBX Spoelbak.

11:11 am

Wim Vermeulen

Is duplicatie van de method eronder. Lijkt me overbodig, want je kan toch SifonLadeMet1Uitsparing() oproepen, hé?

Is duplicatie van de method eronder.
Lijkt me overbodig, want je kan toch SifonLadeMet1Uitsparing() oproepen, hé?

10:38 am

Wim Vermeulen

Klein detail : ik zou hier een ElseIf ... van maken

Klein detail : ik zou hier een ElseIf ... van maken