KRACOEUS-7999: Stack trace when trying to view undisclosed Events

More instances where I converted lookups to DataObjectService, for fetching DevelopmentProposal and ProposalPerson objects.
I agree you @blackcathacker.  I don't think we should ever search for all ProposalPersons with no criteria.  This would likely bring down the app server querying and then bringing all that data into memory.  Now when I look at the code in CoiDisclosureServiceImpl, there are many problems using: findAll(AwardPerson.class), findAll(InstitutionalProposalPerson.class), findAll(ProposalPerson.class), findAll(IacucProtocolPerson.class), findAll(ProtocolPerson.class).  I wonder if perhaps we should fix this and then maybe create a jira to fix the rest of this class and look for other instances of queries without criteria.