CacheAdminA_trunk2010/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: 05 Mar 20
Commits this week: 0
Total Lines of Code (LoC): 76
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:27 pm

Peter Vercoutter

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

- Refactoring

Options
    • -4
    • +4
    ./VariantDescriptionBepaler.cls.xml
  2. … 2 more files in changeset.
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
    • -4
    • +16
    ./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
Monday 19 Feb 2018
10:51 am

Martijn Vierstraete

Saturday 17 Feb 2018
1:38 pm

Wim Vermeulen

  • More
  • CR-100
  • finished reviewing
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
    • -2
    • +2
    ./VariantDescriptionBepaler.cls.xml
  2. … 11 more files in changeset.
Thursday 01 Feb 2018
4:35 pm

Martijn Vierstraete

[UST3545] Kleur ORGALUX en TA'OR Cubics producten ook meegeven in BOMBOL-interface
[UST3545] Kleur ORGALUX en TA'OR Cubics producten ook meegeven in BOMBOL-interface
3:40 pm

MartijnV

Merged revision(s) 49573-49575 from trunk2010:

[UST3545] Kleur ORGALUX en TA'OR Cubics producten ook meegeven in BOMBOL-interface

- nieuwe integratietest cases toegevoegd

........

[UST3545] Kleur ORGALUX en TA'OR Cubics producten ook meegeven in BOMBOL-interface

- tussentijdse commits met mogelijke controles

........

[UST3545] Kleur ORGALUX en TA'OR Cubics producten ook meegeven in BOMBOL-interface

- altijd kleur opvragen (blijkt leeg waar nodig :) )

........

Options
    • -17
    • +7
    ./VariantDescriptionBepaler.cls.xml
  2. … 13 more files in changeset.
1:56 pm

MartijnV

[UST3545] Kleur ORGALUX en TA'OR Cubics producten ook meegeven in BOMBOL-interface

- altijd kleur opvragen (blijkt leeg waar nodig :) )

Options
    • -9
    • +5
    ./VariantDescriptionBepaler.cls.xml
  2. … 15 more files in changeset.
Wednesday 27 Dec 2017
5:46 pm

MartijnV

CRLFs in langtekst
Options
    • -1
    • +1
    ./VariantDescriptionBepaler.cls.xml
  2. … 6 more files in changeset.
Thursday 14 Dec 2017
12:18 pm

Martijn Vierstraete

Wednesday 13 Dec 2017
4:27 pm

Kurt Callebaut

  • More
  • CR-812
  • finished reviewing
1:42 pm

MartijnV

VHIP481 BOM/BOL - langtekst inclusief kleur

- met kleurlabel

Options
    • -2
    • +9
    ./VariantDescriptionBepaler.cls.xml
  2. … 1 more file in changeset.
11:08 am

Martijn Vierstraete

VHIP481 BOM/BOL - langtekst inclusief kleur
VHIP481 BOM/BOL - langtekst inclusief kleur
10:05 am

MartijnV

VHIP481 BOM/BOL - langtekst inclusief kleur
Options
    • -3
    • +16
    ./VariantDescriptionBepaler.cls.xml
  2. … 19 more files in changeset.