LadeKlantEtiket.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
In verband met het statement : "QR-code ... staat op dezelfde plaats op het etiket als de barcode, dus ze mogen niet beide ingevuld zijn" Optie 1 : in de software dit controleren/valideren en een ...

In verband met het statement : "QR-code ... staat op dezelfde plaats op het etiket als de barcode, dus ze mogen niet beide ingevuld zijn"

Optie 1 : in de software dit controleren/valideren en een exceptie smijten wanneer dit zich toch voordoet. De exceptie dan ook in UT opnemen : ZetVerwachteExceptie (je weet wel).
Optie 2 : in de UT tonen dat de software aanvaardt dat beide velden tegelijk ingevuld zijn, dus zonder exceptie te smijten. Hierbij wordt de verantwoordelijkheid dan naar Bartender (in dit geval) doorgeschoven.

Jij hebt hier gekozen voor Optie 2 (bewust of onbewust). Voor mij is dit oké.
Eventueel in die specifieke unit test method toelichten dat dit scenario problemen zal geven in Bartender.

[ICT-3055] VHOSS Etiket QRCode voor De Decker
[ICT-3055] VHOSS Etiket QRCode voor De Decker
[ICT-3055] VHOSS Etiket QRCode voor De Decker

- QRCode op LadeKlantEtiket gezet

  1. … 2 more files in changeset.
is inderdaad onterecht gecommit. In de huidige versie staan de tests weer actief trouwens

is inderdaad onterecht gecommit. In de huidige versie staan de tests weer actief trouwens

dat dit een vhossetiket is dat afleidt van een normaal etiket. De vhossvelden worden achteraan toegevoegd.

dat dit een vhossetiket is dat afleidt van een normaal etiket. De vhossvelden worden achteraan toegevoegd.

is in latere commits al gebeurd. kaartje werd te groot en moest in stukken gekapt worden wat hier en daar een raar gevolg heeft http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/ic...

is in latere commits al gebeurd. kaartje werd te groot en moest in stukken gekapt worden wat hier en daar een raar gevolg heeft

Deze is nogal tricky ... zelf een zoektochtje moeten doen in svn, om te zien dat deze per ongeluk mee gecommit was. Eventueel kan je hier zelf, op voorhand een commentje bij plaatsen.

Deze is nogal tricky ...
zelf een zoektochtje moeten doen in svn, om te zien dat deze per ongeluk mee gecommit was.
Eventueel kan je hier zelf, op voorhand een commentje bij plaatsen.

Volgens de Header is de DnaCode niet achteraan toegevoegd, maar wel ergens tussenin. Ik zou verwachten dat de inhoud van de tabel dan ook verandert (begrijp: opschuift). Wat is uw uitleg? ;-D

Volgens de Header is de DnaCode niet achteraan toegevoegd, maar wel ergens tussenin.
Ik zou verwachten dat de inhoud van de tabel dan ook verandert (begrijp: opschuift).
Wat is uw uitleg? ;-D

Debug writes opkuisen : 100 - 200 - 300 - en hieronder ook nog.

Debug writes opkuisen : 100 - 200 - 300 - en hieronder ook nog.

Ik zie dat je de property hebt toegevoegd, en dezelfde dag nog terug verwijderd. Daarom lijkt me logisch dat je de (ongebruikte) storage ook terug verwijderd.

Ik zie dat je de property hebt toegevoegd, en dezelfde dag nog terug verwijderd.
Daarom lijkt me logisch dat je de (ongebruikte) storage ook terug verwijderd.

[ICT-1048] PA-UB-061: etiketten aanpassen
[ICT-1048] PA-UB-061: etiketten aanpassen
[ICT-1048] PA-UB-061: etiketten aanpassen

- casing juist gezet

[ICT-1048] PA-UB-061: etiketten aanpassen

- extra veld voor dnacode op ladeklantetiket basisklasse

$$$TAB

$$$TAB

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

- VerwijderNietPrintbareKaraktersDoorSpatie() hernoemt naar VervangNietPrintbareKaraktersDoorSpatie()

  1. … 31 more files in changeset.
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)

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

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

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

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

[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

  1. … 34 more files in changeset.
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

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.

Eigenlijk is het niet VerwijderNietPrintbareKarakters, maar VervangNietPrintbareKaraktersDoorSpatie

Eigenlijk is het niet VerwijderNietPrintbareKarakters, maar VervangNietPrintbareKaraktersDoorSpatie

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