CacheAdminA_trunk2010/vhConsistency/Checks/DOM/PM

PrijzenVariatiesViaWebservice.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 19 Oct 18
Commits this week: 0
Total Lines of Code (LoC): 218
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Monday 22 Oct 2018
9:51 am

Tommy Hebb

8:57 am

Wim Vermeulen

8:57 am

Wim Vermeulen

Zou ook kunnen checken op (..GeefAantalMeldingen() = 0) dan had je geen extra property MeldingCounter moeten aanmaken. Deze is niet fout en moet niet aangepast worden. Ik wou je er enkel effe op a...

Zou ook kunnen checken op (..GeefAantalMeldingen() = 0)
dan had je geen extra property MeldingCounter moeten aanmaken.

Deze is niet fout en moet niet aangepast worden. Ik wou je er enkel effe op attent maken.

Friday 19 Oct 2018
2:24 pm

Tommy Hebb

[UW615] Uitzoeken prijzenverschillen PrijzenvariatiesViaWebservice komt dagelijks voor + documentatie in mail:

- Verduidelijking igv verknoeien seperator in csv

Options
    • -3
    • +4
    ./PrijzenVariatiesViaWebservice.cls.xml
1:37 pm

Tommy Hebb

[UW615] Uitzoeken prijzenverschillen PrijzenvariatiesViaWebservice komt dagelijks voor + documentatie...
[UW615] Uitzoeken prijzenverschillen PrijzenvariatiesViaWebservice komt dagelijks voor + documentatie...
1:36 pm

Tommy Hebb

[UW615] Uitzoeken prijzenverschillen PrijzenvariatiesViaWebservice komt dagelijks voor + documentatie in mail:

- Juiste (volledige) instructie in comment aangepast

Options
    • -1
    • +1
    ./PrijzenVariatiesViaWebservice.cls.xml
12:51 pm

Tommy Hebb

[UW615] Uitzoeken prijzenverschillen PrijzenvariatiesViaWebservice komt dagelijks voor + documentatie in mail:
Options
    • -8
    • +65
    ./PrijzenVariatiesViaWebservice.cls.xml
Thursday 20 Sep 2018
10:42 am

Tommy Hebb

[UW589] ConsistencyCheck Prijzen: Fix decimals en aanpassing prijs PRNR 1348833 voor Dovy:

- Prijsaanpassing gebeurd in het csv controlebestand

- Fix decimals door Justify => moet altijd 2 cijfers na de komma zijn

Options
    • -2
    • +2
    ./PrijzenVariatiesViaWebservice.cls.xml
Thursday 06 Sep 2018
9:53 am

Tommy Hebb

[UW578] ConsistencyCheck PrijzenVariatiesViaWebservice: Comment ivm MailRedirect:
Options
    • -1
    • +9
    ./PrijzenVariatiesViaWebservice.cls.xml
Friday 31 Aug 2018
12:20 pm

Tommy Hebb

[UW552] VHIP907: Verkoopprijzen: consistencycheck Accept nakijken:

- Check mailt zelf igv CacheAccept2010 => Eerste opzet => gerevert => forward-rule gebruikt in default mailbox van cacheaccept2010

Options
    • -8
    • +0
    ./PrijzenVariatiesViaWebservice.cls.xml
11:15 am

Tommy Hebb

[UW552] VHIP907: Verkoopprijzen: consistencycheck Accept nakijken:

- Check mailt zelf igv CacheAccept2010 => Eerste opzet

Options
    • -0
    • +8
    ./PrijzenVariatiesViaWebservice.cls.xml
Thursday 12 Jul 2018
10:18 am

Tommy Hebb

