← Back to team overview

dhis2-devs team mailing list archive

Re: Limit characters in Comment

 

Oh!, i am using MySQL. where i can find this sql expression?


On Wed, Apr 9, 2014 at 4:46 PM, Jason Pickering <jason.p.pickering@xxxxxxxxx
> wrote:

> Could be. On postgres, the datavalue table (at least on my system) looks
> like this
>
> CREATE TABLE datavalue
> (
>   dataelementid integer NOT NULL,
>   periodid integer NOT NULL,
>   sourceid integer NOT NULL,
>   categoryoptioncomboid integer NOT NULL,
>   value character varying(255),
>   storedby character varying(100),
>   lastupdated timestamp without time zone,
>   comment character varying(360),
>   followup boolean,
> ....
>
> In that case, not sure then. :-/
>
> Regards,
> Jason
>
>
>
> On Wed, Apr 9, 2014 at 11:42 AM, channara rin <rin.channara@xxxxxxxxx>wrote:
>
>> The default, comment field in datavalue table is longtext type.
>>
>> Best Regards,
>> CHANNARA
>>
>>
>> On Wed, Apr 9, 2014 at 4:40 PM, Jason Pickering <
>> jason.p.pickering@xxxxxxxxx> wrote:
>>
>>> Did you change the database field type from varchar to text?
>>>
>>> Regards,
>>> Jason
>>>
>>>
>>>
>>> On Wed, Apr 9, 2014 at 11:31 AM, channara rin <rin.channara@xxxxxxxxx>wrote:
>>>
>>>> Dear Lars,
>>>> I tried to change in history.js
>>>> but when i enter longer than 360 in comment, and press "Save Comment".
>>>> It always jump:
>>>> CONSOLE
>>>> -------------------------------------
>>>> ReferenceError: i18n_saving_comment_failed_error_code is not defined
>>>>                        history.js:64
>>>> -------------------------------------
>>>>
>>>> history.js
>>>> --------------------------------------
>>>> function handleError( xhr, textStatus, errorThrown )
>>>>     {
>>>>         markComment( COLOR_RED );
>>>>         window.alert( i18n_saving_comment_failed_error_code + '\n\n' +
>>>> xhr.responseText );
>>>>     }
>>>>
>>>>
>>>> On Wed, Apr 9, 2014 at 3:29 PM, Lars Helge Øverland <
>>>> larshelge@xxxxxxxxx> wrote:
>>>>
>>>>> The cell border will be green for fields which have comments (next
>>>>> time you open that form).
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> On Wed, Apr 9, 2014 at 10:27 AM, channara rin <rin.channara@xxxxxxxxx>wrote:
>>>>>
>>>>>> Great, Thank you.
>>>>>> SO i need to modify the 360 in history.js file?
>>>>>>
>>>>>> AND another problem is. when has comment, can show highlight data
>>>>>> value in dataset that has text in that comment? SO it is easy for us(M&E
>>>>>> staff) to view if that data value has comment..
>>>>>>
>>>>>> Thank you
>>>>>> CHANNARA
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> On Wed, Apr 9, 2014 at 3:02 PM, Lars Helge Øverland <
>>>>>> larshelge@xxxxxxxxx> wrote:
>>>>>>
>>>>>>> Hi Channara,
>>>>>>>
>>>>>>> the max is actually 255 characters, backed by a "varchar" column in
>>>>>>> the database. I'm sorry but we don't plan to change it to a "text" type
>>>>>>> column anytime soon - reason is just that most comments are pretty short
>>>>>>> and "text" requires more storage space on disk.
>>>>>>>
>>>>>>> Now that said you are of course free to change the column type to
>>>>>>> text directly in your database, and then apply the patch copied below this
>>>>>>> email to remove the validation.
>>>>>>>
>>>>>>> regards,
>>>>>>>
>>>>>>> Lars
>>>>>>>
>>>>>>> -----
>>>>>>>
>>>>>>>
>>>>>>> === modified file
>>>>>>> 'dhis-2/dhis-web/dhis-web-dataentry/src/main/webapp/dhis-web-dataentry/javascript/history.js'
>>>>>>> ---
>>>>>>> dhis-2/dhis-web/dhis-web-dataentry/src/main/webapp/dhis-web-dataentry/javascript/history.js 2014-02-27
>>>>>>> 04:15:35 +0000
>>>>>>> +++
>>>>>>> dhis-2/dhis-web/dhis-web-dataentry/src/main/webapp/dhis-web-dataentry/javascript/history.js 2014-04-09
>>>>>>> 08:01:57 +0000
>>>>>>> @@ -5,14 +5,6 @@
>>>>>>>
>>>>>>>      var commentValue = $( '#commentTextArea' ).val();
>>>>>>>
>>>>>>> -    if ( commentValue.length > 360 )
>>>>>>> -    {
>>>>>>> -        markComment( COLOR_YELLOW );
>>>>>>> -        window.alert(i18n_value_too_long + " for comment field");
>>>>>>> -
>>>>>>> -        return;
>>>>>>> -    }
>>>>>>> -
>>>>>>>       var commentSaver = new CommentSaver( currentDataElementId,
>>>>>>> currentOptionComboId, commentValue );
>>>>>>>
>>>>>>>      commentSaver.save();
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> On Wed, Apr 9, 2014 at 6:55 AM, channara rin <rin.channara@xxxxxxxxx
>>>>>>> > wrote:
>>>>>>>
>>>>>>>> Dear All,
>>>>>>>> I found that, in the Comment in data value in dataset is
>>>>>>>> limit character around 360characters.
>>>>>>>> Can we extend to unlimited characters for comments?
>>>>>>>>
>>>>>>>>
>>>>>>>> Thank you
>>>>>>>> CHANNARA
>>>>>>>>
>>>>>>>> _______________________________________________
>>>>>>>> Mailing list: https://launchpad.net/~dhis2-devs
>>>>>>>> Post to     : dhis2-devs@xxxxxxxxxxxxxxxxxxx
>>>>>>>> Unsubscribe : https://launchpad.net/~dhis2-devs
>>>>>>>> More help   : https://help.launchpad.net/ListHelp
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>> _______________________________________________
>>>> Mailing list: https://launchpad.net/~dhis2-devs
>>>> Post to     : dhis2-devs@xxxxxxxxxxxxxxxxxxx
>>>> Unsubscribe : https://launchpad.net/~dhis2-devs
>>>> More help   : https://help.launchpad.net/ListHelp
>>>>
>>>>
>>>
>>
>

Follow ups

References