Inject the entity field manager service in DevelGenerateBase as it is used in several DevelGenerate plugins.

Move the logic that adds comments and statistics from the devel_generate.module's 'hook_entity_insert' implementation to ContentDevelGenerate as it was used for node generation.

Change approach to not override constructors in devel_generate plugins.

use dependency injection wherever possible, replace non-extent `\Drupal\Core\Datetime\TimeInterface` with `\Drupal\Component\Datetime\TimeInterface`, make type hints for variables more specific (e.g. `UserStorageInterface` instead of `EntityStorageInterface`), move optional arguments to the end, add some missing types, restore the `DrupalPractice.Objects.GlobalClass.GlobalClass` sniff as it now passes.

Issue #2235767 by chris_hall_hu_cheng, wiifm, luisnicg, juampy, Nick_vh: Default Cache Bin changed to cache.discovery.