Filecolumn: url_for_file_column problem


#1

Calling url_for_file_colum results in a TypeError: can’t convert nil to
string.

The error is in line 148 of filecolumn.rb:
def relative_path(subdir=nil)
File.join(relative_path_prefix,subdir, @filename)
end

If I substitute ‘’ for subdir in the File.join command, then it seems
to work fine. Since this is a plugin, I’m wondering why I need to make
this modification. Is there something wrong with my install or with the
plugin?


#2

Sebastian,
No problem, thanks for the clarification. I’m using Fedora 4.
-W

Sebastian K. removed_email_address@domain.invalid wrote: Hi Wolfgang,

On 12/4/05, Wolfgang B. wrote:

modification. Is there something wrong with my install or with the plugin?
well, even a plugin can have bugs… Even more so if I’m the author :slight_smile:

Thanks for the bug report. I’m wondering which OS you are on as
File.join works on my system even if one of the arguments is nil.
Perhaps other platforms behave in a different way here? Having said
this, I just committed a work-around to the trunk that should fix this
problem.

BTW, it seems you are using the latest version from the trunk. While I
try to keep this as bug-free as possible it is not as stable as an
official releases. You can find them in the repository’s “tags/”
directory.

Sebastian


#3

Hi Wolfgang,

On 12/4/05, Wolfgang B. removed_email_address@domain.invalid wrote:

modification. Is there something wrong with my install or with the plugin?
well, even a plugin can have bugs… Even more so if I’m the author :slight_smile:

Thanks for the bug report. I’m wondering which OS you are on as
File.join works on my system even if one of the arguments is nil.
Perhaps other platforms behave in a different way here? Having said
this, I just committed a work-around to the trunk that should fix this
problem.

BTW, it seems you are using the latest version from the trunk. While I
try to keep this as bug-free as possible it is not as stable as an
official releases. You can find them in the repository’s “tags/”
directory.

Sebastian