Implemented .with_instructions(text) that will fill the .instructions property on the Field element.

This allows you to do (in fieldset.mako for example):

% if field.instructions: <span class="instruction">${field.instructions|n}</span> % endif

Index: fields.py

--- fields.py (révision 738) +++ fields.py (copie de travail) @@ -656,6 +656,8 @@ self.is_pk = False # True iff this Field is a raw foreign key self.is_raw_foreign_key = False + # Instructions + self.instructions = None return False

 def __deepcopy__(self, memo):

@@ -744,6 +746,9 @@ def bind(self, parent): """Return a copy of this Field, bound to a different parent""" return self._modified(parent=parent) + def with_instructions(self, text): + """Return a copy of this Field, with some newly attached instructions""" + return self._modified(instructions=text) def validate(self, validator): """ Add the validator function to the list of validation

hmmm.. Documentation missing.

I'd like to see a complete patch, including templates & docs, before committing.

This patch should fix the issue, and document the changes.

Nothing tests it however, haven't got around to poke with the test suite.

doctests fail.

I'd like to see a test of the template change too before committing.

Now implemented with more generic 'metadata'

ok, this fixes the issue, adds the functionality, covers all the tests, adds templates for both Tempita and Mako, and documents the changes.

sorry I didn't have time to review sooner. looks good; committed. Thanks!