Multiple Joi.object() with className break docs about hapi-swagger

[glennjones]

I am not going to be able to look at this for next three weeks as just about to leave for hols. If you want track down the issue and create a PR great, otherwise I pick it up on my return.

[grabbou]

Also - when multiple routes are defined like so:

• events/{id} GET
• events/{id} POST
• events/{id} DELETE
  And no className is set and the automatic one is generated (eventsid_response) and if responses are different (for example DELETE response returns affected rows), they are mixed up and all of them are the same as the first one ever defined (in this case - GET response schema).

Quick fix can be done by modifying this line:

var responseProperty = internals.validatorToProperty(responseClassName || op.nickname + '_response',

to

var responseProperty = internals.validatorToProperty(responseClassName || op.nickname + '_' + route.method + '_response',

I guess it is because Swagger returns all of models under "models" property, so if the name is the same, they overwrite themselves.

So - the results are:
eventsid_GET_response
eventsid_POST_response
eventsid_DELETE_response

Also I think that "name generation" should be done inside __getClassName, instead of doing || conditional in method args.

Can do PR for this and above as well if you want.

[glennjones]

Hi Mike, sorry taken I while to get back to you

I believe I have fixed the two use case issues you where having. Can you try out v0.6.3 and make sure it works the way you expected.