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
[ICT-3685][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 url i.p.v. product.aspx

- refactor code DocLink() voor oproep PV : oproepen via method GeefProductInfoUrlAspx() van BL.Legacy.HADWIZ

[ICT-1661] SVG-037 Implementatie SVG in CSP (voor SamV)

- Bij defaulting naar SVG (vanuit een order bij ieder product) geen Optie-flag voor VML meer doorgeven.

[ICT-1440] [rvPRA] ICT: Source control: backport Deploy2010 naar Trunk2010

-Backport files van wijzigingen van het jaar 2019

  1. … 40 more files in changeset.
[ICT-1423] PA-UB-097: DottedDosserNummer geeft error bij starten batch

- cachecode geschreven :)

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
[ICT-1423] PA-UB-097: DottedDosserNummer geeft error bij starten batch

- uitgeschakelde code verwijderd.

[ICT-1423] PA-UB-097: DottedDosserNummer geeft error bij starten batch

- wat code in commentaar gezet en vervangen door code die enkel het laatste cijfer als controlekarakter gebruikt

[ICT-1214] [PieterVR] Extra gemapte properties van ^KPR G-node bij generische producten, om via SQL te kunnen raadplegen.
Merged revision(s) 56981-56982 from deploy2010:

HLX planning resource berekenen

........

Klant 4682 Van Hoecke uit StatWeek gehaald

........

  1. … 8 more files in changeset.
Aanpassing ABC Transport versus waarde : sub piece verwisseld
Prod.Product : mapping extra property LabelCode (voor ad hoc onderzoekje voor PM).
  • 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...
[UW432] Zoektocht leveringen met verkeerde grootteorder versus totaalbedrag : GrootteOrde veld op Product.
  • 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)