Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Ook een klein stukje in Econ! (zie svn)

Ook een klein stukje in Econ! (zie svn)

[ICT-1693] TAXEB1 - PM: Maatwerk: BevatGeleiderBevestigingSchroeven = True indien Alpnach:
[ICT-1693] TAXEB1 - PM: Maatwerk: BevatGeleiderBevestigingSchroeven = True indien Alpnach:
Testje bijgezet als AttrVsAdminProdNr niet SalesPriceBOM is => dan moet er betreffende de ProductIdRecycleerder niets gebeuren.

Testje bijgezet als AttrVsAdminProdNr niet SalesPriceBOM is => dan moet er betreffende de ProductIdRecycleerder niets gebeuren.

Ik vond dat nogal overkill, omdat die list dan amper een codelijn of 6 later alweer omgezet zou moeten worden naar een delimited string. Moest de omzetting naar delimited string in een andere metho...

Ik vond dat nogal overkill, omdat die list dan amper een codelijn of 6 later alweer omgezet zou moeten worden naar een delimited string. Moest de omzetting naar delimited string in een andere method gebeuren, dan zou ik het zeker gedaan hebben

Ook zoals aan telefoon besproken... het gaat over de LadeVariant zoals gekend in AX, waarvoor overal in de BOMBOL de AXimpl gebruikt wordt.

Ook zoals aan telefoon besproken... het gaat over de LadeVariant zoals gekend in AX, waarvoor overal in de BOMBOL de AXimpl gebruikt wordt.

Zoals besproken aan de telefoon: Dat Request komt in de BOMBOL-interface ook al binnen als parameter, dus de afhankelijkheid is sowieso al een feit. Bovendien wordt er 1 laagje hoger ook een andere...

Zoals besproken aan de telefoon: Dat Request komt in de BOMBOL-interface ook al binnen als parameter, dus de afhankelijkheid is sowieso al een feit. Bovendien wordt er 1 laagje hoger ook een andere property van uitgelezen

testje(s) voor de andere gevallen?

testje(s) voor de andere gevallen?

ikzelf probeer constructies als deze meestal te vermijden omdat het om de zoveel tijd wel eens kan leiden naar een rariteit met die delimiter. Misschien beter om een list te gebruiken? Kunt ge gewo...

ikzelf probeer constructies als deze meestal te vermijden omdat het om de zoveel tijd wel eens kan leiden naar een rariteit met die delimiter. Misschien beter om een list te gebruiken? Kunt ge gewoon op adden indien nodig. Een list kan daarna 'gewoon' in een delimited string omgezet worden indien nodig.

minor nitpicking: ik las dit als een Event "OnJadajada" en het duurde toch een fractie van een seconde voor ik doorhad dat dat niet echt logisch zou zijn indeze context. Een kleine "B" was misschie...

minor nitpicking: ik las dit als een Event "OnJadajada" en het duurde toch een fractie van een seconde voor ik doorhad dat dat niet echt logisch zou zijn indeze context. Een kleine "B" was misschien logische geweest. Nogmaals: kommaneuken hoor

een beetje in de trand van de vorige opmerking: zou het niet properder zijn om de DOM....LadeVariant te gebruiken (qua hiërarchie hé). Je gaat dan wel een 'converter' moeten opzetten die de AX-vari...

een beetje in de trand van de vorige opmerking: zou het niet properder zijn om de DOM....LadeVariant te gebruiken (qua hiërarchie hé). Je gaat dan wel een 'converter' moeten opzetten die de AX-variant omzet naar een DOM-variant, maar daar kan de LadeVariantBepaler zich mee bezighouden. Op die manier verdwijnt de dependency op AXimpl.* package

Van deze request wordt enkel Request.Attribute gebruikt. Het zou beter zijn om die attribute als parameter mee te geven, anders introduceer je een (extra?) afhankelijkheid tss WSimpl en AXif packages.

Van deze request wordt enkel Request.Attribute gebruikt. Het zou beter zijn om die attribute als parameter mee te geven, anders introduceer je een (extra?) afhankelijkheid tss WSimpl en AXif packages.

[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:
zie andere review van dit kaartje

zie andere review van dit kaartje

ik snap wat je wil zeggen, maar zou je dit niet vervangen door "identnummer van product"? Zonder naar de code te kijken is het precies of ge een random nummer gaat teruggeven http://subversion02.va...

ik snap wat je wil zeggen, maar zou je dit niet vervangen door "identnummer van product"? Zonder naar de code te kijken is het precies of ge een random nummer gaat teruggeven

probeer geen early quits te gebruiken, dat maakt het voor latere debugging/uitbreiding veel lastiger. In principe zijn er maar twee plaatsen waar je quits mag zetten: helemaal bovenaan een method o...

probeer geen early quits te gebruiken, dat maakt het voor latere debugging/uitbreiding veel lastiger. In principe zijn er maar twee plaatsen waar je quits mag zetten: helemaal bovenaan een method om bij trivialiteiten al te quiten en natuurlijk onderaan als er iets gereturned moet worden.

Dus beter een variable "resultaat" of zoiets maken en die dan assignen en returnen op het einde

ipv If $$$Not(xxxx = "") kan je if $$$HasLength(xxx) gebruiken leest wat gemakkelijker In Caché kan je ook met een postconditional werken: ipv If blablabla{ Do zwamzwam } kan je volgende lijn ge...

ipv If $$$Not(xxxx = "") kan je if $$$HasLength(xxx) gebruiken
leest wat gemakkelijker

In Caché kan je ook met een postconditional werken:
ipv
If blablabla

Unknown macro: { Do zwamzwam }


kan je volgende lijn gebruiken

"Do:blablabla zwamzwam"

Aangezien er met VerwachteUitkomst niks gebeurt tot op het einde van de test, is het niet per sé nodig om die hier al te dimmen. Je zou de assert evengoed kunnen reduceren tot Do $$$AssertEquals("O...

Aangezien er met VerwachteUitkomst niks gebeurt tot op het einde van de test, is het niet per sé nodig om die hier al te dimmen. Je zou de assert evengoed kunnen reduceren tot
Do $$$AssertEquals("Omschrijving in het Nederlands"$$$CRLF"Kleur:VertalingKleur",DescriptionBepaler.GeefDescription(ProductMock,RandomTaal))
Hoe dan ook is dat iets waarover je kan blijven discussiëren: langere lijnen en minder variabelen, of kortere lijnen en meer one-time-use variabelen...
Indien je toch gaat voor meer variabelen: de gewoonte is om de #dim's niet allemaal bovenaan te smijten, maar vlak voordat je ze nodig hebt

[ICT - 883] UAT 1422: verkeerde tekst in engels document
[ICT - 883] UAT 1422: verkeerde tekst in engels document
Maak hier maar ineens een Setterke van, want zoals je ziet kan je in dit geval (lange namen) niet zien of er dingen aan de constructor worden meegegeven. Idem voor andere #dim/Set combinaties in de...

Maak hier maar ineens een Setterke van, want zoals je ziet kan je in dit geval (lange namen) niet zien of er dingen aan de constructor worden meegegeven.
Idem voor andere #dim/Set combinaties in deze en de andere test