add log4php and getTournaments::Registrations

gfrenoy / TabT-API

Programming interface for TabT, the free table tennis manager software

http://tabt.frenoy.net

GNU Affero General Public License v3.0

9 stars 5 forks source link

add log4php and getTournaments::Registrations #16

Closed kaising closed 5 years ago

kaising commented 5 years ago

Gaetan, J'ai ajouté log4php dans le projet. Cela permet d'être moins aveugle sur ce que fait le code pendant la phase de dev. J'ai aussi ajouté un 'no-cache' pour le wsdl. Cela peut éviter des surprises ...

Et j'ai bien testé la récupération des "registrations" d'un tournoi en particulier. Cela me semble bon. G

CLAassistant commented 5 years ago

Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

KAISIN Guy seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

gfrenoy commented 5 years ago

Hi Guy,

Thanks for your contribution. I will not merge this for now but feel free to re-open if you have time to implement the discussed remarks:

CLA not signed
log4net is adding a lot of dependencies that will be hard to maintain in the longer run
no-cache must be configurable in the general config file
preserve indentation in wsdl file

kaising commented 5 years ago

gaetan, je reviens de congés... pour la CLA, je l'ai bien signée... Pour log4php, je peux supprimer la lib et les refs... mais quelle solution de log adopter. le code est tres pauvre de ce côté et c'est pas une bonne idée de mon point de vue pour le no-cache, ok, faut une solution plus robuste que cette simple approche et pour le wsdl je vois demon coté... je vais tester avec vs-code. bàt, g