Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Ik zou hier precies ook nog een test schrijven met waarde true voor IsSalesMedewerker. Kwestie van zeker te zijn dat de waarde niet verloren gaat na encrypt/decrypt.

Ik zou hier precies ook nog een test schrijven met waarde true voor IsSalesMedewerker.
Kwestie van zeker te zijn dat de waarde niet verloren gaat na encrypt/decrypt.

[ICT-5110] DATAOPT uitbreiden met extra parameter voor Häfele
[ICT-5110] DATAOPT uitbreiden met extra parameter voor Häfele
Mijn eerste idee was om $Number(Korting) te gebruiken, maar dit werkt niet omdat kortingen "?" en "" ook toegelaten moeten zijn. Daarom heb ik heb ervoor gekozen om de logica lichtjes te veranderen...

Mijn eerste idee was om $Number(Korting) te gebruiken, maar dit werkt niet omdat kortingen "?" en "" ook toegelaten moeten zijn.
Daarom heb ik heb ervoor gekozen om de logica lichtjes te veranderen, dit leek ook de meest cleane oplossing te zijn.

In plaats van waardes 0 en 1-100 toe te laten, laten we nu waardes 0-100 toe (dus ook een halve procent korting bijvoorbeeld). Dit omdat:

  • Niemand ooit een fractie van een procent korting gaat geven
  • Dit volgens Dries ook gewoon zo hoort te zijn
[ICT-4531] MP kortingen: waarde '0.0' doorgeven geeft error
[ICT-4531] MP kortingen: waarde '0.0' doorgeven geeft error
[ICT-3740] RVG: Data OPT extra parameter
[ICT-3740] RVG: Data OPT extra parameter
Misschien best wel, ja: deze worden enkel gebruikt voor kortingen met DataOpt (die kijkt naar de toegelaten ladetypes). Door ze erbij te zetten weet de dataopt dat OL een ding is, maar hoeft er nog...

Misschien best wel, ja: deze worden enkel gebruikt voor kortingen met DataOpt (die kijkt naar de toegelaten ladetypes). Door ze erbij te zetten weet de dataopt dat OL een ding is, maar hoeft er nog niet mee om te gaan. Er is ook al iets in productie dat deze OL gewoon kan negeren indien nodig.
Plus, op CacheAccept staat het al zoals het hier is (openstaande wijziging, mag ik niet vergeten als ik het merge/commit!)

hier juist de bedenking of het klopt dat al die "OL" lijnen hier al mee moeten of niet?

hier juist de bedenking of het klopt dat al die "OL" lijnen hier al mee moeten of niet?

[ICT-2976] MVX Kortingen Instellen
[ICT-2976] MVX Kortingen Instellen
Unit test toegevoegd die gedrag voor niet-Haefele personality test.

Unit test toegevoegd die gedrag voor niet-Haefele personality test.

Deze moet dan False teruggeven. Khad inderdaad wel wat moeite met een goeie naam te kiezen... en deze lijkt nogal afhankelijk van hoe ge het leest. Het idee was: "Niet-BE Haefele Personality", dus ...

Deze moet dan False teruggeven. Khad inderdaad wel wat moeite met een goeie naam te kiezen... en deze lijkt nogal afhankelijk van hoe ge het leest.
Het idee was: "Niet-BE Haefele Personality", dus een Haefele personality die niet HaefeleBE is. Ander idee is bijvoorbeeld "IsHaefelePersonalityMaarNietHaefeleBE" ofzo?

Nuja, het is volledig afhankelijk van hoe verschillende bedrijven die data doorsturen. Als er bvb een nieuwe dat exact zoals Haefele wilt doen dan moet die hier ook toegevoegd worden en dus moet de naam veranderd worden.

zie ook opmerking in kortingservicehelper zelf: ge gaat hier moeten duidelijk maken (met een of meerdere extra testjes) wat het gedrag gaat zijn als er met een willekeurige personality deze method ...

zie ook opmerking in kortingservicehelper zelf: ge gaat hier moeten duidelijk maken (met een of meerdere extra testjes) wat het gedrag gaat zijn als er met een willekeurige personality deze method wordt opgeroepen (leeg, error, defaultgedrag, ....)

wat moet deze doen als bv SFS binnenkomt (of een andere personality die niets met Häfele te maken heeft?

wat moet deze doen als bv SFS binnenkomt (of een andere personality die niets met Häfele te maken heeft?

[ICT-2786] Aparte korting voor Orgalux bij multipersonalities aanmaken:
[ICT-2786] Aparte korting voor Orgalux bij multipersonalities aanmaken:
pietluttig detail, maar hier is een "Do" in principe voldoende. (ook bij de tests hieronder)

pietluttig detail, maar hier is een "Do" in principe voldoende. (ook bij de tests hieronder)

ICT-1473: SFS kortingen meegeven via DataOpt
ICT-1473: SFS kortingen meegeven via DataOpt
[ICT-1082] [rvTVE] EC: VH4: MP: Hahle multipersonality (Finland)
[ICT-1082] [rvTVE] EC: VH4: MP: Hahle multipersonality (Finland)