CacheAdminA_trunk2010/vhUnitTest/WSimpl/AX/CalculatedProduct

VariantDescriptionBepaler

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 17 Jul 17
Commits this week: 0
Total Lines of Code (LoC): 200
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 13 Oct 2022
9:46 am

Peter Vercoutter

[ICT-2929] [rvJBA] INK: Signaletiek opstellen

- Langtekst van te verzagen product toevoegen aan langtekst inkorting product

Options
    • -0
    • +4
    ./DescriptionBepaler/Test.cls.xml
  2. … 6 more files in changeset.
Thursday 05 Mar 2020
3:27 pm

Peter Vercoutter

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

- Refactoring

Options
    • -3
    • +4
    ./DescriptionBepaler/Test.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
9:26 am

Peter Vercoutter

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

- Kleine refactoring in test

Options
    • -5
    • +3
    ./DescriptionBepaler/Test.cls.xml
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
    • -19
    • +9
    ./DescriptionBepaler/Test.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
8:53 am

Peter Vercoutter

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

- Refactoring code

- 2 Testen aangemaakt die de descriptionbepaler zullen testen, indien een omschrijving ontbreekt moet hierdoor de Nederlands omschrijving teruggegeven worden.

Options
    • -0
    • +71
    ./DescriptionBepaler/Test.cls.xml
  2. … 1 more file in changeset.
Thursday 14 Nov 2019
3:08 pm

Jo Willems

[ICT-604][VRB647]PM: Kleuren: andere vertaling van letters S en W voor TAX (enkel langtekst)

- Bombol service aanpassen zodat deze ook de goede vertalingen voor TAX plaatmateriaal gebruikt in geval van TAX losse component rug en bodem

Options
    • -0
    • +3
    ./DescriptionBepaler/Test.cls.xml
  2. … 2 more files in changeset.
Monday 19 Feb 2018
10:51 am

Martijn Vierstraete

Saturday 17 Feb 2018
1:38 pm

Wim Vermeulen

  • More
  • CR-100
  • finished reviewing
Friday 02 Feb 2018
3:16 pm

Martijn Vierstraete

2:50 pm

Wim Vermeulen

  • More
  • CR-106
  • finished reviewing
10:47 am

MartijnV

[UST3545] Kleur ORGALUX en TA'OR Cubics producten ook meegeven in BOMBOL-interface +[UST3622] Identnummer ingeval lossecomponenten meegeven

- puntjes op de i na acceptatietest PM

Options
    • -1
    • +1
    ./DescriptionBepaler/Test.cls.xml
10:46 am

MartijnV

[UST3545] Kleur ORGALUX en TA'OR Cubics producten ook meegeven in BOMBOL-interface +[UST3622] Identnummer ingeval lossecomponenten meegeven

- puntjes op de i na acceptatietest PM

Options
    • -2
    • +2
    ./DescriptionBepaler/Test.cls.xml
  2. … 6 more files in changeset.
Thursday 01 Feb 2018
4:37 pm

Martijn Vierstraete

[UST3622] Identnummer ingeval lossecomponenten meegeven
[UST3622] Identnummer ingeval lossecomponenten meegeven
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
4:11 pm

MartijnV

[UST3622] Identnummer ingeval lossecomponenten meegeven
Options
    • -1
    • +33
    ./DescriptionBepaler/Test.cls.xml
  2. … 3 more files in changeset.
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
    • -28
    • +14
    ./Test.cls.xml
    • -0
    • +47
    ./DescriptionBepaler/Test.cls.xml
  3. ./DescriptionBepaler
  4. … 11 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
  1. ./KleurBepaler
    • -16
    • +18
    ./Test.cls.xml
    • -50
    • +0
    ./KleurBepaler/Test.cls.xml
  4. … 13 more files in changeset.