Skip to content

Commit ed0c722

Browse files
committed
Removed backwards compatible shims from 3.5
1 parent 163baaf commit ed0c722

File tree

3 files changed

+8
-79
lines changed

3 files changed

+8
-79
lines changed

cms/cms_toolbars.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -308,7 +308,7 @@ def page_is_pending(self, page, language):
308308
page.publisher_public.get_publisher_state(language) == PUBLISHER_STATE_PENDING)
309309

310310
def in_apphook(self):
311-
with force_language(self.toolbar.language):
311+
with force_language(self.toolbar.request_language):
312312
try:
313313
resolver = resolve(self.toolbar.request_path)
314314
except Resolver404:
@@ -447,7 +447,7 @@ def add_draft_live_item(self, template='cms/toolbar/items/live_draft.html', extr
447447
self.toolbar.add_item(TemplateItem(template, extra_context=context, side=self.toolbar.RIGHT), position=pos)
448448

449449
def add_page_settings_button(self, extra_classes=('cms-btn-action',)):
450-
url = '%s?language=%s' % (admin_reverse('cms_page_change', args=[self.page.pk]), self.toolbar.language)
450+
url = '%s?language=%s' % (admin_reverse('cms_page_change', args=[self.page.pk]), self.toolbar.request_language)
451451
self.toolbar.add_modal_button(_('Page settings'), url, side=self.toolbar.RIGHT, extra_classes=extra_classes)
452452

453453
# Menus
@@ -529,7 +529,7 @@ def change_admin_menu(self):
529529
if not self._changed_admin_menu and can_change_page:
530530
admin_menu = self.toolbar.get_or_create_menu(ADMIN_MENU_IDENTIFIER)
531531
url = admin_reverse('cms_page_changelist') # cms page admin
532-
params = {'language': self.toolbar.language}
532+
params = {'language': self.toolbar.request_language}
533533
if self.page:
534534
params['page_id'] = self.page.pk
535535
url = add_url_parameters(url, params)
@@ -600,7 +600,7 @@ def add_page_menu(self):
600600
)
601601

