CacheAdminA_trunk2010/APPS/Halux/PPS/TBXZaag/impl

PlaatmateriaalHelperVoorMVX.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 02 Feb 23
Commits this week: 0
Total Lines of Code (LoC): 252
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 02 Feb 2023
11:38 am

Peter Vercoutter

[ICT-3363] [rvJLM] Zaagmachine: Sortering bodemlijst niet goed

- reviewopmerkingen

Options
    • -11
    • +10
    ./PlaatmateriaalHelperVoorMVX.cls.xml
  2. … 2 more files in changeset.
11:37 am

Peter Vercoutter

11:32 am

Peter Vercoutter

GesorteerdePlaatMaterialen As %ListOfObjects = ##class(%ListOfObjects).%New() ? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif

GesorteerdePlaatMaterialen As %ListOfObjects = ##class(%ListOfObjects).%New() ?

11:16 am

Peter Vercoutter

Dat is de bedoeling http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif

Dat is de bedoeling

11:16 am

Peter Vercoutter

Zal niet gebeuren, anders heeft de bombol ook problemen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Zal niet gebeuren, anders heeft de bombol ook problemen

10:03 am

Joannes Lemmens

10:03 am

Joannes Lemmens

Zelfde opmerkingen als bij LBX

Zelfde opmerkingen als bij LBX

10:02 am

Joannes Lemmens

Misschien de volgorde aanpassen waarin de methods staan: eerst deze en dan GeefSorteerSleutel sinds deze de die gebruikt.

Misschien de volgorde aanpassen waarin de methods staan: eerst deze en dan GeefSorteerSleutel sinds deze de die gebruikt.

10:00 am

Joannes Lemmens

Volgens mij was het de bedoeling om "BZ/DG":$$$LeftPad4(10000) te doen bij de $Case? Dit zal ook een andere volgorde geven, sinds nu de volgorde is: 10000 - 6000 - 7000 - 8000 - 9000 (alfabetisch, ...

Volgens mij was het de bedoeling om "BZ/DG":$$$LeftPad4(10000) te doen bij de $Case?
Dit zal ook een andere volgorde geven, sinds nu de volgorde is: 10000 - 6000 - 7000 - 8000 - 9000 (alfabetisch, ascending).

9:51 am

Joannes Lemmens

ReturnType is een TECH.Iterator, maar maakt niet veel uit voor Cache http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif

ReturnType is een TECH.Iterator, maar maakt niet veel uit voor Cache

9:51 am

Joannes Lemmens

Twee opmerkingen: 1) Is de ByRef nodig hier? 2) SequentieNummer wordt niet gebruikt

Twee opmerkingen:
1) Is de ByRef nodig hier?
2) SequentieNummer wordt niet gebruikt

9:50 am

Joannes Lemmens

Stel dat er in de lijst 2 PlaatMaterialen zijn met dezelfde kleur en hoogte, wordt er hier dan niet 1 van overgeschreven door de andere? Als ze enkel unieke plaatmaterialen willen is dit perfect, a...

Stel dat er in de lijst 2 PlaatMaterialen zijn met dezelfde kleur en hoogte, wordt er hier dan niet 1 van overgeschreven door de andere?
Als ze enkel unieke plaatmaterialen willen is dit perfect, anders zou je op het einde nog een unieke identifier moeten hebben (gewoon een index waarschijnlijk).

8:37 am

Peter Vercoutter

[ICT-3363] [rvJLM] Zaagmachine: Sortering bodemlijst niet goed
[ICT-3363] [rvJLM] Zaagmachine: Sortering bodemlijst niet goed
8:37 am

Peter Vercoutter

[ICT-3363] [rvJLM] Zaagmachine: Sortering bodemlijst niet goed

- Sortering van bodemlijst op kleur en dan hoogte

Options
    • -1
    • +39
    ./PlaatmateriaalHelperVoorMVX.cls.xml
  2. … 2 more files in changeset.
Wednesday 30 Nov 2022
11:51 am

Peter Vercoutter

[ICT-3201] Zaagmachinelijst

