nah, not necessary, although I could see seeming more concise:

```
    def valid_substitutions?(contnent_url, substitutions)
      real_path = gsub_vars(contnent_url, substitutions)
      if is_substituable(real_path)
        return false
      else
          if valid_path?(File.join(real_path, "repodata")) || valid_path?(File.join(real_path, "PULP_MANIFEST"))
            return true
          else
           @resource.log :error, "Unable to locate repodata files for #{content_url}"
            return false
          end
      end
    end

def valid_path?(path)
     !! @resource.get(path)
 rescue Errors::NotFound => e
     false
end
```

But regardless, All looks good to me.