Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Je doet een vergelijking van de verschillen, dus je wilt dat er geen verschillen zijn, daarom empty list

Je doet een vergelijking van de verschillen, dus je wilt dat er geen verschillen zijn, daarom empty list

Vanwaar assert empty list?

Vanwaar assert empty list?

wlips mogen weg of het zal trakteren worden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

wlips mogen weg of het zal trakteren worden

[ICT-2932] INK: Productcreatie INK
[ICT-2932] INK: Productcreatie INK
Ik zou deze method naar boven verplaatsen, namelijk vlak achter method BerekenStandaardLaden() Initieel was mijn idee/voorstel om dit op "nog hoger" niveau reeds te splitsen, maar moet da nu niet ...

Ik zou deze method naar boven verplaatsen, namelijk vlak achter method BerekenStandaardLaden()

Initieel was mijn idee/voorstel om dit op "nog hoger" niveau reeds te splitsen, maar moet da nu niet meer veranderen. Bij volgende uitbreiding of aanpassing kan dat alsnog meegenomen worden.
Dus nu alleen verplaatsen aub.

method naam aanpassen, zoals VoerUit...() dus ook hier : HaalOp...() i.p.v. Haal...Op...()

method naam aanpassen, zoals VoerUit...()
dus ook hier :
HaalOp...() i.p.v. Haal...Op...()

[VRB569] PM: Maatwerk: Selectiekenmerken: continu calculatie-checks op standaardlades op Productie...
[VRB569] PM: Maatwerk: Selectiekenmerken: continu calculatie-checks op standaardlades op Productie...
Zou ook kunnen checken op (..GeefAantalMeldingen() = 0) dan had je geen extra property MeldingCounter moeten aanmaken. Deze is niet fout en moet niet aangepast worden. Ik wou je er enkel effe op a...

Zou ook kunnen checken op (..GeefAantalMeldingen() = 0)
dan had je geen extra property MeldingCounter moeten aanmaken.

Deze is niet fout en moet niet aangepast worden. Ik wou je er enkel effe op attent maken.

[UW615] Uitzoeken prijzenverschillen PrijzenvariatiesViaWebservice komt dagelijks voor + documentatie...
[UW615] Uitzoeken prijzenverschillen PrijzenvariatiesViaWebservice komt dagelijks voor + documentatie...
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: