Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
E
edx-platform-release
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Wiki
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Deploy
Releases
Package Registry
Model registry
Operate
Terraform modules
Monitor
Incidents
Service Desk
Analyze
Value stream analytics
Contributor analytics
Repository analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
Hsin-Yu Chien
edx-platform-release
Commits
bd7740a8
Commit
bd7740a8
authored
13 years ago
by
David Ormsbee
Browse files
Options
Downloads
Patches
Plain Diff
removed conditional loading of apps
parent
d267812f
Loading
Loading
No related merge requests found
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
settings2/common.py
+39
-41
39 additions, 41 deletions
settings2/common.py
with
39 additions
and
41 deletions
settings2/common.py
+
39
−
41
View file @
bd7740a8
...
...
@@ -10,7 +10,11 @@ We should make a method that calls all these config methods so that you just
make one call at the end of your site-specific dev file to reset all the
dependent variables (like INSTALLED_APPS) for you.
TODO:
IMMEDIATE TODO:
1. Fix logging()
2.
Longer TODO:
1. Right now our treatment of static content in general and in particular
course-specific static content is haphazard.
2. We should have a more disciplined approach to feature flagging, even if it
...
...
@@ -314,43 +318,37 @@ MIDDLEWARE_CLASSES = (
)
################################### APPS #######################################
def
installed_apps
():
"""
If you want to get a different set of INSTALLED_APPS out of this, you
'
ll
have to set ASKBOT_ENABLED and COURSEWARE_ENABLED to True/False and call
this method. We can
'
t just take these as params because other pieces of the
code check fo the value of these constants.
"""
# We always install these
STANDARD_APPS
=
[
'
django.contrib.auth
'
,
'
django.contrib.contenttypes
'
,
'
django.contrib.humanize
'
,
'
django.contrib.sessions
'
,
'
django.contrib.sites
'
,
'
django.contrib.messages
'
,
'
django.contrib.staticfiles
'
,
'
track
'
,
'
util
'
]
COURSEWARE_APPS
=
[
'
circuit
'
,
'
courseware
'
,
'
student
'
,
'
static_template_view
'
,
'
staticbook
'
,
'
simplewiki
'
,
'
perfstats
'
]
ASKBOT_APPS
=
[
'
django.contrib.sitemaps
'
,
'
django.contrib.admin
'
,
'
south
'
,
'
askbot.deps.livesettings
'
,
'
askbot
'
,
'
keyedcache
'
,
'
robots
'
,
'
django_countries
'
,
'
djcelery
'
,
'
djkombu
'
,
'
followit
'
]
return
tuple
(
STANDARD_APPS
+
(
COURSEWARE_APPS
if
COURSEWARE_ENABLED
else
[])
+
(
ASKBOT_APPS
if
ASKBOT_ENABLED
else
[]))
INSTALLED_APPS
=
installed_apps
()
INSTALLED_APPS
=
(
# Standard ones that are always installed...
'
django.contrib.auth
'
,
'
django.contrib.contenttypes
'
,
'
django.contrib.humanize
'
,
'
django.contrib.messages
'
,
'
django.contrib.sessions
'
,
'
django.contrib.sites
'
,
'
django.contrib.staticfiles
'
,
'
south
'
,
# Our courseware
'
circuit
'
,
'
courseware
'
,
'
perfstats
'
,
'
student
'
,
'
static_template_view
'
,
'
staticbook
'
,
'
simplewiki
'
,
'
track
'
,
'
util
'
,
# For Askbot
'
django.contrib.sitemaps
'
,
'
django.contrib.admin
'
,
'
django_countries
'
,
'
djcelery
'
,
'
djkombu
'
,
'
askbot
'
,
'
askbot.deps.livesettings
'
,
'
followit
'
,
'
keyedcache
'
,
'
robots
'
,
)
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment