dhis2-devs team mailing list archive
-
dhis2-devs team
-
Mailing list archive
-
Message #03444
Re: Should add one more property in patient attribute Object
For me, breaking the address into pieces is better than the long/short text
approach. The advantage with the current system is we have flexibility,
micro level filtering, ... in addition we are dealing most of the patient
parameters (except demographics - age,sex,...) through attributes and
address shouldn't be treated differently - may be we can think of grouping
of attributes.
More importantly, from the current approach, we can always generate the
longer text you want by combining different fields/attributes (from address
group). But if we opted for the long text approach searching/sorting based
on address will be a little difficult (especially if we want more filtering
from the address).
2009/12/3 Trí Trần Thanh <tranthanhtri84@xxxxxxxxx>
> Hi Bob and Abyot. Thanks
> Your idea is good.
> Until now we have 2 way:
> - Using javascript: check input text, if it longer than x character ,
> input filed will automatically switch to textarea.
> - Define Value type are shortText and longText.
>
> I think using javascript is better than. How are you ?
>
> ================================
> Tran Thanh Tri
> HISP Viet Nam
> Cell phone: +84903670967
> Website: http://tringuyenvn.com
> ================================
>
>
>
> 2009/12/2 Abyot Gizaw <abyota@xxxxxxxxx>:
> > Breaking the address into different sub-addresses gives more sense than
> > having one long address. That is how address is approached in the system
> -
> > things like housenumber, street address, province, landmark and the like
> can
> > provide detail description of the person address/location.
> >
> > What do you think Tri? Do you think the above approach is not appropriate
> > for your context? Is it possible to break the address, from your case,
> into
> > smaller parameters?
> >
> >
> >
> > 2009/12/2 Bob Jolliffe <bobjolliffe@xxxxxxxxx>
> >>
> >> Hi Tri
> >>
> >> Sorry I have not looked at this module in detail so forgive me if I am
> >> misinterpreting something. But ...
> >>
> >> It is generally a bad idea to have view related information as
> properties
> >> of the model. Model-View-Control separation is very important. When
> you
> >> compromise on this it almost always comes back to bite you eventually.
> I
> >> feel PatientAttributes should as much as possible relate to attributes
> of
> >> the patient - what to display rather than how to display on a form.
> >>
> >> But I do see your problem. What is the workaround? I don't know off
> hand
> >> without greater familiarity with the module. Two suggestions:
> >> 1. at least for address, it is normal to break into a number of text
> >> fields on a form - address line1, line2, line2 etc or house, street,
> town
> >> etc. Does it make sense to create separate attributes for these address
> >> parts? How you choose these address parts mostly depends on what the
> >> address is used for (physical location for house visits, extra
> >> identification, postal delivery point etc). And also different
> countries do
> >> use different address standards.
> >> 2. here's a nifty idea for a javascript widget. You say it becomes a
> >> problem for addresses >50 characters. It is possible to have a
> javascript
> >> listener dynamically replace text field with textarea when the amount of
> >> text goes over a threshold - say 50. And similarly convert textarea to
> text
> >> field when contents drop below a threshold.
> >>
> >> While 2 is interesting, I think 1 is better. And can be implemented
> >> easily in patient attribute configuration.
> >>
> >> Regards
> >> Bob
> >>
> >> 2009/12/2 Trí Trần Thanh <tranthanhtri84@xxxxxxxxx>
> >>>
> >>> Hi Lars and Abyot.
> >>> I mean is :
> >>> - Current PatientAttribute class have some property is : Name -
> >>> Descriptions - Value Type.
> >>> In my case: i am trying to create one patient attribute is address.
> >>> Really. The address is so long (>50 char ). If we use text field , it
> is
> >>> small and short. Really difficult to see what was text entered into.
> >>> - We can add one property in PatientAttribute class. If user choose
> >>> Value Type is text so they can choose text filed for short text or
> textarea
> >>> for longer text.
> >>> Thanks
> >>>
> >>> ================================
> >>> Tran Thanh Tri
> >>> HISP Viet Nam
> >>> Cell phone: +84903670967
> >>> Website: http://tringuyenvn.com
> >>> ================================
> >>>
> >>>
> >>> 2009/12/2 Lars Helge Øverland <larshelge@xxxxxxxxx>
> >>>>
> >>>>
> >>>> 2009/12/2 Abyot Gizaw <abyota@xxxxxxxxx>
> >>>>>
> >>>>> Hi Tri,
> >>>>>
> >>>>> You want to add one more attribute called controlType?
> >>>>>
> >>>>> What are you going to store with it? Can you give me an example where
> >>>>> you will need controlType attribute? have you seen PatientAttribute
> module?
> >>>>>
> >>>>> Abyot.
> >>>>>
> >>>>
> >>>> We do want to customize it for Vietnam so thanks for the feedback Tri.
> >>>> Please notice that, like Abyot says, the PatientAttribute
> functionality lets
> >>>> you define any attribute/property you like for a Patient.
> >>>>
> >>>> Lars
> >>>
> >>>
> >>> _______________________________________________
> >>> Mailing list: https://launchpad.net/~dhis2-devs<https://launchpad.net/%7Edhis2-devs>
> >>> Post to : dhis2-devs@xxxxxxxxxxxxxxxxxxx
> >>> Unsubscribe : https://launchpad.net/~dhis2-devs<https://launchpad.net/%7Edhis2-devs>
> >>> More help : https://help.launchpad.net/ListHelp
> >>>
> >>
> >>
> >> _______________________________________________
> >> Mailing list: https://launchpad.net/~dhis2-devs<https://launchpad.net/%7Edhis2-devs>
> >> Post to : dhis2-devs@xxxxxxxxxxxxxxxxxxx
> >> Unsubscribe : https://launchpad.net/~dhis2-devs<https://launchpad.net/%7Edhis2-devs>
> >> More help : https://help.launchpad.net/ListHelp
> >>
> >
> >
>
References