Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-4466] Revego Vijsjes work around

  1. … 2 more files in changeset.
[ICT-4439] [rvPVR] Ophalen BOMBOL uit Encoway:

- Verplaatsen VHConfig van DOM naar APPS laag

  1. … 25 more files in changeset.
[ICT-4361] MP Haefele Spanje opzetten
[ICT-4361] MP Haefele Spanje opzetten
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

[ICT-3983] OrderSync in cache uitbreiden voor encowayconfiguraties

- Zoeken naar dossiercode is ook maatwerktype gebonden uiteraard

  1. … 2 more files in changeset.
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

Idd, is ook niet belangrijk voor het valideren of het uniek is

Idd, is ook niet belangrijk voor het valideren of het uniek is

Ik zou deze in een config item plaatsen

Ik zou deze in een config item plaatsen

Er zijn precies geen unittesten, ik zou deze precies toch voorzien http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Er zijn precies geen unittesten, ik zou deze precies toch voorzien

De werkpost wordt hier niet ingevuld?

De werkpost wordt hier niet ingevuld?

Hier zit nog een wlipke http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Hier zit nog een wlipke

Ik dacht eerst dat dit niet nodig was, maar heb het achter aan configsettings gestoken http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif.

Ik dacht eerst dat dit niet nodig was, maar heb het achter aan configsettings gestoken .

[ICT-4302] Wijzigingen Caché en VHintra TST - Lijn 5:
[ICT-4302] Wijzigingen Caché en VHintra TST - Lijn 5:
[ICT-4302] Wijzigingen Caché en VHintra TST - Lijn 5:

- Implementatie voor lijn 5 achter config setting plaatsen

    • -0
    • +6
    ./DOM/common/ProjectSettingsAPI.cls.xml
  1. … 4 more files in changeset.
[ICT-3715] VHLOG MES M1 - Orgalux-code van OBJT naar Cache (via NodeRed)
[ICT-3715] VHLOG MES M1 - Orgalux-code van OBJT naar Cache (via NodeRed)
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)