Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
3 keer is scheepsrecht http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

3 keer is scheepsrecht

twijfelgevalleke: zondert ge de gemeenschappelijke code af in private method of niet... Ja: want het is een beperkte exacte copy Nee: 't staat er maar twee keer... your call

twijfelgevalleke: zondert ge de gemeenschappelijke code af in private method of niet...
Ja: want het is een beperkte exacte copy
Nee: 't staat er maar twee keer...

your call

best geen vervolglijnen op dezelfde indent laten beginnen als de beginlijn (hier de if). volgens mij is het ook properder om zo'n kloeft in een private methodje af te zonderen. dat bevordert de lee...

best geen vervolglijnen op dezelfde indent laten beginnen als de beginlijn (hier de if).
volgens mij is het ook properder om zo'n kloeft in een private methodje af te zonderen. dat bevordert de leesbaarheid enorm

[UST3682] VHIP903: PickingCompleet: dubbels teveel verwijderd:
[UST3682] VHIP903: PickingCompleet: dubbels teveel verwijderd:
[UST3682] VHIP903: PickingCompleet: dubbels teveel verwijderd:

- InventoryOutConverter => Producten die een AxConfigId hebben worden niet meer tegengehouden als LinkType = S (=> geldt enkel indien geen AxConfigId, zoals voor de SyncAll)

  1. … 5 more files in changeset.
  • More
  • CR-953
  • finished reviewing
[UST3606] VHIP903 InventSync: Uitfilteren ongewenste producten:
[UST3606] VHIP903 InventSync: Uitfilteren ongewenste producten:
[UST3606] VHIP903 InventSync: Uitfilteren ongewenste producten:

- Extra in AX aangemaakt service-product hardcoded bij de lijst van Service-producten gezet in IsOngewenstProductInMigratieFilter => ProductNr 1829171

- AxProductFilter Mockable gemaakt

- Ongewenste producten (zoals service-producten) worden nu niet meer geInventSynct

  1. … 7 more files in changeset.
  • More
  • CR-936
  • finished reviewing
ongeveer ja http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

ongeveer ja

Best duidelijk zijn. Neemt niet weg dat het gewoon Set DynIterator = ##class().... moeten zijn

Best duidelijk zijn. Neemt niet weg dat het gewoon Set DynIterator = ##class().... moeten zijn

Is dat niet dezelfde test als: "Test: Generate wordt NIET aangeroepen indien enkel items met verkeerde hoofdnode aanwezig"() ?

Is dat niet dezelfde test als: "Test: Generate wordt NIET aangeroepen indien enkel items met verkeerde hoofdnode aanwezig"() ?

Mag deze niet gedimd worden als TECH.Iterator? Maakt niet uit zeker, wegens toch maar een dimmeke?

Mag deze niet gedimd worden als TECH.Iterator? Maakt niet uit zeker, wegens toch maar een dimmeke?

Waarom hier IsGeldig en niet IsValid, zoals in alle filters? Dit dan nog de "Valid"-ator http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Waarom hier IsGeldig en niet IsValid, zoals in alle filters? Dit dan nog de "Valid"-ator

[UST3571] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX
[UST3571] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX
[UST3571] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX

- Validatiecode in common

- Voorlopig geen test wegen aardige objecten van TommyH

  1. … 2 more files in changeset.
[UST3571] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX

- filter gecopieerd van AXimpl.Admin.DOM.PM.ProductIteratorFilter omdat die resultsetbased is ipv stringbased

- gemeenschappelijke code moet nog afgezonderd worden

    • -0
    • +51
    ./AxProductFilter.cls.xml