On 05/09/2006, at 01:46 AM, Toshio Kuratomi wrote:
On Mon, 2006-09-04 at 15:55 -0400, Elliot Lee wrote:
> On 29/08/2006, at 01:13 AM, Toshio Kuratomi wrote:
>> Are the comments in this file accurate? Can you explain what the
>> "status" fields in the *History classes represent?
> It tells you the new status of the item at that point in history. For
> example, this lets you tell who changed a package from 'UNDER_REVIEW'
> to 'APPROVED', and when.
Ah. Any reason not to combine that with the "action" column?
There are some actions (currently add and delete) that don't affect
the status. Overloading the action column to store the status as well
doesn't seem particularly clean to me, but I suppose it could be done.
The "right" way to do things might be to serialize the whole object
into history records, instead of trying to copy parts of its state
into the history fields.