Amend regex to prevent incorrect boolean return type

src/block/function.ts

@@ -86,7 +86,7 @@ export default class FunctionBlock extends Block
      */
     public getReturnFromName(name:string):string
     {
-        if (/^(is|has|can)/.test(name)) {
+        if (/^(is|has|can|should)(?:[A-Z0-9_]|$)/.test(name)) {
             return TypeUtil.instance.getFormattedTypeByName('bool');
         }
 

test/fixtures/functions.php

@@ -162,16 +162,61 @@ public function isSomething()
     {
     }
 
+    ////=> is-void
+    public function isotope()
+    {
+    }
+
+    ////=> is-only
+    public function is()
+    {
+    }
+
     ////=> has
     public function hasValue()
     {
     }
-
+
+    ////=> has-void
+    public function hashed()
+    {
+    }
+
+    ////=> has-only
+    public function has()
+    {
+    }
+
     ////=> can
     public function canValue()
     {
     }
 
+    ////=> can-void
+    public function cancel()
+    {
+    }
+
+    ////=> can-only
+    public function can()
+    {
+    }
+
+    ////=> should
+    public function shouldDoSomething()
+    {
+    }
+
+    ////=> should-void
+    public function shouldasdf()
+    {
+    }
+
+    ////=> should-only
+    public function should()
+    {
+    }
+
     ////=> debug-info
     public function __debugInfo()
     {

test/fixtures/functions.php.json

@@ -421,6 +421,22 @@
             "params": []
         }
     },
+    {
+        "key": "is-void",
+        "name": "Check is prefix - void",
+        "result": {
+            "return": "void",
+            "params": []
+        }
+    },
+    {
+        "key": "is-only",
+        "name": "Check is prefix - only",
+        "result": {
+            "return": "boolean",
+            "params": []
+        }
+    },
     {
         "key": "has",
         "name": "Check has prefix",
@@ -429,6 +445,22 @@
             "params": []
         }
     },
+    {
+        "key": "has-void",
+        "name": "Check has prefix - void",
+        "result": {
+            "return": "void",
+            "params": []
+        }
+    },
+    {
+        "key": "has-only",
+        "name": "Check has prefix - only",
+        "result": {
+            "return": "boolean",
+            "params": []
+        }
+    },
     {
         "key": "can",
         "name": "Check can prefix",
@@ -437,6 +469,46 @@
             "params": []
         }
     },
+    {
+        "key": "can-void",
+        "name": "Check can prefix - void",
+        "result": {
+            "return": "void",
+            "params": []
+        }
+    },
+    {
+        "key": "can-only",
+        "name": "Check can prefix - only",
+        "result": {
+            "return": "boolean",
+            "params": []
+        }
+    },
+    {
+        "key": "should",
+        "name": "Check should prefix",
+        "result": {
+            "return": "boolean",
+            "params": []
+        }
+    },
+    {
+        "key": "should-void",
+        "name": "Check should prefix - void",
+        "result": {
+            "return": "void",
+            "params": []
+        }
+    },
+    {
+        "key": "should-only",
+        "name": "Check should prefix - only",
+        "result": {
+            "return": "boolean",
+            "params": []
+        }
+    },
     {
         "key": "debug-info",
         "name": "Debug info return type",