Activity CacheAdminA_trunk2010/APPS/VKP/Offerte

12:12 pm

Jo Willems

More
69152
changed 20 files
root:

[ICT-2320] MP Martens opzetten

- decrypteer call werkte niet

- volgende reeks visitor die van dom.common.personalityunishopvisitor afhangen

- -0
- +6
./impl/BesteldeOfferteMelder/MailVerzender/Handel.cls.xml
- History
- Source
- Diff
./impl/.../Handel.cls.xml
… 19 more files in changeset.

5:24 pm

Jo Willems

More
69131
changed 24 files
root:

[ICT-2320] MP Martens opzetten

- decrypteer call werkte niet

- Personality visitor aangepast en alle daarvan ervende visitors

- -0
- +6
./impl/BesteldeOfferteMelder.cls.xml
- History
- Source
- Diff
./impl/BesteldeOfferteMelder.cls.xml
… 23 more files in changeset.

9:26 am

Jo Willems

More
68424
changed 2 files
root:

[ICT-3026] Tijdelijke Webservice voor nieuwe SFS httpost

- eigenlijke class oproepen

- evt fout info naar AX teruggeven.

- -1
- +5
./impl/BesteldeOfferteMelder/HttpVerzender/SFS.cls.xml
- History
- Source
- Diff
./impl/BesteldeOfferteMelder/.../SFS.cls.xml
… 1 more file in changeset.

3:25 pm

Jo Willems

More
68420
changed 5 files
root:

[ICT-3026] Tijdelijke Webservice voor nieuwe SFS httpost

- nieuwe webservice + wsdl controle

- implementatie van versturen bestand op de SFS implementatie van de HttpVerzender/SFS.cls.xml

- override van de generieke verzendhttpmessage omdat we geen offerte etc. hebben, waarnaar gerefereerd wordt

- -0
- +48
./impl/BesteldeOfferteMelder/HttpVerzender/SFS.cls.xml
- History
- Source
- Diff
./impl/BesteldeOfferteMelder/.../SFS.cls.xml
… 4 more files in changeset.

11:50 am

Wim Vermeulen

More
CR-2419
started review

[ICT-2813] PM: Kostenplaatsen invullen : Schaduw voor grote prijswijziging op 01/05/2022

4:44 pm

Wim Vermeulen

More
67249
changed 1 file
root: CR-2419

[ICT] KlantBestellingAanLeverancierOfferteKoppelaar signature is zeer misleidend

- de twee function parameters van method KoppelAanOfferteVanHandel() omgewisseld (vogire commit), want nu zijn de benamingen wel conform met de context. Dit kan je een half uur aan troubleshooting besparen :-P

- comment in een oude impl gezet. Dus niet de code aangepast.

- -1
- +1
./impl/KlantBestellingNaarLeverancierOfferteConverterVhisie3.cls.xml
- History
- Source
- Diff
./impl/KlantBestellingNaarLeverancierOfferteConverterVhisie3.cls.xml

4:43 pm

Wim Vermeulen

More
67248
changed 3 files
root: CR-2419

[ICT] KlantBestellingAanLeverancierOfferteKoppelaar signature is zeer misleidend

- de twee function parameters van method KoppelAanOfferteVanHandel() omgewisseld, want nu zijn de benamingen wel conform met de context. Dit kan je een half uur aan troubleshooting besparen :-P

- Mock-klasse, interface mee aangepast.

- -1
- +1
./KlantBestellingAanLeverancierOfferteKoppelaar.cls.xml
- History
- Source
- Diff
./KlantBestellingAanLeverancierOfferteKoppelaar.cls.xml
- -3
- +5
./impl/KlantBestellingAanLeverancierOfferteKoppelaar.cls.xml
- History
- Source
- Diff
./impl/KlantBestellingAanLeverancierOfferteKoppelaar.cls.xml
… 1 more file in changeset.

12:21 pm

Joannes Lemmens

More
67229
changed 67 files
root:

[ICT-2829] Haefele NO multipersonality:

- Overige bestanden gewijzigd (zoals in kaartje ICT-2223)

- -0
- +8
./impl/BesteldeOfferteMelder.cls.xml
- History
- Source
- Diff
./impl/BesteldeOfferteMelder.cls.xml
- -1
- +1
./impl/OfferteKopieerder.cls.xml
- History
- Source
- Diff
./impl/OfferteKopieerder.cls.xml
… 65 more files in changeset.

8:27 am

Joannes Lemmens

More
67227
changed 29 files
root:

[ICT-2829] Haefele NO multipersonality:

- Initiële commit (Personality toegevoegd en alles errond dat het doet werken)

- -1
- +2
./impl/BesteldeOfferteMelder/HttpVerzender/dto/OfferteGenerator.cls.xml
- History
- Source
- Diff
./impl/.../OfferteGenerator.cls.xml
… 28 more files in changeset.

5:23 pm

Jo Willems

More
66769
changed 1 file
root:

[ICT-2704] Noorse vertalingen in httppost met taalcode 'no' ipv 'nb'

- Noor effectief ook toevoegen aan de omschrijvingen van maatwerk en producten.

- -0
- +2
./impl/BesteldeOfferteMelder/HttpVerzender/dto/OfferteGenerator.cls.xml
- History
- Source
- Diff
./impl/.../OfferteGenerator.cls.xml

1:29 pm

Peter Vercoutter

More
CR-2397
Closed

12:49 pm

Wim Vermeulen

More
CR-2397
finished reviewing

12:49 pm

Wim Vermeulen

More
Re: CR-2397
commented on review

FYI : Bij een volgende uitbreiding van deze If-else, zullen we best de impl afzonderen in aparte method (of klasse). Bvb. wanneer Revego (of eenvoudig maatwerk) erbij komt, kan de eerste if zijn : ...

FYI :
Bij een volgende uitbreiding van deze If-else, zullen we best de impl afzonderen in aparte method (of klasse).
Bvb. wanneer Revego (of eenvoudig maatwerk) erbij komt, kan de eerste if zijn :

if Not(GeefFamilie() = "Lade") then ... else ...

Of als je dat nu reeds wil doen, feel free

12:28 pm

Wim Vermeulen

More
Re: CR-2397
commented on review

FYI: We hadden samen bekeken, in welke klasse we Mat-materiaal best zouden toevoegen. Het is nu pas, tijdens review, dat me opvalt dat deze klasse (met "...Omschrijving" in de naam) NIET taalafhank...

FYI:
We hadden samen bekeken, in welke klasse we Mat-materiaal best zouden toevoegen.
Het is nu pas, tijdens review, dat me opvalt dat deze klasse (met "...Omschrijving" in de naam) NIET taalafhankelijk is, maar dat is dus eerder toevallig, omdat LBX, TAX en TBX in alle talen hetzelfde is.

Het had zeker niet verkeerd geweest om deze klasse verder uit te breiden met vertaling en MatMateriaal. My bad!
Voorlopig zullen we het hierbij laten. De huidige oplossing is zeker ook oké.

10:32 am