skttl / umbraco-fulltextsearch8

Full Text indexing and searching for Umbraco 8 and Examine.
MIT License
19 stars 22 forks source link

Added logging for HttpPageRenderer #112

Closed enkelmedia closed 1 year ago

enkelmedia commented 1 year ago

Hi!

This PR referenced this issue #111, to add more logging in the HttpPageRenderer. Used LogLevel Debug and adjusted parameter naming to be PascalCase as this is used trough out the code base in other places.

There are other places (around 25) in the code base where _logger.IsEnabled() is not used. I would maybe call this a "micro-optimization" in this context as it's only executed when editors work in the backoffice. To keep the PR small and concise I decided not to touch the other places.

skttl commented 1 year ago

Thanks :)