Product.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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
voilà zie, weer iets bijgeleerd. mercikes

voilà zie, weer iets bijgeleerd. mercikes

Als ik het goed begrijp, is het controlecijfer enkel het laatste digit, terwijl het algoritme meer algemeen geschreven was. maar het oude algoritme (For-loop) gaat mis, omdat er (sinds lange tijd) ...

Als ik het goed begrijp, is het controlecijfer enkel het laatste digit, terwijl het algoritme meer algemeen geschreven was.
maar het oude algoritme (For-loop) gaat mis, omdat er (sinds lange tijd) ook cijfers in de dossiercode voorkomen. Right?

Anyway, er is wel een properdere manier om "last char" te pakken te krijgen : $Extract(string, *)
Zie mijn voorbeeldjes :

 
write $Extract("ABCDE",*)
E
write $Extract("ABCDE",*-1)
D
write $Extract("ABCDE",*-2,*)
CDE


Dit werkt helaas niet op CacheV5

http://localhost:57772/csp/docbook/DocBook.UI.Page.cls?KEY=RCOS_fextract

Eventuele oplossing door het proper weg te steken in de TECH.StringUtils (met nen UT)

Eventuele oplossing door het proper weg te steken in de TECH.StringUtils (met nen UT)

Hier ook : write $Extract("ABCDE",1,*-1) ABCD

Hier ook :

write $Extract("ABCDE",1,*-1)
ABCD
[ICT-1423] PA-UB-097: DottedDosserNummer geeft error bij starten batch
[ICT-1423] PA-UB-097: DottedDosserNummer geeft error bij starten batch
  • More
  • CR-948
  • finished reviewing
[UW432] Zoektocht leveringen met verkeerde grootteorder versus totaalbedrag : AantalEenheid veld op...
[UW432] Zoektocht leveringen met verkeerde grootteorder versus totaalbedrag : AantalEenheid veld op...
  • More
  • CR-450
  • finished reviewing
Je gebruikt principieel en principaal door elkaar, best vervangen door principaal (doe een CTRL-F, het zijn vijf files)

Je gebruikt principieel en principaal door elkaar, best vervangen door principaal (doe een CTRL-F, het zijn vijf files)

kan in 1 lijn (hierboven)

kan in 1 lijn (hierboven)

%New in constructor

%New in constructor

Beter een $IsObject ipv $$$HasLength als het over objecten gaat ipv strings. (jaja, untyped enz, maar toch)

Beter een $IsObject ipv $$$HasLength als het over objecten gaat ipv strings. (jaja, untyped enz, maar toch)

[UST2729] VHIP903: Period. voorraad: SoprContract opnemen voor elke SOPR-variant:
[UST2729] VHIP903: Period. voorraad: SoprContract opnemen voor elke SOPR-variant: