Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-4439] [rvPVR] Ophalen BOMBOL uit Encoway:

- Verplaatsen VHConfig van DOM naar APPS laag

    • -32
    • +0
    ./Maatwerk/Calc/Common/VhConfig/KostItem.cls.xml
    • -4
    • +6
    ./impl/DataM/DataMProductCreator.cls.xml
    • -17
    • +0
    ./Maatwerk/Calc/Common/VhConfig/Kost.cls.xml
    • -166
    • +0
    ./Maatwerk/Calc/Common/VhConfigHelper.cls.xml
    • -43
    • +0
    ./Maatwerk/Calc/Common/VhConfig.cls.xml
  1. … 16 more files in changeset.
Correct! Heb ik aangepast

Correct! Heb ik aangepast

Vraagje: als dwarsverdelingtype = dubbel, wordt de UitvoeringInfo die in den if gezet is geweest dan niet direct overschreven door deze regel?

Vraagje: als dwarsverdelingtype = dubbel, wordt de UitvoeringInfo die in den if gezet is geweest dan niet direct overschreven door deze regel?

Zijn er geen integratietesten van dat document?

Zijn er geen integratietesten van dat document?

Klopt het dat deze nu niet meer gefilterd worden op productieroute?

Klopt het dat deze nu niet meer gefilterd worden op productieroute?

[ICT-4439] [rvPVR] Ophalen BOMBOL uit Encoway:

- Dto's van maken in plaats van registred objects

    • -1
    • +1
    ./Maatwerk/Calc/Common/VhConfig.cls.xml
[ICT-4380] [rvJLM] [PAS] Dwarsverdeling en dwarsverdelinghouder - dubbele picking indicatie
    • -4
    • +4
    ./Maatwerk/Filter/enu/FilterType.cls.xml
[ICT-4380] [rvJLM] [PAS] Dwarsverdeling en dwarsverdelinghouder - dubbele picking indicatie
[ICT-4380] [rvJLM] [PAS] Dwarsverdeling en dwarsverdelinghouder - dubbele picking indicatie
[ICT-4380] [rvJLM] [PAS] Dwarsverdeling en dwarsverdelinghouder - dubbele picking indicatie

- kolom type uitbreiden zodat langere tekst erin zou passen

- nieuwe filter voorzien voor dwarsverdeling van het onderdelenpickingblad te halen indien nodig

- Zijwandtype hardgecodeerd op Pure gezien taor geen zijwandtypekenmerk heeft

    • -2
    • +9
    ./Maatwerk/Filter/enu/FilterType.cls.xml
    • -0
    • +62
    ./Maatwerk/Filter/impl/GeenFlatpackVerpakkingFilter.cls.xml
  1. … 4 more files in changeset.
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

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

Ik zou deze in een config item plaatsen

Ik zou deze in een config item plaatsen

[ICT-4403] [rvPVR] VHLOG MES M1 - Nieuwe orderoptie 'Fysiekeladehoogteinmm':
[ICT-4403] [rvPVR] VHLOG MES M1 - Nieuwe orderoptie 'Fysiekeladehoogteinmm':
[ICT-4403] [rvPVR] VHLOG MES M1 - Nieuwe orderoptie 'Fysiekeladehoogteinmm':

- Uitbreiden met 'X38'

[ICT-4403] [rvPVR] VHLOG MES M1 - Nieuwe orderoptie 'Fysiekeladehoogteinmm':

- TBX Ladeutils klasse toevoegen

    • -0
    • +66
    ./Maatwerk/Lade/LadeUtils/impl/TBXLadeUtils.cls.xml
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