Activity CacheAdminACommon_trunk2010/vhLib/WebServiceLogger/impl

10:39 am

Jelle Baele

More
Re: CR-2258
replied to Wim Vermeulen

Ok top, bedankt!!

4:06 pm

inderdaad. T eerste deel was het belangrijkste om te begrijpen : code afzonderen (in dit geval via composition) om enerzijds de basisklasse WS.WebService niet telkens te belasten, en anderzijds om ...

inderdaad. T eerste deel was het belangrijkste om te begrijpen : code afzonderen (in dit geval via composition) om enerzijds de basisklasse WS.WebService niet telkens te belasten, en anderzijds om de complexere logica om te loggen los te trekken van de klasse zelf.

Dat je de rest niet meteen kan volgen, is geen probleem hoor. Is te begrijpen, want is idd complexere logica.
Om je toch een beetje wegwijs te maken:
de complexiteit was/is : hoe geraak ik aan de opgeroepen WebMethod naam? Want deze willen we loggen. Er zijn verschillende gebruikers die een WebService van Caché oproepen : SoapUI, Vhisie4, AX, Fop DocBase, een andere Caché-server, ...
Onlangs is er een nieuwe gebruiker om aan dit lijstje toe te voegen : de eCon-configurator. Deze gebruikt (nog) een andere manier om WS request naar Caché te sturen. Deze request heb ik geanalyseerd, en op basis daarvan, vond ik in de structuur dat de webmethod vervat zat in de property MsgClass. En in de meeste gevallen lijkt dit zelfs de beste manier om de juiste webMethod naam te bepalen.
Tot zover de uitleg voor vandaag

3:45 pm

Wim Vermeulen

More
Re: CR-2258
replied to Jelle Baele

Ja, oude gewoonte om $G() te schrijven. Kzal het aanpassen. Reden : in dit geval hou ik rekening met andere scenario's waarbij de nieuwe parameter nog niet wordt doorgegeven. Begrijp dus als : om ...

Ja, oude gewoonte om $G() te schrijven. Kzal het aanpassen.

Reden :
in dit geval hou ik rekening met andere scenario's waarbij de nieuwe parameter nog niet wordt doorgegeven. Begrijp dus als : om backward-compatible te
zijn. Anders zou het hier een <UNDEFINED> crash geven.

N.B. het is een public method, dus kan van eender welke plaats opgeroepen worden. Ik hou er dus rekening mee dat ik 1 of meerdere gebruikers kan gemist hebben.

12:37 pm

Jelle Baele

More
CR-2258
finished reviewing

12:37 pm

Jelle Baele

More
Re: CR-2258
commented on review

Volgens mij is hier alles juist voor zover ik mee ben. Ik snap wel het principe van het afzonderen naar de WebServiceHelper door het object via $this mee te geven zodat er minder grote compileerti...

Volgens mij is hier alles juist voor zover ik mee ben.

Ik snap wel het principe van het afzonderen naar de WebServiceHelper door het object via $this mee te geven zodat er minder grote compileertijden zullen zijn bij het aanpassen van iets in te toekomst. Over de algemene werking zelf snap ik wel niet perfect wat er juist allemaal gebeurt (bv. waar de 'MsgClass' van de webservice zelf voor staat (Die wordt opgeroepen in 'vhLib.WebServiceLogger.Logger^GeefWebMethodFromService'))

12:22 pm

Jelle Baele

More
Re: CR-2258
commented on review

Hier hetzelfde, $G -> $Get?

12:21 pm

Jelle Baele

More
Re: CR-2258
commented on review

Moet volgens de coding conventions hier niet '$Get' geschreven worden in plaat van $G? Snap ook niet zo goed waarom hier een $Get gebruikt wordt?

Moet volgens de coding conventions hier niet '$Get' geschreven worden in plaat van $G?

Snap ook niet zo goed waarom hier een $Get gebruikt wordt?

11:58 am

Wim Vermeulen

More
CR-2258
started review

[ICT-1756] WebServiceLogger geeft "Unknown webmethod"

11:50 am

Wim Vermeulen

More
3750
changed 5 files
root: CR-2258

[ICT-1756] WebServiceLogger geeft "Unknown webmethod"

- Webservice-object doorgeven aan LogStream method

- nieuwe method GeefWebMethodFromService() met fallback naar de oude methode (zou normaal gezien niet meer nodig zijn, maar ja, ge weet maar nooit hé)

- impl.VhLibLogger en BestandLogger aangepast, roepen nieuwe methode op.

Aparte commit (later) : klasse vhLib.WebService , want compilatie van deze klasse zal alle andere WS-klassen mee compileren.