ConsistencyCheck

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Do ##super() vergeten !!!

Do ##super() vergeten !!!

Zelfde reactie als 'ginder' ;-P

Zelfde reactie als 'ginder' ;-P

3 --> ..#DataStartRij

3 --> ..#DataStartRij

Ik ben het volledig eens met Martijn : Zolang er geen test (of andere nood aan Injection) is, zet je gewoon : Set ..Console = ##class(TECH.Console.impl.TerminalConsole).%New() Wanneer later iemand ...

Ik ben het volledig eens met Martijn :
Zolang er geen test (of andere nood aan Injection) is, zet je gewoon :
Set ..Console = ##class(TECH.Console.impl.TerminalConsole).%New()
Wanneer later iemand toch iets moet injecten, dan zal hij zelf (alleen het gewenste object) aanpassen naar $$$Inject()
De meeste onder ons hebben de gewoonte om op deze laatste manier te werken. Eerder dan alles meteen inject-ready te maken
Deze code terugdraaien hoeft niet voor mij ... op voorwaarde dat je in de toekomst onze manier overneemt :-D

Zelfde reply als 'ginder'

Zelfde reply als 'ginder'

De JoC heeft mij da zo geleerd, als zijnde 'good coding convention', voor mocht er in de toekomst iemand eens een andere resource willen meegeven

De JoC heeft mij da zo geleerd, als zijnde 'good coding convention', voor mocht er in de toekomst iemand eens een andere resource willen meegeven

Niet echt http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Had die idd beter niet opgenomen in de review, sorry

Niet echt Had die idd beter niet opgenomen in de review, sorry

Is deze klasse nog relevant?

Is deze klasse nog relevant?

Zelfde opmerking als bij BestandUploader

Zelfde opmerking als bij BestandUploader

Er zijn geen tests op deze klasse, waarom constructor injections?

Er zijn geen tests op deze klasse, waarom constructor injections?

[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:
[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:
[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:

- De properties en niet de parameters gebruiken voor de check of bestand bestaat en CheckParameters is afgezonderd

- Iets meer duiding via console aan user over wat er is gedaan

[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:

- FullPath en Filename geparametriseerd en gedefault

- Check of het bestand wel bestaat => anders melding via console

[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:

- Filename en Delimiter worden nu niet meer meegegeven aan de constructor van de FileReader die geinjecteerd wordt, maar nadien gezet => nog te parametriseren

[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:

- Aanpassing Count zodat deze overeenstemt met rangschikking lijnen in bestand

- Zaten soms spaties in SimPPL => Translate gedaan

[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:

- Hang is niet meer nodig door aanpassing Close van FileReader

- De Global wordt nu eerst gekilld zodat zeker met een verse lei begonnen wordt

- Write mag weg

[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:

- Voorlopige code om bestand van PV in global te zetten