[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:

- Na review => Super uitvoeren in constructor en SkipCheck mag nu weg

- Parameter DataStartRij gebruiken waar mogelijk

Options
    • -1
    • +1
    ./PrijzenVariatiesViaWebservice.cls.xml
  2. … 1 more file in changeset.
9:48 am

Tommy Hebb

Wednesday 11 Jul 2018
2:32 pm

Wim Vermeulen

2:26 pm

Wim Vermeulen

Do ##super() vergeten !!!

Do ##super() vergeten !!!

2:20 pm

Wim Vermeulen

Zelfde reactie als 'ginder' ;-P

Zelfde reactie als 'ginder' ;-P

2:11 pm

Wim Vermeulen

3 --> ..#DataStartRij

3 --> ..#DataStartRij

2:08 pm

Wim Vermeulen

Ik ben het volledig eens met Martijn : Zolang er geen test (of andere nood aan Injection) is, zet je gewoon : Set ..Console = ##class(TECH.Console.impl.TerminalConsole).%New() Wanneer later iemand ...

Ik ben het volledig eens met Martijn :
Zolang er geen test (of andere nood aan Injection) is, zet je gewoon :
Set ..Console = ##class(TECH.Console.impl.TerminalConsole).%New()
Wanneer later iemand toch iets moet injecten, dan zal hij zelf (alleen het gewenste object) aanpassen naar $$$Inject()
De meeste onder ons hebben de gewoonte om op deze laatste manier te werken. Eerder dan alles meteen inject-ready te maken
Deze code terugdraaien hoeft niet voor mij ... op voorwaarde dat je in de toekomst onze manier overneemt :-D

Tuesday 10 Jul 2018
10:27 am

Tommy Hebb

Zelfde reply als 'ginder'

Zelfde reply als 'ginder'

10:27 am

Tommy Hebb

De JoC heeft mij da zo geleerd, als zijnde 'good coding convention', voor mocht er in de toekomst iemand eens een andere resource willen meegeven

De JoC heeft mij da zo geleerd, als zijnde 'good coding convention', voor mocht er in de toekomst iemand eens een andere resource willen meegeven

10:24 am

Tommy Hebb

Niet echt http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Had die idd beter niet opgenomen in de review, sorry

Niet echt Had die idd beter niet opgenomen in de review, sorry

10:16 am

Martijn Vierstraete

10:16 am

Martijn Vierstraete

Is deze klasse nog relevant?

Is deze klasse nog relevant?

10:07 am

Martijn Vierstraete

Zelfde opmerking als bij BestandUploader

Zelfde opmerking als bij BestandUploader

10:04 am

Martijn Vierstraete

Er zijn geen tests op deze klasse, waarom constructor injections?

Er zijn geen tests op deze klasse, waarom constructor injections?

9:21 am

Tommy Hebb

Thursday 05 Jul 2018
3:37 pm

Tommy Hebb

[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:
[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:
3:18 pm

Tommy Hebb

[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:

- Reset van GlobalMultiDimIterator voor het geval hij meermaals na elkaar uitgevoerd wordt zonder een nieuwe instance van de klasse te nemen

Options
    • -0
    • +2
    ./PrijzenVariatiesViaWebservice.cls.xml
2:31 pm

Tommy Hebb

[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:

- Toevoegen SkipGlobalHeader om eerste Next te doen op de GlobalMultiDimIterator

- Meldingen uit comment gehaald en writejes weg

- Continue weggehaald uit alle vier testjes => falen van de ene hoeft testen van andere niet te annuleren => resultaat is toch al opgehaald

- PS => Continue bij check op KLNr en op PRNr moeten blijven staan => indien foutief => verder verloop van iteratie zou toch falen

Options
    • -17
    • +14
    ./PrijzenVariatiesViaWebservice.cls.xml
12:37 pm

Tommy Hebb

[UST3585] VHIP907: Verkoopprijzen: Consist check uitbreiden: voorgemont. lade en:

- Injectie DecimalSeperatorInput en ThousandSeperatorInput doorgeven aan de private properties gaat helpen

Options
    • -0
    • +2
    ./PrijzenVariatiesViaWebservice.cls.xml