added check for type field on attribute

[JoshPJackson]

Added error exception giving a bit more detail on which field is missing the type declaration. The current error message detailed here (#57 (comment)) makes is very hard to debug

[jarrodparkes]

related to #61

[jarrodparkes]

@JoshPJackson can you remember when you were getting errors that caused you to want to make this change? I'm about to make a similar change in #59 but instead of throwing an error, I was going to safely break over objects that don't have a type defined since we just want those key/properties to map to their given schema (leave it unchanged)

[jarrodparkes]

See here:

spectator/src/Validation/AbstractValidator.php

Lines 69 to 72 in c65ab51

	// Before we check recursive cases, make sure this object defines a "type".
	if (! isset($attributes->type)) {
	break;
	}

[JoshPJackson]

@jarrodparkes sorry for the late reply. I can't remember exactly, I just remember it took me a long time to work out where the error was coming from. It think it is worth adding some form of message or something so the user knows that is has been skipped

[jarrodparkes]

@JoshPJackson this should be closed by #78