CacheAdminA_trunk2010/vhTest/Mock/APPS

common

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 11
Last Commit: 22 Oct 15
Commits this week: 0
Total Lines of Code (LoC): 386
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
Friday 07 Feb
4:57 pm

Jo Willems

[ICT-5311][rvPVR] Vervangen carrier connection voor DBSchenker via transsmart. De oude wordt vervangen.

- Voor deze carrier stuurt transsmart het label als een embedded bitmap in zpl. de lengte kan langer zijn dan MAX_STRING.

- in de dto was labelData een Base64Binary,wat inprincipe onbeperkt van lengte is en in één keer decodeert, maar na correlatie loopt toch fout omdat er nog ergens in String variabele tussen zit.

- GlobalCharacterstreeam maken van property labelData die de Base64 encoded stream moet gaan bevatten.

- method bij op ZPLPrinter die Stream in stukken base64 decodeert (lengte van stuk moet veelvoud van 3 zijn!), en dat naar de printer device dumpt.

- bijkomend, aangezien de nieuwe verbinding online is (synchroon) met Schenker is, manifest bij sluiten transport uitschakelen.

Options
    • -0
    • +6
    ./Print/ZPLPrinter.cls.xml
  2. … 4 more files in changeset.
Friday 25 Oct 2024
11:09 am

Wim Vermeulen

[ICT-5051][rvNVT] FOP: DocBase Upgrade - Clean up caché-code na de go-live

- Opkuis Mock en Fake van klasse APPS.common.Document.impl.DocBase.WebServerUrlBepaler

- De klasse zelf zal ook verwijderd worden. De enige gebruiker (in method GenereerProductDetailBlad() ) is ook al aangepast

Options
    • --17
    • +0
    ./Document/impl/DocBase/WebServerUrlBepaler.cls.xml
  2. … 4 more files in changeset.
Monday 30 May 2022
4:41 pm

Tommy Hebb

Tuesday 24 May 2022
1:09 pm

Wim Vermeulen

1:09 pm

Wim Vermeulen

FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt http://sub...

FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt )
De impl in deze afgeleide klasse is wel correct.

De implementatie voor "Lengte" hieronder zou ik wel in een private method steken, analoog aan GeefFrontPlaatHoogte()
Zo blijft deze method simpel leesbaar en overzichtelijk. En dan is de scope ook duidelijk, nl. dat Kleur W7 enkel impact heeft op de kenmerk "Lengte, en niet op de "Breedte" van het Front.

10:08 am

Wim Vermeulen

copy-paste foutje : "... kan niet gemaild worden." (ook al is de conditie altijd false :-P )

copy-paste foutje :
"... kan niet gemaild worden."
(ook al is de conditie altijd false :-P )

Wednesday 20 Apr 2022
3:37 pm

Tommy Hebb

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
Tuesday 19 Apr 2022
1:37 pm

Wim Vermeulen

1:07 pm

Peter Vercoutter

Friday 15 Apr 2022
4:52 pm

Wim Vermeulen

[ICT-2743][rvPVR] PM: Maatwerk: ALU Kaderdeur detailblad PDF genereren via FopScheduler+Watchfolder
[ICT-2743][rvPVR] PM: Maatwerk: ALU Kaderdeur detailblad PDF genereren via FopScheduler+Watchfolder
Wednesday 13 Apr 2022
5:03 pm

Tommy Hebb

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Toevoeging van en uitbreiding met PDFDocumentMailer

Options
    • -0
    • +25
    ./Document/DocumentMailer.cls.xml
    • -2
    • +7
    ./Document/DocumentAPI.cls.xml
  3. … 5 more files in changeset.
Friday 18 Feb 2022
5:04 pm

Admin

[ICT-2743][rvPVR] PM: Maatwerk: ALU Kaderdeur detailblad PDF genereren via FopScheduler+Watchfolder

- WebServerUrlBepaler (voor FOP) methods toegevoegd voor NetwerkLocatieViaSharedMap()

- !!! LET OP: de shared map is alleen aanwezig op Fop !

- Mock en Fake aangepast

- Unittest uitgebreid

Options
    • -0
    • +7
    ./Document/impl/DocBase/WebServerUrlBepaler.cls.xml
  2. … 3 more files in changeset.
5:00 pm

Admin

[ICT-2743][rvPVR] PM: Maatwerk: ALU Kaderdeur detailblad PDF genereren via FopScheduler+Watchfolder

- Mock WebServerUrlBepaler implementatie up-to-date gebracht (vooraleer ik mijn aanpassingen doorvoer)

Options
    • -29
    • +5
    ./Document/impl/DocBase/WebServerUrlBepaler.cls.xml
Monday 29 Nov 2021
9:39 am

Tommy Hebb

9:10 am

Wim Vermeulen

Tuesday 23 Nov 2021
10:40 am

Tommy Hebb

