Add psalm checks

.phan/config.php

@@ -0,0 +1,95 @@
+<?php
+
+/**
+ * This configuration will be read and overlaid on top of the
+ * default configuration. Command line arguments will be applied
+ * after this file is read.
+ *
+ * @see src/Phan/Config.php
+ * See Config for all configurable options.
+ *
+ * A Note About Paths
+ * ==================
+ *
+ * Files referenced from this file should be defined as
+ *
+ * ```
+ *   Config::projectPath('relative_path/to/file')
+ * ```
+ *
+ * where the relative path is relative to the root of the
+ * project which is defined as either the working directory
+ * of the phan executable or a path passed in via the CLI
+ * '-d' flag.
+ */
+
+// use Phan\Config;
+
+return [
+	// If true, missing properties will be created when
+	// they are first seen. If false, we'll report an
+	// error message.
+	"allow_missing_properties" => false,
+
+	// Allow null to be cast as any type and for any
+	// type to be cast to null.
+	"null_casts_as_any_type" => false,
+
+	// Backwards Compatibility Checking
+	'backward_compatibility_checks' => true,
+
+	// Run a quick version of checks that takes less
+	// time
+	"quick_mode" => false,
+
+	// Only emit critical issues to start with
+	// (0 is low severity, 5 is normal severity, 10 is critical)
+	"minimum_severity" => 10,
+
+	// default false for include path check
+	"enable_include_path_checks" => true,
+	"include_paths" => [
+	],
+	'ignore_undeclared_variables_in_global_scope' => true,
+
+	"file_list" => [
+	],
+
+	// A list of directories that should be parsed for class and
+	// method information. After excluding the directories
+	// defined in exclude_analysis_directory_list, the remaining
+	// files will be statically analyzed for errors.
+	//
+	// Thus, both first-party and third-party code being used by
+	// your application should be included in this list.
+	'directory_list' => [
+		// Change this to include the folders you wish to analyze
+		// (and the folders of their dependencies)
+		'.'
+		// 'www',
+		// To speed up analysis, we recommend going back later and
+		// limiting this to only the vendor/ subdirectories your
+		// project depends on.
+		// `phan --init` will generate a list of folders for you
+	],
+
+
+	// A list of directories holding code that we want
+	// to parse, but not analyze
+	"exclude_analysis_directory_list" => [
+	],
+	'exclude_file_list' => [
+	],
+
+	// what not to show as problem
+	'suppress_issue_types' => [
+		// 'PhanUndeclaredMethod',
+		'PhanEmptyFile',
+	],
+
+	// Override to hardcode existence and types of (non-builtin) globals in the global scope.
+	// Class names should be prefixed with `\`.
+	//
+	// (E.g. `['_FOO' => '\FooClass', 'page' => '\PageClass', 'userId' => 'int']`)
+	'globals_type_map' => [],
+];

phpstan.neon

@@ -0,0 +1,7 @@
+# PHP Stan Config
+
+parameters:
+	tmpDir: /tmp/phpstan-codeblocks-dotenv
+	level: max
+	paths:
+		- %currentWorkingDirectory%

psalm.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0"?>
+<psalm
+    errorLevel="1"
+    resolveFromConfigFile="true"
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    xmlns="https://getpsalm.org/schema/config"
+    xsi:schemaLocation="https://getpsalm.org/schema/config vendor/vimeo/psalm/config.xsd"
+>
+    <projectFiles>
+        <directory name="src" />
+        <ignoreFiles>
+        </ignoreFiles>
+    </projectFiles>
+</psalm>

src/read_env_file.php

@@ -15,16 +15,13 @@
  *
  * @param  string $path     Folder to file, default is __DIR__
  * @param  string $env_file What file to load, default is .env
- * @return int              -1 other error
- *                          0 for success full load
+ * @return int              0 for success full load
  *                          1 for file loadable, but no data inside
  *                          2 for file not readable
  *                          3 for file not found
  */
 function readEnvFile(string $path = __DIR__, string $env_file = '.env'): int
 {
-	// default -1;
-	$status = -1;
 	$env_file_target = $path . DIRECTORY_SEPARATOR . $env_file;
 	// this is not a file -> abort
 	if (!is_file($env_file_target)) {
@@ -42,7 +39,7 @@ function readEnvFile(string $path = __DIR__, string $env_file = '.env'): int
 	$var = '';
 	while ($line = fgets($fp)) {
 		// main match for variable = value part
-		if (preg_match("/^\s*([\w_]+)\s*=\s*((\"?).*)/", $line, $matches)) {
+		if (preg_match("/^\s*([\w_.]+)\s*=\s*((\"?).*)/", $line, $matches)) {
 			$var = $matches[1];
 			$value = $matches[2];
 			$quotes = $matches[3];
@@ -74,6 +71,7 @@ function readEnvFile(string $path = __DIR__, string $env_file = '.env'): int
 				$line = $matches[1];
 			}
 			// strip line of slashes
+			/** @psalm-suppress MixedOperand */
 			$_ENV[$var] .= stripslashes($line);
 		}
 	}

test/.env

@@ -8,6 +8,8 @@ FAILURE = ABC
 SIMPLEBOX= A B  C
 TITLE=1
 FOO=1.2
+SOME.TEST=Test Var
+SOME.LIVE=Live Var
 Test="A"
 TEST="B"
 TEST="D"