CacheAdminA_trunk2010/APPS/Halux/common/impl

ProductieGroepBepalerLBX.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 8
Last Commit: 06 May
Commits this week: 0
Total Lines of Code (LoC): 2.747
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 22 Sep 2020
11:18 am

Peter Vercoutter

11:13 am

Tom Vermeulen

Wednesday 02 Sep 2020
3:42 pm

Peter Vercoutter

Integratietest voor productieaansturingbestanden, Productiegroep+prodlijnbepaler(Hoort bij ICT-1589)
Integratietest voor productieaansturingbestanden, Productiegroep+prodlijnbepaler(Hoort bij ICT-1589)
Monday 09 Mar 2020
11:00 am

Tom Vermeulen

9:58 am

Laura Landuyt

9:58 am

Laura Landuyt

Verwachte productiegroep is probox maar in testnaam staat er pallet? En ook test voorzien waarbij die in productiegroep specials komt ?

Verwachte productiegroep is probox maar in testnaam staat er pallet?

En ook test voorzien waarbij die in productiegroep specials komt ?

9:57 am

Laura Landuyt

Ook test voorzien waarbij die in productiegroep specials komt ?

Ook test voorzien waarbij die in productiegroep specials komt ?

9:54 am

Laura Landuyt

Mag deze test weg?

Mag deze test weg?

Friday 06 Mar 2020
8:35 am

Tom Vermeulen

[ICT-1324] Klant Interfurn TBX in specialsKlant
[ICT-1324] Klant Interfurn TBX in specialsKlant
Friday 14 Feb 2020
10:32 am

Laura Landuyt

Thursday 13 Feb 2020
12:11 pm

Wim Vermeulen

Thursday 06 Feb 2020
1:10 pm

Wim Vermeulen

Zelfde opmerking als bij TBX : OpstartfaseBepaler als eerste conditie laten staan aub.

Zelfde opmerking als bij TBX : OpstartfaseBepaler als eerste conditie laten staan aub.

1:10 pm

Wim Vermeulen

Zelfde opmerking als bij TBX : OpstartfaseBepaler als eerste conditie laten staan aub.

Zelfde opmerking als bij TBX : OpstartfaseBepaler als eerste conditie laten staan aub.

Tuesday 04 Feb 2020
11:31 am

Laura Landuyt

[ICT-968][+WV] HX: Planning: Probox-lades voor klant De Vliegher (2856) in -Specials- "Andere productiegroep" laten terechtkomen
[ICT-968][+WV] HX: Planning: Probox-lades voor klant De Vliegher (2856) in -Specials- "Andere productiegroep" laten terechtkomen
Tuesday 15 Oct 2019
9:34 am

Kurt Callebaut

Friday 11 Oct 2019
3:55 pm

Wim Vermeulen

3:54 pm

Wim Vermeulen

LET OP: ALTIJD wanneer je storage-data aanpast rechtstreeks in de globals, dan nakijken of deze klasse indexen bevat. In dat geval kan je best de ...%BuilldIndices(,1) oproepen, zelfs wanneer dat g...

LET OP: ALTIJD wanneer je storage-data aanpast rechtstreeks in de globals, dan nakijken of deze klasse indexen bevat.
In dat geval kan je best de ...%BuilldIndices(,1) oproepen, zelfs wanneer dat geen verschil zou geven. Play safe

3:20 pm

Wim Vermeulen

Ook al is het maar een deploy-klasse, toch zou ik duidelijk maken dat ID 35 vast is. Bijvoorbeeld door als variabele te definieren : #dim ActivieitGroepId As %Integer = 35 ; Hard-coded, want vast...

Ook al is het maar een deploy-klasse, toch zou ik duidelijk maken dat ID 35 vast is.
Bijvoorbeeld door als variabele te definieren :

