Edgewall Software

Ticket #2733 (new enhancement)

Opened 2 years ago

Last modified 17 months ago

Push some more behaviour to the vc backend

Reported by: Lele Gaifax Owned by: cboos
Priority: normal Milestone: 0.13
Component: version control Version: 0.9.3
Severity: normal Keywords:
Cc:

Description

In order to get my new TracDarcs plugin working, I had to apply this patch that

  1. Fixes #2731
  2. Adds an "hash" field to the revision table
  3. Moves the actual insertion of the cache data from the CachedRepository class to the Changeset one

Point 3 allows the backend to augment the cached data as needed, simply by overriding a Changeset method (see for example).

Of course, I could have reimplemented the whole sync() machinery in a subclass of CachedRepository, but DRY approach suggests that's not the right way.

Attachments

Change History

Changed 2 years ago by mgood

It would help if you explained the purpose of the "hash" column since it's not actually used in the patch anywhere.

What changes are necessary in the cache? The classes in trac.versioncontrol.api are meant to be decoupled from the cache, so if it's necessary to extend the cache, this should be done in the cache module, not api.

Changed 17 months ago by cboos

  • owner changed from cmlenz to cboos
  • milestone set to 0.12

Add/Change #2733 (Push some more behaviour to the vc backend)

Author



Change Properties
<Author field>
Action
as new
as The resolution will be set. Next status will be 'closed'
to The owner will change. Next status will be 'new'
The owner will change to anonymous. Next status will be 'assigned'
 
Note: See TracTickets for help on using tickets.