wadqc / WAD_Interface

NVKF WAD webinterface
3 stars 1 forks source link

Admin functionaliteit logfiles #41

Closed dickrsch closed 9 years ago

dickrsch commented 10 years ago

Feature request:

Ik zou graag een knopje zien om vanuit de admin sectie van de wad de verschillende logfiles te kunnen openen. Ik zit nu vaak (tijdens testen / debuggen) heen en weer te gaan als er iets mis is.

Die logfiles staan op een platform afhankelijke locatie dus dat zal in de config moeten komen als variabel pad.

tcdewit commented 9 years ago

Ik zat zelf te denken aan een proces-specifieke logfile in de module-output folder. Met een paar eenvoudige aanpassingen aan de processor is me dit al gelukt. Probleem is echter om deze logfiles te kunnen tonen vanuit de interface (bijv in de processor status tabel een link maken van de status indien gelijk aan "error"); die weet namelijk niet waar de XML folder zich bevindt. Mogelijke oplossingen:

Voorkeur iemand?

wadqc commented 9 years ago

Optie 1 (entry in database als object met link naar logfile) heeft mijn voorkeur. Qua implementatie het meest transparant lijkt mij.

tcdewit commented 9 years ago

Geïmplementeerd in commit https://github.com/wadqc/WAD_Services/commit/ab2f4b15b4124c02d7ed5f92bdff7ccec8cefc81 (processor) en commit 4d4dd7b3f5d03cdda12589abd3d769d7bfd486b2 (interface). @dickrsch, @wadqc: graag even testen en als ie naar behoren werkt, mergen in de master branch.

tcdewit commented 9 years ago

De resultaat object-thumbnails bleken ineens niet meer te werken bij Dennis (vanwege andere apache/site configuratie). Gefixt in commit 51b255c3357f3a5f5a58c368a6e7eb7f37f37539 (Tim branch).

tcdewit commented 9 years ago

Momenteel wordt de logfile altijd opgeslagen en ook altijd in de DB ge-insert, ongeacht status. Alleen bij status "Error" zie je een clickable linkje. Is dit gewenst gedrag? Hoor 't graag als dit anders moet. Verder is er ook een mogelijkheid om de DB insert alleen te doen bij status "Error".

dickrsch commented 9 years ago

In de Tim branch getest en de beschreven functionaliteit werkt. Voorstel om alleen bij status error een clickable linkje te hebben is prima, ik heb geen idee/voorkeur over de DB insert. Kan verder wat mij betreft de main branch in.