I'm using the following code (From https://gist.github.com/sniffdk/7600822) that spits out an UmbracoContext for a few background tasks. They are not always backgrounded, so UmbracoContext.Current is sometimes available.

public static UmbracoContext GetOrCreateContext()
{
    if (UmbracoContext.Current != null)
    {
        return UmbracoContext.Current;
    }

    var httpContext = new HttpContextWrapper(HttpContext.Current ?? new HttpContext(new SimpleWorkerRequest("temp.aspx", "", new StringWriter())));

    return UmbracoContext.EnsureContext(
        httpContext,
        ApplicationContext.Current,
        new WebSecurity(httpContext, ApplicationContext.Current),
        UmbracoConfig.For.UmbracoSettings(),
        UrlProviderResolver.Current.Providers,
        false);
}

I notice that UmbracoContext is IDisposable. My instinct is that if I have created something that is disposable then I should dispose of it, is that a correct assumption?

I'm also guessing that I shouldn't dispose of UmbracoContext.Current so I've put this in place, is it a sensible approach?

    public static void CompleteActionWithUmbracoContext(Action<UmbracoContext> action)
    {
        if (UmbracoContext.Current != null)
        {
            action(UmbracoContext.Current);
        }
        else
        {
            using (var ctx = GetOrCreateContext())
            {
                action(ctx);
            }
        }
    }