CacheAdminA_trunk2010/APPS/VisiDot/impl

VHossService.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 20 Feb 20
Commits this week: 0
Total Lines of Code (LoC): -38
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 20 Feb 2020
10:06 am

Tom Vermeulen

[ICT-1243] Vhosskasten recepteren kuist de database niet proper op

- extra queries om locations en carriers op te kuisen

Options
    • -2
    • +16
    ./VHossService.cls.xml
Thursday 09 Jan 2020
2:44 pm

Tom Vermeulen

[ICT-1057] Nog niet alle vhosskasten worden geregistreerd

- while ipv if om oude records ook te verwijderen

Options
    • -16
    • +17
    ./VHossService.cls.xml
Tuesday 07 Jan 2020
11:49 am

Tom Vermeulen

[ICT-1057] Nog niet alle vhosskasten worden geregistreerd

- oneliner gemaakt

Options
    • -3
    • +1
    ./VHossService.cls.xml
Tuesday 03 Dec 2019
1:48 pm

Laura Landuyt

Tuesday 26 Nov 2019
8:32 am

Tom Vermeulen

Monday 25 Nov 2019
2:58 pm

Tom Vermeulen

Dees klopt niet (sorry, niet gezien bij orginele review) De method CheckVhossOp... is geen classmethod. Aangezien de klasse ...VHossService nog niet geïnstantieerd werd, geeft het een fout at runti...

Dees klopt niet (sorry, niet gezien bij orginele review)
De method CheckVhossOp... is geen classmethod. Aangezien de klasse ...VHossService nog niet geïnstantieerd werd, geeft het een fout at runtime.
Oplossing: ##class(APPS.VisiDot.impl.VHossService).%New().CheckVhossOpMeerdereLocaties()

nog een kleintje: als je #dim doet, zet er dan een "As %Boolean" bij (in dit geval). Op zich zal het even goed werken want het is een untyped language, maar het is een betere manier van werken.

Thursday 21 Nov 2019
11:22 am

Tom Vermeulen

11:22 am

Tom Vermeulen

[ICT-999] LBX Vhoss Kasten
[ICT-999] LBX Vhoss Kasten
9:37 am

Tom Vermeulen

naamgevingnitpicking: een IndienMethodCall wordt enkel voor "sturing" van het SUT gebruikt, dus is het eigenlijk een EwmsLocationsStub ipv ...Mock. Bij een VerwachtMethodCall is het wel een ...Mock...

naamgevingnitpicking: een IndienMethodCall wordt enkel voor "sturing" van het SUT gebruikt, dus is het eigenlijk een EwmsLocationsStub ipv ...Mock. Bij een VerwachtMethodCall is het wel een ...Mock gelijk hieronder. In principe is dat ook niet 100% correct, maar dat is een beetje de conventie zoals ze aangehouden wordt in deze gevallen.
Waarom hier ook geen VerwachtMethodCall gebruikt trouwens? In principe is het hetzelfde gedrag als de test hieronder, enkel de returnvalue is anders

9:15 am

Wim Vermeulen

9:14 am

Wim Vermeulen

Opgelet met de indexen van de data, wanneer je rechtstreeks in de globals gaat aanpassen. In dit geval is het geen probleem. Anderszijds zie ik echt geen zinvolle reden waarom je hier niet gewoon ...

Opgelet met de indexen van de data, wanneer je rechtstreeks in de globals gaat aanpassen.
In dit geval is het geen probleem.

Anderszijds zie ik echt geen zinvolle reden waarom je hier niet gewoon het object opent (het ObjectId heb je al), property wijzigt en een propere .Save() uitvoert.
Wijzigen via globals zou ik alleen "toelaten" wanneer een andere methode echt (te) complex zou zijn. Wat hier zeker niet het geval is, hé.

9:08 am

Wim Vermeulen

TIP : ivm $Listbuild : om enkel het 3e list-item te wijzigen Set $LI(varOfGlobal, 3) = NewValue Dit moet je hier zeker (leren) gebruiken!

