fix: extractBrush parameter type (#2670)

Update type in method `extractBrush`

A color value should never be undefined or null.
There is a check implemented like this: `!fill && typeof fill !==
'number'`

You can easily run an example application with Test2670, which I've
created.

| OS      | Implemented |
| ------- | :---------: |
| iOS     |    ✅      |
| MacOS   |    ✅      |
| Android |    ✅      |
| Web     |    ✅      |

---------

Author: bohdanprog

apps/common/test/Test2670.tsx

@@ -0,0 +1,70 @@
+import React, {useState} from 'react';
+import {View, Button, Text} from 'react-native';
+import {Svg, Path} from 'react-native-svg';
+
+const PathChangingFillFromValueToUndefined = () => {
+  const [fill, setFill] = useState<string | undefined>(undefined);
+
+  const changeFill = () => {
+    setFill(prevFill => (prevFill ? undefined : 'red'));
+  };
+
+  return (
+    <View
+      style={{
+        width: 200,
+        margin: 50,
+        aspectRatio: 1,
+        alignItems: 'center',
+        justifyContent: 'center',
+      }}>
+      <Text>Path which on a button click sets or resets fill to undefined</Text>
+      <Button onPress={changeFill} title="Change fill" />
+      <Svg height="100" width="100">
+        <Path
+          d="M25 10 L98 65 L70 25 L16 77 L11 30 L0 4 L90 50 L50 10 L11 22 L77 95 L20 25"
+          fill={fill}
+        />
+      </Svg>
+    </View>
+  );
+};
+
+const PathChangingFillFromValueToFillNone = () => {
+  const [fill, setFill] = useState<string | 'none'>('none');
+
+  const changeFill = () => {
+    setFill(prevFill => (prevFill !== 'none' ? 'none' : 'red'));
+  };
+
+  return (
+    <View
+      style={{
+        width: 200,
+        margin: 50,
+        aspectRatio: 1,
+        alignItems: 'center',
+        justifyContent: 'center',
+      }}>
+      <Text>
+        Path which on a button click sets or resets fill to none value
+      </Text>
+      <Button onPress={changeFill} title="Change fill" />
+      <Svg height="100" width="100">
+        <Path
+          d="M25 10 L98 65 L70 25 L16 77 L11 30 L0 4 L90 50 L50 10 L11 22 L77 95 L20 25"
+          fill={fill}
+        />
+      </Svg>
+    </View>
+  );
+};
+
+export default function App() {
+  return (
+    <View style={{justifyContent: 'center', alignItems: 'center', flex: 1}}>
+      <PathChangingFillFromValueToUndefined />
+      <PathChangingFillFromValueToFillNone />
+    </View>
+  );
+}

apps/common/test/index.tsx

@@ -35,6 +35,7 @@ import Test2417 from './Test2417';
 import Test2455 from './Test2455';
 import Test2471 from './Test2471';
 import Test2520 from './Test2520';
+import Test2670 from './Test2670';
 
 export default function App() {
   return <ColorTest />;

src/lib/extract/extractBrush.ts

@@ -8,8 +8,8 @@ const currentColorBrush = { type: 2 };
 const contextFillBrush = { type: 3 };
 const contextStrokeBrush = { type: 4 };
 
-export default function extractBrush(color?: ColorValue) {
-  if (!color || color === 'none') {
+export default function extractBrush(color: ColorValue) {
+  if (color === 'none') {
     return null;
   }