Skip to content

Commit cb1c562

Browse files
author
David Kline (ANALOG)
committed
svc locator now using non-data provider IMixedRealityServiceRegistrar methods
1 parent a492a23 commit cb1c562

File tree

12 files changed

+143
-236
lines changed

12 files changed

+143
-236
lines changed

Assets/MixedRealityToolkit.Services/InputSystem/FocusProvider.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ float IMixedRealityFocusProvider.GlobalPointingExtent
3939
{
4040
get
4141
{
42-
if (MixedRealityToolkit.HasActiveProfile &&
42+
if (MixedRealityToolkit.Instance.HasActiveProfile &&
4343
MixedRealityToolkit.Instance.ActiveProfile.IsInputSystemEnabled &&
4444
MixedRealityToolkit.Instance.ActiveProfile.InputSystemProfile.PointerProfile != null)
4545
{
@@ -59,7 +59,7 @@ public LayerMask[] FocusLayerMasks
5959
{
6060
if (focusLayerMasks == null)
6161
{
62-
if (MixedRealityToolkit.HasActiveProfile &&
62+
if (MixedRealityToolkit.Instance.HasActiveProfile &&
6363
MixedRealityToolkit.Instance.ActiveProfile.IsInputSystemEnabled &&
6464
MixedRealityToolkit.Instance.ActiveProfile.InputSystemProfile.PointerProfile != null)
6565
{

Assets/MixedRealityToolkit.Services/SpatialAwarenessSystem/MixedRealitySpatialAwarenessSystem.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ public override void Enable()
5959
}
6060

6161
// Get the collection of registered observers.
62-
List<Core.Interfaces.IMixedRealityService> services = MixedRealityToolkit.Instance.GetActiveServices(typeof(IMixedRealitySpatialAwarenessObserver));
62+
IReadOnlyList<IMixedRealitySpatialAwarenessObserver> services = MixedRealityToolkit.Instance.GetServices<IMixedRealitySpatialAwarenessObserver>();
6363
for (int i = 0; i < services.Count; i++)
6464
{
6565
observers.Add(services[i] as IMixedRealitySpatialAwarenessObserver);

Assets/MixedRealityToolkit.Tests/Core/TestFixture_01_MixedRealityToolkitTests.cs

Lines changed: 59 additions & 59 deletions
Large diffs are not rendered by default.

Assets/MixedRealityToolkit.Tests/InputSystem/TestFixture_03_InputSystemTests.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,9 @@ public void Test01_CreateMixedRealityInputSystem()
3737
MixedRealityToolkit.Instance.RegisterService<IMixedRealityInputSystem>(new MixedRealityInputSystem());
3838

3939
// Tests
40-
Assert.IsNotEmpty(MixedRealityToolkit.ActiveSystems);
41-
Assert.AreEqual(1, MixedRealityToolkit.ActiveSystems.Count);
42-
Assert.AreEqual(0, MixedRealityToolkit.RegisteredMixedRealityServices.Count);
40+
Assert.IsNotEmpty(MixedRealityToolkit.Instance.ActiveSystems);
41+
Assert.AreEqual(1, MixedRealityToolkit.Instance.ActiveSystems.Count);
42+
Assert.AreEqual(0, MixedRealityToolkit.Instance.RegisteredMixedRealityServices.Count);
4343
}
4444

4545
[Test]

Assets/MixedRealityToolkit.Tests/TestUtilities.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ public static void InitializeMixedRealityToolkitScene(bool useDefaultProfile = f
3333
// Tests
3434
Assert.IsTrue(MixedRealityToolkit.IsInitialized);
3535
Assert.IsNotNull(MixedRealityToolkit.Instance);
36-
Assert.IsFalse(MixedRealityToolkit.HasActiveProfile);
36+
Assert.IsFalse(MixedRealityToolkit.Instance.HasActiveProfile);
3737

3838
var configuration = useDefaultProfile
3939
? GetDefaultMixedRealityProfile<MixedRealityToolkitConfigurationProfile>()

Assets/MixedRealityToolkit/Inspectors/Profiles/MixedRealityToolkitConfigurationProfileInspector.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ protected override void OnEnable()
8989
return;
9090
}
9191

92-
if (!MixedRealityToolkit.HasActiveProfile)
92+
if (!MixedRealityToolkit.Instance.HasActiveProfile)
9393
{
9494
return;
9595
}

Assets/MixedRealityToolkit/Inspectors/PropertyDrawers/InputActionPropertyDrawer.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ public class InputActionPropertyDrawer : PropertyDrawer
1919

2020
public override void OnGUI(Rect rect, SerializedProperty property, GUIContent content)
2121
{
22-
if (!MixedRealityToolkit.IsInitialized || !MixedRealityToolkit.HasActiveProfile)
22+
if (!MixedRealityToolkit.IsInitialized || !MixedRealityToolkit.Instance.HasActiveProfile)
2323
{
2424
profile = null;
2525
actionLabels = new[] { new GUIContent("Missing Mixed Reality Toolkit") };

Assets/MixedRealityToolkit/Inspectors/Utilities/CanvasEditorExtension.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ public class CanvasEditorExtension : Editor
2323

2424
private bool hasUtility = false;
2525

26-
private static bool IsUtilityValid => MixedRealityToolkit.HasActiveProfile && MixedRealityToolkit.Instance.ActiveProfile.IsInputSystemEnabled && MixedRealityToolkit.InputSystem?.FocusProvider != null;
26+
private static bool IsUtilityValid => MixedRealityToolkit.Instance.HasActiveProfile && MixedRealityToolkit.Instance.ActiveProfile.IsInputSystemEnabled && MixedRealityToolkit.InputSystem?.FocusProvider != null;
2727

2828
private void OnEnable()
2929
{

Assets/MixedRealityToolkit/Inspectors/Utilities/MixedRealityInspectorUtility.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ public static bool CheckMixedRealityConfigured(bool showHelpBox = true)
4343
MixedRealityToolkit.ConfirmInitialized();
4444
}
4545

46-
if (!MixedRealityToolkit.HasActiveProfile)
46+
if (!MixedRealityToolkit.Instance.HasActiveProfile)
4747
{
4848
if (showHelpBox)
4949
{

Assets/MixedRealityToolkit/Interfaces/Registrars/IMixedRealityServiceRegistrar.cs

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ public interface IMixedRealityServiceRegistrar
1919
/// </summary>
2020
/// <typeparam name="T">The interface type of the service to be registered (ex: IMixedRealityBoundarySystem).</typeparam>
2121
/// <param name="serviceInstance">An instance of the service to be registered.</param>
22-
bool RegisterService<T>(IMixedRealityService serviceInstance) where T : IMixedRealityService;
22+
bool RegisterService<T>(T serviceInstance) where T : IMixedRealityService;
2323

2424
/// <summary>
2525
/// Registers a service of the specified type.
@@ -68,18 +68,13 @@ bool RegisterService<T>(
6868
/// <returns>The registered service instance as the requested type.</returns>
6969
T GetService<T>(string name = null, bool showLogs = true) where T : IMixedRealityService;
7070

71-
/// <summary>
72-
/// Gets the collection of the registered service instances matching the requested type.
73-
/// </summary>
74-
/// <returns>Read-only collection of the service instances, as IMixedRealityService.</returns>
75-
IReadOnlyList<IMixedRealityService> GetServices();
76-
7771
/// <summary>
7872
/// Gets the collection of the registered service instances matching the requested type.
7973
/// </summary>
8074
/// <typeparam name="T">The interface type of the service (ex: IMixedRealityBoundarySystem).
75+
/// <param name="name">Friendly name of the service.</param>
8176
/// <returns>Read-only collection of the service instances, as tye requested type.</returns>
82-
IReadOnlyList<T> GetServices<T>() where T : IMixedRealityService;
77+
IReadOnlyList<T> GetServices<T>(string name = null) where T : IMixedRealityService;
8378

8479
#endregion IMixedRealityServce registration
8580

0 commit comments

Comments
 (0)