MaatwerkLijnBuilder.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Bedoeling is dat de project setting terug verdwijnt. Heeft dynamisch maken dan zin?

Bedoeling is dat de project setting terug verdwijnt. Heeft dynamisch maken dan zin?

Deze tests met (te)véél opzet zijn een reden temeer om de ZetDoorHandelNietMeeleverens af te zonderen

Deze tests met (te)véél opzet zijn een reden temeer om de ZetDoorHandelNietMeeleverens af te zonderen

ProjectSettingsApi newen

ProjectSettingsApi newen

ProjectSettingsApi newen

ProjectSettingsApi newen

om mee te nemen in de refactor http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif : set --> Set #Dim --> #dim if ---> If

om mee te nemen in de refactor :
set --> Set
#Dim --> #dim
if ---> If

Api *het ophalen van een projectsetting hoeft niet van de domeincontext
  • Api
  • het ophalen van een projectsetting hoeft niet van de domeincontext
Rekent deze klasse op de domeincontext?

Rekent deze klasse op de domeincontext?

Is het niet logischer/performanter dat de 2 bovenstaande lijnen gewisseld worden?

Is het niet logischer/performanter dat de 2 bovenstaande lijnen gewisseld worden?

Met een kleine moeite kan dit dynamisch gemaakt worden zodat slechts één setting nodig is voor eventueel toekomstige personalities met hetzelfde gedrag.

Met een kleine moeite kan dit dynamisch gemaakt worden zodat slechts één setting nodig is voor eventueel toekomstige personalities met hetzelfde gedrag.

Naamgeving bij nieuwe code: ...Api

Naamgeving bij nieuwe code: ...Api

voorstel: de ZetDoorHandelNietMeeleverens afzonderen naar een aparte klasse, en voor elke variant een afgeleide maken. Dan kun je gebruik maken van de autocomplete-enums. Do ..ZetDoorHandelNietMeel...

voorstel: de ZetDoorHandelNietMeeleverens afzonderen naar een aparte klasse, en voor elke variant een afgeleide maken.
Dan kun je gebruik maken van de autocomplete-enums.

Do ..ZetDoorHandelNietMeeleverensVoorRol(Lijn, HalffabItemsBoom, ..Enum.Rol.TiponBlumotionSynchronisatieStang)


Of minder werk: in elke method de juiste autocomplete-enum newen

#dim Rol As DOM.PM.Maatwerk.Calc.HF.enu.LBXRol.AutoComplete = ##class(DOM.PM.Maatwerk.Calc.HF.enu.LBXRol.AutoComplete).%New()
[BUS734] EC: MP: Daro: niet meeleveren aanvinken bij doorbestellen, en lijstje componenten mailen...
[BUS734] EC: MP: Daro: niet meeleveren aanvinken bij doorbestellen, en lijstje componenten mailen...
[BUS734] EC: MP: Daro: niet meeleveren aanvinken bij doorbestellen, en lijstje componenten mailen (zoals bij Beslag Design)

- nietmeeleverens opzetten voor Daro uitschakelen ondervoorwaarde van projectsetting

- constructors mocks

  1. … 3 more files in changeset.
[DEF681] VH4: BUDI: HerberekenMaatwerk =} Mail offerte overzicht: oude offertes blijven dagelijks...
[DEF681] VH4: BUDI: HerberekenMaatwerk =} Mail offerte overzicht: oude offertes blijven dagelijks...
Bij de verwachtmethodcall op de WinkelkarServiceMock heb je overal een lege string meegegeven wat op zich niet fout is aangezien deze een default waarde heeft. Andere parameters die voor deze metho...

Bij de verwachtmethodcall op de WinkelkarServiceMock heb je overal een lege string meegegeven wat op zich niet fout is aangezien deze een default waarde heeft. Andere parameters die voor deze method ook optioneel zijn, worden ingevuld met een random waarde. Mss kan je naar analogie ook hier een RandomIsGeldig meegeven.

Idem opmerking hierboven

Idem opmerking hierboven

Bij voorkeur #dim (mét type), Set wordt ook toegelaten, een tussenvorm zou ik vermijden.

Bij voorkeur #dim (mét type), Set wordt ook toegelaten, een tussenvorm zou ik vermijden.

[DEF681] VH4: BUDI: HerberekenMaatwerk =} Mail offerte overzicht: oude offertes blijven dagelijks vermeld (door herberekenmaatwerk probleem met ongeldige lades)

- Mock en de Fake van de DOM.EC maatwerklijnbuilder ook aangepast

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