#dim ActivieitGroepId As %Integer = 35   ; Hard-coded, want vaste ID
Set ^APPS.HaluxA368.ActiviteitGDA7FD(ActivieitGroepId)=$lb("~APPS.Halu...
3:02 pm

Wim Vermeulen

Voor mij persoonlijk is dit één stapje te ver gerefactored. De method ZetProductGegevens() zou ik niet gebruiken en MaatwerkProduct niet als property maar gewoon als variabele (wordt slechts 1x geb...

Voor mij persoonlijk is dit één stapje te ver gerefactored.
De method ZetProductGegevens() zou ik niet gebruiken en MaatwerkProduct niet als property maar gewoon als variabele (wordt slechts 1x gebruikt). Voor ingegevenKenmerken wel O.K.

Dan zou de code er zo uit zien : 

#dim MaatwerkProduct As DOM.PM.MaatwerkProduct = ..ProductTypeApi.GeefMaatwerkProduct(ProductId)
Set ..IngegevenKenmerken = MaatwerkProduct.GeefIngegevenKenmerken()
Thursday 10 Oct 2019
9:53 am

Kurt Callebaut

ICT-930: Refactoring nodig voor klassen van activiteiten. (vooral spoelbak is meer dan enkel spoelbak.)
ICT-930: Refactoring nodig voor klassen van activiteiten. (vooral spoelbak is meer dan enkel spoelbak.)
Friday 27 Sep 2019
8:16 am

Kurt Callebaut

7:54 am

Kurt Callebaut

De stat juist, Heb ik bewust gewijzigd. Anders krijgen we standaard het gedrag dat spoelbak en smalle lade altijd in spoelbakgroep terecht komen en default gedrag is dat dit niet gebeurd. daarom he...

De stat juist, Heb ik bewust gewijzigd. Anders krijgen we standaard het gedrag dat spoelbak en smalle lade altijd in spoelbakgroep terecht komen en default gedrag is dat dit niet gebeurd. daarom heb ik ook de altijdsplitser moeten toevoegen.
maw indien true, faalde de bestaande testen op gewone aantallen als er geen splitser meegegeven wordt..

(Ik vind het nog steeds een gevaarlijk iets om te beslissen dat de groepering per kenmerk vervangen wordt door groepering op productiegroep, terwijl het in dit geval toevallig hetzelfde is.)

Thursday 26 Sep 2019
2:12 pm

Wim Vermeulen

2:11 pm

Wim Vermeulen

Kritische vraag (misschien om in de groep te gooien) : Is dit een project-setting? --> volgens mij enkel bedoeld om iets nieuw te kunnen "infaseren". Nadien wordt een project-setting opgekuist (in ...

Kritische vraag (misschien om in de groep te gooien) :
Is dit een project-setting? --> volgens mij enkel bedoeld om iets nieuw te kunnen "infaseren". Nadien wordt een project-setting opgekuist (in theorie althans )
Is dit dan eerder een gewone ConfigItem? --> is eigenlijk bedoeld om onderscheid te kunnen maken tussen de verschillende omgevingen (bvb. andere waarde op PROD dan op DEV)
Is het voldoende om deze gewoon als Class Parameter te definieren? --> volgens mij wel. Dit is een single-responsibility klasse, dus een wijziging van de waarde committen en mergen, kan supersnel.

Willen we dit toch wegtrekken uit onze code, dan misschien toch een aparte data-structuur voor maken. Functioneel is config-item hiervoor ook wel geschikt, maar ik vraag me af of we den boel dan niet te veel gaan vervuilen?

1:10 pm

Wim Vermeulen

Ook testmethod maken met VerwachtMethodCall("IsNodigTeSplitsen" --> dan Return False

Ook testmethod maken met
VerwachtMethodCall("IsNodigTeSplitsen" --> dan Return False

12:29 pm

Wim Vermeulen

ToeleveringSplitserMock.Verifieer()

ToeleveringSplitserMock.Verifieer()

12:07 pm

Wim Vermeulen

AltijdSplitser kan best in de constructor, want nu krijg je een nieuwe instantie bij iedere "VoegToe"-call.

AltijdSplitser kan best in de constructor, want nu krijg je een nieuwe instantie bij iedere "VoegToe"-call.

11:38 am

Wim Vermeulen

Quit $$$True (want ik zie dat de UT-resultaten gewijzigd zijn, en dat was niet de bedoeling http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif )

Quit $$$True
(want ik zie dat de UT-resultaten gewijzigd zijn, en dat was niet de bedoeling )

11:12 am

Wim Vermeulen

Deze klasse mag weg, indien geen Config-item gebruikt wordt. (zie opmerking bij ToeleveringSplitser)

Deze klasse mag weg, indien geen Config-item gebruikt wordt. (zie opmerking bij ToeleveringSplitser)

10:53 am

Wim Vermeulen

Deze klasse is niet meer nodig, denk.

Deze klasse is niet meer nodig, denk.