Rubberduck: Extract Interface should be disabled for document modules

Created on 13 Jan 2020  路  1Comment  路  Source: rubberduck-vba/Rubberduck

Extract Interface is currently (v2.5.0.x) offered in the code-behind of document modules.

While Implements statements on Access forms apparently aren't an issue, the vast majority of our users would work in Excel, and having Implements IAnything on a worksheet's code-behind module is a sure-shot way to crash the host.

Rubberduck should leave that command disabled for document modules.

bug feature-refactorings
Source
picture retailcoder

Most helpful comment

IMO, it doesn't make sense to disable a feature just because it doesn't work for a host which happens to be the majority. After all, the minority that develop solutions in Access or indeed any other VBA hosts might end up filing a bug report about this being disabled.

A more sensible solution is to disable this on host-specific basis, IMO. We already have HostRequired attribute; we only need to have HostExcluded attribute, right?

picture bclothier on 13 Jan 2020
👍2

>All comments

IMO, it doesn't make sense to disable a feature just because it doesn't work for a host which happens to be the majority. After all, the minority that develop solutions in Access or indeed any other VBA hosts might end up filing a bug report about this being disabled.

A more sensible solution is to disable this on host-specific basis, IMO. We already have HostRequired attribute; we only need to have HostExcluded attribute, right?

bclothier picture bclothier on 13 Jan 2020
👍2
Was this page helpful?
0 / 5 - 0 ratings

Related issues

eteichm picture eteichm  路  4Comments
retailcoder picture retailcoder  路  3Comments
bclothier picture bclothier  路  3Comments
connerk picture connerk  路  3Comments
DecimalTurn picture DecimalTurn  路  3Comments