Skip to content

Commit 06e460b

Browse files
committed
Housekeeping: removed celery dependency and some unused code.
1 parent 864d45a commit 06e460b

27 files changed

+78
-376
lines changed

AUTHORS.md

+7-2
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
Authors
22
-------
33

4-
* Łukasz Rekucki <[email protected]>
5-
* Marek Stępniowski <marek@stepniowski>
4+
* Łukasz Rekucki
5+
* Marek Stępniowski
6+
* Radek Czajka
7+
* Łukasz Anwajler
8+
* Marcin Koziej
9+
* Aleksander Łukasz
10+
* Jan Szejko

README.md

+7-18
Original file line numberDiff line numberDiff line change
@@ -12,37 +12,26 @@ Instalacja i użycie
1212
#. Przejdź do katalogu serwisu w konsoli
1313
#. Zainstaluj wymagane biblioteki komendą::
1414

15-
pip install -r requirements.txt
15+
pip install -r requirements/requirements.txt
1616

17-
#. Wypełnij bazę danych (Django poprosi o utworzenie pierwszego użytkownika)::
17+
#. Wypełnij bazę danych::
1818

19-
./project/manage.py syncdb
19+
src/manage.py migrate
2020

21-
#. Skopiuj zawartość pliku `project/localsettings.sample` do `project/localsettings.py` i zmień go zgodnie ze swoimi potrzebami.
21+
#. Skopiuj zawartość pliku `src/redakcja/localsettings.sample` do `src/redakcja/localsettings.py` i zmień go zgodnie ze swoimi potrzebami.
2222

2323
#. Uruchom serwer deweloperski::
2424

25-
./project/manage.py runserver
25+
src/manage.py runserver
2626

27-
#. Zalecane jest serwowanie aplikacji przez `modwsgi <http://code.google.com/p/modwsgi/>`_ na serwerze `Apache2 <http://httpd.apache.org/>`_ przy pomocy załączonego skryptu `dispatch.fcgi`. Inne strategie wdrożeniowe opisane są w `Dokumentacji Django <http://docs.djangoproject.com/en/dev/howto/deployment/#howto-deployment-index>`_.
27+
#. Strategie wdrożeniowe opisane są w `Dokumentacji Django <http://docs.djangoproject.com/en/dev/howto/deployment/#howto-deployment-index>`_.
2828

29-
Wdrożenie
30-
=========
31-
#. Ściągnij i zainstaluj `fabric <http://docs.fabfile.org/>`_
32-
#. Przejdź do katalogu serwisu w konsoli
33-
#. Aby wdrożyć serwis na serwer deweloperski wpisz komendę::
34-
35-
fab staging deploy
36-
37-
Aby wdrożyć serwis na serwer produkcyjny wpisz::
38-
39-
fab production deploy
4029

4130
Testy
4231
====
4332

4433
$ pip install -r requirements-test.txt
45-
$ python src/manage.py test --settings=redakcja.settings.test
34+
$ make test
4635

4736
JavaScript (wymagany node.js i xsltproc):
4837

redakcja-celery.conf

-22
This file was deleted.

requirements/requirements.txt

-3
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,4 @@ django-gravatar2
2121
python-slugify
2222
django-extensions==1.5.7
2323

24-
celery>=3.1.12,<3.2
25-
kombu>=3.0,<3.1
26-
2724
raven

src/catalogue/helpers.py

-1
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,6 @@ def merge(self):
8787
self.src_size = len(files_other)
8888

8989
if files and files_other:
90-
print "compare %s with %s" % (files[-1], files_other[0])
9190
if filecmp.cmp(
9291
join(self.path(self.dest), files[-1]),
9392
join(self.path(self.src), files_other[0]),

src/catalogue/management/commands/assign_from_redmine.py

-151
This file was deleted.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
# -*- coding: utf-8 -*-
2+
# Generated by Django 1.11.20 on 2019-03-07 15:43
3+
from __future__ import unicode_literals
4+
5+
from django.db import migrations
6+
7+
8+
class Migration(migrations.Migration):
9+
10+
dependencies = [
11+
('catalogue', '0001_initial'),
12+
]
13+
14+
operations = [
15+
migrations.RemoveField(
16+
model_name='book',
17+
name='_short_html',
18+
),
19+
migrations.RemoveField(
20+
model_name='chunk',
21+
name='_short_html',
22+
),
23+
migrations.RemoveField(
24+
model_name='image',
25+
name='_short_html',
26+
),
27+
]
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
# -*- coding: utf-8 -*-
2+
# Generated by Django 1.11.20 on 2019-03-07 15:48
3+
from __future__ import unicode_literals
4+
5+
from django.db import migrations, models
6+
7+
8+
class Migration(migrations.Migration):
9+
10+
dependencies = [
11+
('catalogue', '0002_auto_20190307_1543'),
12+
]
13+
14+
operations = [
15+
migrations.AddField(
16+
model_name='chunk',
17+
name='_new_publishable',
18+
field=models.NullBooleanField(editable=False),
19+
),
20+
]

src/catalogue/models/book.py

+1-18
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
from catalogue.helpers import cached_in_field, GalleryMerger
1616
from catalogue.models import BookPublishRecord, ChunkPublishRecord, Project
1717
from catalogue.signals import post_publish
18-
from catalogue.tasks import refresh_instance, book_content_updated
1918
from catalogue.xml_tools import compile_text, split_xml
2019
from cover.models import Image
2120
import os
@@ -36,7 +35,6 @@ class Book(models.Model):
3635
parent_number = models.IntegerField(_('parent number'), null=True, blank=True, db_index=True, editable=False)
3736

3837
# Cache
39-
_short_html = models.TextField(null=True, blank=True, editable=False)
4038
_single = models.NullBooleanField(editable=False, db_index=True)
4139
_new_publishable = models.NullBooleanField(editable=False)
4240
_published = models.NullBooleanField(editable=False)
@@ -326,10 +324,6 @@ def is_single(self):
326324
return len(self) == 1
327325
single = cached_in_field('_single')(is_single)
328326

329-
@cached_in_field('_short_html')
330-
def short_html(self):
331-
return render_to_string('catalogue/book_list/book.html', {'book': self})
332-
333327
def book_info(self, publishable=True):
334328
try:
335329
book_xml = self.materialize(publishable=publishable)
@@ -363,25 +357,14 @@ def refresh_dc_cache(self):
363357
Book.objects.filter(pk=self.pk).update(**update)
364358

365359
def touch(self):
366-
# this should only really be done when text or publishable status changes
367-
book_content_updated.delay(self)
368-
369360
update = {
370361
"_new_publishable": self.is_new_publishable(),
371362
"_published": self.is_published(),
372363
"_single": self.is_single(),
373364
"_on_track": self.get_on_track(),
374-
"_short_html": None,
375365
}
376366
Book.objects.filter(pk=self.pk).update(**update)
377-
refresh_instance(self)
378-
379-
def refresh(self):
380-
"""This should be done offline."""
381-
self.short_html
382-
self.single
383-
self.new_publishable
384-
self.published
367+
self.refresh_dc_cache()
385368

386369
# Materializing & publishing
387370
# ==========================

0 commit comments

Comments
 (0)