ActionManager context variable is never used.

[julianstirling]

The code currently has two action manager dependencies:

1. ActionManagerDep - a standard dependency that returns the Action manger
2. ActionManagerContextDep - A dependency that calls the above dependency to set the action manger to a context variable. Allowing ActionManagerContext.get()

The code always ActionManagerContextDep, however it also always uses the the action manager directly returned by it rather than using ActionManagerContext.get().

I don't really see the advantage of the context var here. I feel the code is more explicit and testable without a context variable, as ActionManagerContext.get() is never used it should be simple to remove ActionManagerContextDep and change all call to it to ActionManagerDep?

[rwb27]

Yes, this is vestigial code: it used to be the case that Blob objects would need to use it to generate their URL. This is no longer the case, so we should delete it. Thanks :)

[rwb27]

We will need a tidy-up of dependency code once we've released the migration release that includes both dependencies and the new-style alternatives. I suggest we remove this then.