Refactor to get record loader via class method.

[samuli]

This makes it easier to provide a customized loader by overriding getRecordLoader.

[demiankatz]

I have no objection to this change since it improves readability a bit (at least until we eventually refactor the controllers as discussed in VUFIND-966).

However, I'd suggest that if you want to override the record loader, it would probably be safer to override the service in your module configuration than to override the getRecordLoader method.

[samuli]

However, I'd suggest that if you want to override the record loader, it would probably be safer to override the service in your module configuration than to override the getRecordLoader method.

Yes, I forgot to mention that we need to init our customized loader (after it has been constructed) differently depending on the controller.

[demiankatz]

Makes sense!