KenmerkNaarDtoVisitor.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1830] Convertoren aanmaken voor MVX
[ICT-1830] Convertoren aanmaken voor MVX
[VRB535] HX: Klantetiket: DKG lijnref in barcode (generische maatwerkaansturing)
[VRB535] HX: Klantetiket: DKG lijnref in barcode (generische maatwerkaansturing)
Deze code was dus niet correct...http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif is intussen gewijzigd. Later misschien nog te refactoren.

Deze code was dus niet correct... is intussen gewijzigd. Later misschien nog te refactoren.

ik ga ondanks de beslissing om hier niet aan verder te werken, toch een kaartje aanmaken.. ge weet maar nooit.

ik ga ondanks de beslissing om hier niet aan verder te werken, toch een kaartje aanmaken.. ge weet maar nooit.

  • More
  • CR-580
  • finished reviewing
LogActie lijkt mij toch niet echt correct te zijn, het hoort allemaal in de detail te staan. Actie lijkt mij eerder ControleKenmerken - Kwestie dat de logging beetje ordentelijk is

LogActie lijkt mij toch niet echt correct te zijn, het hoort allemaal in de detail te staan.

Actie lijkt mij eerder ControleKenmerken - Kwestie dat de logging beetje ordentelijk is

DOM logica

DOM logica

DOM logica

DOM logica

APPS logica

APPS logica

APPS logica

APPS logica

APPS logica

APPS logica

APPS logica

APPS logica

DOM Logica

DOM Logica

DOM Logica

DOM Logica

DOM Logica

DOM Logica

  • More
  • CR-594
  • finished reviewing
"Voor Wand ... " Het is jammer dat je de foute schrijfwijze hebt voortgezet. Ik weet dat je voorstander bent van consistente spelling, maar in dit geval (van nieuwe implementatie) had je gerust kun...

"Voor Wand ... "
Het is jammer dat je de foute schrijfwijze hebt voortgezet.
Ik weet dat je voorstander bent van consistente spelling, maar in dit geval (van nieuwe implementatie) had je gerust kunnen omschakelen.
Zeker als je weet dat een Klasse-rename bijna ONmogelijk is als het alleen om een hoofdletter gaat.
Met andere woorden, we zullen nooit meer van die "VoorWant" vanaf geraken

Anyway, het staat nu zo, dus laat maar staan. Geen onnodige tijd meer aan verliezen!
Verder is de volledige review OK, denk

[EPIC52][UST2878] Conversie VoorWandType (DOM <> APP):
[EPIC52][UST2878] Conversie VoorWandType (DOM <> APP):
[UST2867] Vh4_SVC: Snelheid sync winkelkar naar Caché controleren en tot aanvaardbaar punt brengen
[UST2867] Vh4_SVC: Snelheid sync winkelkar naar Caché controleren en tot aanvaardbaar punt brengen
Code is allemaal weg en daarom ook de testjes weggesmeten

Code is allemaal weg en daarom ook de testjes weggesmeten

allez hup, gaan we nu al commentaar geven op lijnen die er al sinds 12/12/2013 inzitten? tof, tof, ...

allez hup, gaan we nu al commentaar geven op lijnen die er al sinds 12/12/2013 inzitten? tof, tof, ...

  • More
  • CR-50
  • finished reviewing
Het lijkt mij duidelijker om zulke structuur op te zetten #dim TbxKenmerken As DOM... = ##class(vhTest.Utils...).StandaardLade() set TbxKenmerken.LadeKleur = "onbestaandeKleur" ; <----- deze lijn ...

Het lijkt mij duidelijker om zulke structuur op te zetten

#dim TbxKenmerken As DOM... = ##class(vhTest.Utils...).StandaardLade()
set TbxKenmerken.LadeKleur = "onbestaandeKleur" ; <----- deze lijn is niet echt noodzakelijk
set TbxKenmerken.LegacyKenmerken.ItemKB = "onbestaandekleur"