DataMBouwsteenCreator.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Ik had deze in de klassenaam zelf misschien ook "converter" bijgezet. Nu is dit HalffabItemsboom, maar is op die manier niet echt duidelijk dat het om een converter gaat zonder naar de folder te ki...

Ik had deze in de klassenaam zelf misschien ook "converter" bijgezet. Nu is dit HalffabItemsboom, maar is op die manier niet echt duidelijk dat het om een converter gaat zonder naar de folder te kijken (zelfde voor LBXKenmerken)

Zijn deze lijnen in commentaar nog relevant?

Zijn deze lijnen in commentaar nog relevant?

Samengesteld ipv SamenGesteld

Samengesteld ipv SamenGesteld

'Dotnet' -> Encoway

'Dotnet' -> Encoway

Rol doet niets, mag weg

Rol doet niets, mag weg

ProductApi is nergens gedefinieerd als input parameter, dus deze inject is niet nodig

ProductApi is nergens gedefinieerd als input parameter, dus deze inject is niet nodig

Ik zou dit niet als constructor porperty meegeven, want nu moet je overal waar je de VHConfigHelper wilt gebruiken, eerst hem initialiseren in de code zelf. Op die manier kan je moeilijker de vhcon...

Ik zou dit niet als constructor porperty meegeven, want nu moet je overal waar je de VHConfigHelper wilt gebruiken, eerst hem initialiseren in de code zelf. Op die manier kan je moeilijker de vhconfighelper injecteren.

Er is wel de method 'ZetVhConfig', maar die moet je dan eerst aanroepen vooraleer je de andere methods kan gebruiken. Ergens lijkt mij dat wat zot, omdat je dan snel fouten kan maken

Ik zou deze in een config item plaatsen

Ik zou deze in een config item plaatsen

In enu steken en hier ook naar verwijzen?

In enu steken en hier ook naar verwijzen?

Mijn gevoel is om dit ook in een aparte klasse te plaatsen en onder test te steken (ook al zal deze klasse dan vrij klein zijn)

Mijn gevoel is om dit ook in een aparte klasse te plaatsen en onder test te steken (ook al zal deze klasse dan vrij klein zijn)

Talen kunnen eventueel in een aparte enu?

Talen kunnen eventueel in een aparte enu?

Variabele naam aanpassen naar Encoway ipv dotnet

Variabele naam aanpassen naar Encoway ipv dotnet

Variabele naam aanpassen naar Encoway ipv dotnet

Variabele naam aanpassen naar Encoway ipv dotnet

[ICT-3982] Productcreatie in cache voor encowayconfiguraties
[ICT-3982] Productcreatie in cache voor encowayconfiguraties
Deze moet je ook met "BSS" of ..BSNode voorzien. Dit is het antwoord op de vraag in je mail http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Deze moet je ook met "BSS" of ..BSNode voorzien. Dit is het antwoord op de vraag in je mail

GRRR !! cosmetica (hoofdletters en spatieringen, etc.) apart committen aub, zo kan de reviewer die revision skippen.

GRRR !!
cosmetica (hoofdletters en spatieringen, etc.) apart committen aub, zo kan de reviewer die revision skippen.

Ik zou hier ook de property "BSNode" toevoegen met waarde "BS" of "BSS" Dan kan je hieronder heel wat extra "If schaduw"-constructies, met duplicatie van code, wegwerken.

Ik zou hier ook de property "BSNode" toevoegen met waarde "BS" of "BSS"
Dan kan je hieronder heel wat extra "If schaduw"-constructies, met duplicatie van code, wegwerken.

Opmerking 2 (zonder extra werk) : Ik merk nu pas dat we niet de BekereningSpecificatie doorgeven naar .NET, maar enkel KlantID en Aantal, wat de eerste 2 parameters van de constructor ervan zijn. V...

Opmerking 2 (zonder extra werk) :
Ik merk nu pas dat we niet de BekereningSpecificatie doorgeven naar .NET, maar enkel KlantID en Aantal, wat de eerste 2 parameters van de constructor ervan zijn.
Voorlopig is dit voldoende, maar later zal dit toch nog moeten uitgebreid worden (o.a. BerekeningWijze, BerekenDatum, BerekenTag). Dan zullen we moeten beslissen of we de parameterkes toevoegen, of toch omschakelen naar het object BerekeningSpecificatie.

Effe opmerken dat uw voorgangers hier gezondigd hebben tegen de VH-spellingsconventies! "BerekeningSpecificatie" moet normaal zonder tussen-"s" zijn http://subversion02.vanhoecke.be/static/ogdo0b/2...

Effe opmerken dat uw voorgangers hier gezondigd hebben tegen de VH-spellingsconventies!
"BerekeningSpecificatie" moet normaal zonder tussen-"s" zijn
Tis echter veel te veel werk om dit nu nog te gaan aanpassen, dus laat maar zo ... "even slikken en doorgaan" :-P

[ICT-2359] TBX BOMBOL builders porten naar .NET
[ICT-2359] TBX BOMBOL builders porten naar .NET
Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronde...

Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronder verifiëren?

overbodige comment?

overbodige comment?

[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten
[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten