oliverdavies.uk/content/node.7064860b-42de-47c3-b5ac-3560d941fe0f.json

{
    "uuid": [
        {
            "value": "7064860b-42de-47c3-b5ac-3560d941fe0f"
        }
    ],
    "langcode": [
        {
            "value": "en"
        }
    ],
    "type": [
        {
            "target_id": "daily_email",
            "target_type": "node_type",
            "target_uuid": "8bde1f2f-eef9-4f2d-ae9c-96921f8193d7"
        }
    ],
    "revision_timestamp": [
        {
            "value": "2025-05-11T09:00:20+00:00"
        }
    ],
    "revision_uid": [
        {
            "target_type": "user",
            "target_uuid": "b8966985-d4b2-42a7-a319-2e94ccfbb849"
        }
    ],
    "revision_log": [],
    "status": [
        {
            "value": true
        }
    ],
    "uid": [
        {
            "target_type": "user",
            "target_uuid": "b8966985-d4b2-42a7-a319-2e94ccfbb849"
        }
    ],
    "title": [
        {
            "value": "Which level is right for you?"
        }
    ],
    "created": [
        {
            "value": "2024-02-20T00:00:00+00:00"
        }
    ],
    "changed": [
        {
            "value": "2025-05-11T09:00:20+00:00"
        }
    ],
    "promote": [
        {
            "value": false
        }
    ],
    "sticky": [
        {
            "value": false
        }
    ],
    "default_langcode": [
        {
            "value": true
        }
    ],
    "revision_translation_affected": [
        {
            "value": true
        }
    ],
    "path": [
        {
            "alias": "\/daily\/2024\/02\/20\/which-level-is-right-for-you",
            "langcode": "en"
        }
    ],
    "body": [
        {
            "value": "\n          <p>Today, whilst working on <a href=\"\/daily\/2024\/02\/19\/introducing-versa\">Versa<\/a>, I was experimenting with different PHPStan levels.<\/p>\n\n<p>Level 1 is the least strict level, and applies the fewest rules and returns the fewest results.<\/p>\n\n<p>As you increase the level, the stricter PHPStan is.<\/p>\n\n<h2 id=\"levelling-up-to-9\">Levelling up to 9<\/h2>\n\n<p>Here is the code to get the values of the <code>--extra-args<\/code> and <code>--working-dir<\/code> options:<\/p>\n\n<pre><code class=\"language-php\">$extraArgs = $input-&gt;getOption('extra-args');\n$workingDir = $input-&gt;getOption('working-dir');\n<\/code><\/pre>\n\n<p>This passed PHPStan level 8, but these are the errors I get when running level 9:<\/p>\n\n<pre><code class=\"language-plain\">------ -------------------------------------------------------------------------------------------------------\n Line   versa\n------ -------------------------------------------------------------------------------------------------------\n 61     Parameter $extraArgs of static method App\\Process\\Process::create() expects string|null, mixed given.\n 62     Parameter $workingDir of static method App\\Process\\Process::create() expects string, mixed given.\n 72     Parameter $extraArgs of static method App\\Process\\Process::create() expects string|null, mixed given.\n 73     Parameter $workingDir of static method App\\Process\\Process::create() expects string, mixed given.\n 84     Parameter $extraArgs of static method App\\Process\\Process::create() expects string|null, mixed given.\n 85     Parameter $workingDir of static method App\\Process\\Process::create() expects string, mixed given.\n 94     Parameter $extraArgs of static method App\\Process\\Process::create() expects string|null, mixed given.\n 95     Parameter $workingDir of static method App\\Process\\Process::create() expects string, mixed given.\n 104    Parameter $extraArgs of static method App\\Process\\Process::create() expects string|null, mixed given.\n 105    Parameter $workingDir of static method App\\Process\\Process::create() expects string, mixed given.\n 119    Parameter $extraArgs of static method App\\Process\\Process::create() expects string|null, mixed given.\n 120    Parameter $workingDir of static method App\\Process\\Process::create() expects string, mixed given.\n------ -------------------------------------------------------------------------------------------------------\n\n[ERROR] Found 12 errors\n<\/code><\/pre>\n\n<p>The issue is that <code>$input-&gt;getOption()<\/code> from Symfony's <code>InputInterface<\/code> returns <code>mixed<\/code> - even though it always returns <code>null<\/code> or a string.<\/p>\n\n<p>If I did <code>--working-dir 2<\/code>, it would return the string <code>\"2\"<\/code>.<\/p>\n\n<p>An empty string throws an error, and an empty value (if there are no extra arguments) returns <code>null<\/code>.<\/p>\n\n<p>So, I know <code>$workingDir<\/code> will always be a string and <code>$extraArgs<\/code> will either a string or <code>null<\/code>.<\/p>\n\n<h2 id=\"passing-level-8\">Passing level 8<\/h2>\n\n<p>To pass level 8, PHPStan needs to be sure the variables are what I think they are.<\/p>\n\n<p>Here's the code I can use to get it to pass:<\/p>\n\n<pre><code class=\"language-php\">$extraArgs = $input-&gt;getOption('extra-args');\n$workingDir = $input-&gt;getOption('working-dir');\nassert(is_string($extraArgs) || is_null($extraArgs));\nassert(is_string($workingDir));\n<\/code><\/pre>\n\n<p>By using <code>assert()<\/code> and throwing an Exception if the condition fails, PHPStan is now happy with the code and my code passes level 9.<\/p>\n\n<h2 id=\"here%27s-the-thing\">Here's the thing<\/h2>\n\n<p>Although the extra code gets PHPStan to pass, it it worth it?<\/p>\n\n<p>Is this extra code adding value? Does it make the code more readable or is it likely to prevent a bug?<\/p>\n\n<p>In this case, I know the value will always be the type I expect.<\/p>\n\n<p>I can work around this using a baseline or annotations 
            "format": "full_html",
            "processed": "\n          <p>Today, whilst working on <a href=\"http:\/\/default\/daily\/2024\/02\/19\/introducing-versa\">Versa<\/a>, I was experimenting with different PHPStan levels.<\/p>\n\n<p>Level 1 is the least strict level, and applies the fewest rules and returns the fewest results.<\/p>\n\n<p>As you increase the level, the stricter PHPStan is.<\/p>\n\n<h2 id=\"levelling-up-to-9\">Levelling up to 9<\/h2>\n\n<p>Here is the code to get the values of the <code>--extra-args<\/code> and <code>--working-dir<\/code> options:<\/p>\n\n<pre><code class=\"language-php\">$extraArgs = $input-&gt;getOption('extra-args');\n$workingDir = $input-&gt;getOption('working-dir');\n<\/code><\/pre>\n\n<p>This passed PHPStan level 8, but these are the errors I get when running level 9:<\/p>\n\n<pre><code class=\"language-plain\">------ -------------------------------------------------------------------------------------------------------\n Line   versa\n------ -------------------------------------------------------------------------------------------------------\n 61     Parameter $extraArgs of static method App\\Process\\Process::create() expects string|null, mixed given.\n 62     Parameter $workingDir of static method App\\Process\\Process::create() expects string, mixed given.\n 72     Parameter $extraArgs of static method App\\Process\\Process::create() expects string|null, mixed given.\n 73     Parameter $workingDir of static method App\\Process\\Process::create() expects string, mixed given.\n 84     Parameter $extraArgs of static method App\\Process\\Process::create() expects string|null, mixed given.\n 85     Parameter $workingDir of static method App\\Process\\Process::create() expects string, mixed given.\n 94     Parameter $extraArgs of static method App\\Process\\Process::create() expects string|null, mixed given.\n 95     Parameter $workingDir of static method App\\Process\\Process::create() expects string, mixed given.\n 104    Parameter $extraArgs of static method App\\Process\\Process::create() expects string|null, mixed given.\n 105    Parameter $workingDir of static method App\\Process\\Process::create() expects string, mixed given.\n 119    Parameter $extraArgs of static method App\\Process\\Process::create() expects string|null, mixed given.\n 120    Parameter $workingDir of static method App\\Process\\Process::create() expects string, mixed given.\n------ -------------------------------------------------------------------------------------------------------\n\n[ERROR] Found 12 errors\n<\/code><\/pre>\n\n<p>The issue is that <code>$input-&gt;getOption()<\/code> from Symfony's <code>InputInterface<\/code> returns <code>mixed<\/code> - even though it always returns <code>null<\/code> or a string.<\/p>\n\n<p>If I did <code>--working-dir 2<\/code>, it would return the string <code>\"2\"<\/code>.<\/p>\n\n<p>An empty string throws an error, and an empty value (if there are no extra arguments) returns <code>null<\/code>.<\/p>\n\n<p>So, I know <code>$workingDir<\/code> will always be a string and <code>$extraArgs<\/code> will either a string or <code>null<\/code>.<\/p>\n\n<h2 id=\"passing-level-8\">Passing level 8<\/h2>\n\n<p>To pass level 8, PHPStan needs to be sure the variables are what I think they are.<\/p>\n\n<p>Here's the code I can use to get it to pass:<\/p>\n\n<pre><code class=\"language-php\">$extraArgs = $input-&gt;getOption('extra-args');\n$workingDir = $input-&gt;getOption('working-dir');\nassert(is_string($extraArgs) || is_null($extraArgs));\nassert(is_string($workingDir));\n<\/code><\/pre>\n\n<p>By using <code>assert()<\/code> and throwing an Exception if the condition fails, PHPStan is now happy with the code and my code passes level 9.<\/p>\n\n<h2 id=\"here%27s-the-thing\">Here's the thing<\/h2>\n\n<p>Although the extra code gets PHPStan to pass, it it worth it?<\/p>\n\n<p>Is this extra code adding value? Does it make the code more readable or is it likely to prevent a bug?<\/p>\n\n<p>In this case, I know the value will always be the type I expect.<\/p>\n\n<p>I can work around this using a base
            "summary": null
        }
    ]
}