CacheAdminA_trunk2010/APPS/Halux/PPS/TBXZaag/impl

PlaatmateriaalHelperVoorLBX.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): 55
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
    ./PlaatmateriaalHelperVoorLBX.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
    • -0
    • +39
    ./PlaatmateriaalHelperVoorLBX.cls.xml
  2. … 2 more files in changeset.
Wednesday 30 Nov 2022
11:51 am

Peter Vercoutter

[ICT-3201] Zaagmachinelijst

- Terechte reviewopmerking

Options
    • -23
    • +8
    ./PlaatmateriaalHelperVoorLBX.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
    • -4
    • +27
    ./PlaatmateriaalHelperVoorLBX.cls.xml
  2. … 1 more file in changeset.