Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken. Alvast bedankt om het aantal testfiles te filteren. Het geheel lijkt me we...

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken.
Alvast bedankt om het aantal testfiles te filteren.
Het geheel lijkt me wel oké te zijn.

Ook in eCon enkele commits gereviewed.

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten) In combinatie met svn rev. 69413 is dit wel een correcte overgang. Hierbij is deze cha...

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten)
In combinatie met svn rev. 69413 is dit wel een correcte overgang.

Hierbij is deze change nagekeken en goedgekeurd!

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om ui...

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om uit te maken a.d.h.v. de changesets of het al dan niet over code gaat of testfiles.
P.s.: Ook kleine change in Econ (zie svn).

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- Nieuwe testbestanden - deel 8 van...

=> Aanpassing na toevoeging vertaling Moventokoppeling

    • -5
    • +5
    ./Bodem/TestLosseComponentenInBulkAfwijkendeKantenband_Langteksten.txt
  1. … 2 more files in changeset.
[ICT] TAX Eindkap kleur : vertalingen gewijzigd (aanpassing door PM)

- vhTestFiles aangepast (o.a. "Alu look" is vervangen door "Aluminium Silbergrau")

    • -1
    • +1
    ./Front/TestLosseComponentenInBulkAfwijkendeKantenband_Langteksten.txt
    • -1
    • +1
    ./FrontLBX/TestLosseComponentenInBulkAfwijkendeKantenband_Langteksten.txt
  1. … 18 more files in changeset.
[ICT-1482] TA’ORBOX front op een LEGRABOX lade
[ICT-1482] TA’ORBOX front op een LEGRABOX lade
[ICT-1482] TA’ORBOX front op een LEGRABOX lade

- Langtekst verder fixen + bijhorende integratietesten

- vertaling toevoegen voor "voor LEGRABOX"

    • -0
    • +48
    ./FrontLBX/TestLosseComponentenInBulkAfwijkendeKantenband_Langteksten.txt
  1. … 5 more files in changeset.
[ICT] wijzigingen Taor kleuren. (ref JB)
    • -4
    • +4
    ./Bodem/TestLosseComponentenInBulkAfwijkendeKantenband_Langteksten.txt
    • -8
    • +8
    ./Rug/TestLosseComponentenInBulkAfwijkendeKantenband_Langteksten.txt
    • -12
    • +12
    ./ZijkantLinks/TestLosseComponentenInBulkAfwijkendeKantenband_Langteksten.txt
    • -12
    • +12
    ./ZijkantRechts/TestLosseComponentenInBulkAfwijkendeKantenband_Langteksten.txt
    • -4
    • +4
    ./Front/TestLosseComponentenInBulkAfwijkendeKantenband_Langteksten.txt
  1. … 155 more files in changeset.
De ladekleur in deze testcase is fineer. De kantenband wordt dus nu ook fineer, de ingevulde data is vergelijkbaar met andere fineer-data

De ladekleur in deze testcase is fineer. De kantenband wordt dus nu ook fineer, de ingevulde data is vergelijkbaar met andere fineer-data

Methodnaam best aanpassen naar ... iets zoals : Test: Positie 17-18: Front - Kantenband volgt ladekleur ipv kantenbandkleur

Methodnaam best aanpassen naar ... iets zoals :

Test: Positie 17-18: Front - Kantenband volgt ladekleur ipv kantenbandkleur 
Pass 3 en 4 zijn er nu bijgekomen (lees: ingevuld). Zou dit door het verschil tussen melamine- en fineer-kantenband zijn?

Pass 3 en 4 zijn er nu bijgekomen (lees: ingevuld). Zou dit door het verschil tussen melamine- en fineer-kantenband zijn?

Exit na Pass4. Dit klopt met vorige opmerking.

Exit na Pass4. Dit klopt met vorige opmerking.

Printer bij Pass1 i.p.v. bij Pass2 --> te checken.

Printer bij Pass1 i.p.v. bij Pass2 --> te checken.

Tja, geen idee of de wijzigingen voor de Kantenband-data perfect zijn. Dit lijkt me het normale gevolg te zijn van de aanpassing voor het front, ik zou durven denken dat de software hier universeel...

Tja, geen idee of de wijzigingen voor de Kantenband-data perfect zijn.
Dit lijkt me het normale gevolg te zijn van de aanpassing voor het front, ik zou durven denken dat de software hier universeel geschreven is (voor rug, zijkant, front).

Hoor eens bij Pieter, of rechtstreeks bij de SQL-specialist van de kantenband machine

Voor de method GenereerKenmerken() zou je ook de ##super(...) kunnen gebruiken. Nu heb je de method keihard overschreven, en zal dus nooit meer de aanpassingen van de parent-class volgen (tenzij du...

Voor de method GenereerKenmerken() zou je ook de ##super(...) kunnen gebruiken.
Nu heb je de method keihard overschreven, en zal dus nooit meer de aanpassingen van de parent-class volgen (tenzij dubbel onderhoud)

Wat denk je van deze:

	#dim SelectieKenmerken As %ArrayOfDataTypes = ##super(IngegevenKenmerken)
	If IngegevenKenmerken.IsBinnenlade {
		Do SelectieKenmerken.SetAt(IngegevenKenmerken.LadeKleur, "LadeKleur")
	}
	Quit SelectieKenmerken
[BUS705] TAX: Fronten: kantenband volgt frontkleur (met spatie)
[BUS705] TAX: Fronten: kantenband volgt frontkleur (met spatie)
[BUS705] TAX: Fronten: kantenband volgt frontkleur
    • -5
    • +0
    ./Front/TestLosseComponentenInBulkAfwijkendeKantenband_Langteksten.txt
  1. … 5 more files in changeset.
  • More
  • CR-430
  • finished reviewing
[ICT286] TAX LC (losse componenten): falende test EAN-code
[ICT286] TAX LC (losse componenten): falende test EAN-code
[ICT286] TAX LC (losse componenten): falende test EAN-code

- Standaard TAX binnenlade kan geen schroefkoppeling hebben, unittestjes moeten volgen

    • -5
    • +5
    ./Bodem/TestLosseComponentenInBulkAfwijkendeKantenband_Langteksten.txt
  1. … 32 more files in changeset.
  • More
  • CR-101
  • finished reviewing