Skip to content

Commit d37476f

Browse files
committed
Wrapped the designer in od a VBComponent as a UserFrom.
1 parent d2d5e16 commit d37476f

File tree

1 file changed

+8
-18
lines changed

1 file changed

+8
-18
lines changed

Rubberduck.VBEEditor/SafeComWrappers/VBA/VBComponent.cs

Lines changed: 8 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -43,11 +43,11 @@ public IControls Controls
4343
{
4444
var designer = IsWrappingNullReference
4545
? null
46-
: Target.Designer as VB.Forms.UserForm;
46+
: new UserForm(Target.Designer as VB.Forms.UserForm);
4747

4848
return designer == null
4949
? new Controls(null)
50-
: new Controls(designer.Controls);
50+
: designer.Controls;
5151
}
5252
}
5353

@@ -57,11 +57,11 @@ public IControls SelectedControls
5757
{
5858
var designer = IsWrappingNullReference
5959
? null
60-
: Target.Designer as VB.Forms.UserForm;
60+
: new UserForm(Target.Designer as VB.Forms.UserForm);
6161

6262
return designer == null
6363
? new Controls(null)
64-
: new Controls(designer.Selected);
64+
: designer.Selected;
6565
}
6666
}
6767

@@ -73,9 +73,10 @@ public bool HasDesigner
7373
{
7474
return false;
7575
}
76-
var designer = Target.Designer;
77-
var hasDesigner = designer != null;
78-
return hasDesigner;
76+
using (var designer = new UserForm(Target.Designer as VB.Forms.UserForm))
77+
{
78+
return !designer.IsWrappingNullReference;
79+
}
7980
}
8081
}
8182

@@ -188,17 +189,6 @@ private string ExportToTempFile()
188189
Export(path);
189190
return path;
190191
}
191-
//public override void Release(bool final = false)
192-
//{
193-
// if (!IsWrappingNullReference)
194-
// {
195-
// DesignerWindow().Release();
196-
// Controls.Release();
197-
// Properties.Release();
198-
// CodeModule.Release();
199-
// base.Release(final);
200-
// }
201-
//}
202192

203193
public override bool Equals(ISafeComWrapper<VB.VBComponent> other)
204194
{

0 commit comments

Comments
 (0)