Re: [Gnumed-devel] Advance directives (DNR, Do not resuscitate)

[Busser, Jim]

On 2012-05-17, at 2:22 AM, Karsten Hilbert wrote:

>> No, that's the (say, line) template to replace fields of
>> the tag business object into, like
>> 
>>    $<patient_tags::%(l10n_description)s: %(comment)s//\n::60>$

Ah, ok, that was partly my misunderstanding because 'template' is sometimes a 
reference to some object defined elsewhere (such as the reports generated from 
a .tex template). I had further not realized that you had provided (by default) 
for all tags to be included -- which I think is a good idea, and needs to be 
preserved.

In any event these are the reasons that I had misunderstood 'template' to refer 
to a specific tag reference, IMO something like 'formula' or 'expression' might 
be better. Is there any objection in git client/wxpython/gmMacro.py to change 
'template'

108     u'patient_tags',        # "args" holds: <template>//<separator>

to any of

        u'patient_tags',        # "args" holds: <line template>//<separator>
        u'patient_tags',        # "args" holds: <template 
expression>//<separator>

====================================
Question about the 'fields' that are available to be used in any one 
placeholder... are these best looked up in one of

- (currently) lines 1461-1489 in gmMacro.py within the section def 
test_placeholder() where it appears you store some formatted placeholders for 
testing, and

- lower down, in the file gmMacro.py, the related 'def' inside the class 
definition, for example in this case

        def _get_variant_patient_tags

  which references

        gmDemographicsWidgets.select_patient_tags(patient = self.pat)

and look up there. ??

====================================


One last thing... I noticed in line 109
        
#       u'patient_tags_table',                                  # "args" holds: 
no args

which I gather is an as-yet unimplemented tag which you think can be helpful (I 
agree).

-- Jim