Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5256][Caché] Mapping verzendwijze 'Opo' bij syncen van klant OPO Duitsland
[ICT-5256][Caché] Mapping verzendwijze 'Opo' bij syncen van klant OPO Duitsland
[ICT-4716] [rvJBA] fout bij afdrukken etiketten met "
[ICT-4716] [rvJBA] fout bij afdrukken etiketten met "
Ergens zou deze method naam moeten duidelijk maken dat het originele product verwijderd wordt lijkt mij?

Ergens zou deze method naam moeten duidelijk maken dat het originele product verwijderd wordt lijkt mij?

Is dit niet 'gevaarlijk'? Moet je niet checken of er 1 product aanwezig is en anders een error gooien?

Is dit niet 'gevaarlijk'? Moet je niet checken of er 1 product aanwezig is en anders een error gooien?

Is écht muggenziften, maar deze 'if' mag op lijn 566. Ik durf het bijna niet typen :-p

Is écht muggenziften, maar deze 'if' mag op lijn 566. Ik durf het bijna niet typen :-p

[ICT-3983] OrderSync in cache uitbreiden voor encowayconfiguraties
[ICT-3983] OrderSync in cache uitbreiden voor encowayconfiguraties
Je doet een vergelijking van de verschillen, dus je wilt dat er geen verschillen zijn, daarom empty list

Je doet een vergelijking van de verschillen, dus je wilt dat er geen verschillen zijn, daarom empty list

Vanwaar assert empty list?

Vanwaar assert empty list?

wlips mogen weg of het zal trakteren worden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

wlips mogen weg of het zal trakteren worden

[ICT-2932] INK: Productcreatie INK
[ICT-2932] INK: Productcreatie INK
Displaylist en valuelist zijn uitgebreid.

Displaylist en valuelist zijn uitgebreid.

aan de valuelist (en hieronder bij de displaylist, ook nog het item toevoegen) (en hier ook op de spaties letten)

aan de valuelist (en hieronder bij de displaylist, ook nog het item toevoegen)

(en hier ook op de spaties letten)

[ICT-2962] MaakVerzendWijze Gefaald: Nieuwe Verzendwijze nodig in Cache:
[ICT-2962] MaakVerzendWijze Gefaald: Nieuwe Verzendwijze nodig in Cache:
Is inderdaad helemaal niet mijn keuze(KPR) momenteel wordt de extra enum enkel gebruikt om zo snel mogelijk te vertalen naar een 'klassieke' verzendwijze. Als er natuurlijk ergens stukken code zij...

Is inderdaad helemaal niet mijn keuze(KPR)

momenteel wordt de extra enum enkel gebruikt om zo snel mogelijk te vertalen naar een 'klassieke' verzendwijze. Als er natuurlijk ergens stukken code zijn die die valuelist itereren en er iets zots mee doen... maar op het eerste zicht is dat niet direct een risico. Valuelist wordt in een paar AdresUtils methods en nog wat andere dingen gebruikt.

Het is misschien niet uw eigen keuze, maar "DiHF" bevat kleine letter i. Persoonlijk zou ik kiezen om alles in hoofdletters te zetten. Maar wellicht is dat nu al te laat. Tweede bedenking : is het...

Het is misschien niet uw eigen keuze, maar "DiHF" bevat kleine letter i.
Persoonlijk zou ik kiezen om alles in hoofdletters te zetten. Maar wellicht is dat nu al te laat.

Tweede bedenking : is het overal oké om een code van 4 letters te nemen? Dit is de eerste in de lijst, zie ik

[ICT-2806] Interface klanten - verzendwijze DiHF
[ICT-2806] Interface klanten - verzendwijze DiHF
De vhUtils standaard-kenmerken kiezen is soms nogal tricky: *liefst zo standaard mogelijk (behalve bij speciallekes zoals LadeMetVeelExtras) *liefst zo ondubbelzinnig mogelijk : bvb. X en Y versc...

De vhUtils standaard-kenmerken kiezen is soms nogal tricky:

  • liefst zo standaard mogelijk (behalve bij speciallekes zoals LadeMetVeelExtras)
  • liefst zo ondubbelzinnig mogelijk : bvb. X en Y verschillend bij sifonlade --> check.
  • in de meeste gevallen zo werkelijkheidsgetrouw mogelijk


in deze method vind ik enige verwarring :

  • StandaardMat heeft LadeType=TAX, die heeft (nog) geen spoelbak.
  • Spoelbak voor LBX? of eerder voor TBX/MVX? --> wellicht zijn beide methods zinvol, en dan zou ik één method expliciet MatVoorLbxSpoelbakLade() noemen.
  • bij deze LBX moeten de uitsparingBreedtes X en Y dan best wel de juiste afmeting bevatten, want dat is ook een "vaste maat"


In eerdere gesprekken had ik al wel aangegeven dat een goede keuze van kenmerkwaarden, niet zo vanzelfsprekend is, en dat ik je hierbij zeker wel zou bijstaan. Bij deze, call me