Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT718] [Kanban UST4262] TAXFP1: Kostprijsberekening - Tijden: Kostenplaatsen:
[ICT718] [Kanban UST4262] TAXFP1: Kostprijsberekening - Tijden: Kostenplaatsen:
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.

[ICT718] [Kanban UST4262] TAXFP1: Kostprijsberekening - Tijden: Kostenplaatsen:

- Aangepaste kosten-tijden

  1. … 2 more files in changeset.
[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

  1. … 3 more files in changeset.
[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.
Check http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Check

#dim met .%New( ... ) :-/ Zeker met een lange klassenaam zoals hierboven (en hieronder) staat de relevante code gegarandeerd buiten beeld. Ik zie hiervan echt geen meerwaarde. Als de nuttige info i...

#dim met .%New( ... ) :-/
Zeker met een lange klassenaam zoals hierboven (en hieronder) staat de relevante code gegarandeerd buiten beeld.
Ik zie hiervan echt geen meerwaarde.
Als de nuttige info in beeld kan staan (korte klassenaam) dan laat ik da aan mij voorbijgaan tijdens reviewen. Maar in dit geval vind ik da écht hopeloos zinloos!

N.B.: ik ga ervan uit dat je dit ook maar gekopieerd hebt van daarboven hé.

Effe ter info : door het aanpassen van de #dim in dezelfde svn-commit is het reviewen van deze klasse behoorlijk lastig geworden. Best volgende keer hoofdletters en andere cosmetische wijzigingen a...

Effe ter info :
door het aanpassen van de #dim in dezelfde svn-commit is het reviewen van deze klasse behoorlijk lastig geworden.
Best volgende keer hoofdletters en andere cosmetische wijzigingen apart committen.
In deze klasse zou de review verbazend simpel worden. Echt waar

Graag effe feedback aub.

Voor één waarde te vergelijken geen .Equals() method gebruiken; dit is overkill. Voor 2 (of meer) zeker wel gerechtvaardigd, en tevens mijn persoonlijke voorkeur, eerder dan (value = "A") || (value...

Voor één waarde te vergelijken geen .Equals() method gebruiken; dit is overkill.
Voor 2 (of meer) zeker wel gerechtvaardigd, en tevens mijn persoonlijke voorkeur, eerder dan

(value = "A") || (value = "B")
IngegevenKenmerkenTypeAPI --> IsFlatpackProboxVerpakkingBepaler --> complexe constructie, veel omwegen om uiteindelijk iets simpel op te vragen, iets dat enkel bij TAX zal voorkomen. Bij nader inz...

IngegevenKenmerkenTypeAPI --> IsFlatpackProboxVerpakkingBepaler --> complexe constructie, veel omwegen om uiteindelijk iets simpel op te vragen, iets dat enkel bij TAX zal voorkomen.

Bij nader inzien geldt dezelfde opmerking voor de "IsEenLosseComponentenVerpakkingBepaler". Wellicht heg je je daarop gebaseerd.
Dan is je actie min of meer gerechtvaardigd.

P.S.: ik had deze review-opmerking gewoon kunnen wissen, want je moet dus niks doen. Maar ik wou mijn bedenking toch effe delen.

[UST4235] TAXFP1: Kostprijsberekening - Tijden:
[UST4235] TAXFP1: Kostprijsberekening - Tijden:
[UST4235] TAXFP1: Kostprijsberekening - Tijden:

- Kostenplaats TA'OR VPK Flatpack Probox - implementatie

  1. … 4 more files in changeset.
TAOR "specifiek" in de algemene "common" KostItemBuilder ... Alleen als je deze method met (zeer) weinig moeite kan verplaatsen naar andere klasse, anders is het voorlopig oké.

TAOR "specifiek" in de algemene "common" KostItemBuilder ...

Alleen als je deze method met (zeer) weinig moeite kan verplaatsen naar andere klasse, anders is het voorlopig oké.

[UST4169] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Kostprijs:...
[UST4169] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Kostprijs:...
[UST4169] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Kostprijs:

- Tijden kartonmachine => Niet meer nodig voor zijkant, rug en front

[UST1271] TAX DuoColor - correctie type
[UST1186] TAX LC: evaluator IsEenLosseComponentenVerpakking zodat tijden voor kartonmachine aangerekend worden
  1. … 9 more files in changeset.
[UST1249] TAX LC: rug/bodem/zijkant/front: juiste items in boom

- Rol Front is terug

  1. … 21 more files in changeset.
[UST1249] TAX LC: rug/bodem/zijkant/front: juiste items in boom

- Rol rug is terug

  1. … 40 more files in changeset.
[UST1242] TAX-LC : HalffabItemsBoom is veranderd door IDynamischSamengesteldHF

- AddHalffabItem niet meer

- Kosten voor zijkant LI/RE i.p.v. zijkantplaat

- BouwsteenCreator verwerkt samengesteldHF

- BouwsteenmappingxTAXVerpakkingen

  1. … 5 more files in changeset.