CacheAdminA_trunk2010/vhConsistency/Checks/DOM/PM/PrijzenVariatiesViaWebservice

BestandUploader.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 12 Jul 18
Commits this week: 0
Total Lines of Code (LoC): 140
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
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
    ./BestandUploader.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:
2:47 pm

Tommy Hebb

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

- Toevoegen Parameter DataStartRij => om duidelijk te maken wat de verhouding is tussen Count en AantalRecords

Options
    • -2
    • +7
    ./BestandUploader.cls.xml
Wednesday 04 Jul 2018
12:59 pm

Tommy Hebb

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

- BestandUploader => De beschrijvende 1e record in de Global wordt pas weggeschreven als er minstens 1 datalijn zal volgen

- PrijzenVariatiesViaWebservice => De AtEnd geeft hier niet gewenste resultaat => implementatie veranderd => check of Global bestaat

Options
    • -2
    • +5
    ./BestandUploader.cls.xml
  2. … 1 more file in changeset.
12:03 pm

Tommy Hebb

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

- Implementatie VoerUit => voorlopig SkipCheck = 1 en writejes ipv meldingen => voor testen en debuggen op Accept

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

Tommy Hebb

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

- Afzondering PopuleerGlobalVolgensBestand naar eigen klasse BestandUploader

Options
    • -0
    • +132
    ./BestandUploader.cls.xml
  2. … 2 more files in changeset.