fixing #8491 - removing puppet modules list from content view version api

This takes the CVV index api from ~5 minutes down to 17 seconds
for 47 content view versions, 10 having 100 puppet modules, the rest having
5
It does introduce a minor change, but the performance at this level just is not acceptable.  Changes that remove an entire api or remove an input parameter are easier to handle (as you can deprecate them and later remove).  However removing a part of return data is much more difficult to do.  The UI and Hammer all use the correct method for fetch puppet modules (the puppet modules controller), so i think its okay to introduce this change with a release note.