FIX: DontIncludeInteractions being ignored (#1069).

Author: Rene Damm

Assets/Tests/InputSystem/CoreTests_Actions.cs

@@ -4845,7 +4845,7 @@ public void Actions_WhenGettingDisplayTextForBindingsOnAction_InteractionsAreSho
 
         // Can suppress.
         Assert.That(action1.GetBindingDisplayString(InputBinding.DisplayStringOptions.DontIncludeInteractions),
-            Is.EqualTo("Hold " + GamepadState.ButtonSouthShortDisplayName));
+            Is.EqualTo(GamepadState.ButtonSouthShortDisplayName));
     }
 
     [Test]

Packages/com.unity.inputsystem/CHANGELOG.md

@@ -18,6 +18,7 @@ however, it has to be formatted properly to pass verification tests.
 - Leaving play mode no longer leaves state change monitors lingering around from enabled actions.
 - Enabling action maps with bindings that do not refer to an existing action in the map no longer leads to asserts and exceptions when input on the bindings is received ([case 1213085](https://issuetracker.unity3d.com/issues/input-system-input-actions-cause-exceptions-and-should-not-get-here-errors-to-appear-after-deleting-an-action-map)).
 - `PressInteraction` no longer misses the next button press if it gets reset from within the `performed` callback ([case 1205285](https://issuetracker.unity3d.com/issues/inputsystem-problem-with-button-state-after-deactivating-and-reactivating-an-action-map)).
+- `InputBinding.DisplayStringOptions.DontIncludeInteractions` is now properly respected.
 - Reading the value of a composite binding no longer causes processors from the last active part binding to be applied rather than the processors of the composite itself, if any ([case 1207082](https://issuetracker.unity3d.com/issues/input-system-invert-processors-have-no-effect-on-the-inputaction-dot-callbackcontext-value)).
 
 ## [1.0.0-preview.5] - 2020-02-14

Packages/com.unity.inputsystem/InputSystem/Actions/InputBinding.cs

@@ -694,7 +694,7 @@ public string ToDisplayString(out string deviceLayoutName, out string controlPat
 
             var result = InputControlPath.ToHumanReadableString(pathToUse, out deviceLayoutName, out controlPath, readableStringOptions, control);
 
-            if (!string.IsNullOrEmpty(effectiveInteractions))
+            if (!string.IsNullOrEmpty(effectiveInteractions) && (options & DisplayStringOptions.DontIncludeInteractions) == 0)
             {
                 var interactionString = string.Empty;
                 var parsedInteractions = NameAndParameters.ParseMultiple(effectiveInteractions);