Right, that's if you use giraffesoft-is_taggable. As far as I can tell is_taggable doesn't work with github as the source either, right?

Does config.gem 'is_taggable' actually work? Doesn't look like there's any such gem registered on rubyforge.

Damn! Sorry, my epic fail was what resulted in the lack of this method :-/

Yeah. The reason I think it probably makes sense that they always escape it is just because at the end of the day, it’s designed for websites, and it’s usually a good idea to either escape or modify URIs like that.

Ah, fair point. Silly me. What is really needed is a CGI::escape on the correct part of the URI (i.e., the filename part). Unfortunately, that’s probably not possible. Unless you could do something like (pseudocode):

def escaped_filename
  file_path.gsub filename, CGI::escape(filename)
end

Or some such. Still, that would double-escape attachment_fu’s output, so it would have to be placed in the plugin-specific modules anyway. Might be worth filing a ticket with Paperclip, don’t know if getting the file path unescaped is necessarily a good idea.

You could also use CGI::escape (https://apidock.com/ruby/CGI/escape/class) or URI::escape (https://apidock.com/ruby/URI/Escape/escape) to be more thorough here.

Assuming this isn’t personal preference, you could use /^https?: (instead of {0,1}).

Hahaha. Awesome.

Bah. Damn. That is complicated.

Hm.. So is there a way to reconstruct the correct filename for S3 without using public_filename? That would let us successfully upload to Scribd.

Is this really a good idea? If we upload after save, (a) the validations are guaranteed not to work (the file hasn’t actually been uploaded before validating), and (b) we may end up with a record that has been created but never got scribd, which could be quite dangerous depending on how important it is that the scribd phase works.

I think it was originally after_save, but I ultimately decided to make it before_validation so that the validations would be effective and there would be a guarantee that the file was in scribd if the save succeeded.

Wtf indeed. I must have forgotten to deal with the trailing options hash when I made the merges… Looks like alt_text_if_paperclip should be alt_text_or_options and a new parameter options_if_paperclip should be added. Then a little more logic in the disambiguation conditional and we should actually have an options collection again…