Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1494] HLX: TAX - aangeven hoeveelste tray van batch aan robots

- Reviewopmerkingen behandelen

  1. … 1 more file in changeset.
Zoals elders gezegd... geen waarden via parameters doorgeven in een constructor, dat moet je dan via een aparte public method doen! De ZetTekstLinks en ZetTekstMidden mogen wel blijven staan, omdat...

Zoals elders gezegd... geen waarden via parameters doorgeven in een constructor, dat moet je dan via een aparte public method doen!
De ZetTekstLinks en ZetTekstMidden mogen wel blijven staan, omdat het louter defaultering is.

Ik zie dat dit een copy/paste is van hierboven, maar dan toch even de casing aanpassen aub http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif For Set $Orde...

Ik zie dat dit een copy/paste is van hierboven, maar dan toch even de casing aanpassen aub
For
Set
$Order
Quit

En ook graag een leeg lijntje tussen de For-loop en de uiteindelijk Quit
Misschien gelijk ook de casing elders in deze klasse aanpassen, is niet zo veel.

Oeps! Dit is een beetje tegen de schenen schoppen van onze code-conventions http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif De constructor gebruiken we ...

Oeps! Dit is een beetje tegen de schenen schoppen van onze code-conventions
De constructor gebruiken we ENKEL voor het intantiëren van een object. De énige parameters die daar dus aan meegegeven mogen worden zijn degene die voor die intantiëring essentieel zijn, zoals dependencies.
Wat we zeker NIET doen in constructors, of toch zeker zoveel als mogelijk proberen te vermijden, is het doorgeven van waarden (als parameters) om properties te zetten. Dat maakt namelijk een knoeiboel van eventuele testen.
Wat je dus eigenlijk moet doen is die 2 parameters er weer uithalen en in de klasse waarvan je hier een object maakt een public method maken om het zetten van die properties te regelen.
Dat maakt dan wel dat je hier 2 coderegels gaat hebben: Eentje om een object te instantiëren en direct daarna een tweede om op die instantie die public method te callen met deze 2 parameters, bv:
Do VoetTekst.ZetPaginaNrEnTotaalAantalBladzijden(HuidigPaginaNr, TotaalAantalBladzijden)

[ICT-1494] HLX: TAX - aangeven hoeveelste tray van batch aan robots
[ICT-1494] HLX: TAX - aangeven hoeveelste tray van batch aan robots
[ICT-1494] HLX: TAX - aangeven hoeveelste tray van batch aan robots

- In de koptekst wordt nu aangegeven welke tray huidige tray is en hoeveel er in totaal zijn

- in de voettekst klopt de paginanummer nu wel en wordt ook aangegeven hoeveel paginas er zijn in totaal

  1. … 23 more files in changeset.
[UST4053] HX: TAX zijkant: pickinglijst voor frezen: exacte zaagmaat i.p.v. nominale diepte (onderscheid...
[UST4053] HX: TAX zijkant: pickinglijst voor frezen: exacte zaagmaat i.p.v. nominale diepte (onderscheid...
[UST4053] HX: TAX zijkant: pickinglijst voor frezen: exacte zaagmaat i.p.v. nominale diepte (onderscheid Baliko/Vällinge duidelijk maken)

- aanpassing en al enkele testjes

- resultaat al met GBS besproken en was ok

  1. … 2 more files in changeset.
  • More
  • CR-298
  • finished reviewing
[BUS639] : TAOR : Nieuw Wit & Zwart voor iedereen
[BUS639] : TAOR : Nieuw Wit & Zwart voor iedereen
[BUS639] : TAOR : Nieuw Wit & Zwart voor iedereen
  1. … 65 more files in changeset.
  • More
  • CR-289
  • finished reviewing
VerwachtMethodCall in de testjes?

VerwachtMethodCall in de testjes?

[BUS633] - TAX: wit: Alpnach-uitzondering uitschakelen te gepasten tijde
[BUS633] - TAX: wit: Alpnach-uitzondering uitschakelen te gepasten tijde
Merged revision(s) 42407 from trunk2010:

[BUS633] - TAX: wit: Alpnach-uitzondering uitschakelen te gepasten tijde

maw reverting [BUS626] - TAX: wit: Alpnach tijdelijk andere kleurcode naar productie ( met uitzondering van de aanpassingen in mocks en verwijderen van duplicate lijn )

  1. … 3 more files in changeset.
  • More
  • CR-269
  • finished reviewing
[BUS626] - TAX: wit: Alpnach tijdelijk andere kleurcode naar productie

- Fix van bepaling in voor kantenband en plaatmateriaal

- Kleur vermelding op etiket.

- ZijkantPlaatPicking document kleur vermelding.

  1. … 3 more files in changeset.
[BUS626] - TAX: wit: Alpnach tijdelijk andere kleurcode naar productie
[BUS626] - TAX: wit: Alpnach tijdelijk andere kleurcode naar productie
[UST1301] TAX-PAS-Bi-Colour: Picking zijkantplaten doc aanpassen

- opnieuw enkele spaties toegevoegd

[UST1301] TAX-PAS-Bi-Colour: Picking zijkantplaten doc aanpassen

- extra spaties weggehaald

  1. … 15 more files in changeset.
[UST1301] TAX-PAS-Bi-Colour: Picking zijkantplaten doc aanpassen

- property Kleur ZijkantPlaatPickingUitvoeringInfo als String ipv enumeratie

- indien bi-colour worden 4 kleuren getoond

- aangepaste integratietests

  1. … 19 more files in changeset.