jazzband / django-embed-video

Django app for easy embedding YouTube and Vimeo videos and music from SoundCloud.
http://django-embed-video.rtfd.org
MIT License
383 stars 137 forks source link

Template_context_processors has been depreciated in django 1.8 #71

Closed Basiczombie closed 7 years ago

Basiczombie commented 7 years ago
TEMPLATE_CONTEXT_PROCESSORS = (
    'django.core.context_processors.request',
)

This was depreciated in Django 1.8. It is saying that Template Dictionaries take precedent. Do you have a work around to place the needed code within the Templates -> options -> context_processors in the settings post 1.8?

TEMPLATES = [
    {
        'BACKEND': 'django.template.backends.django.DjangoTemplates',
        'DIRS': [
            'templates',
        ],
        'APP_DIRS': True,
        'OPTIONS': {
            'context_processors': [
                'django.template.context_processors.debug',
                'django.template.context_processors.request',
                'django.template.context_processors.media',
                'django.contrib.auth.context_processors.auth',
                'django.contrib.messages.context_processors.messages',
            ],
        },
    },
]
SalahAdDin commented 7 years ago

Which exactly the problem?

Basiczombie commented 7 years ago

The setting TEMPLATE_CONTEXT_PROCESSORS has been depreciated in Django 1.8. It still works but my server keeps yelling at me to put it into the TEMPLATES setting which when added to the context_processors in the Options breaks the module.

thijstriemstra commented 7 years ago

how does it break? any error message?

SalahAdDin commented 7 years ago

@Basiczombie can you put the warning message here please?

Basiczombie commented 7 years ago

WARNINGS: ?: (18.W001) The standalone TEMPLATE* settings were deprecated in Django 1.8 and the TEMPLATES dictionary takes precedence. You must put the values of the following settings into your default TEMPLATES dict: TEMPLATE_CONTEXT_PROCESSORS.

SalahAdDin commented 7 years ago

But, doesn't ıt show the exactly fıle and lıne that gıve you the error?

İ revıewed the package oversıde and the settıngs fıle hasn't these optıon, so, the problem ıs an error ın the documentatıon, ısn't ıt?

cedriccarrard commented 7 years ago

@Basiczombie fixed with new version ?