CacheAdminA_trunk2010/BL/PPS/TBX/sub

pxLabelV1.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 27 Jan 20
Commits this week: 0
Total Lines of Code (LoC): 242
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 30 Jan 2020
11:39 am

Tommy Hebb

11:10 am

Tom Vermeulen

Tuesday 28 Jan 2020
1:41 pm

Tommy Hebb

[ICT-590] [VRB529] LBX Free zonder inschuifelem. : op zaagetiket:
[ICT-590] [VRB529] LBX Free zonder inschuifelem. : op zaagetiket:
Monday 27 Jan 2020
5:48 pm

Tommy Hebb

[ICT-590] [VRB529] LBX Free zonder inschuifelem. : op zaagetiket:

- Property ZijkantInfo was overkill => weggehaald

- GeefZijkantInfo nu als extra bijkomend veld geimplementeerd ipv ter vervanging van bestaand => nummring velden in Bartender respecteren

- Beetje refactor

- Beschrijvende comments bij nu in Bartender ongebruikte velden => die echter nog wel doorgegeven moeten blijven worden om de TABs te respecteren

- Aangepast testbestand

Options
    • -19
    • +10
    ./pxLabelV1.cls.xml
  2. … 1 more file in changeset.
Friday 24 Jan 2020
5:05 pm

Tommy Hebb

[ICT-590] [VRB529] LBX Free zonder inschuifelem. : op zaagetiket:

- Velden m.b.t. zijkant zijn uitgebreid met ZI (Zonder Inschuifelement) en zijn nu verzameld in 1 property ZijkantInfo

Options
    • -4
    • +32
    ./pxLabelV1.cls.xml
Friday 10 Jan 2020
9:32 am

Peter Vercoutter

Wednesday 18 Dec 2019
11:14 am

Tom Vermeulen

10:31 am

Tom Vermeulen

$$$TAB

$$$TAB

Monday 16 Dec 2019
4:38 pm

Tom Vermeulen

[ICT-1056] Bartender opdrachtenbestanden: CR LF TAB (nog iets?) uit filteren, want Bartender gaat daar niet correct mee om.

- VerwijderNietPrintbareKaraktersDoorSpatie() hernoemt naar VervangNietPrintbareKaraktersDoorSpatie()

Options
    • -18
    • +18
    ./pxLabelV1.cls.xml
  2. … 31 more files in changeset.
10:40 am

Tom Vermeulen

