dhis2-devs team mailing list archive

[Mark Polak <mark@xxxxxxxxx>]

Hey Paolo,

We have made this action unavailable in the UI, as like Jason says, this is
generally not something you would want.

I would say the test case is indeed correct. We should enforce the same
logic on the server.

On Mon, Oct 10, 2016 at 6:32 AM, Jason Pickering <
jason.p.pickering@xxxxxxxxx> wrote:

> Hi Paulo,
>
> I would expect that this operation should be disallowed by the server. The
> reason for this is that once you have established that a given option set
> has a type "INTEGER" , you should not be able to change this afterwards to
> a different type. It seems the test is correct to me, but from the results
> of the test, it seems it is possible to update.
>
> Maybe the developers can comment more?
>
> Regards,
> Jason
>
>
> On Sun, Oct 9, 2016 at 5:58 PM, Paulo Grácio <paulogracio@xxxxxxxxx>
> wrote:
>
>> Hi devs,
>>
>> should it be possible to update existent Option Set type from 'INTEGER'
>> to ‘TEXT'?
>>
>> Test Case: https://github.com/dhis2/api-tests/blob/master/modules/data-
>> admin/optionSet.js#L77
>> Failing Job: https://travis-ci.org/dhis2/api-tests/jobs/166211140
>>
>> BR,
>> Paulo Grácio
>>
>> _______________________________________________
>> 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
>>
>>
>
>
> --
> Jason P. Pickering
> email: jason.p.pickering@xxxxxxxxx
> tel:+46764147049
>
> _______________________________________________
> 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
>
>


-- 
Regards,


Mark Polak
Software developer, DHIS 2
University of Oslo
http://www.dhis2.org <https://www.dhis2.org/>
mark@xxxxxxxxx