Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2966] [rvTHB] Aanpassingen Lodder sturing

- Een extra underscore gezet tussen ImportFile en SpecialWorkshop

    • -1
    • +1
    ./OptimalisatieBestandGenerator.cls.xml
  1. … 10 more files in changeset.
Extra integratietest is hiervoor aangemaakt: "TestUitsluitendSpecialWorkshop". Momenteel bevat deze enkel een W7 Front, mochten er later meer SpecialWorkshop zaken bijkomen mogen deze hier uiteraar...

Extra integratietest is hiervoor aangemaakt: "TestUitsluitendSpecialWorkshop".
Momenteel bevat deze enkel een W7 Front, mochten er later meer SpecialWorkshop zaken bijkomen mogen deze hier uiteraard ook bij.

Het probleem is dat je hier echt aan het randje zit van wat Windows aankan met de volledige bestandsnaam (247 karakters). Als dit langer is wordt er helemaal geen bestand aangemaakt. Voor deze IT ...

Het probleem is dat je hier echt aan het randje zit van wat Windows aankan met de volledige bestandsnaam (247 karakters). Als dit langer is wordt er helemaal geen bestand aangemaakt.

Voor deze IT (BinnenladeLadeKleurW7) heb ik alvast de 006 terug toegevoegd. Dan zijn er in totaal 246 karakters voor de special workshop ImportFile met 100-999 aantal stuks.
Voor de LosseComponenten: volgens mij had deze oorspronkelijk ook geen nummering? Hoe dan ook zou dit in totaal dan 247 karakters zijn met een nummering, toch een beetje gevaarlijk lijkt mij (hoewel nipt mogelijk).

Ik heb de indruk dat er nu soms ook een leeg bestand wordt gegenereerd (als alles W7 -front is, dan wordt "het originele" bestand ook nog gegenereerd. Klopt dat? Toch precies wel volgens de testfil...

Ik heb de indruk dat er nu soms ook een leeg bestand wordt gegenereerd (als alles W7 -front is, dan wordt "het originele" bestand ook nog gegenereerd. Klopt dat? Toch precies wel volgens de testfiles). Dat lijkt mij onwenselijk.

ProductTypeApi en ToeleveringService mogen terug verwijderd worden wegens uiteindelijk niet gebruikt (zie ook verderop). P.s.: Ook als parameter en als property te verwijderen! En eveneens niet ve...

ProductTypeApi en ToeleveringService mogen terug verwijderd worden wegens uiteindelijk niet gebruikt (zie ook verderop).
P.s.: Ook als parameter en als property te verwijderen!

En eveneens niet vergeten de nodige aanpassingen door te voeren in bijhorende testcase: vhUnitTest.APPS.Halux.PPS.Activiteit.impl.TAOR.LijstVerwerkers.Opdeelzaag.OptimalisatieBestandenGenerator.VoerUit.TestProductGroep

Ik mis nog een test met als rol "Front".

Ik mis nog een test met als rol "Front".

Ai, dat is spijtig, dat je hier die "006" hebt weggelaten. Hierdoor staat de W7-specifieke test niet meer proper net onder de niet-W7-specifieke test. Dat zie je overzichtelijk als je deze klasse o...

Ai, dat is spijtig, dat je hier die "006" hebt weggelaten. Hierdoor staat de W7-specifieke test niet meer proper net onder de niet-W7-specifieke test. Dat zie je overzichtelijk als je deze klasse opzoekt via SVN. Die test-nummering heeft een doel
Hetzelfde geldt tevens ook voor de LosseComponenten-test!

Ik zou "ProductieGroep" niet afkorten in de variabelenaam, hij wordt door deze ingreep al meer dan voldoende ingekort http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emotic...

Ik zou "ProductieGroep" niet afkorten in de variabelenaam, hij wordt door deze ingreep al meer dan voldoende ingekort
Dus: ProductieGroepEnum.

Bovenstaande private method BevatBatchProductMetBewerkingInSpecialWorkshop mag verwijderd worden, wegens uiteindelijk toch niet gebruikt (zie ook hogerop)

Bovenstaande private method BevatBatchProductMetBewerkingInSpecialWorkshop mag verwijderd worden, wegens uiteindelijk toch niet gebruikt (zie ook hogerop)

Property IsSpecialWorkshop mag terug verwijderd worden, want wordt uiteindelijk toch niet gebruikt (zie ook verderop).

Property IsSpecialWorkshop mag terug verwijderd worden, want wordt uiteindelijk toch niet gebruikt (zie ook verderop).

Opkuis: Lijn in comment doet niet veel http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Opkuis: Lijn in comment doet niet veel

[ICT-2966] [rvTHB] Aanpassingen Lodder sturing
[ICT-2966] [rvTHB] Aanpassingen Lodder sturing
[ICT-2966] [rvTHB] Aanpassingen Lodder sturing

- Bestandsnaam toch "SpecialWorkshop" laten bevatten, maar integratietests hernoemd zodat de limiet niet bereikt wordt

    • -1
    • +1
    ./OptimalisatieBestandGenerator.cls.xml
  1. … 5 more files in changeset.
[ICT-2966] [rvTHB] Aanpassingen Lodder sturing

- Bestandsnaam korter gemaakt omdat we boven de Windows-limiet zaten van 247 karakters (inclusief pad)

    • -1
    • +1
    ./OptimalisatieBestandGenerator.cls.xml
[ICT-2966] [rvTHB] Aanpassingen Lodder sturing

- IsSpecialWorkshop toegevoegd

- method GeefAantalStuksSpecialWorkshop uitgewerkt en unit test voor gemaakt

    • -5
    • +16
    ./OptimalisatieBestandGenerator.cls.xml
  1. … 2 more files in changeset.
[ICT-2966] [rvTHB] Aanpassingen Lodder sturing

- OptimalisatieDataLijst als parameter gemaakt in de method, zodat die opgesplitst kan worden en meegegeven

- Extra underscore toegevoegd op het einde van de BestandsNaamPrefix

    • -1
    • +1
    ./OptimalisatieBestandGenerator.cls.xml
  1. … 1 more file in changeset.
[ICT-2966] [rvTHB] Aanpassingen Lodder sturing

- Method 'GeefBestandNaamPrefix' aangemaakt

    • -2
    • +19
    ./OptimalisatieBestandGenerator.cls.xml
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- OptimalisatieData.Kleur wordt elders ook nog gebruikt en daar wordt die S ongewenst geconverteerd naar S8 in dat geval, dus dto'ke uitgebreid met boolean IsKleurOudZwart om onderscheid te kunnen maken wanneer die conversie wel en niet gewenst is

  1. … 2 more files in changeset.
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- Op juiste plaatsen OudZwart converteren naar S zijnde Zwart

  1. … 2 more files in changeset.
Zoals besproken blijkbaar ongewild in deze review terechtgekomen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Zoals besproken blijkbaar ongewild in deze review terechtgekomen

is deze wel nodig?

is deze wel nodig?

  • More
  • CR-746
  • finished reviewing
[BUS661] -TA'OR : Baliko concept -> andere afmetingen
[BUS661] -TA'OR : Baliko concept -> andere afmetingen
[BUS661] -TA'OR : Baliko concept -> andere afmetingen

- Refactoring quick en dirty aanpassingen

  1. … 1 more file in changeset.