An extension to show associated family parameters

[CADBIMDeveloper]

Summary of the Pull Request

What is this about: Added an ability to show associated family parameters Description:

Quality Checklist

• [x] My code follows the style guidelines of this project
• [x] I have performed a self-review of my own code
• [ ] I have made corresponding changes to the documentation
• [x] My changes generate no new warnings

Issues (the PR is a draft now)

1. it requires the magic to make the family parameter snoopable:

   public sealed class FamilyParameterDescriptor : Descriptor, IDescriptorResolver // <-- !!!
   {
   ...
   public ResolveSet Resolve(Document context, string target, ParameterInfo[] parameters)
   {
       return null; // <-- !!!
   }
   }

   otherwise it shows a non-snoopable text like:

if I remove FamilyParameterDescriptor registration (what I obviously don't want to do :-)) https://github.com/jeremytammik/RevitLookup/blob/f52d2fef57c27159d0029866daba12bf482c5351/RevitLookup/Core/ComponentModel/DescriptorMap.cs#L68, it also becomes snoopable:

@Nice3point , you know the codebase much deeper than me, probably, it's better if you'll fix it :-)

2. As a suggestion would be nice to extend IExtensionManager or DescriptorExtension to be able to control when the extension should be added, for example adding another extension registration method like:

void Register<T>(T value, Action<DescriptorExtension<T>> extension, Func<DescriptorExtension<T>, bool> isActive);

So we'll avoid adding GetAssociateFamilyParameters extension in non-family editor contexts

[Nice3point]

@CADBIMDeveloper

1. реализуй IDescriptorCollector вместо IDescriptorResolver https://github.com/jeremytammik/RevitLookup/blob/dev/Contributing.md#idescriptorcollector

   Это работает потому что IDescriptorResolver уже реализует IDescriptorCollector и позволяет снупить объект https://github.com/jeremytammik/RevitLookup/blob/dev/Contributing.md#idescriptorcollector

   Ну а удаление строчки из DescriptorMap позволяет снупить потому что этот класс FamilyParameter наследуется от ApiObject, который уже и так зарегистрирован, лукап использует более базовый класс, вот и вся магия

2. Можно обойтись и без этой перегрузки, а просто добавить обычный if перед регистрацией расширения

   
   if(_parameter.Element.Document.IsFamilyDocument)
   {
       manager.Register(...);
   }

[CADBIMDeveloper]

Ок, завтра)

P.S. IDescriptorCollector проморгал как-то

[Nice3point]

Да давай) без проблем

[CADBIMDeveloper]

@Nice3point , привет.

я чуток по другому сделал вместо:

if(_parameter.Element.Document.IsFamilyDocument)

IExtensionManager теперь имеет публичное свойство документа.

Мотивация: не у всего в API можно достучаться до документа, т.е. либо передавать в Descriptor ещё и документ (что ИМХО, не очень) либо не иметь возможности делать больше Extension-ов.

Пример - тот самый FamilyParameter. Там есть простор для того, чтобы расширять его Extension-ами с доступом через FamilyManager

[Nice3point]

Привет, да я тоже думал там документ свойством потом сделать именно для этого. Спасибо что сделал это сам. Я гляну Pr чуть позже как за комп сяду)

[Nice3point]

@jeremytammik hey Jeremy, what about checking the PR and getting everything agreed? (

[jeremytammik]

hi roman, i did not translate the russian, and assumed that my agreement was required purely pro forma. the RevitLookup coding has reached a point now that exceeds my competence. i trust and hope that i can leave it in your capable hands to check. is that OK?

[CADBIMDeveloper]

Sorry, Jeremy :-) @Nice3point , we should use English here

[Nice3point]

@jeremytammik Yes, Jeremy no problem, we will discuss it in English, it was just quicker to solve the problem in the native language. I can merge the PRs myself, it's fine)

[jeremytammik]

Thank you! That is a relief!