SciPyCentral
SciPyCentral copied to clipboard
Django 1.5 - DO NOT MERGE
The below are the changes made (some minor changes might have been missed but can be found clearly in commit messages)
- Upgrade
{% url %}
template tag - Use django-haystack 2.1.0, xapian-haystack 2.0.0, Whoosh 2.5.1 -- make respective changes in
search_settings.py
andsearch_indexes.py
- Use django-registration 1.0 -- Also move
registration.backends.default.urls
fromscipy_central.urls
toscipy_central.person.urls
- DVCS (windows specific) error fixed
- Use django transaction middleware, clickjacking middleware
- Make
pages.views.page_404_error
default 404 handler - Add wsgi.py, change manage.py, upgrade
slugify
import, upgrade url conf import - Fix os.path.normpath import
- Use paginator module for finding "next" or "prev" objects in Revision, Submission module
- Make UserProfile model compatible with django-1.5
[5] Django transaction middleware raises https://github.com/scipy/SciPyCentral/issues/132
Note: A deprecation warning is observed when starting the server regarding hashlib
import. This is caused in django transactions middleware (It will be deprecated in 1.7. However, we require it at the moment until we have our own Middleware)
@ksurya I think you included a lot of changes, that although probably necessary, are not at all related to Django 1.5. Why don't you create more PRs, each with their commits, neatly separated?
I was thinking django-1.5
was more like improving existing code. Its why I added lots of code and simultaneously fixed very necessary bugs found.
Since each of the individual bug-fixing or improving code are quite small and confined to upgrading the packages and fixing relevant code, I added them in the same branch. Moreover, I discovered them when I started reading django-docs and other respective docs.
No, django-1.5 is updating the code to work correctly with Django-1.5. Remove all commits that are not part of that, and create a PR for each one of them so it can be decided separately if they are needed and when to merge.
Problems:
- The error I told you about in Search pages. Are you sure you are not getting it? Why could it be that I see it?
- I can't upload images on submissions. I get error "Error on server, please try again decoder jpeg not available". Please fix both the error message, and whatever is causing this.
I will look at the code now.
yeah, you can't upload images. This is fixed in https://github.com/scipy/SciPyCentral/pull/141 PR
I think commits 30c73e8 and a320450 shouldn't be here. Create new PRs for those.
Also, there are conflicts between this PR and master, which means it can't be automatically merged. Fix them.
@lomegor I have found why you were getting server error on search query.
django-haystack
requires another module xapian-haystack
and xapian-haystack bindings for python.
If you install those, the search is going to work.
pip install xapian-haystack
installs 1st module I mentioned. Unfortunately, xapian bindings are highly system dependent and we need to install packages based on it.
As requirements.txt
never mentioned these packages, quickstart.py
does not install and thus you get this error
I have tested the commit you mentioned https://github.com/scipy/SciPyCentral/commit/a320450
The new changes in slugify()
are causing UnicodeDecode error. I moved these changes to https://github.com/scipy/SciPyCentral/commit/983c851081d6d8898e15264671d369ec69a65fd8
These changes are required in the PR