Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1187] PA-UB-073: DnaCodes voor taor binnenlade flatpack
[ICT-1187] PA-UB-073: DnaCodes voor taor binnenlade flatpack
[ICT-1187] PA-UB-073: DnaCodes voor taor binnenlade flatpack

- Bij het tweede etiket dat wordt afgedrukt in dit scenario wordt het bepalen van een DnaCode onderdrukt.

  1. … 2 more files in changeset.
[ICT756] [Kanban UST4279] TAXFP1: Bijsturing en extra implementatie:

- Na review => KlantEtiketInput-object per insert => Vooraf gedimd want dimmen en setten in 1 keer zorgde voor 2 bestanden ipv 1 => fluke van Cache wellicht

    • -17
    • +18
    ./PerVerpakking/FlatpackV1.cls.xml
[ICT756] [Kanban UST4279] TAXFP1: Bijsturing en extra implementatie:

- Na review => KlantEtiketInput-object per insert => copy-paste-errorke rechtgezet

[ICT756] [Kanban UST4279] TAXFP1: Bijsturing en extra implementatie:

- Na review => KlantEtiketInput-object per insert

  1. … 1 more file in changeset.
In vhUnitTest.APPS.Halux.PPS.Activiteit.impl.TAOR.Verpakking.WerklijstItemsMaker.MaakWerklijstItems.TestFlatpackV1 moet je volgende 2 regels toevoegen. Dat zal bewijzen dat uw implementatie hierbov...

In vhUnitTest.APPS.Halux.PPS.Activiteit.impl.TAOR.Verpakking.WerklijstItemsMaker.MaakWerklijstItems.TestFlatpackV1
moet je volgende 2 regels toevoegen. Dat zal bewijzen dat uw implementatie hierboven niet correct is.

	Do $$$AssertEquals(WerklijstItems.GetAt(1).UitvoeringInfo.KlantEtiketData.LadeAfbeeldingLocatie, "GENERIEK\TAOR_BOX\VET\TAOR_BOX_C_Z.JPG")
	Do $$$AssertEquals(WerklijstItems.GetAt(2).UitvoeringInfo.KlantEtiketData.LadeAfbeeldingLocatie, "GENERIEK\TAOR_BOX\VET\TAOR_BOX_C_Z_Front.JPG")
Ooow, een foutje !! In de review van [UST4237] is dit ontsnapt aan het oog van de reviewer :-O Voor iedere Insert() moet je een nieuw KlantEtiketInput-object maken ==> KlantEtiketInput = ##class(A...

Ooow, een foutje !!
In de review van [UST4237] is dit ontsnapt aan het oog van de reviewer :-O

Voor iedere Insert() moet je een nieuw KlantEtiketInput-object maken ==> KlantEtiketInput = ##class(APPS.Halux.PPS.Activiteit.impl.TAOR.Verpakking.KlantEtiketInput).%New()
anders gaat het fout.

Ik zal sebiet dus ook de UT's kritisch moeten nakijken :-P

Hmm, dus dit is eigenlijk het enige argument dat pleit voor de term "probox" en tégen de term "V1", hé. Maar ik volg verder de redenering van PM wel http://subversion02.vanhoecke.be/static/ogdo0b/2...

Hmm, dus dit is eigenlijk het enige argument dat pleit voor de term "probox" en tégen de term "V1", hé.
Maar ik volg verder de redenering van PM wel

Dit gewoon ter info.

[ICT756] [Kanban UST4279] TAXFP1: Bijsturing en extra implementatie:
[ICT756] [Kanban UST4279] TAXFP1: Bijsturing en extra implementatie:
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.

[ICT756] [Kanban UST4279] TAXFP1: Bijsturing en extra implementatie:

- Juiste afbeelding op KlantEtiket igv Front => Is losse component

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

- Move class vanwege naamswijziging => FlatpackV1

    • -47
    • +0
    ./PerVerpakking/FlatpackProbox.cls.xml
  1. … 1 more file in changeset.
Gezien de .Reset() wordt opgeroepen, zou ik de .%New() verplaatsen naar boven de While() Uw voorgangers hebben dat blijkbaar ook niet gedaan, doe deze maar gerust wel.

Gezien de .Reset() wordt opgeroepen, zou ik de .%New() verplaatsen naar boven de While()
Uw voorgangers hebben dat blijkbaar ook niet gedaan, doe deze maar gerust wel.

[UST4237] TAXFP1: Productie-aansturing - Etiketten:

- Correctie foute implementatie => extra KlantEtiket indien binnenlade => voor extra doos van Front

    • -28
    • +24
    ./PerVerpakking/FlatpackProbox.cls.xml
  1. … 1 more file in changeset.
[UST4237] TAXFP1: Productie-aansturing - Etiketten:
[UST4237] TAXFP1: Productie-aansturing - Etiketten:
[UST4237] TAXFP1: Productie-aansturing - Etiketten:

- Eigen WerklijstItemsMaker voor FlatpackProbox => KlantEtiket per HalffabItem

    • -0
    • +51
    ./PerVerpakking/FlatpackProbox.cls.xml
  1. … 1 more file in changeset.
Ter info : 1) Alles in één testklasse is oké; maar je kan ook meerdere testklasses maken, met naam .TestLosseComponentInBulk.cls en .TestBOG.cls 2) Testklasse bevat meestal ook de naam van de publi...

Ter info :
1) Alles in één testklasse is oké; maar je kan ook meerdere testklasses maken, met naam .TestLosseComponentInBulk.cls en .TestBOG.cls
2) Testklasse bevat meestal ook de naam van de public method die je aan 't testen bent. Maar misschien werd de naam dan te lang.
Je hoeft dit hier niet te veranderen, maar is taak van reviewer om dit op te merken

Deze method doet exact hetzelfde als ..BestelEenheidLosseComponentenInBulkBepaler.GeefBestelEenheidZRFLosseComponentenInBulk() Uiteraard klopt dat ook qua resultaat, maar waarom staat de code dan d...

Deze method doet exact hetzelfde als ..BestelEenheidLosseComponentenInBulkBepaler.GeefBestelEenheidZRFLosseComponentenInBulk()
Uiteraard klopt dat ook qua resultaat, maar waarom staat de code dan dubbel? --> dubbel onderhoud.
Misschien kan je hier (BOG) de implementatie van LosCompInBulkk laten oproepen.