TIP : ivm $Listbuild : om enkel het 3e list-item te wijzigen
Set $LI(varOfGlobal, 3) = NewValue

Dit moet je hier zeker (leren) gebruiken!

9:01 am

Wim Vermeulen

##class(APPS.VisiDot.impl.Tech).IsVhossID( ) oproepen

##class(APPS.VisiDot.impl.Tech).IsVhossID( ) oproepen

Wednesday 20 Nov 2019
4:02 pm

LauraL

[ICT-974]MAG: Voorraad: zbcheck VHOSS op meerdere plaatsen

Extra test met mock of EWMS/Locations

Options
    • -1
    • +1
    ./VHossService.cls.xml
  2. … 2 more files in changeset.
11:02 am

LauraL

[ICT-974]MAG: Voorraad: zbcheck VHOSS op meerdere plaatsen

Edit VhossService to use mock and alter test

Options
    • -3
    • +19
    ./VHossService.cls.xml
  2. … 2 more files in changeset.
Tuesday 19 Nov 2019
12:04 pm

Tom Vermeulen

mocks gebruiken

mocks gebruiken

12:04 pm

Tom Vermeulen

TECH.Mockable maken en de gegenereerde mocks in de testvhossservice gebruiken

TECH.Mockable maken en de gegenereerde mocks in de testvhossservice gebruiken

12:02 pm

Tom Vermeulen

zie opmerking andere test

zie opmerking andere test

12:02 pm

Tom Vermeulen

01... gebruiken ipv TT....

01... gebruiken ipv TT....

12:00 pm

Tom Vermeulen

data best opkuisen in OnAfterAllTests

data best opkuisen in OnAfterAllTests

11:57 am

Tom Vermeulen

misschien XX in een aparte variabele steken voor de leesbaarheid

misschien XX in een aparte variabele steken voor de leesbaarheid

11:53 am

Tom Vermeulen

Ge moogt gerust spaties gebruiken in testnamen

Ge moogt gerust spaties gebruiken in testnamen

11:43 am

Laura Landuyt

[ICT-974]MAG: Voorraad: zbcheck VHOSS op meerdere plaatsen
[ICT-974]MAG: Voorraad: zbcheck VHOSS op meerdere plaatsen
11:26 am

LauraL

[ICT-974]MAG: Voorraad: zbcheck VHOSS op meerdere plaatsen

VHossService aangepast met call naar query om vhosslocaties te checken

Options
    • -0
    • +10
    ./VHossService.cls.xml
  2. … 1 more file in changeset.
Monday 18 Nov 2019
11:20 am

Tom Vermeulen

[ICT-999] LBX Vhoss Kasten

- ZijkantAfhandelaar moet enkel VhossId en niet FrameId persisteren

Options
    • -6
    • +2
    ./VHossService.cls.xml
  2. … 2 more files in changeset.
9:25 am

Tom Vermeulen

[ICT-999] LBX Vhoss Kasten

- VHossService laten werken met korte vorm

- tests: beetje boyscout principe toegepast (nav uitleg aan Laura)

Options
    • -26
    • +2
    ./VHossService.cls.xml
  2. … 1 more file in changeset.
Tuesday 05 Jun 2018
10:24 am

Jo Willems

Friday 01 Jun 2018
11:31 am

Kurt Callebaut

Tuesday 29 May 2018
1:55 pm

Jo Willems

[VRB500] MAG: Transport: bij terugkomst VHOSS-kast: scan en registreer als staande bij VH/HX
[VRB500] MAG: Transport: bij terugkomst VHOSS-kast: scan en registreer als staande bij VH/HX
1:23 pm

Jo Willems

[VRB500] MAG: Transport: bij terugkomst VHOSS-kast: scan en registreer als staande bij VH/HX

- pattern check vhoskast id

- kunnen opvragen welke id een vhoskast geregistreerd is adhv de gescande nummer (kan ander type of andere kant van de kast zijn).

Options
    • -0
    • +32
    ./VHossService.cls.xml