Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
True!

True!

Mocht de uiteindelijke layout anders zijn dan wat nu gebruikt wordt ("ProboxInhoud") zou ik dit gewoon in BL.Etiket.ProboxInhoud zetten ipv hier te veranderen.

Mocht de uiteindelijke layout anders zijn dan wat nu gebruikt wordt ("ProboxInhoud") zou ik dit gewoon in BL.Etiket.ProboxInhoud zetten ipv hier te veranderen.

[ICT-4335] [rvTVE] Proboxinhoudetiket - Wijzigen naar formaat 8x11 + vervolgetiket
[ICT-4335] [rvTVE] Proboxinhoudetiket - Wijzigen naar formaat 8x11 + vervolgetiket
[ICT-4335] [rvTVE] Proboxinhoudetiket - Wijzigen naar formaat 8x11 + vervolgetiket

- Testen toevoegen

[ICT] Fixen ROUTINELOAD op de unittest van ProboxinhoudEtiket

- NIET DOEN : een classmethod oproepen meteen na de .%New() --> nu is de .%New weggelaten :-)

[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket

- Rariteit in caché: Geen TECH.ListOfObjects maar %ListOfObjects gebruiken

[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket

- Verwerken reviewopmerkingen

detail, maar hier kan je gewoon Quit ##class(TECH.ListIterator).%New(DnaCodeList) gebruiken. Het dimmen in een aparte variabele heeft niet echt meerwaarde omdat de dim bij cache enkel zorgt voor co...

detail, maar hier kan je gewoon Quit ##class(TECH.ListIterator).%New(DnaCodeList) gebruiken. Het dimmen in een aparte variabele heeft niet echt meerwaarde omdat de dim bij cache enkel zorgt voor code-completion

[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket

- Testen dat voor product de projectreferentie wordt vermeld

[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket

- Unittest uitbreiden om standaardgedrag met ingevulde OrderLijnId te testen

[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket

- Refactor unittest

[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket

- Implementeren originele manier opdat in tussenfase ook nog lijnreferenties worden opgehaald

- Unittest opnieuw op oude manier

  1. … 2 more files in changeset.
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket

- Aanpassen test om nu ook rekening te houden met het ophalen van een referentie via OrderlijnId

[ICT-2055] Proboxinhoudetiket samenvoegen

- Code opkuisen naar aanleiding uitleg unit tests

  1. … 1 more file in changeset.
[ICT-1197] PA-UB-074: ProboxinhoudEtiket afdrukken op basis van DnaCode tabel

- reviewopmerkingske

nice catch http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif is trouwens gevolg van copy-paste...

nice catch
is trouwens gevolg van copy-paste...

Heb het nog veranderd naar Quit:(ProboxEtiketAfdrukkenRequest.DnaCodes.Count() = 0) ##class(TECH.ListIterator).%New() kwestie van hoe dan ook een (lege) iterator terug te geven ipv helemaal niets

Heb het nog veranderd naar
Quit:(ProboxEtiketAfdrukkenRequest.DnaCodes.Count() = 0) ##class(TECH.ListIterator).%New()
kwestie van hoe dan ook een (lege) iterator terug te geven ipv helemaal niets

Moeten de 3 methods van de Helper ook niet onder UT steken? Als je ze hier uitmockt, wil dit zeggen dat ze blindelings te vertrouwen zijn. #ikVoorspelEenCommandErrorinMethodX :-P

Moeten de 3 methods van de Helper ook niet onder UT steken?
Als je ze hier uitmockt, wil dit zeggen dat ze blindelings te vertrouwen zijn.

#ikVoorspelEenCommandErrorinMethodX :-P

Finishing touch in $Case : zet na iedere komma een spatie. Dat verhoogt de leesbaarheid http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Finishing touch in $Case : zet na iedere komma een spatie. Dat verhoogt de leesbaarheid

Early-quit in het midden van uw code ... kan ik niet goedkeuren ... zeker niet omdat er een simpel alternatief is, dat wel oké is http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/...

Early-quit in het midden van uw code ... kan ik niet goedkeuren ... zeker niet omdat er een simpel alternatief is, dat wel oké is
Zet als eerste lijn van de method :

        Quit:(ProboxEtiketAfdrukkenRequest.DnaCodes.Count() = 0)