CacheAdminA_trunk2010/APPS/Halux/PPS/Activiteit

ProductieSequentieAwareActiviteitGroep.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 16 Dec 20
Commits this week: 0
Total Lines of Code (LoC): 10
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 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 16 Dec 2020
5:49 pm

Tommy Hebb

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

- In ProductieSequentieItemService => Ophalen HalffabItemsBoom in DotNet in TryCatch gezet

- dto.ProductieUitvoeringInfo => Wordt generisch gebruikt => LadeHoogte as string ipv als LBX-enum => E-hoogte van MVX bestaat niet in LBX en was sowieso niet correct => moet mogelijks nog voor andere hier gebruikte enums gebeuren ook

- ProductieVoorbereidingZaag.Utils => Enkel Probox is al voorzien in VerpakkingType-enum van MVX

- VoorbereidingZaag => Nog een paar voorlopig onbestaande enums gedefault naar False

- ProductieSequentieAwareActiviteitGroep => Uitbreiding voorzien voor MVX => Merivobox overzicht batch

Options
    • -1
    • +1
    ./ProductieSequentieAwareActiviteitGroep.cls.xml
  2. … 4 more files in changeset.
Tuesday 22 Sep 2020
3:50 pm

Peter Vercoutter

3:47 pm

Peter Vercoutter

Ik denk dat dit veel extra werk zou opleveren en niet waard zou zijn kosten/baten

Ik denk dat dit veel extra werk zou opleveren en niet waard zou zijn kosten/baten

1:38 pm

Peter Vercoutter

1:38 pm

Peter Vercoutter

[ICT-1589] Implementatie productieaansturing SpaceStep

- Reviewopmerkingen

Options
    • -1
    • +1
    ./ProductieSequentieAwareActiviteitGroep.cls.xml
  2. … 1 more file in changeset.
1:02 pm

Peter Vercoutter

Deze klasse wordt afgeleid van een abstracte klasse dus onNew zal niet lukken

Deze klasse wordt afgeleid van een abstracte klasse dus onNew zal niet lukken

11:07 am

Tom Vermeulen

11:07 am

Tom Vermeulen

detailke: ge kunt in uw assert evengoed zetten ...width=""25"">"ImageUrl"</ ... op die manier valt het direct op dat ge daar de exact zelfde output wilt zien als in de input idem bij andere test

detailke: ge kunt in uw assert evengoed zetten ...width=""25"">"ImageUrl"</ ... op die manier valt het direct op dat ge daar de exact zelfde output wilt zien als in de input

idem bij andere test

11:04 am

Tom Vermeulen

DocumentApi in %OnNew zetten (en als property), zodat ge hem hier niet elke keer moet ophalen/newen en hij uitmockbaar is (en al de dependencies duidelijk zichtbaar worden in constructor). Dat gel...

DocumentApi in %OnNew zetten (en als property), zodat ge hem hier niet elke keer moet ophalen/newen en hij uitmockbaar is (en al de dependencies duidelijk zichtbaar worden in constructor).

Dat geldt ook voor die andere klasses hieronder

11:02 am

Tom Vermeulen

als ge zoveel vergelijkingen hebt staan, kunt ge beter met de TECH.StringUtils.Contains() werken

als ge zoveel vergelijkingen hebt staan, kunt ge beter met de TECH.StringUtils.Contains() werken

11:00 am

Tom Vermeulen

ge zoudt hier een onlinerke van kunnen maken

ge zoudt hier een onlinerke van kunnen maken

Tuesday 08 Sep 2020
11:57 am

Tom Vermeulen

11:54 am

Tom Vermeulen

Door de bestaande testen aan te passen heb je het gedrag waarbij de laatste parameter de default is, weggemoffeld. Je moet de originele testen laten staan en een paar nieuwe testen maken die True a...

Door de bestaande testen aan te passen heb je het gedrag waarbij de laatste parameter de default is, weggemoffeld. Je moet de originele testen laten staan en een paar nieuwe testen maken die True als laatste parameter hebben.

11:53 am

Tom Vermeulen

{$$$False} ipv 0

{$$$False} ipv 0

11:48 am

Tom Vermeulen

Kan die nieuwe property eigenlijk niet mee op de UitvoeringInfo gestoken worden? In principe staan daar nog allerlei gegevens die nodig zijn om (in dit geval) etiketten af te drukken. Op die manier...

Kan die nieuwe property eigenlijk niet mee op de UitvoeringInfo gestoken worden? In principe staan daar nog allerlei gegevens die nodig zijn om (in dit geval) etiketten af te drukken. Op die manier houden we de interface(s) proper (zeker als er ooit nog andere parameters bij moeten komen).

Dit geldt ook voor sommige andere methods/klasses in deze review

11:47 am

Tom Vermeulen

comments weg

comments weg