Only display dblog message if module is enabled

+ if (\Drupal::moduleHandler()->moduleExists('block')) {

I think this should be checking for the dblog module instead of block? :)

Other than that it looks good to me.

+++ b/src/Form/SchedulerCronForm.php
@@ -120,11 +120,11 @@ class SchedulerCronForm extends ConfigFormBase {
-    try {
+    if (\Drupal::moduleHandler()->moduleExists('dblog')) {
       $url = Url::fromRoute('dblog.overview')->toString();

Should be injected

Nope! I just reviewed your patch at #7.

I didn't provide the patch as it was assigned to you. Now assigning myself :)

Something like this @joshi.rohit100?

Sorry to jump in there but I've never used DI in Drupal 8 so I thought it would be a good opportunity to try :)

1. +++ b/src/Form/SchedulerCronForm.php
   @@ -19,6 +20,30 @@ use Symfony\Component\Routing\Exception\RouteNotFoundException;
   +  /**
   +   * Creates a SchedulerCronForm.
   +   *
   

   should be "Creates a SchedulerCronForm instance."

2. +++ b/src/Form/SchedulerCronForm.php
   @@ -19,6 +20,30 @@ use Symfony\Component\Routing\Exception\RouteNotFoundException;
   +   * @var \Drupal\Core\Extension\ModuleHandlerInterface
   +   *   The module handler service.
   +   */
   

   missing parameter name

Missing the use statement.

Also it would be nice if we just change module_handler to moduleHandler (only property not in constructor) as core uses this name.

Patch is there, so unassigning!

Thanks for the feedback @joshi.rohit100.

This is much better, nice improvement! Thanks a lot guys!

I removed an unused use statement and fixed some minor coding standards before committing.

Thanks, good work.

As this issue was originally raised at 7.x I think it will be worth making the fix there too. Seems like the original patch at #1 is correct? Are you ok if I commit it?

Yes, absolutely! I didn't intend to bypass this for D7. Patch looks good!

Done. Thanks marcelovani for the original 7.x patch

Ignore the failed patch above. It was queued but not run until the 8.x committed codebase passed all tests. That happened with my commit a few minutes ago, and then the untested patches have suddenly come to life and been run. This issue is already fixed.