I'm a bit confused by this and have a couple of approaches to correcting this in my head.  Tell me if either one are remotely close to what you think would be ideal:

**Idea 1:** Instead of ```@art_files = @job.art_files.separations```, there would be a method on the Jobs model like:

```
def separations
    art_files.where(upload_type: 'separations')
end
```

And then I'd call ```@art_files = @job.separations```, and similarly for other types.

**Idea 2:** Move the entire load_art_files method to the job model and have it return a list of art_files of the appropriate type.  So, the method on the Job model would look something like:

```
def get_art_files(upload_type)
    art_files.where(upload_type: upload_type)
end
```

**Idea 3:** Move load_art_files method to job model like I mention above, but leave the separations/final_files/source_files filter methods/scopes on the ArtFile model.

So, the Job model method would look something like this:

```
def get_art_files(upload_type)
    if upload_type == 'separations' then
        art_files.separations
    ...
    end
end
```

And the controller would look like ```@art_files = @job.get_art_files('separations')```

Thoughts?