search

sergant210 / controlErrorLog

Adds the ability to control the MODX Revo error log and view it in a popup window.
MIT License
13 stars 5 forks source link

Улучшение призводительности #12

Closed kudashevs closed 7 years ago

kudashevs commented 7 years ago

Обратил внимание, что данный плагин влияет на время отработки страниц вне админки. Посмотрел код и понял, что он в любом случае проверяет наличие лог файла, вне зависимости от того, будем пользоваться функцией оповещения об изменении лог файла или нет. Предлагаю немного измененную логику, в которой проверка наличия лог файла осуществляется после проверки настроек почты. Если мне не нужно пользоваться функцией отправки почты, то я не буду заполнять email адрес в настройках. С такой логикой время отработки сократилось почти до нуля :)

sergant210 commented 7 years ago

Логично. Только проверять нужно настройку control_frontend, которая как раз и отвечает за этот функционал.

kudashevs commented 7 years ago

Мне кажется, что логичнее проверять заполненность почты, так как по умолчанию настройка control_frontend установлена в true. При этом почта по умолчанию не заполнена, и пока она не будет заполнена, данный функционал не заработает. Но Вам, как разработчику виднее как лучше.

Мое дело только предложить :)