uses of drupal_set_message()

@berdir as phpstan also reports this issue
As per #2 do we need to remove this form in the next release?

There is no need for DI here, it just adds more phpstan messages at the moment. This is an entity form that uses the messenger trait, you can use messenger(), especially on dead code that nobody should be using anymore.

Yeah it's a dead code ~~ so don't know whether this needs to be merged or whole form gets removed!!

You must use the messenger() method, not the property directly.

It is deprecated and scheduled for removal, but that will require a new major version, which will likely not happen soon if ever.

Used \Drupal::messenger(); moving into review

No, shouldn't be \Drupal::messenger(), should be $this->messenger(). Read what @berdir wrote in #11.

Ahh I got bit confused with messenger, Thanks @TR