Merge pull request #241 from audionerd/master

Clarify the documentation of a page's "eponymous" directory
This commit is contained in:
Thomas Reynolds 2012-01-16 18:08:50 -08:00
commit 5a5ebdc37f

View file

@ -159,19 +159,21 @@ module Middleman::Sitemap
end end
end end
# Whether this page is a directory index # Whether this page either a directory index, or has the same name as an existing directory in the source
# @return [Boolean] # @return [Boolean]
def directory_index? def directory_index?
path.include?(app.index_file) || path =~ /\/$/ || eponymous_directory? path.include?(app.index_file) || path =~ /\/$/ || eponymous_directory?
end end
# Whether this page is a eponymous_directory # Whether the page has the same name as a directory in the source
# (e.g., if the page is named 'gallery.html' and a path exists named 'gallery/', this would return true)
# @return [Boolean] # @return [Boolean]
def eponymous_directory? def eponymous_directory?
!!Dir.exists?(File.join(app.source_dir, eponymous_directory_path)) !!Dir.exists?(File.join(app.source_dir, eponymous_directory_path))
end end
# The path for the eponymous_directory of this page # The path for this page if it were a directory, and not a file
# (e.g., for 'gallery.html' this would return 'gallery/')
# @return [String] # @return [String]
def eponymous_directory_path def eponymous_directory_path
path.sub('.html', '/').sub(/\/$/, "") + "/" path.sub('.html', '/').sub(/\/$/, "") + "/"