- Terechte reviewopmerking

Options
    • -23
    • +7
    ./PlaatmateriaalHelperVoorMVX.cls.xml
  2. … 2 more files in changeset.
11:50 am

Peter Vercoutter

Thursday 24 Nov 2022
5:56 pm

Jelle Baele

5:56 pm

Jelle Baele

Ik zie dat er een generieke "APPS.Halux.PPS.TBXZaag.impl.PlaatMateriaalHelper" klasse bestaat. Mij persoonlijk lijkt het 'cleaner' om deze en die voor LBX ervan te laten overerven, de methods te ov...

Ik zie dat er een generieke "APPS.Halux.PPS.TBXZaag.impl.PlaatMateriaalHelper" klasse bestaat. Mij persoonlijk lijkt het 'cleaner' om deze en die voor LBX ervan te laten overerven, de methods te overschrijven en de nieuwe method 'GeefLijstZonderDubbels()' daar te plaatsen, aangezien dit nu tweemaal dezelfde code is

Tuesday 22 Nov 2022
4:31 pm

Peter Vercoutter

[ICT-3201] Zaagmachinelijst
[ICT-3201] Zaagmachinelijst
4:13 pm

Peter Vercoutter

[ICT-3201] Zaagmachinelijst

- Dubbels uit lijst halen. Dubbels komen er in voor omdat ze meerdere values hebben in de selectiekenmerken (multivalues)

Options
    • -3
    • +26
    ./PlaatmateriaalHelperVoorMVX.cls.xml
  2. … 1 more file in changeset.
Tuesday 16 Feb 2021
10:00 am

Tommy Hebb

8:24 am

Tom Vermeulen

En eigenlijk moet die in de constructor genewed worden. Ik weet het, aan de ene kant "deze klasse gebruikt die manier van werken nog niet" en aan de andere kant het "boyscout principe". Your call

En eigenlijk moet die in de constructor genewed worden. Ik weet het, aan de ene kant "deze klasse gebruikt die manier van werken nog niet" en aan de andere kant het "boyscout principe". Your call

Monday 15 Feb 2021
12:57 pm

Tom Vermeulen

12:50 pm

Tommy Hebb

Done!

Done!

12:23 pm

Tommy Hebb

= Code van Wim. Wellicht zijn er teveel verschillende mogelijkheden om een al te specifieke te gebruiken hé.

= Code van Wim. Wellicht zijn er teveel verschillende mogelijkheden om een al te specifieke te gebruiken hé.

12:20 pm

Tommy Hebb

Goed gezien :-s Bij deze aangepast!

Goed gezien :-s Bij deze aangepast!

Friday 12 Feb 2021
2:01 pm

Tom Vermeulen

ik veronderstel dat dit een copy paste van lbx is, maar misschien toch de ' door $$$Not vervangen

ik veronderstel dat dit een copy paste van lbx is, maar misschien toch de ' door $$$Not vervangen

2:00 pm

Tom Vermeulen

Als er ooit vhossen bij mvx komen, gaat ge het probleem hebben dat er geen vhossetiketten worden afgedrukt. Dat is een uitbreiding die dan nog moet toegevoegd worden op de KlantEtiket.PrinterFactory

Als er ooit vhossen bij mvx komen, gaat ge het probleem hebben dat er geen vhossetiketten worden afgedrukt. Dat is een uitbreiding die dan nog moet toegevoegd worden op de KlantEtiket.PrinterFactory

1:27 pm

Tom Vermeulen

is het niet beter om een specifiekere exception te gebruiken? CommunicationErrorException ofzo

is het niet beter om een specifiekere exception te gebruiken? CommunicationErrorException ofzo

Thursday 11 Feb 2021
12:09 pm

Tom Vermeulen

moet hier ook geen ...Merivobox voorzien worden (zelfs als die gelijk is aan de Legrabox)? Kwestie van consequent namen te gebruiken

moet hier ook geen ...Merivobox voorzien worden (zelfs als die gelijk is aan de Legrabox)? Kwestie van consequent namen te gebruiken