ProboxEtiketAfdrukkenHelper.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Is het interessant om als het om een herafdruk ga te checken of er een batchid werd meegeven? Om zo een nauwkeurigere error log te hebben. Of wordt misschien al gedaan in de ZoekViaBatchIdEnSequent...

Is het interessant om als het om een herafdruk ga te checken of er een batchid werd meegeven? Om zo een nauwkeurigere error log te hebben. Of wordt misschien al gedaan in de ZoekViaBatchIdEnSequentienummer!?

juist een bedenking. Moet deze test ook voor klantetiketten?

juist een bedenking. Moet deze test ook voor klantetiketten?

[ICT-1702] PA-UB-139: Planscan: TAOR etiketten afdrukken
[ICT-1702] PA-UB-139: Planscan: TAOR etiketten afdrukken
[ICT-1702] PA-UB-139: Planscan: TAOR etiketten afdrukken

- proboxafdrukkenlogica uit de dnacodeservice verwijderd

    • -33
    • +0
    ./ProboxEtiketAfdrukkenHelper.cls.xml
  1. … 6 more files in changeset.
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)
Commentaar is tegenstrijdig aan de assert-waarde ?

Commentaar is tegenstrijdig aan de assert-waarde ?

Kan je hier niet gewoon de ##class(TECH.FrequentieTabel) gebruiken ?

Kan je hier niet gewoon de ##class(TECH.FrequentieTabel) gebruiken ?

Hiere wa muggeziften voor gevorderden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif Voor een buitenstaander is de code meteen begrijpbaar als je het g...

Hiere wa muggeziften voor gevorderden

Voor een buitenstaander is de code meteen begrijpbaar als je het gedrag in een boolean steekt : bvb.

 
  #dim ToonLijnReferentieOpTweedeRegel As %Boolean = ($$$HasLength(LijnReferentie))
  If ToonLijnReferentieOpTweedeRegel { ... 

en de "if" hieronder uiteraard ook.

[ICT-1197] PA-UB-074: ProboxinhoudEtiket afdrukken op basis van DnaCode tabel
[ICT-1197] PA-UB-074: ProboxinhoudEtiket afdrukken op basis van DnaCode tabel
[ICT-1197] PA-UB-074: ProboxinhoudEtiket afdrukken op basis van DnaCode tabel

- helpkerklasse mockable gemaakt

    • -0
    • +33
    ./ProboxEtiketAfdrukkenHelper.cls.xml
  1. … 2 more files in changeset.