-
-
trunk/reviewboard/rooturl.py (Diff revision 1) Probably settings.SITE_ROOT could be referenced here as well ?
Create the possibility to install reviewboard in a subdirectory of the webserver.
Review Request #235 — Created Feb. 10, 2008 and submitted
Information | |
---|---|
pouakai68 | |
Review Board SVN (deprecated) | |
trunk | |
Reviewers | |
reviewboard | |
Several requests have been seen on the forums for that review board be able to be installed on the webserver in other than the root directory. A variable 'SITE_ROOT' has been added to settings.py and where possible all absolute references to the root directory have been modified to reference this variable. Paths in the .css files have been made relative. A 'shim URLconf' (rooturl.py) has been added to absorb the subdirectory before the normal URLconfs take effect. An example apache.conf is added to the contrib directory.
Normal site walk through looking for broken links and features only - no unit testing.
PO
-
-
trunk/reviewboard/accounts/decorators.py (Diff revision 1) Should be more like: "%saccount/preferences/?%s=%s" % (settings.SITE_ROOT, ...)
-
trunk/reviewboard/accounts/views.py (Diff revision 1) just settings.SITE_ROOT. settings.SITE_ROOT should have a "/" appended in the settings file (and we should document this). That's the convention used elsewhere in Django.
-
-
trunk/reviewboard/htdocs/scripts/comments.js (Diff revision 1) Remove the / on /api This should be changed for all other uses as well. Not going to list each one individually, because that will just take too long :)
-
trunk/reviewboard/htdocs/scripts/rb/common.js (Diff revision 1) You'll need to make STAR_*_IMG not have a leading /
-
-
-
trunk/reviewboard/rooturl.py (Diff revision 1) I need to think through it some more, but I'm not convinced we need or want to use this file. Since URL entries are relative to their root, can't we just use Apache to tell Review Board that its root is /reviewboard/ ? The problem with hard-coding "reviewboard/" is that some installs might want something else ("reviews/" maybe). This should probably do r'^%s' % (settings.SITE_ROOT) This is especially true on setups where you don't have a prefix. I think right now, this will break all installs that use /
-
trunk/reviewboard/rooturl.py (Diff revision 1) Indentation issues. Line 8 should be indented 4 spaces, and 9 not at all.
-
PO
-
-
/trunk/reviewboard/context_processors.py (Diff revision 5) I have created this file in order to expose SITE_ROOT, in the templates especially accounts/login, reviews/star, & reviews/dashboard_entry which seemed unable to access settings.SITE_ROOT despite most other templates having no problems