Edgewall Software

Changes between Initial Version and Version 1 of Ticket #11609


Ignore:
Timestamp:
May 10, 2014, 4:50:36 AM (10 years ago)
Author:
Ryan J Ollos
Comment:

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.

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #11609

    • Property Summary Add realm attribute to Model and ResourceManager classes → Add realm attribute to Model classes

  • Ticket #11609 – Description

    initial v1  
    1 The proposed change is a refactoring to add a `realm` attribute to Model and !ResourceManager classes. In some cases, the attribute already exists: [browser:/tags/trac-1.0.1/trac/wiki/model.py?marks=33#L30].
     1The proposed change is a refactoring to add a `realm` attribute to Model ~~and !ResourceManager~~ classes. In some cases, the attribute already exists: [browser:/tags/trac-1.0.1/trac/wiki/model.py?marks=33#L30].
    22
    33Previously discussed in th:comment:8:ticket:11706.