It's not always convenient to pull up the conf files in use by a particular index (or to verify they are what you think).

We should add a listing page to the reports section to display the conf file XML.

CommentFileSizeAuthor
#8 704190-8.patch3.88 KBjpmckinney
#6 704190.patch2.97 KBrobertdouglass
#3 704190-conf-file-display-3.patch3.52 KBpwolanin
#2 704190-conf-file-display-2.patch3.49 KBpwolanin
#1 704190-conf-file-display-1.patch3.43 KBpwolanin

Comments

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new 704190-conf-file-display-1.patch3.43 KB

proof of concept

pwolanin’s picture

pwolanin commented
StatusFileSize
new 704190-conf-file-display-2.patch3.49 KB

slightly improved patch.

Geting ugly output for file=mapping-ISOLatin1Accent.txt - some kind of character encoding issue that I can't fix even by starting java with -Dfile.encoding=UTF8, though that partially fixes it.

pwolanin’s picture

pwolanin commented
StatusFileSize
new 704190-conf-file-display-3.patch3.52 KB

sort the list too.

pwolanin’s picture

pwolanin commented
Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Needs review » Patch (to be ported)

I think this is good enough to let people try it out. Committed to 6.x-1.x

patch doesn't apply, so needs to be ported to 6.x-2.x and HEAD.

robertdouglass’s picture

robertdouglass commented
StatusFileSize
new 704190.patch2.97 KB

#704190 by pwolanin: Added Add page callbacks to display conf files in the index in reports.
committed to 6.2.

pwolanin’s picture

pwolanin commented
Version: 6.x-2.x-dev » 7.x-1.x-dev
jpmckinney’s picture

jpmckinney commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new 704190-8.patch3.88 KB

Just had to change the theme() call.

jpmckinney’s picture

jpmckinney commented
Status: Needs review » Fixed

Fixed.

Status: Fixed » Closed (fixed)

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