Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Duplicates of records #421

Closed
ppeczek opened this issue Feb 21, 2017 · 8 comments
Closed

Duplicates of records #421

ppeczek opened this issue Feb 21, 2017 · 8 comments

Comments

@ppeczek
Copy link

ppeczek commented Feb 21, 2017

I am getting duplicates of table record with no exception, but warning in server:
/usr/local/lib/python2.7/dist-packages/django_tables2/rows.py:143: UnicodeWarning: Unicode equal comparison failed to convert both arguments to Unicode - interpreting them as being unequal if value in bound_column.column.empty_values:

@jieter
Copy link
Owner

jieter commented Feb 24, 2017

Interesting, can you tell more about your environment? OS, python, django and django-tables2 versions?

@ppeczek
Copy link
Author

ppeczek commented Feb 24, 2017

This occured when I migrated from django==1.7 to django==1.10 and from django_tables2==1.0.4 to current version. I also started to work on docker with ubuntu:trusty. Python version is 2.7.
I managed to fix it by commenting one boolean column which had visible=False, but was used for ordering table. What is more concerning I have second similar table which doesn't produce this error. I'm still working on it and will report when find the very reason of this strange behaviour.

@jieter
Copy link
Owner

jieter commented Feb 24, 2017

Is it possibly related to this issue: #361 ?

@ppeczek
Copy link
Author

ppeczek commented Feb 24, 2017

Oh, that's correct. I am sorry for duplicating issue. But this doesn't happen in all cases.

@jieter
Copy link
Owner

jieter commented Feb 24, 2017

Never mind, the more information on this the better. I'm really interested in your findings on the issue, let's keep it in this separate issue until we see things more clearly.

jieter added a commit that referenced this issue Feb 27, 2017
jieter added a commit that referenced this issue Feb 27, 2017
jieter added a commit that referenced this issue Feb 27, 2017
@jieter
Copy link
Owner

jieter commented Feb 27, 2017

@ppeczek I just pushed some commits to master which might fix this. Can you verify?

intiocean pushed a commit to intiocean/django-tables2 that referenced this issue Feb 27, 2017
@ppeczek
Copy link
Author

ppeczek commented Feb 27, 2017

Works as a charm!

@ppeczek ppeczek closed this as completed Feb 27, 2017
@jieter
Copy link
Owner

jieter commented Feb 27, 2017

released 1.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants