[Opensim-dev] On Mantis #1370 (avatar persistence)

[バンション ヨハン]

<snip>
>> 2. GetAppearance() is using InventoryFolder to get asset IDs even though
>> the inventory asset IDs are available in 'avatarappearance' in the DB.
>> The information seems to be duplicated. Can't we use the DB IDs
>> directly? In that case we don't need to synchronize GetInventory() and
>> GetAppearance().
>This was added to deal with the "Can't find your clothes" issue.  It
>turns out that we were missing an update on appearance editing that
>meant the asset ids were out of date.  It seemed more logical to look it
>up via inventory.  If we can track down where we were loosing that
>update, we could rely on the asset fields again. 

So there was a reason :)

>> 3. One of the GetAppearance() calls seem redundant, but which? Should we
>> call GetApperance after UseCircuitCode or CompletedAgentMovement?
>>
>> We think it would work if we removed the GetInventory() dependency in
>> GetAppearance() and the second call to GetApperance(), but we are
>> reluctant to commit the change since we don't understand the design
>> completely. Hope you can enlighten us.
>Honestly, I don't understand the design completely either. :)  And worse
>than that, confirming that it works on Standalone + Grid / accross
>multiple dbs tends to show that one approach doesn't work in one of
>those environments.  At this point, any help in getting this pulled
>together would be appreciated.
>    -Sean

Good to know that you are as confused as we are. We'll have another look at the code to see if we can come up with a better solution.

Cheers,
Johan