Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
commentaar mag weg

commentaar mag weg

[ICT-1767] [rvTVE] [rvJWI] EC: MP: ORGALUX: productlijnen mee kopiëren van MP-winkelkar naar VH-winkelkar...
[ICT-1767] [rvTVE] [rvJWI] EC: MP: ORGALUX: productlijnen mee kopiëren van MP-winkelkar naar VH-winkelkar...
[ICT-1767] [rvTVE] [rvJWI] EC: MP: ORGALUX: productlijnen mee kopiëren van MP-winkelkar naar VH-winkelkar

-GeefLijnenIterator toegevoegd voor het veralgemenen van de methodes zo dat zowel productlijnen als maatwerklijnen worden geiterate ;)

  1. … 2 more files in changeset.
[ICT-1546] PM: Maatwerk: Space Step: Ingegeven Kenmerken Caché-kant inbouwen

- Fakes en Mocks committen

    • -9
    • +5
    ./GroepNaamVoorIngegevenKenmerkenBepaler.cls.xml
  1. … 2 more files in changeset.
[ICT-1546] PM: Maatwerk: Space Step: Ingegeven Kenmerken Caché-kant inbouwen

- Fakes en Mocks committen

    • -5
    • +9
    ./GroepNaamVoorIngegevenKenmerkenBepaler.cls.xml
  1. … 3 more files in changeset.
Hier 4 talen onder test, moeten we kaartje maken om testen voor andere talen toe te passen ?

Hier 4 talen onder test, moeten we kaartje maken om testen voor andere talen toe te passen ?

Vraagje : bodem en rug zijn dan nog steeds oude kleurbenaming ondanks dit plaatmateriaal is ?

Vraagje : bodem en rug zijn dan nog steeds oude kleurbenaming ondanks dit plaatmateriaal is ?

wordt natuurlijk al een lange constructor method, maar deze zou naar constructor mogen. Dan kan hij uitgemockt worden. Taal parameter wordt dan opgevangen door een "lazyload" toe te passen.

wordt natuurlijk al een lange constructor method, maar deze zou naar constructor mogen. Dan kan hij uitgemockt worden.
Taal parameter wordt dan opgevangen door een "lazyload" toe te passen.

[ICT-604][VRB647]PM: Kleuren: andere vertaling van letters S en W voor TAX (enkel langtekst)
[ICT-604][VRB647]PM: Kleuren: andere vertaling van letters S en W voor TAX (enkel langtekst)
[ICT-604][VRB647]PM: Kleuren: andere vertaling van letters S en W voor TAX (enkel langtekst)

- GroepNaamVoorIngegevenKenmerkenBepaler gerlateerd die al in vorige commit mee zat.

    • -0
    • +18
    ./GroepNaamVoorIngegevenKenmerkenBepaler.cls.xml
  1. … 2 more files in changeset.
[UW749] Syncen code
  1. … 9 more files in changeset.
Reverse merged revision(s) 56420 from trunk2010:

Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

revert van de revert omdat mijn wijzigingen wel goed waren, maar de rest van de ventjes niet :)

  1. … 65 more files in changeset.
Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

  1. … 65 more files in changeset.
[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...
  1. … 65 more files in changeset.
ja, maar code vliegt er over twee weken toch weer uit

ja, maar code vliegt er over twee weken toch weer uit

nope, code vliegt er over twee weken toch weer uit

nope, code vliegt er over twee weken toch weer uit

Eén instantie maken van legacypartij lijkt me voldoende.

Eén instantie maken van legacypartij lijkt me voldoende.

Hier ook 2 dezelfde testen met enkel alpnach en alpnachold als verschil ? vb één assert maken voor beide testen en klantid meegeven.

Hier ook 2 dezelfde testen met enkel alpnach en alpnachold als verschil ? vb één assert maken voor beide testen en klantid meegeven.

Los van de wijzigingen. Deze method Arrange() zegt niet veel, best hernoemen naar een beter naam.

Los van de wijzigingen. Deze method Arrange() zegt niet veel, best hernoemen naar een beter naam.

"hoe dan ook" lijkt me wat vaag. Bij welke actie gebeurd dit. Ik zou eerder opteren voor : Geeft steeds 35 weer bij het opvragen van klantsessiedata ? of overschrijft korting bij het zetten of opvr...

"hoe dan ook" lijkt me wat vaag. Bij welke actie gebeurd dit. Ik zou eerder opteren voor : Geeft steeds 35 weer bij het opvragen van klantsessiedata ? of overschrijft korting bij het zetten of opvragen van klantsessiedata.

Duplicate testcode op klantid na ?

Duplicate testcode op klantid na ?

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Dubbele test enkel met verschil alpnachoud(). zou ik samenvoegen om duplicate code te vermijden en klantid meegeven als parameter. (result meegeven is in dit geval zelfs niet nodig bij AssertBevest...

Dubbele test enkel met verschil alpnachoud(). zou ik samenvoegen om duplicate code te vermijden en klantid meegeven als parameter. (result meegeven is in dit geval zelfs niet nodig bij AssertBevestigingschroevenVoorKlantID().

Wil dat zeggen dat de hardcoded value 35 in andere klassen zoals APPS.EC.impl.KlantSessieDataBepaler vervangen moet worden door deze ingestelde waarde ?

Wil dat zeggen dat de hardcoded value 35 in andere klassen zoals APPS.EC.impl.KlantSessieDataBepaler vervangen moet worden door deze ingestelde waarde ?