Replaced deprecated functions

+++ b/src/Controller/GISController.php
@@ -133,8 +133,8 @@ class GISController extends ControllerBase {
+          \Drupal::database()->update('google_image_sitemap')

Can you use the Inject DB object for any Database related stuff in this class.

Will work on this.

I'll review this!

Steps performed:
(1) Installed module
(2) Reproduced the issue.
(3) Applied patch.
(4) Code review on changes.
(5) Tested again with patch, issue resolved.
The commit removed all deprecated code in this module, and the module still working well as far as I can test it.

Hi,

Please see if you can improve the patch.

1. +++ b/src/Controller/GISController.php
   @@ -133,8 +133,8 @@ class GISController extends ControllerBase {
   +          \Drupal::database()->update('google_image_sitemap')
   

   Can you please try to use Dependency Injection for DB.

2. +++ b/src/Form/GoogleImageSitemapCreateForm.php
   @@ -70,11 +70,11 @@ class GoogleImageSitemapCreateForm extends FormBase {
   +      $sitemap['created'] = \Drupal::time()->getRequestTime();
   

   Same here.

3. +++ b/src/Form/GoogleImageSitemapCreateForm.php
   @@ -70,11 +70,11 @@ class GoogleImageSitemapCreateForm extends FormBase {
   +    $sitemap['last_updated'] = \Drupal::time()->getRequestTime();
   

   Same here.

1.) This is already fixed in the latest version:

+++ b/src/Controller/GISController.php
@@ -133,8 +133,8 @@ class GISController extends ControllerBase {
+          \Drupal::database()->update('google_image_sitemap')

2.) Here is the patch for the dependency injection fix for points 2&3

I'll review this!

I've just added the patch in MR, by the way the patch added dependency injection correctly. How was said in #11 This is already fixed in the latest version:

+++ b/src/Controller/GISController.php
@@ -133,8 +133,8 @@ class GISController extends ControllerBase {
+          \Drupal::database()->update('google_image_sitemap')

Moving it to RTBC!

Thanks for the PR, it has been merged.