602602
for title, params, has_perm in add_page_menu_modal_items:
603-
params.update(language=self.toolbar.language)
603+
params.update(language=self.toolbar.request_language)
604604
add_page_menu.add_modal_item(
605605
title,
606606
url=add_url_parameters(add_page_url, params),
@@ -609,7 +609,7 @@ def add_page_menu(self):
609609

610610
add_page_menu.add_modal_item(
611611
_('Duplicate this Page'),
612-
url=add_url_parameters(duplicate_page_url, {'language': self.toolbar.language}),
612+
url=add_url_parameters(duplicate_page_url, {'language': self.toolbar.request_language}),
613613
disabled=not can_add_sibling_page,
614614
)
615615

@@ -621,13 +621,13 @@ def add_page_menu(self):
621621
current_page_menu.add_link_item(_('Edit this Page'), disabled=edit_mode, url=page_edit_url)
622622

623623
# page settings
624-
page_settings_url = add_url_parameters(page_settings_url, language=self.toolbar.language)
624+
page_settings_url = add_url_parameters(page_settings_url, language=self.toolbar.request_language)
625625
settings_disabled = not edit_mode or not can_change
626626
current_page_menu.add_modal_item(_('Page settings'), url=page_settings_url, disabled=settings_disabled,
627627
on_close=refresh)
628628

629629
# advanced settings
630-
advanced_url = add_url_parameters(advanced_url, language=self.toolbar.language)
630+
advanced_url = add_url_parameters(advanced_url, language=self.toolbar.request_language)
631631
can_change_advanced = self.page.has_advanced_settings_permission(self.request.user)
632632
advanced_disabled = not edit_mode or not can_change_advanced
633633
current_page_menu.add_modal_item(_('Advanced settings'), url=advanced_url, disabled=advanced_disabled)
@@ -656,7 +656,7 @@ def add_page_menu(self):
656656
# page type
657657
if not self.page.is_page_type:
658658
page_type_url = admin_reverse('cms_pagetype_add')
659-
page_type_url = add_url_parameters(page_type_url, source=self.page.pk, language=self.toolbar.language)
659+
page_type_url = add_url_parameters(page_type_url, source=self.page.pk, language=self.toolbar.request_language)
660660
page_type_disabled = not edit_mode or not can_add_root_page
661661
current_page_menu.add_modal_item(_('Save as Page Type'), page_type_url, disabled=page_type_disabled)
662662

cms/models/pagemodel.py

Lines changed: 0 additions & 50 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
# -*- coding: utf-8 -*-
22
import copy
3-
import warnings
43
from collections import OrderedDict
54
from logging import getLogger
65
from os.path import join
@@ -180,55 +179,6 @@ def _clear_internal_cache(self):
180179
if hasattr(self, '_prefetched_objects_cache'):
181180
del self._prefetched_objects_cache
182181

183-
@property
184-
def parent(self):
185-
warnings.warn(
186-
'Pages no longer have a "parent" field. '
187-
'To get the parent object of any given page, use the "parent_page" attribute. '
188-
'This backwards compatible shim will be removed in version 3.6',
189-
UserWarning,
190-
stacklevel=2,
191-
)
192-
return self.parent_page
193-
194-
@property
195-
def parent_id(self):
196-
warnings.warn(
197-
'Pages no longer have a "parent_id" attribute. '
198-
'To get the parent id of any given page, '
199-
'call "pk" on the "parent_page" attribute. '
200-
'This backwards compatible shim will be removed in version 3.6',
201-
UserWarning,
202-
stacklevel=2,
203-
)
204-
if self.parent_page:
205-
return self.parent_page.pk
206-
return None
207-
208-
@property
209-
def site(self):
210-
warnings.warn(
211-
'Pages no longer have a "site" field. '
212-
'To get the site object of any given page, '
213-
'call "site" on the page "node" object. '
214-
'This backwards compatible shim will be removed in version 3.6',
215-
UserWarning,
216-
stacklevel=2,
217-
)
218-
return self.node.site
219-
220-
@property
221-
def site_id(self):
222-
warnings.warn(
223-
'Pages no longer have a "site_id" attribute. '
224-
'To get the site id of any given page, '
225-
'call "site_id" on the page "node" object. '
226-
'This backwards compatible shim will be removed in version 3.6',
227-
UserWarning,
228-
stacklevel=2,
229-
)
230-
return self.node.site_id
231-
232182
@cached_property
233183
def parent_page(self):
234184
return self.get_parent_page()

cms/toolbar/toolbar.py

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
from collections import OrderedDict
33
import functools
44
import operator
5-
import warnings
65

76
from cms import __version__
87
from cms.api import get_page_draft
@@ -36,16 +35,6 @@ class BaseToolbar(ToolbarAPIMixin):
3635
structure_mode_url_on = get_cms_setting('CMS_TOOLBAR_URL__BUILD')
3736
disable_url = get_cms_setting('CMS_TOOLBAR_URL__DISABLE')
3837

39-
@property
40-
def language(self):
41-
# Backwards compatibility
42-
warnings.warn(
43-
"toolbar.language has been deprecated "
44-
"and will be removed in django CMS 3.6",
45-
PendingDeprecationWarning
46-
)
47-
return self.request_language
48-
4938
@cached_property
5039
def site_language(self):
5140
cms_page = get_page_draft(self.request.current_page)
@@ -211,16 +200,6 @@ def init_toolbar(self, request, request_path=None):
211200
self.toolbars[key].request = self.request
212201
self.request_path = request_path or request.path
213202

214-
@property
215-
def edit_mode(self):
216-
# Backwards compatibility
217-
warnings.warn(
218-
"toolbar.edit_mode has been deprecated "
219-
"and will be removed in django CMS 3.6",
220-
PendingDeprecationWarning
221-
)
222-
return self.edit_mode_active
223-
224203
@cached_property
225204
def user_settings(self):
226205
return self.get_user_settings()

0 commit comments

Comments
 (0)