[SOLVED] Re: Type of CommandType (short or long) ?

[Julien Nabet]

Thank you Lionel for your quick feedback!

As you may have seen, I submitted 
https://gerrit.libreoffice.org/c/core/+/114659

If this one is ok, there are other similar cases to take a look.

Julien

On 26/04/2021 13:26, Lionel Élie Mamane wrote:
> Hi,
>
> According to
> https://api.libreoffice.org/docs/idl/ref/namespacecom_1_1sun_1_1star_1_1sdb_1_1CommandType.html
> it should be an IDL long.
>
> Best Regards,
>
> Lionel
>
> On Mon, Apr 26, 2021 at 11:20:29AM +0200, Julien Nabet wrote:
>
>> warn:unotools.config:30371:30371:unotools/source/config/configitem.cxx:487:
>> Exception from PutProperties com.sun.star.lang.IllegalArgumentException
>> message: configmgr inappropriate property value
>> /home/julien/lo/libreoffice/configmgr/source/access.cxx:1497
>> ArgumentPosition: -1
>   
>> Taking a look at utl::ConfigItem::PutProperties, here what I got:
>>
>> (gdb) p rNames
>> $54 = uno::Sequence of length 8 = {"CurrentDataSource/DataSourceName",
>> "CurrentDataSource/Command", "CurrentDataSource/CommandType",
>> "BeamerHeight", "ViewHeight", "QueryText", "QueryField",
>>    "ShowColumnAssignmentWarning"}
>> (gdb) p rValues
>> $55 = uno::Sequence of length 8 = {uno::Any("string": "Bibliography"),
>> uno::Any("string": "biblio"), uno::Any("long": 0), uno::Any("hyper": 0),
>> uno::Any("hyper": 0), uno::Any("string": ""),
>>    uno::Any("string": "Identifier"), uno::Any("boolean": 1 '\001')}
>>
>> Then for:
>>
>> #3  0x00007f49e39db589 in configmgr::Access::replaceByName (this=0x77282a0,
>> aName="CommandType", aElement=uno::Any("long": 0)) at
>> configmgr/source/access.cxx:1068
>> 1068                child->setProperty(aElement, &localMods);
>>
>> I saw that the throw was during:
>>
>> #1  0x00007f49e39dd4b5 in configmgr::Access::checkValue (this=0xb86d9a0,
>> value=uno::Any("long": 0), type=configmgr::TYPE_SHORT, nillable=true) at
>> configmgr/source/access.cxx:1497
>> 1497            throw css::lang::IllegalArgumentException(
>>
>> git grepping about "CommandType", most of the times it's considered as long:
>>
>>
>> but sometimes CommandType is considered as a short:
>>