ExterneWinkelkarService

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Zelfde opmerking als bij LBX

Zelfde opmerking als bij LBX

[UST3778] TAX-TB-001 : niet meeleveren "synchro koppelstuk" mag weg.
[UST3778] TAX-TB-001 : niet meeleveren "synchro koppelstuk" mag weg.
[UST3778] TAX-TB-001 : niet meeleveren "synchro koppelstuk" mag weg.

  1. … 88 more files in changeset.
En als ik ze er expliciet bij zet, dan zijn ze waarschijnlijk overbodig. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

En als ik ze er expliciet bij zet, dan zijn ze waarschijnlijk overbodig.

  • More
  • CR-126
  • finished reviewing
Ik zou echter alles testen door de APPS.EC.Winkelkar.impl.ExterneWinkelkarService.KenmerkenAanpasser ipv expliciet de APPS.EC.Winkelkar.impl.ExterneWinkelkarService.KenmerkenAanpasser.LBX aan te ro...

Ik zou echter alles testen door de APPS.EC.Winkelkar.impl.ExterneWinkelkarService.KenmerkenAanpasser ipv expliciet de APPS.EC.Winkelkar.impl.ExterneWinkelkarService.KenmerkenAanpasser.LBX aan te roepen.
Nu ontbreken we de stap dat de KenmerkenAanpasser de juiste aanpassing doet indien het om LBX gaat.

Test is voldoende ipv TestPasAan cfr TAX equivalent

Test is voldoende ipv TestPasAan cfr TAX equivalent

Klasse kan best hetzelfde overervings traject volgen als de TAX equivalent dan is de LBXEnum niet nodig

Klasse kan best hetzelfde overervings traject volgen als de TAX equivalent dan is de LBXEnum niet nodig

privatiseren ( maar zal uiteindelijk weg mogen ) daar de eerste versie van deze test wel duidelijker was

privatiseren ( maar zal uiteindelijk weg mogen ) daar de eerste versie van deze test wel duidelijker was

Eerste versie van de test leek mij wel iets duidelijker .. ik zou er wel een extra arrange-assert in zetten do $$$AssertTrue(LBXKenmerken.NietMeeleveren.Find(Rol)) ; hiermee verzeker je dat de rol...

Eerste versie van de test leek mij wel iets duidelijker .. ik zou er wel een extra arrange-assert in zetten

do $$$AssertTrue(LBXKenmerken.NietMeeleveren.Find(Rol)) ; hiermee verzeker je dat de rol er initieel in zat.

Do Actie

do $$$AssertFalse(LBXKenmerken.NietMeeleveren.Find(Rol)) ; hiermee verzeker je dat de rol er uit is.

??????

??????

[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop- afdekkaprugwand
[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop- afdekkaprugwand
[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop

- Dezelfde rol gebruiken.

[DEF563] Niet meeleveren afdekkapRugwandhouder voor LBX verwijderen van webshop

- afdekkaprugwandhouder niet tonen op webshop. (niet meeleveren.)

    • -0
    • +24
    ./KenmerkenAanpasser/LBX.cls.xml
[UST1942] EDI bestellingen met enkel voorgedefinieerde producten

- ervoor zorgen dat de productlijnen in een aparte groep komen te staan (gebeurt nu in de ProductLijnBuilder.MaakLijn...)

  1. … 9 more files in changeset.
[UST1437] WinkelkarService apart voor Web en VHintra:

- WinkekarServiceUtils -> KenmerkAanpasser als kenmerken visitor

    • -0
    • +36
    ./KenmerkenAanpasser.cls.xml
    • -0
    • +52
    ./KenmerkenAanpasser/TAX.cls.xml
  1. … 10 more files in changeset.