Opened 18 years ago

Closed 18 years ago

#4290 closed (fixed)

misleading comment about trailing slash on MEDIA_URL in settings.py

Reported by: mw@… Owned by: Adrian Holovaty
Component: Core (Management commands) Version: dev
Severity: Keywords: MEDIA_URL django-admin
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Currently it says:

# URL that handles the media served from MEDIA_ROOT.
# Example: "http://media.lawrence.com"

Which is an error, since the MEDIA_URL has to end with a "/" as far as I know.
So I add this little patch that corrects this comment.

It's no code change, but a real pitfall for new users coming to django.

Attachments (1)

conf_project_template_settings.py.diff​ (518 bytes ) - added by mw@… 18 years ago.
patch for the documentation-comment

Download all attachments as: .zip

Change History (3)

by mw@…, 18 years ago

Attachment: conf_project_template_settings.py.diff​ added

patch for the documentation-comment

comment:1 by Simon G. <dev@…>, 18 years ago

Summary: misleading comment in conf/project_template/settings.py → misleading comment about trailing slash on MEDIA_URL in settings.py
Triage Stage: Unreviewed → Ready for checkin

This is only true if the url has a path component, but this is the type of thing that can confuse people. It may be worth removing the / for ​http://media.lawrence.com but keeping the other example, but I'll leave that up to the checker-in.

comment:2 by Malcolm Tredinnick, 18 years ago

Resolution: → fixed
Status: new → closed

(In [5404]) Fixed #4290 -- Added a warning about trailing slashes to the project settings
file. Thanks, mw@….

Note: See TracTickets for help on using tickets.
Back to Top