CacheAdminA_trunk2010/vhTest/Mock/WSimpl/AX/CalculatedProduct

VariantDescriptionBepaler.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 24 Feb 20
Commits this week: 0
Total Lines of Code (LoC): 27
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 05 Mar 2020
3:26 pm

Peter Vercoutter

2:20 pm

Peter Vercoutter

Wednesday 04 Mar 2020
8:24 am

Tom Vermeulen

8:20 am

Tom Vermeulen

zie andere review van dit kaartje

zie andere review van dit kaartje

8:17 am

Tom Vermeulen

8:17 am

Tom Vermeulen

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

8:16 am

Tom Vermeulen

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

Tuesday 03 Mar 2020
10:02 am

Tom Vermeulen

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"

Monday 02 Mar 2020
8:56 am

Tom Vermeulen

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

Monday 24 Feb 2020
11:50 am

Peter Vercoutter

[ICT - 883] UAT 1422: verkeerde tekst in engels document
[ICT - 883] UAT 1422: verkeerde tekst in engels document
11:48 am

Peter Vercoutter

[ICT - 883] UAT 1422: verkeerde tekst in engels document

- Code refactoren en uitbreiden zodat geen description meer wordt teruggeven i.p.v. een Nederlandstalige description indien deze niet bestaat in de vertalingen.

Options
    • -0
    • +7
    ./VariantDescriptionBepaler.cls.xml
  2. … 3 more files in changeset.
Wednesday 19 Feb 2020
3:31 pm

Tom Vermeulen

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

3:21 pm

Tom Vermeulen

om goed te zijn zou je ook een test moeten voorzien die laat zien dat er geen Nederlands gebruikt wordt indien de omschrijving in andere taal niet ontbreekt

om goed te zijn zou je ook een test moeten voorzien die laat zien dat er geen Nederlands gebruikt wordt indien de omschrijving in andere taal niet ontbreekt

9:45 am

Peter Vercoutter

[ICT-883] UAT 1422: verkeerde tekst in engels document
[ICT-883] UAT 1422: verkeerde tekst in engels document
Friday 16 Feb 2018
9:37 am

Martijn Vierstraete

Tuesday 13 Feb 2018
3:06 pm

Wim Vermeulen

  • More
  • CR-338
  • finished reviewing
Wednesday 07 Feb 2018
1:52 pm

Martijn Vierstraete

[UST3614] VHIP481: BOMBOL: Gewicht meegeven
[UST3614] VHIP481: BOMBOL: Gewicht meegeven
11:32 am

MartijnV

[UST3614] VHIP481: BOMBOL: Gewicht meegeven: hergenereren ==> ADUConfigVariantId ipv RetailVariantId
Options
    • -1
    • +1
    ./VariantDescriptionBepaler.cls.xml
  2. … 11 more files in changeset.
Friday 01 Sep 2017
3:16 pm

Martijn Vierstraete

2:48 pm

Wim Vermeulen

  • More
  • CR-341
  • finished reviewing
2:30 pm

Martijn Vierstraete

2:30 pm

Martijn Vierstraete

[UST2612] VHIP481: BOMBOM: ConfigVariantId invullen met ItemId
[UST2612] VHIP481: BOMBOM: ConfigVariantId invullen met ItemId
Tuesday 22 Aug 2017
9:38 am

Martijn Vierstraete

Thursday 17 Aug 2017
10:48 am

Wim Vermeulen

  • More
  • CR-340
  • finished reviewing
Friday 11 Aug 2017
11:55 am

Martijn Vierstraete

11:54 am

Martijn Vierstraete

Volgorde is niet belangrijk

Volgorde is niet belangrijk

Tuesday 18 Jul 2017
2:55 pm

Jo Claes

  • More
  • CR-341
  • finished reviewing
2:54 pm

Jo Claes

Is de volgorde belangrijk ?? of is het voldoende om te testen dat ze het lijstje zitten ? Als de volgorde belangrijk is, dan gelieve een extra test te maken dat de volgorde van de teksten belangri...

Is de volgorde belangrijk ?? of is het voldoende om te testen dat ze het lijstje zitten ?

Als de volgorde belangrijk is, dan gelieve een extra test te maken dat de volgorde van de teksten belangrijk is

2:51 pm

Jo Claes

sowieso try catchen want het zou kunnen dat het niet meer bestaat tussen de twee calls http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif aangezien we toc...

sowieso try catchen want het zou kunnen dat het niet meer bestaat tussen de twee calls

aangezien we toch enkel blue sky scenario doen is zelfs in eerste versie de bestaat al te veel

Monday 17 Jul 2017
2:57 pm

Jo Claes

  • More
  • CR-340
  • finished reviewing