ImageLink.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5609] Parametriseren Images path in cache code

- code met expliciete verwijzing naar \\notes01\images aangepast met configItem, met default fallback op Notes01 indien niet gezet.

  1. … 22 more files in changeset.
[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...
[ICT-3847][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 : code productie-waardig maken

- Klasse BL.Legacy.HADWIZ kan ProductApi uitmocken + ProductInfoAesKey en -Domein via config-item opgehaald

- Klasse BL.Prod.ImageLink kan instance van BL.Legacy.HADWIZ injecteren

- vhUnitTest.BL.Prod.TestImageLink aangepast : Product en ProductApi uitgemockt; Resultaat geeft nieuwe url naar vhisie4

  1. … 3 more files in changeset.
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
[ICT-3685][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 url i.p.v. product.aspx

- refactor code GetInfoPageURL() voor FOP : oproepen via method GeefProductInfoUrlAspx() van BL.Legacy.HADWIZ

[ICT-3111] RVG: Afbeeldingcode bepalen
[ICT-3111] RVG: Afbeeldingcode bepalen
[ICT-3111] RVG: Afbeeldingcode bepalen
Je hebt een aantal imagecodes gecontroleerd, dat is oké. Wanneer er nu een nieuwe MatKleur (of materiaal) bijkomt, dan zal deze test niet beginnen te falen. Logisch! Als we zoiets wel willen, dan m...

Je hebt een aantal imagecodes gecontroleerd, dat is oké.
Wanneer er nu een nieuwe MatKleur (of materiaal) bijkomt, dan zal deze test niet beginnen te falen. Logisch!
Als we zoiets wel willen, dan moeten we daarvoor een "SmokeTest" schrijven. Ik heb effe nagedacht, maar dat zou niet zo simpel zijn (gezien de combinatie van Materiaal en Kleur). Dus ik stel voor om het zo te laten, tenzij je zelf een briljant idee hebt.

Commentaar opkuisen :-P

Commentaar opkuisen :-P

... geeft lege AfbeeldingCode."

... geeft lege AfbeeldingCode."

Ter informatie (om misverstanden in de toekomst te vermijden) : Bij materiaal bestaat "antislip" en "antislip-Fiber" Voor deze testmethods (naam) maakt het echt ni uit (dus ge hoeft het ni per sé ...

Ter informatie (om misverstanden in de toekomst te vermijden) :
Bij materiaal bestaat "antislip" en "antislip-Fiber"

Voor deze testmethods (naam) maakt het echt ni uit (dus ge hoeft het ni per sé aan te passen) maar let wel op als ge met andere mensen spreekt over "Antislip" terwijl ge "AntislipFiber" zou bedoelen.

$$$Implementation( MATImageCodeBepaler (i.p.v. "SST...") en ook : in nieuwe code gebruiken we $$$Inject i.p.v. $$$implementation

$$$Implementation( MATImageCodeBepaler (i.p.v. "SST...")

en ook :
in nieuwe code gebruiken we $$$Inject i.p.v. $$$implementation

FormalSpec is copy-paste van LBX --> aanpassen naar MATImageCodeBepaler

FormalSpec is copy-paste van LBX --> aanpassen naar MATImageCodeBepaler

[ICT-2419] Matjes hebben geen afbeelding op documenten
[ICT-2419] Matjes hebben geen afbeelding op documenten
[ICT-2419] Matjes hebben geen afbeelding op documenten

- beetje bypass logica voor matjes

[ICT-2419] Matjes hebben geen afbeelding op documenten

- imagelink data toevoegen conform andere producten, dit keer met betere namen

Ik vermoed dat vhUnitTest.APPS.Halux.Docs.TestDataEtiketten nergens anders gebruikt wordt en dus mag opgekuist worden hé http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emo...

Ik vermoed dat vhUnitTest.APPS.Halux.Docs.TestDataEtiketten nergens anders gebruikt wordt en dus mag opgekuist worden hé