Fix Drupal coding standards

Working on it.

One issue left to fix, the LICENSE is needed for GitHub:

FILE: /root/repos/pareviewsh/pareview_temp/src/GuardianManager.php
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 12 WARNINGS AFFECTING 12 LINES
--------------------------------------------------------------------------
24 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
47 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
50 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
73 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
76 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
83 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
90 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
92 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
115 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
145 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
176 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
177 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
--------------------------------------------------------------------------

Looks really good!

Some small things:
* drupal_set_message is deprecated with D8.5
* the user storage is only needed within the Manager, replacing entityTypeManager with userStorage make the code cleaner
* r129 $site = $this->configFactory->getEditable('system.site')->get('name'); isn't an editable variable, $this->configFactory->get('system.site')->... is more than enough

Thanks for the new patch :)