Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
OEPS !!! Great call!!! Thx!!!

OEPS !!! Great call!!! Thx!!!

Het oude productId komt nog op 2 andere plaatsen voor in caché studio. Deze best ook aanpassen. Searching for 1118152 in *.cls,*.mac,*.inc APPS.PM.Maatwerk.MaatwerkModel.Modellen.TaorboxModel.Rolle...

Het oude productId komt nog op 2 andere plaatsen voor in caché studio. Deze best ook aanpassen.

Searching for 1118152 in *.cls,*.mac,*.inc
APPS.PM.Maatwerk.MaatwerkModel.Modellen.TaorboxModel.Rollen.ZijkantBevestigingVeer.cls(GeefRol+2): ##class(APPS.PM.Maatwerk.MaatwerkModel.dto.SpecifiekProductBuilderType).%New(1118152)
DOM.PM.Maatwerk.Syncs.DotNet.MaatwerkCalculator.impl.ProductInfo.cls(GeefLijstVanProductIdVanNietSelectieKenmerken+102): Do ProductIds.Insert(1118152) 
Found 2 occurrence/s in 2 file/s.
[ICT-3034] [rvWV] Andere veer Vällinge koppeling TA'OR:
[ICT-3034] [rvWV] Andere veer Vällinge koppeling TA'OR:
Vreemd dat deze klasse in de review van UST3954 zit, want is gecommit onder UST3942 :-? Hoe dan ook : nieuwe UT toegevoegd, maar ook gecommit onder UST3942 . De review heb ik wel hier toegevoegd, w...

Vreemd dat deze klasse in de review van UST3954 zit, want is gecommit onder UST3942 :-?
Hoe dan ook : nieuwe UT toegevoegd, maar ook gecommit onder UST3942 .
De review heb ik wel hier toegevoegd, want gij hebt er achter gevraagd :-D :-D

Er is een testcase waarbij verpakking probox moet geconverteerd worden. Dit moet uiteraard probox blijven! *vhUnitTest.EXT.Unishop.PurchaseOrder001.ProductConverter.ExtNaarEdi.Tax.Test.cls(Test: ...

Er is een testcase waarbij verpakking probox moet geconverteerd worden. Dit moet uiteraard probox blijven!

  • vhUnitTest.EXT.Unishop.PurchaseOrder001.ProductConverter.ExtNaarEdi.Tax.Test.cls(Test: TAX LadeMetVeelExtras)
    Maar ik maak een nieuwe testmethod bij, die controleert of verpakking "" wordt geconverteerd naar V1.
in volgende fase wordt dit opgekuist http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

in volgende fase wordt dit opgekuist

Bij volgende update mag dit definitief verdwijnen.

Bij volgende update mag dit definitief verdwijnen.

Klopt, maar is in een ander kanban-kaartje, dus ook aparte review

Klopt, maar is in een ander kanban-kaartje, dus ook aparte review

moet de omgekeerde in APPS.PM.Maatwerk.impl.NaarObjectConverterTAX ook niet nog aangepast worden?

moet de omgekeerde in APPS.PM.Maatwerk.impl.NaarObjectConverterTAX ook niet nog aangepast worden?

idem

idem

mag dit niet gewoon weg?

mag dit niet gewoon weg?

die comments mogen weg zeker?

die comments mogen weg zeker?

in principe zouden hier nog wat testjes rond moeten komen

in principe zouden hier nog wat testjes rond moeten komen

oude logica laten staan in dit geval. alpnach stuurt nl die tag niet door en dat zou dus voor verrassingen kunnen zorgen. en in principe is het een sifon-gerelateerd kaartje en zou de werking van v...

oude logica laten staan in dit geval. alpnach stuurt nl die tag niet door en dat zou dus voor verrassingen kunnen zorgen.
en in principe is het een sifon-gerelateerd kaartje en zou de werking van verpakkingtypes niet aangepast moeten worden (ander kaartje nodig, kleine kaartjes enzo)

[UST3954] TAX-SY: herschikken van de LBX-testen (converters)
[UST3954] TAX-SY: herschikken van de LBX-testen (converters)
  • More
  • CR-321
  • finished reviewing
[BUS666] TAX: Huls insteekpen geleider
[BUS666] TAX: Huls insteekpen geleider
De klasse wordt maar éénmalig gegenereerd en ik vind het niet correct om foutieve code te laten staan, ook al is het een gegenereerde klasse en ook al wordt deze overridden in een andere klasse. Di...

De klasse wordt maar éénmalig gegenereerd en ik vind het niet correct om foutieve code te laten staan, ook al is het een gegenereerde klasse en ook al wordt deze overridden in een andere klasse. Dit kan enkel maar verwarrend werken en kans geven op fouten als anderen de gegenereerde gaan gebruiken.

SpecifiekeProductieWijze wordt nergens in de code gebruikt, dan zouden we de rest ook moeten refactoren naar specifieke...

SpecifiekeProductieWijze wordt nergens in de code gebruikt, dan zouden we de rest ook moeten refactoren naar specifieke...

Standaardbinnenlade wordt getest met ---> CopyListOfDataTypes(..StandaardLadeItems(),..StandaardBinnenladeItems()) maw gebruikt standaardlade en binnenlade bevat geen setschroeven.

Standaardbinnenlade wordt getest met ---> CopyListOfDataTypes(..StandaardLadeItems(),..StandaardBinnenladeItems()) maw gebruikt standaardlade
en binnenlade bevat geen setschroeven.

  • More
  • CR-91
  • finished reviewing
Waarom is er geen versie voor FlatpackV1 ??

Waarom is er geen versie voor FlatpackV1 ??

Waarom is deze niet toegevoegd bij de StandaardLadeItems()

Waarom is deze niet toegevoegd bij de StandaardLadeItems()