hasgeek / nodular

Reusable content objects for HasGeek apps
http://nodular.readthedocs.org/
BSD 2-Clause "Simplified" License
1 stars 0 forks source link

Revisioning: add test for assignment of revision labels #5

Open jace opened 11 years ago

jace commented 11 years ago

What should the API be for assigning labels? Is it as simple as this:

rev.workflow_label = 'foo'

... which then automatically removes the label from previous uses, or should it be:

rev.workflow_label = 'foo'
oldrev.workflow_label = None

... which puts the onus on the caller, but making it possible to make mistakes. Or:

rev.assign_label('foo')

... which then finds the older revision and removes the label there. The difference from the first approach being that this does not override the lower level attribute assignment API.