nu niet zo belangrijk, maar een geval als dit mag je natuurlijk altijd op meerdere lijnen zetten (zoals bij vele andere bestanden) (opmerking geldt ook voor een paar andere bestanden met lange lij...

nu niet zo belangrijk, maar een geval als dit mag je natuurlijk altijd op meerdere lijnen zetten (zoals bij vele andere bestanden)

(opmerking geldt ook voor een paar andere bestanden met lange lijnen)

10:10 am

Tom Vermeulen

geen $translate in deze klasse maar ook de method van de pxLabel gebruiken het originele probleem is ook nog steeds aanwezig: je gaat eerst tabs toevoegen aan TeSchrijven om ze dan in de laatste li...

geen $translate in deze klasse maar ook de method van de pxLabel gebruiken
het originele probleem is ook nog steeds aanwezig: je gaat eerst tabs toevoegen aan TeSchrijven om ze dan in de laatste lijn weer te verwijderen

9:11 am

Tom Vermeulen

ik zou er nog "Vervang..." ipv "Verwijder..." van maken http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

ik zou er nog "Vervang..." ipv "Verwijder..." van maken

9:10 am

Tom Vermeulen

Als het over een oneliner gaat, mag je ook "Quit $Translate(...." gebruiken

Als het over een oneliner gaat, mag je ook "Quit $Translate(...." gebruiken

Friday 13 Dec 2019
8:43 am

Tom Vermeulen

[ICT-1056] Bartender opdrachtenbestanden: CR LF TAB (nog iets?) uit filteren, want Bartender gaat daar niet correct mee om.

- In alle bestanden de naam van de methode ‘VerwijderNietPrintbareKarakters’ veranderd naar ‘VerwijderNietPrintbareKaraktersDoorSpatie’

- Waar mogelijk *9,$C(9), $Char(9) aangepast naar $$$TAB

- Waar mogelijk $C(10,13) aangepast naar $$$CRLF

- Ook in de 2 mac files VerwijdernietPrintbareKaraktersDoorSpatie() toegevoegd waar nodig

Options
    • -23
    • +23
    ./pxLabelV1.cls.xml
  2. … 34 more files in changeset.
Thursday 12 Dec 2019
10:30 am

Tom Vermeulen

de comments die bij de laatste files staan gelden voor de andere bestanden ook, maar ik heb het niet overal gekopieerd

de comments die bij de laatste files staan gelden voor de andere bestanden ook, maar ik heb het niet overal gekopieerd

10:29 am

Tom Vermeulen

voor de volledigheid zou je ook een test moeten voorzien met een lege string eentje met enkel een te vervangen karakter, eentje met het karakter in het begin en eentje in het midden. Vervolgens ook...

voor de volledigheid zou je ook een test moeten voorzien met een lege string eentje met enkel een te vervangen karakter, eentje met het karakter in het begin en eentje in het midden. Vervolgens ook nog eens een test met meerdere te vervangen karakters (dezelfde/verschillende). Op die manier zijn alle mogelijkheden afgedekt. Aja ook een testje waarbij er geen karakter te vervangen is.

10:27 am

Tom Vermeulen

Eigenlijk is het niet VerwijderNietPrintbareKarakters, maar VervangNietPrintbareKaraktersDoorSpatie

Eigenlijk is het niet VerwijderNietPrintbareKarakters, maar VervangNietPrintbareKaraktersDoorSpatie

10:25 am

Tom Vermeulen

we gebruiken nooit All Caps de geneste $Translates kan je wijzigen in 1 translate

we gebruiken nooit All Caps
de geneste $Translates kan je wijzigen in 1 translate

10:24 am

Tom Vermeulen

de wijzigingen voor de andere files waar etiketten worden gegenereerd zitten er nog niet tussen

de wijzigingen voor de andere files waar etiketten worden gegenereerd zitten er nog niet tussen

8:30 am

Tom Vermeulen

Voor de leesbaarheid misschien ook ineens de *9 door $$$Tab vervangen Het kan wel zijn dat die macro niet beschikbaar is in deze klasse. (misschien even uitzoeken hoe je die wel kan gebruiken?) In ...

Voor de leesbaarheid misschien ook ineens de *9 door $$$Tab vervangen
Het kan wel zijn dat die macro niet beschikbaar is in deze klasse. (misschien even uitzoeken hoe je die wel kan gebruiken?)
In dat geval is het misschien toch nog iets leesbaarder om de *9 te vervangen door $Char(9), wat de link met een TAB-karakter iets duidelijk maakt.
Up to you

oudste vorm: *9
minder oude vorm: $Char(9) of $C(9)
huidige vorm: $$$TAB

hetzelfde geldt natuurlijk ook voor CR en LF (13 en 10)

8:19 am

Tom Vermeulen

deze moet eigenlijk niet mee in de review, maar dat maakt nu niet zoveel uit

deze moet eigenlijk niet mee in de review, maar dat maakt nu niet zoveel uit

8:19 am

Tom Vermeulen

Hier gaat ge een licht probleem krijgen: de TeSchrijven variabele bevat tabs ($Char(9)) als veldseparator. Door ze er weer uit te halen gaat alles in 1 veld komen. Zelfde opmerking voor de CRLF die...

Hier gaat ge een licht probleem krijgen: de TeSchrijven variabele bevat tabs ($Char(9)) als veldseparator. Door ze er weer uit te halen gaat alles in 1 veld komen.
Zelfde opmerking voor de CRLF die aan TeSchrijven wordt toegevoegd, ge gaat die ook kwijt geraken en alles op 1 lijn zetten
Vervang en passant die $Char(9) door $$$Tab

8:16 am

Tom Vermeulen

Quit ipv QUIT

Quit ipv QUIT

8:16 am

Tom Vermeulen

Quit ipv QUIT

Quit ipv QUIT

Wednesday 11 Dec 2019
1:15 pm

Peter Vercoutter

[ICT-1056] Bartender opdrachtenbestanden: CR LF TAB (nog iets?) uit filteren, want Bartender gaat daar niet correct mee om.
[ICT-1056] Bartender opdrachtenbestanden: CR LF TAB (nog iets?) uit filteren, want Bartender gaat daar niet correct mee om.
Tuesday 10 Dec 2019
4:08 pm

Tom Vermeulen

[ICT-1056]

- Bij iedere WRITE, die te maken heeft met de bonnen voor Bartender, de methode VerwijderNietPrintbareKarakters() geplaatst om zo de niet afdrukbare tekens te verwijderen

Options
    • -23
    • +19
    ./pxLabelV1.cls.xml
  2. … 28 more files in changeset.
Wednesday 04 Dec 2019
5:12 pm

Tommy Hebb

[ICT-738] HX: Klantetiket: VRB666 Sp+Si ofwel releasen, ofwel revert op trunk:

- Aanpassing naamgeving Property TiponCount => TiponCountBefore

- Idem getter SetTiponCount => SetTiponCountBefore

- Idem LabelMock

- Idem TestCases

Options
    • -6
    • +6
    ./pxLabelV1.cls.xml
  2. … 4 more files in changeset.
Monday 02 Dec 2019
5:15 pm

Tommy Hebb

[ICT-738] HX: Klantetiket: VRB666 Sp+Si ofwel releasen, ofwel revert op trunk:

- Fix testen die uitgeschakeld waren => Extra properties LadenAanPersCount en LadenAanPersCountAfter stonden al in productie => heeft nooit gewerkt => terug weggehaald om consistentie te verzekeren

=> tabs moeten blijven staan ivm latere toevoeging van property Opm4

Options
    • -40
    • +0
    ./pxLabelV1.cls.xml
  2. … 1 more file in changeset.
3:01 pm

Tommy Hebb

[ICT-738] HX: Klantetiket: VRB666 Sp+Si ofwel releasen, ofwel revert op trunk:

- Fix testen die uitgeschakeld waren

Options
    • -0
    • +30
    ./pxLabelV1.cls.xml
  2. … 1 more file in changeset.
12:37 pm

Tommy Hebb

[ICT-738] HX: Klantetiket: VRB666 Sp+Si ofwel releasen, ofwel revert op trunk:

- Reset code volgens versie op productie

Options
    • -4
    • +24
    ./pxLabelV1.cls.xml