Activity CacheAdminA_trunk2010/AX/Uitgaand/PM/Product/impl/ArticleIDCHUI.cls.xml

9:59 am

Joannes Lemmens

More
68632
changed 1 file
root: CR-2470

[ICT-] [rvWV] PM - lijst van artikelnummers uit Admin krijgen

- Method hernoemd naar GenereerAantalArtikelNummers omdat het niets teruggeeft

- -2
- +2
./ArticleIDCHUI.cls.xml
- History
- Source
- Diff
./ArticleIDCHUI.cls.xml

12:02 pm

Joannes Lemmens

More
68631
changed 1 file
root: CR-2470

[ICT-] [rvWV] PM - lijst van artikelnummers uit Admin krijgen

- Refactor: GeefInAantal en GenereerLijstVanArtikelNummers zijn nu ProcedureBlocks, Aantal is een property en andere variabelen geNewed

- -23
- +45
./ArticleIDCHUI.cls.xml
- History
- Source
- Diff
./ArticleIDCHUI.cls.xml

8:07 am

Joannes Lemmens

More
68590
changed 1 file
root:

[ICT-] [rvWV] PM - lijst van artikelnummers uit Admin krijgen

- Typo's gefixed en 'BestandVolledigeNaam' ipv 'BestandNaamMetPad'

- Indien BestandVolledigeNaam leeg is een foutmelding op het bericht laten verschijnen

- -9
- +15
./ArticleIDCHUI.cls.xml
- History
- Source
- Diff
./ArticleIDCHUI.cls.xml

5:32 pm

Wim Vermeulen

More
Re: CR-2470
commented on review

variabelenaam is zeker geen verkeerde keuze, maar "BestandVolledigeNaam" is best te gebruiken volgens onze conventies (maar ik vind ze zelf ook niet terug op confluence)

variabelenaam is zeker geen verkeerde keuze, maar
"BestandVolledigeNaam" is best te gebruiken volgens onze conventies (maar ik vind ze zelf ook niet terug op confluence)

5:24 pm

Ik ben geen voorstander om veel methods in "ProcedureBlock = 0" te schrijven. Enkel daar waar het nodig is. In dit geval zit je in een Admin-scherm, dus is er sowieso al een deel "ProcedureBlock = ...

Ik ben geen voorstander om veel methods in "ProcedureBlock = 0" te schrijven. Enkel daar waar het nodig is.
In dit geval zit je in een Admin-scherm, dus is er sowieso al een deel "ProcedureBlock = 0".

Maar als je deze wil wegwerken, moet je zeker opletten want soms is dat nogal tricky.
Geef me een seintje als je hierover meer uitleg wenst.

5:20 pm

Wim Vermeulen

More
Re: CR-2470
commented on review

Typo : Wacht om (A)f te sluiten http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Typo : Wacht om (A)f te sluiten

5:19 pm

Wim Vermeulen

More
Re: CR-2470
commented on review

Typo op meerdere plaatsen : artikkel met 1 "k"

10:49 am

Joannes Lemmens

More
CR-2470
started review

[ICT-1793] [rvWV] PM - lijst van artikelnummers uit Admin krijgen

10:47 am

Joannes Lemmens

More
68559
changed 1 file
root: CR-2470

[ICT-1793] [rvWV] PM - lijst van artikelnummers uit Admin krijgen

- Kleine wijziging: nu moet er expliciet aantal 1 ingegeven worden om 1 artikelnummer te genereren. Als er geen getal of 0 wordt ingegeven wordt er niets gegenereerd.