🐛 Review form, fix onChange handlers (#1839) (#1848)

On the review form, fix the `onChange` handler for the following
`NumberInput` fields:
  - criticality
  - priority

Note: This fix is a follow up to the fix in #1831. A grep across the
codebase showed 2 more of the same problems.

Resolves: #1844
Backport-of: #1839

Signed-off-by: Scott J Dickerson <sdickers@redhat.com>
Co-authored-by: Ian Bolton <ibolton@redhat.com>
Signed-off-by: Cherry Picker <noreply@github.com>

client/src/app/pages/review/components/review-form/review-form.tsx

@@ -222,13 +222,11 @@ export const ReviewForm: React.FC<IReviewFormProps> = ({
             value={value}
             min={1}
             max={10}
-            onMinus={() => {
-              onChange((value || 0) - 1);
-            }}
-            onChange={() => onChange}
-            onPlus={() => {
-              onChange((value || 0) + 1);
-            }}
+            onMinus={() => onChange((value || 0) - 1)}
+            onChange={(event) =>
+              onChange((event.target as HTMLInputElement).value)
+            }
+            onPlus={() => onChange((value || 0) + 1)}
           />
         )}
       />
@@ -247,13 +245,11 @@ export const ReviewForm: React.FC<IReviewFormProps> = ({
             value={value}
             min={1}
             max={10}
-            onMinus={() => {
-              onChange((value || 0) - 1);
-            }}
-            onChange={() => onChange}
-            onPlus={() => {
-              onChange((value || 0) + 1);
-            }}
+            onMinus={() => onChange((value || 0) - 1)}
+            onChange={(event) =>
+              onChange((event.target as HTMLInputElement).value)
+            }
+            onPlus={() => onChange((value || 0) + 1)}
           />
         )}
       />