Opened 16 years ago

Closed 16 years ago

#4691 closed (invalid)

Request for Model History to be merged into trunk (updated code supplied)

Reported by: Thom Linton <thom.linton@…> Owned by: Adrian Holovaty
Component: Contrib apps Version: dev
Severity: Keywords: fullhistory, modelhistory
Cc: thom.linton@…, massimiliano.ravelli@… Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


The 'fullhistory' branch has been lying dormant for a few months - so I've recently updated and implemented leftover functionality in a modified version of the branch. It is undoubtedly imperfect, but it should work in common use cases that the are checked by the included tests. The embedded docs should shed light on slight differences; however, tracking of additions, updates, deletions and reverts is done.

The tar consists of just the contrib subdirectory, should extract directly into django/contrib and 'work out of the box.'

Two issues that I think should be considered and debated w.r.t model history:

(1) it should probably provide some means (if end user-desired) of providing distinct tables for each revisioned object (for performance reasons)
(2) although it seems to function properly when embedded directly as django.contrib.modelhistory, I noticed some issues where it appeared that multiple

processes were calling the provided dispatcher.connect(...), in effect, doubling (or tripling) the changelog output.

Hopefully, these issues can be dealt with quickly, and this (imho) highly useful contrib app will be pushed into trunk soon.

Feedback/being-put-in-my-place greatly appreciated.

Change History (5)

comment:1 Changed 16 years ago by Thom Linton <thom.linton@…>

It seems I am unable to upload an attachment; solution forthcoming ...

comment:2 Changed 16 years ago by Thom Linton <thom.linton@…>

To obtain the source please use the following url:

comment:3 Changed 16 years ago by Thom Linton <thom.linton@…>

Cc: thom.linton@… added

comment:4 Changed 16 years ago by Massimiliano Ravelli <massimiliano.ravelli@…>

Cc: massimiliano.ravelli@… added

comment:5 Changed 16 years ago by Malcolm Tredinnick

Resolution: invalid
Status: newclosed

Please start a thread on django-developers for this. Trac is not a very good place to have design discussions. You should also ensure to include the original (and current) maintainer of that branch, Uros Trebec, in the discussions.

Note: See TracTickets for help on using tickets.
Back to Top