[ICT-2581] [rvWV] HX: TAX: Movento-koppeling: Aanpassing ProductCreatie volgens KoppelingType:
[ICT-2581] [rvWV] HX: TAX: Movento-koppeling: Aanpassing ProductCreatie volgens KoppelingType:
Tuesday 09 Nov 2021
10:39 am

Tommy Hebb

[ICT-2581] [rvWV] HX: TAX: Movento-koppeling: Aanpassing ProductCreatie volgens KoppelingType:

- HeeftFrontHoogteVerstelling niet meer via KlantInstellingenService => mag dus weer weg => Is ondertussen volgens KoppelingType = Movento

Options
    • -7
    • +0
    ./KlantInstellingenService.cls.xml
  2. … 2 more files in changeset.
Tuesday 05 Oct 2021
12:49 pm

Tommy Hebb

12:21 pm

Tom Vermeulen

11:09 am

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
Thursday 23 Sep 2021
4:17 pm

Tommy Hebb

Zoals besproken blijkbaar ongewild in deze review terechtgekomen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Zoals besproken blijkbaar ongewild in deze review terechtgekomen

4:06 pm

Tom Vermeulen

is deze wel nodig?

is deze wel nodig?

Thursday 12 Aug 2021
4:14 pm

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- Uitbreiding Klant-enum met 3626 OlBeslag => teruggedraaid, want toch niet nodig

- Uitbreiding KlantInstellingenService met MagKlantTAORLadeKleurOudZwartBestellen => Enkel klant OlBeslag geeft true => teruggedraaid, want toch niet nodig

Options
    • -7
    • +0
    ./KlantInstellingenService.cls.xml
  2. … 5 more files in changeset.
Tuesday 10 Aug 2021
5:20 pm

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- Uitbreiding Klant-enum met 3626 OlBeslag

- Uitbreiding FreesProgramma-enum met X GeenBewerking

- Uitbreiding KlantInstellingenService met MagKlantTAORLadeKleurOudZwartBestellen => Enkel klant OlBeslag geeft true

- Uitbreiding KantenbandData met LadeKleur => in latere commit nodig voor KantenbandBepaler

Options
    • -0
    • +7
    ./KlantInstellingenService.cls.xml
  2. … 9 more files in changeset.
Tuesday 16 Mar 2021
4:47 pm

Tommy Hebb

4:46 pm

Tom Vermeulen

Thursday 11 Mar 2021
4:43 pm

Tommy Hebb

Heum... waarom? Die verandert toch nooit en hoeft tussen UTs toch ook niet eventueel te veranderen? Toch zeker niet sinds er in de constructor ervan niets meer gebeurt. Ik dacht dat een OnBeforeOne...

Heum... waarom? Die verandert toch nooit en hoeft tussen UTs toch ook niet eventueel te veranderen? Toch zeker niet sinds er in de constructor ervan niets meer gebeurt.
Ik dacht dat een OnBeforeOneTest bedoeld is voor als er een gevaar is dat hij door eerdere UTs zou kunnen bevuild worden, wat hier niet het geval is.

3:13 pm

Tommy Hebb

Bwa, is een beetje zo gegroeid door refactoren... Waarom een Bepaler new'en per UnitTest, als je hem gewoon 1 keer kunt new'en in een OnBeforeAllTests? Daarna de afweging: Waarom dan nog de call on...

Bwa, is een beetje zo gegroeid door refactoren...
Waarom een Bepaler new'en per UnitTest, als je hem gewoon 1 keer kunt new'en in een OnBeforeAllTests?
Daarna de afweging: Waarom dan nog de call onder test speciaal gaan afzonderen? En bovendien: Waarom beschikken we anders over een $$$AssertTrue en $$$AssertFalse?
Het was bovendien daardoor ook niet meer nodig om een betekenisvolle, extra lokale variabele te voorzien in de UnitTesten (VerwachtInOpstartfase), dus ook al een regel minder per UT, zonder aan leesbaarheid in te boeten (eerder het tegendeel).
Doe de nieuwe versie eens open in studio en zie eens hoe leesbaar die is
Misschien moeten we dan eerder (mettertijd, als die klassen eens onder change komen) die van LBX en TBX aanpassen naar dit model?

8:57 am

Wim Vermeulen

Wednesday 10 Mar 2021
7:51 pm

Wim Vermeulen

Ik had verwacht dat "Product" of "ProductId" reeds werd doorgeven via de LadeInfo, maar dat blijkt toch niet het geval. Als dit wel zo was, dan had het niet nodig geweest om de extra parameter (pro...

Ik had verwacht dat "Product" of "ProductId" reeds werd doorgeven via de LadeInfo, maar dat blijkt toch niet het geval.
Als dit wel zo was, dan had het niet nodig geweest om de extra parameter (productieSequentie) toe te voegen.
Na al je inspanningen, keur ik deze oplossing goed