Js dsc 242

Of course. I did see it was just a reformat of the existing code. I was really just imagining what it must have felt like to have to try and construct the statement in the first place :-)

From: Jonathan Shaw <notifications@github.com<mailto:notifications@github.com>>
Reply-To: nla/banjo <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Thursday, 13 November 2014 2:50 PM
To: nla/banjo <banjo@noreply.github.com<mailto:banjo@noreply.github.com>>
Cc: Shaun Coen <scoen@nla.gov.au<mailto:scoen@nla.gov.au>>
Subject: Re: [banjo] Js dsc 242 (#826)


In src/banjo/controllers/JobsController.java:

>
> -        if (jobFile == null || jobFile.getJob().getId() != id) {
> -            throw new RecordNotFoundException();
> -        }
> -
> -        File thumb = new File(jobFile.getPath().toFile().getParentFile()
> -                .toPath().resolve("thumb-" + jobFile.getId() + "-" + width + ".jpg")
> -                .toString());
> +            File thumb = new File(jobFile.getPath().toFile().getParentFile().toPath()
> +                    .resolve("thumb-" + jobFile.getId() + "-" + width + ".jpg").toString());


Possibly. I just reformatted what was there. and its only the internally cached thumbnails there, so not tooo horrible.

—
Reply to this email directly or view it on GitHub<https://github.com/nla/banjo/pull/826/files#r20271242>.