Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-4503] [rvJBA] Detailpapier afdekkappen zonder logo opdruk "zonder logo"

- Uitbreiden enumeratieklasse

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

[ICT-4439] [rvPVR] Ophalen BOMBOL uit Encoway:
[ICT-4439] [rvPVR] Ophalen BOMBOL uit Encoway:
[ICT-4303] RVG: Aanpassingen Blum release 1/2/2024
[ICT-4303] RVG: Aanpassingen Blum release 1/2/2024
[ICT-4439] [rvPVR] Ophalen BOMBOL uit Encoway:

- Klasses verwijderen uit DOM laag naar verplaatsen naar APPS laag (blijkbaar kwamen deze niet in de svn changes naar boven)

[ICT-4380] [rvJLM] [PAS] Dwarsverdeling en dwarsverdelinghouder - dubbele picking indicatie

- Nieuwe code terugdraaien want het is properder als de productieroute in .NET wordt aangepast.

- Correctie op dwarsverdeling inkorten document

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