Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
dto ...MetKar hier gebruikt in de impl van BuilderZonderKar? Indien de definitie/implementatie hetzelfde is voor "metKar" en "ZonderKar", dan best de klassenaam van de dto renamen naar iets gemeens...

dto ...MetKar hier gebruikt in de impl van BuilderZonderKar?
Indien de definitie/implementatie hetzelfde is voor "metKar" en "ZonderKar", dan best de klassenaam van de dto renamen naar iets gemeenschappelijk, bvb. zonder "Kar" (if you know what I mean :-D )

[ICT-2564] [rvWV] HX: TAX: Movento-koppeling: Builders:
[ICT-2564] [rvWV] HX: TAX: Movento-koppeling: Builders:
[ICT-2564] [rvWV] HX: TAX: Movento-koppeling: Builders:

- Uitbreiding in Koppeling-builder => ProductieRoute voor Movento-Koppeling

- Kleine refactor in Koppeling-Builder => IsGemonteerdeLade gebruikt ipv opsomming verpakkingen voor gemonteerd => waar bovendien PalletVerpakking nog miste

- Toevoeging KoppelingAdapter-builder voor MoventoKoppeling

  1. … 9 more files in changeset.
[ICT-717] [ICT502] [rvWV] UT VerpakkingTypes: IsgemonteerdeLade() uitmocken:

- Na review => Eerst Super uitvoeren in constructors

- Rechtzetting in naamgeving UnitTest

  1. … 4 more files in changeset.
Aangezien daar geen automatische testen voor zijn, doen we dat af en toe maar eens manueel hé http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif Stom copy-p...

Aangezien daar geen automatische testen voor zijn, doen we dat af en toe maar eens manueel hé
Stom copy-paste-foutje hé

Tekst in methodnaam klopt niet meer. Is nog steeds "Onderdelenpicking. om de reviewer te testen, zekers? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Tekst in methodnaam klopt niet meer. Is nog steeds "Onderdelenpicking.

om de reviewer te testen, zekers?

Graa deze aanpassen naar de nieuwere afspraak. Dus eerste lijn Do ##super() en op t einde Quit $$$OK

Graa deze aanpassen naar de nieuwere afspraak.
Dus eerste lijn Do ##super() en op t einde Quit $$$OK

Zie opmerking bij AfdekkapBovenV000: Quit $$$OK

Zie opmerking bij AfdekkapBovenV000:
Quit $$$OK

Deze stond nog volgens de "oude standaard" http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/sad.gif Graa deze aanpassen naar de nieuwere afspraak. Dus eerste lijn ...

Deze stond nog volgens de "oude standaard"
Graa deze aanpassen naar de nieuwere afspraak.
Dus eerste lijn Do ##super() en op t einde Quit $$$OK

Bij %OnNew() doen we ALTIJD eerst do ##super() en op t einde ne Quit $$$OK --> de ##super moet hier sowieso als eerste lijn van deze method gebeuren. Dit is al verschillende keren ter sprake gekom...

Bij %OnNew() doen we ALTIJD eerst do ##super() en op t einde ne Quit $$$OK
--> de ##super moet hier sowieso als eerste lijn van deze method gebeuren.

Dit is al verschillende keren ter sprake gekomen (bij andere collega's), maar het moet zo blijven.
Kan je dit dus terugzetten zoals het er stond?
Uiteraard mag uw lijn met $$$Inject wel blijven hé.

[ICT-717] [ICT502] [rvWV] UT VerpakkingTypes: IsgemonteerdeLade() uitmocken:
[ICT-717] [ICT502] [rvWV] UT VerpakkingTypes: IsgemonteerdeLade() uitmocken:
[ICT-717] [ICT502] [rvWV] UT VerpakkingTypes: IsgemonteerdeLade() uitmocken:

- IngegevenKenmerkenTypeApi uitgemockt i.g.v. HFKoppelingv001 => Volledige refactor TestCase

- In Base IngegevenKenmerken uit parameter halen en niet de property

  1. … 2 more files in changeset.
Is bij de laatste wijzigingen terug F geworden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Is bij de laatste wijzigingen terug F geworden

Blijft deze code "P" behouden? Of toch de "F" van oorspronkelijke "FlatpackV1" ? Best toch effe checken bij JohanB.

Blijft deze code "P" behouden? Of toch de "F" van oorspronkelijke "FlatpackV1" ?
Best toch effe checken bij JohanB.

[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:
[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:
[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:

- Verheuis code FlatpackProbox naar FlatpackV1

- Nog wat opkuis hier en daar

  1. … 87 more files in changeset.
[UST4238] TAXFP1: Productie-aansturing - Documenten:

- Na review => kleine refactor van If-structuur voor leesbaarheid en uitbreidbaarheid

Zoals besproken : De meest propere, zuivere en correcte oplossing is door de IngegevenKenmerkenTypeApi uit te mocken (1x Returns True en 1x Returns False) en dan is het niet meer nodig om over alle...

Zoals besproken :
De meest propere, zuivere en correcte oplossing is door de IngegevenKenmerkenTypeApi uit te mocken (1x Returns True en 1x Returns False) en dan is het niet meer nodig om over alle verpakkingstypes te lopen (uitz. FlatpackProbox).

Deze "refactor" zal in een aparte story opgenomen worden.
Voorlopig dus geen aanpassingen nodig

Complexe If-structuur --> moeilijk begrijpbaar, en zeer lastig voor verdere (latere) uitbreidingen. Beter om hier af te stappen van de compacte versie en iets meer uiteen te halen. Mijn voorstel : ...

Complexe If-structuur --> moeilijk begrijpbaar, en zeer lastig voor verdere (latere) uitbreidingen. Beter om hier af te stappen van de compacte versie en iets meer uiteen te halen.
Mijn voorstel :

	If (IngegevenKenmerken.VerpakkingType = ..Enum.VerpakkingType.FlatpackProbox) {
		Set ProductieRoute = ..Enum.ProductieRoute.OnderdelenPicking
	} ElseIf (IngegevenKenmerken.IsBinnenlade || ..API.IngegevenKenmerkenTypeAPI.IsGemonteerdeLade(IngegevenKenmerken)) {
		Set ProductieRoute = ..Enum.ProductieRoute.TeMonteren
	} Else {
		Set ProductieRoute = ..Enum.ProductieRoute.OnderdelenPicking
	}
[UST4238] TAXFP1: Productie-aansturing - Documenten:
[UST4238] TAXFP1: Productie-aansturing - Documenten:
[UST4238] TAXFP1: Productie-aansturing - Documenten:

- I.g.v. IsBinnenlade => indien FlatpackProbox => OnderdelenPicking

[BUS560] Daro: onderdelen niet meeleveren

- review opmerkingen: enus gebruiken

  1. … 1 more file in changeset.