0001-general-add-model-to-store-redirections-20760.patch
| combo/data/migrations/0034_redirect.py | ||
|---|---|---|
| 1 |
# -*- coding: utf-8 -*- |
|
| 2 |
# Generated by Django 1.11.12 on 2018-07-12 18:44 |
|
| 3 |
from __future__ import unicode_literals |
|
| 4 | ||
| 5 |
from django.db import migrations, models |
|
| 6 |
import django.db.models.deletion |
|
| 7 | ||
| 8 | ||
| 9 |
class Migration(migrations.Migration): |
|
| 10 | ||
| 11 |
dependencies = [ |
|
| 12 |
('data', '0033_auto_20180401_1300'),
|
|
| 13 |
] |
|
| 14 | ||
| 15 |
operations = [ |
|
| 16 |
migrations.CreateModel( |
|
| 17 |
name='Redirect', |
|
| 18 |
fields=[ |
|
| 19 |
('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
|
|
| 20 |
('old_url', models.CharField(max_length=512)),
|
|
| 21 |
('page', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='data.Page')),
|
|
| 22 |
], |
|
| 23 |
), |
|
| 24 |
] |
|
| combo/data/models.py | ||
|---|---|---|
| 409 | 409 |
return page |
| 410 | 410 | |
| 411 | 411 | |
| 412 |
class Redirect(models.Model): |
|
| 413 |
old_url = models.CharField(max_length=512) |
|
| 414 |
page = models.ForeignKey(Page) |
|
| 415 | ||
| 416 | ||
| 412 | 417 |
class CellMeta(MediaDefiningClass, ModelBase): |
| 413 | 418 |
pass |
| 414 | 419 | |
| combo/public/views.py | ||
|---|---|---|
| 44 | 44 |
else: |
| 45 | 45 |
get_idps = lambda: [] |
| 46 | 46 | |
| 47 |
from combo.data.models import CellBase, PostException, Page, ParentContentCell, TextCell, PageSnapshot |
|
| 47 |
from combo.data.models import (CellBase, PostException, Page, Redirect, |
|
| 48 |
ParentContentCell, TextCell, PageSnapshot) |
|
| 48 | 49 |
from combo.profile.models import Profile |
| 49 | 50 |
from combo.apps.search.models import SearchCell |
| 50 | 51 |
from combo import utils |
| ... | ... | |
| 353 | 354 |
except Page.DoesNotExist: |
| 354 | 355 |
if Page.objects.count() == 0 and parts == ['index']: |
| 355 | 356 |
return empty_site(request) |
| 356 |
else: |
|
| 357 |
if not url.endswith('/') and settings.APPEND_SLASH:
|
|
| 358 |
# this is useful to allow /login, /manage, and other non-page |
|
| 359 |
# URLs to work. |
|
| 360 |
return HttpResponsePermanentRedirect(url + '/') |
|
| 361 |
raise Http404() |
|
| 357 |
page = None |
|
| 362 | 358 | |
| 363 |
if page.get_online_url() != url: |
|
| 359 |
if page is None or page.get_online_url() != url:
|
|
| 364 | 360 |
if not url.endswith('/') and settings.APPEND_SLASH:
|
| 361 |
# this is useful to allow /login, /manage, and other non-page |
|
| 362 |
# URLs to work. |
|
| 365 | 363 |
return HttpResponsePermanentRedirect(url + '/') |
| 364 |
try: |
|
| 365 |
redirect = Redirect.objects.get(old_url=url) |
|
| 366 |
except Redirect.DoesNotExist: |
|
| 367 |
pass |
|
| 368 |
else: |
|
| 369 |
return HttpResponsePermanentRedirect(redirect.page.get_online_url()) |
|
| 366 | 370 |
raise Http404() |
| 367 | 371 | |
| 368 | 372 |
return publish_page(request, page) |
| tests/test_public.py | ||
|---|---|---|
| 16 | 16 | |
| 17 | 17 |
from combo.wsgi import application |
| 18 | 18 |
from combo.data.models import (Page, CellBase, TextCell, ParentContentCell, |
| 19 |
FeedCell, LinkCell, ConfigJsonCell) |
|
| 19 |
FeedCell, LinkCell, ConfigJsonCell, Redirect)
|
|
| 20 | 20 |
from combo.apps.family.models import FamilyInfosCell |
| 21 | 21 | |
| 22 | 22 |
pytestmark = pytest.mark.django_db |
| ... | ... | |
| 584 | 584 | |
| 585 | 585 |
resp = app.get('/foo/', status=200)
|
| 586 | 586 |
assert resp.body.count('data-ajax-cell-must-load="true"') == 1
|
| 587 | ||
| 588 |
def test_redirects(app): |
|
| 589 |
Page.objects.all().delete() |
|
| 590 |
page = Page(title='Home', slug='index', template_name='standard') |
|
| 591 |
page.save() |
|
| 592 | ||
| 593 |
page2 = Page(title='Second', slug='second', template_name='standard') |
|
| 594 |
page2.save() |
|
| 595 | ||
| 596 |
page3 = Page(title='Third', slug='third', template_name='standard', parent=page2) |
|
| 597 |
page3.save() |
|
| 598 | ||
| 599 |
app.get('/whatever/', status=404)
|
|
| 600 | ||
| 601 |
redirect = Redirect(old_url='/whatever/', page=page3) |
|
| 602 |
redirect.save() |
|
| 603 | ||
| 604 |
assert app.get('/whatever/', status=301).location == '/second/third/'
|
|
| 605 |
assert app.get('/whatever', status=301).location == '/whatever/'
|
|
| 587 |
- |
|