Improved check on using, format, indexed and multiple

[phorward]

parameters in accordance to recordBones

[tsteinruecken]

What's the actual improvement here? Why should the parameter using have a default value that can never be used (None is not valid)? Would rather recommend removing the default value for format also (though this parameter is prefilled in other bones where it works correctly).

[phorward]

What's the actual improvement here? Why should the parameter using have a default value that can never be used (None is not valid)?

The reason is only for better usability:

On the wrong bone definition

test = recordBone(descr="test")

the exception message

TypeError: __init__() takes at least 2 arguments (1 given)

is less informative then

NotImplementedError: A recordBone must have 'using' and 'format' parameter, 'multiple=True' and 'indexed=False' is mandatory

as it is the case with this pull request.

[tsteinruecken]

That's the correct behavior. There's a mandatory parameter that you didn't specify. I'd rather recommend removing the default value for format also. (I thought it was a parameter already specified in baseBone, but that's not the case, it is also unique to relational/recordBones)