Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
VRB583 voor aangemaakt.

VRB583 voor aangemaakt.

LergacyPartijApi gebruiken

LergacyPartijApi gebruiken

2 opmerkingskes: *geen testjes voor deze method *deze werking laat gaten toe, als ik het goe gezien heb -> problemen voor HoutenLaden (met veeeeeeel leveradressen?)

2 opmerkingskes:

  • geen testjes voor deze method
  • deze werking laat gaten toe, als ik het goe gezien heb -> problemen voor HoutenLaden (met veeeeeeel leveradressen?)
[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:
[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:
ik zou nog kunnen beginnen neuten over de casing van adres/Adres. Het probleem is dat het in feite mer kleine a moet, want het is een repo van Leveradressen. Het is wel zo dat historisch gezien er ...

ik zou nog kunnen beginnen neuten over de casing van adres/Adres. Het probleem is dat het in feite mer kleine a moet, want het is een repo van Leveradressen. Het is wel zo dat historisch gezien er altijd (foutief) met een grote A geschreven werd (zie andere lijntjes in deze klasse). Doe er mee wat ge wilt

Eigenlijk hoort dit niet in de repository. Een repository moet enkel bewaren, geven zoeken enz en krijgt/geeft een object van het type waarover de repository het heeft. Dit hoort eerder in een Leve...

Eigenlijk hoort dit niet in de repository. Een repository moet enkel bewaren, geven zoeken enz en krijgt/geeft een object van het type waarover de repository het heeft.
Dit hoort eerder in een LeveradresService. Pas op, ik wil hier niet beweren dat dat momenteel altijd en overal zo te goei gedaan wordt!

[UST4044] VHIP900 - KlantSync: Opkuisen juiste leveradres igv dubbele aanwezigheid AxLocationId:
[UST4044] VHIP900 - KlantSync: Opkuisen juiste leveradres igv dubbele aanwezigheid AxLocationId:
Primary ipv Primory (ook hieronder en hierboven)

Primary ipv Primory (ook hieronder en hierboven)

[UST4026] VHIP900 - Business & primary - 1 wijziging terugdraaien:
[UST4026] VHIP900 - Business & primary - 1 wijziging terugdraaien:
[UST3980] VHIP900 SyncKlant - Business & Primary bij adressen:
[UST3980] VHIP900 SyncKlant - Business & Primary bij adressen:
Hier zou ik denken, leeg adres mee te geven in de result en IsAxHoofdadresmeegegeven te zetten adhv isobject(leveradres), dan hoef je de boolean niet mee te ge ven byref.

Hier zou ik denken, leeg adres mee te geven in de result en IsAxHoofdadresmeegegeven te zetten adhv isobject(leveradres), dan hoef je de boolean niet mee te ge ven byref.

Volgens mij heb je hier moeilijk gemaakt door de byref parameter mee te geven. Enerzijds kan je een klasse maken met twee properties die zowel leveradres als de boolean IsAxHoofdadres als returnva...

Volgens mij heb je hier moeilijk gemaakt door de byref parameter mee te geven.

Enerzijds kan je een klasse maken met twee properties die zowel leveradres als de boolean IsAxHoofdadres als returnvalue geeft.

Beter nog is de 2 dingen die je doet in 1 method opsplitsen in 2 methods. nl bepalen of er een hoofdadres meegegeven is en het zetten ervan in een andere method.

[UST3689] VHIP900: Sync klant: onderzoek landcode Oostenrijk AT of AU:
[UST3689] VHIP900: Sync klant: onderzoek landcode Oostenrijk AT of AU:
Aangezien de oproepende functie het doorgeeft via SQL is dat nog niet echt nodig. (en code moest rap werkend zijn http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/...

Aangezien de oproepende functie het doorgeeft via SQL is dat nog niet echt nodig. (en code moest rap werkend zijn )

Zal het er wel bijsteken voor de volledigheid

Moet er geen + voor OrderHeader.CustAccount staan, zodat de voorloopnul die klantnummers in AX eventueel hebben eraf gehaald wordt?

Moet er geen + voor OrderHeader.CustAccount staan, zodat de voorloopnul die klantnummers in AX eventueel hebben eraf gehaald wordt?

Moet deze niet manueel verwijderd worden?

Moet deze niet manueel verwijderd worden?

Moet deze niet manueel verwijderd worden?

Moet deze niet manueel verwijderd worden?

KlantId of KlantNummer?

KlantId of KlantNummer?