Removes unused copy/paste params (and undocumented) from the invalidate function
Fix setError to use setMessage (In order to do this I had to fractionally rework the logic in invalidate and invalidateAll when no items were selected - there should be no overall effect on this)
Use $this->app instead of the factory application in the controller
Before this patch note many functionalities of requests (such as invalidating requests and deleting items in requests) do not work. After the patch they do.
I don't think so as it's coming from the model email sending. I skipped the email stuff when testing as i was on the train :/ If it's just that mark it as successful and I'll come back to that. At a quick code review I think somethings wrong inside the mailer itself
Not sure if it is related but when I try to email a data request I get a 500 error with the following call stack