Opened 15 years ago
Closed 14 years ago
#8453 closed enhancement (wontfix)
Hide the Wiki Mechanics from people not logged in
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | lowest | Milestone: | |
Component: | wiki system | Version: | 0.12dev |
Severity: | minor | Keywords: | wiki trac.ini |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
To make a trac site look more like a normal website, and not like a wiki, some hiding has to be done. Probably useless for normal project pages, but I'm hosting a personal site on the same server I host my projects, and didn't want to waste resources with a big other web framework.
It's basically three added trac.ini options:
[ctxtnav] hide_ctxtnav_from_anonymous (does exactly what it says)
[wiki] uri_prefix (tie wiki requestHandler and link generator to another prefix than /wiki) wiki_start (allows the default wiki page to have another name)
Attachments (1)
Change History (6)
by , 15 years ago
Attachment: | hide_wiki.diff added |
---|
follow-up: 2 comment:1 by , 15 years ago
Milestone: | 0.12 |
---|
comment:2 by , 15 years ago
Figured it wouldn't get adopted. It's the sensible thing not to include it. Just wanted to put it out there though, in case someone wants that.
And you are right, the hiding of the ctxtmenu could have been easily done in a Theme. And the WikiStart could have been hidden then pretty easily as well. But tinkering with the wiki base path is impossible to do via add-on, since they were just not meant to be changed in any way.
comment:3 by , 15 years ago
Milestone: | → 2.0 |
---|---|
Severity: | normal → minor |
The configuration options should go to the WikiSystem
component, not to the Environment
directly.
I'm not sure the changes in trac/web/api.py
, trac/web/main.py
and trac/web/href.py
are needed. I'd rather not modify the /wiki
links to something else, rather prevent such links to be created by using the appropriate prefix whenever needed.
The test in trac/templates/theme.html
should be reversed (it's cheaper to test authname != anonymous
first) - but as the comments above suggest, it's perhaps better to leave that part out.
comment:5 by , 14 years ago
Milestone: | triaging |
---|---|
Resolution: | → wontfix |
Status: | new → closed |
implementation diff