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.

Sign up for GitHub

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

Jump to bottom

Triggers not created from abstract models #80

Closed
Naddiseo opened this issue Jun 13, 2017 · 0 comments
Closed

Triggers not created from abstract models #80

Naddiseo opened this issue Jun 13, 2017 · 0 comments

Comments

@Naddiseo
Copy link
Contributor

This may also be related to issue #78

From what I can tell, it may be related to having a TriggerVersionField in an abstract base class, which causes django to create all the version fields with the same creation_counter. The creation_counter is used in __hash__, which is called via set(_TRIGGERS) in the create_triggers function.

I've been having trouble writing a specific test for this, but I did manage to reproduce in another repo
saxix added a commit that referenced this issue Jun 14, 2017
@saxix
fixes issue #80
90c8710
saxix added a commit that referenced this issue Jun 21, 2017
@saxix
Merge branch 'feature/issue_80' into develop
8a422d1 
* feature/issue_80:
  updates CHANGES
  fixes issue #80
@saxix saxix closed this as completed Jun 21, 2017
saxix added a commit that referenced this issue Mar 28, 2019
@saxix
Merge branch 'release/1.4'
491b761 
* release/1.4: (24 commits)
  updates CHANGES
  fixes py3.4 tests
  enable ConcurrentModelAdmin.check only on django>=1.11
  closes issue #81
  removes some doctests
  minor tests cleanup
  fixes wrong import
  some code clean, removes old compatibility related code
  updates docs
  updates docs
  fixes docs pagination
  add test for issue #54 (just asa reference)
  updates CHANGES
  fixes issue #80
  Fix warnings on django 1.11 with Python 3.6 -Wall
  updates docs
  updates tests
  removes unused tests requirements
  removes double requirement in testing
  pep8 and minor updates
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@saxix @Naddiseo