Feature #1458

Several listing enhancements

Added by hr about 7 years ago. Updated over 5 years ago.

Status:FixedStart date:
Priority:NormalDue date:
Assignee:jan% Done:

100%

Category:mod_dirlisting
Target version:1.4.23
Missing in 1.5.x:

Description

The start point for this patch is the will to get an official ubuntu repository (fr.archive.ubuntu.com) from apache to lighty. For this, we have to keep some of the mechanisms ubuntu gives us and transfer them from apache to lighty. I tried to implement some of the mod_autoindex things in lighty mod_dirlisting, keeping in mind that it should stay light.

The patch attached basically changes the following things :
  • add dir-listing.encode-header and dir-listing.encode-readme, these options are booleans. It gives the ability to provide a valid html file as HEADER.txt and README.txt, these files are just printed before and after the actual listing. A further enhancement would be to choose the name of those files.
  • add dir-listing.auto-layout, this option is a boolean. When disabled, it will remove the html part lighty was automatically outputing as the start of the page and the end (from <DOCTYPE to <body> and </body></html>). This lets the possibility to take care of those parts in the header and readme files.
  • add dir-listing.pre-list, this option is a boolean. When enable, it will replace the <div></div> around the listing by < pre > < /pre > keeping the same css class.

mod_dirlisting_layout_patch-0_1.diff Magnifier (9.16 KB) hr, 2007-11-15 17:10


Related issues

Related to Feature #1605: patch for dirlisting.extra-head confvar Wontfix
Related to Feature #2063: dirlisting: allow HTML documents for HEADER/README New 2009-09-06

Associated revisions

Revision 2478
Added by stbuehler over 5 years ago

Add some dirlisting enhancements (fixes #1458)

History

#1 Updated by stbuehler about 6 years ago

  • Target version changed from 1.4.20 to 1.4.21

#2 Updated by icy almost 6 years ago

  • Target version changed from 1.4.21 to 1.4.22
  • Patch available set to Yes

#3 Updated by stbuehler over 5 years ago

  • Target version changed from 1.4.22 to 1.4.23

#4 Updated by stbuehler over 5 years ago

I don't see why we need pre instead of div; and we should drop the footer if auto-layout is disabled.

#5 Updated by stbuehler over 5 years ago

  • Status changed from New to Fixed
  • % Done changed from 0 to 100

Applied in changeset r2478.

Also available in: Atom