Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3847][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 : code productie-waardig maken...
[ICT-3847][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 : code productie-waardig maken...
OnAfterAllTests() toegevoegd.

OnAfterAllTests() toegevoegd.

puur theoretische kommaneukerij, maar stel dat een test crasht, dan komen we niet op die Kill dacht ik. Beter om ook nog eens te killen in een OnAfterAllTests. En ik weet het, ik zie in het gebrui...

puur theoretische kommaneukerij, maar stel dat een test crasht, dan komen we niet op die Kill dacht ik.
Beter om ook nog eens te killen in een OnAfterAllTests.

En ik weet het, ik zie in het gebruik van die %-variabele in de andere klassen geen rariteiten, maar ik wou gewoon iƩts schrijven

@Tom : deze klasse is een vereenvoudigde kopie van "APPS.common.Document.impl.DocBase.WebServerUrlBepaler" voor FOP. Geen inheritance of uncle Bob-principes toegepast http://subversion02.vanhoecke....

@Tom : deze klasse is een vereenvoudigde kopie van "APPS.common.Document.impl.DocBase.WebServerUrlBepaler" voor FOP.
Geen inheritance of uncle Bob-principes toegepast

[ICT-5021] FOP: DocBase Upgrade - Set up Web-folder (IIS) op DEV/ACP/PRD + aanpassen caché-code
[ICT-5021] FOP: DocBase Upgrade - Set up Web-folder (IIS) op DEV/ACP/PRD + aanpassen caché-code
Productie-omgeving nog niet beschikbaar bij iO (of toch nog niet gemeld aan mij :P ) -> Staat niet vermeld in jira : Klopt, enerzijds heb ik wel iets geschreven in de svn commit message; anderzijd...

Productie-omgeving nog niet beschikbaar bij iO (of toch nog niet gemeld aan mij :P )

-> Staat niet vermeld in jira :
Klopt, enerzijds heb ik wel iets geschreven in de svn commit message; anderzijds beetje bewust gedaan omdat ik zelf wist dat er nog andere review-opmerkingen zouden zijn.
Verdere afhandeling in vervolg-kaartje jira ICT-3847

Klopt. Wordt verder afgewerkt in vervolg-kaartje jira ICT-3847

Klopt. Wordt verder afgewerkt in vervolg-kaartje jira ICT-3847

Is deze URL die naar www-test wijst wel correct in productie? Om te ontwikkelen en voor validatie kan ik deze begrijpen, maar voor productie vind ik dit vreemd. In het Jira ticket staat hier ook ni...

Is deze URL die naar www-test wijst wel correct in productie?
Om te ontwikkelen en voor validatie kan ik deze begrijpen, maar voor productie vind ik dit vreemd.
In het Jira ticket staat hier ook niets over vermeld.

Is het hard coderen van een encryption key in code geen 'bad practice'?

Is het hard coderen van een encryption key in code geen 'bad practice'?

[ICT-3709][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 url i.p.v. product.aspx (deel...
[ICT-3709][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 url i.p.v. product.aspx (deel...
[ICT-3573][rvJLM] Watchfolder SVG: settings KAD omschakelen naar SVG + configItems naar print01 i.p.v....
[ICT-3573][rvJLM] Watchfolder SVG: settings KAD omschakelen naar SVG + configItems naar print01 i.p.v....
No big deal. Codebase in t Nederlands, maar StringUtils.Replace() is Engels, dus wellicht geen taal-reset van mijn brains http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/em...

No big deal.
Codebase in t Nederlands, maar StringUtils.Replace() is Engels, dus wellicht geen taal-reset van mijn brains

Maar in feite zetten we geen commentaar in onze code (= clean code principle!)
Hier, in dit specifiek geval is het echter om te benadrukken dat ALLE spaties moeten vervangen worden.
Uitzondering bevestigt de regel.

Algemene vraag: Ik heb als PM tester enkele opmerkingen in het Jira ticket geplaatst. Moet ik hier al reviewer rekening mee houden tijdens de review? De uitleg van Tom was dat de reviewer enkel d...

Algemene vraag:

Ik heb als PM tester enkele opmerkingen in het Jira ticket geplaatst.
Moet ik hier al reviewer rekening mee houden tijdens de review?

De uitleg van Tom was dat de reviewer enkel de kwaliteit van de code moet beoordelen.
Maar er is een optie 'factually incorrect' in de tool, en mogelijks vallen enkele van mijn opmerkingen onder deze noemer?

Graag advies.

Style: *Codebase is volledig in het Nederlands, waarom deze comment dan in het Engels?

Style:

  • Codebase is volledig in het Nederlands, waarom deze comment dan in het Engels?
[ICT-3685][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 url i.p.v. product.aspx
[ICT-3685][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 url i.p.v. product.aspx
ik heb het opgekuist. Dat komt wel goed zonder die controle.

ik heb het opgekuist. Dat komt wel goed zonder die controle.

Script zal normaal gezien nog 1x manueel getriggerd worden, en is nadien niet meer nodig. Zal ook niet als geplande taak (bvb. wekelijks) in de background lopen. Dus eigenlijk zou dit script zelfs ...

Script zal normaal gezien nog 1x manueel getriggerd worden, en is nadien niet meer nodig. Zal ook niet als geplande taak (bvb. wekelijks) in de background lopen.
Dus eigenlijk zou dit script zelfs mogen gearchiveerd worden.
Anderszijds is dit wel handig (als template) voor toekomstige gelijkaardige gevallen. Daarom zal ik het laten staan.

Bepalen van klantnummer is inderdaad niet alledaags, maar als het werkt..! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/biggrin.gif

Bepalen van klantnummer is inderdaad niet alledaags, maar als het werkt..!