> Could this just return and omit the need for the success variable?

Yes, that's a good idea. Reading it, I'm not sure why the return false is there, since angular's forEach doesn't work the same way as underscore's find