Edgewall Software

Opened 10 years ago

Last modified 10 years ago

#11609 closed enhancement

Add realm attribute to Model classes — at Version 1

Reported by: Ryan J Ollos Owned by: Ryan J Ollos
Priority: normal Milestone: 1.1.2
Component: general Version:
Severity: normal Keywords: model
Cc: Jun Omae Branch:
Release Notes:
API Changes:
Internal Changes:

Description (last modified by Ryan J Ollos)

The proposed change is a refactoring to add a realm attribute to Model and ResourceManager classes. In some cases, the attribute already exists: tags/trac-1.0.1/trac/wiki/model.py?marks=33#L30.

Previously discussed in th:comment:8:ticket:11706.

Change History (1)

comment:1 by Ryan J Ollos, 10 years ago

Description: modified (diff)
Summary: Add realm attribute to Model and ResourceManager classesAdd realm attribute to Model classes

Adding realm to IResourceManager classes doesn't seem to make sense given that they can be associated with multiple realms: tags/trac-1.0.1/trac/versioncontrol/api.py@:367-369#L364.

However the change for model classes probably makes sense. In Bloodhound there is a ModelBase class with a realm attribute, which is the first point that had me looking into this change.

Proposed changes in log:rjollos.git:t11609.

Note: See TracTickets for help on using tickets.