Opened 17 years ago

Closed 17 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

Description

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 by Thom Linton <thom.linton@…>, 17 years ago

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

comment:2 by Thom Linton <thom.linton@…>, 17 years ago

To obtain the source please use the following url: http://pineapple.binrock.net/django/modelhistory.tar.bz2

comment:3 by Thom Linton <thom.linton@…>, 17 years ago

Cc: thom.linton@… added

comment:4 by Massimiliano Ravelli <massimiliano.ravelli@…>, 17 years ago

Cc: massimiliano.ravelli@… added

comment:5 by Malcolm Tredinnick, 17 years ago

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