Add entity type to FieldException messages thrown in SqlContentEntityStorageSchema

Been a few weeks. Lets go with optionB!

Changed #5 according to Option B

This seems better to me.

+++ b/core/lib/Drupal/Core/Entity/Sql/SqlContentEntityStorageSchema.php
@@ -2063,7 +2063,7 @@ protected function getSharedTableFieldSchema(FieldStorageDefinitionInterface $st
-      throw new FieldException("Illegal field column names on {$storage_definition->getName()}");
+      throw new FieldException("Illegal field column names on {$storage_definition->getName()} in {$storage_definition->getTargetEntityTypeId()} entity");

@@ -2330,7 +2330,7 @@ protected function getDedicatedTableSchema(FieldStorageDefinitionInterface $stor
-      throw new FieldException("Illegal field column names on {$storage_definition->getName()}");
+      throw new FieldException("Illegal field column names on '{$storage_definition->getName()}' on entity type '{$storage_definition->getTargetEntityTypeId()}'");

I think we can improve this one further too, by letting the user know which column names are illegal.

Also, it would be worth making them consistent whilst we're here (one has single quotes around the field name and entity type, the other does not).

Thanks folks

Addressed #16 Feedback.

Also, it would be worth making them consistent whilst we're here (one has single quotes around the field name and entity type, the other does not).

Not changing status #16 remaining part.