There should be an option to specify a custom filename for the KML output. I've attached a patch that just does that and fixes some whitespace issues and removes a few unused variables.

CommentFileSizeAuthor
#6 928758_6.patch729 bytesWill White
#5 custom_feed_name.patch2.03 KBAnonymous (not verified)
#3 custom_kml_feed_name.patch795 bytesAnonymous (not verified)
#1 928758_2.patch2.23 KBWill White
add_custom_filenames.patch5.7 KBAnonymous (not verified)

Comments

Will White’s picture

Will White commented
StatusFileSize
new 928758_2.patch2.23 KB

Updated patch to remove unrelated whitespace adjustments and fix KML output.

tmcw’s picture

tmcw commented
Status: Needs review » Fixed

Great, committed: http://drupal.org/cvs?commit=430916

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Needs review
StatusFileSize
new custom_kml_feed_name.patch795 bytes

The above patch doesn't change the KML feed name if a custom filename is set. I've attached a patch that checks to see if there is a custom name set before writing the KML name.

tmcw’s picture

tmcw commented

Hmm, should that always be the case? Views titles can have spaces, but in some cases filenames shouldn't.

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new custom_feed_name.patch2.03 KB

Good point. I think it makes sense then to have a separate field in the style options for the feed name that way you can set a custom filename and feed name. I've attached a patch that does just that.

Will White’s picture

Will White commented
StatusFileSize
new 928758_6.patch729 bytes

I think just using the view title (instead of its name or a new custom field) would make the most sense.

Will White’s picture

Will White commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.