Updates constructor to use reference for job parameter

Vítězslav Dvořák · Vítězslav Dvořák · commit 42424bed6948 · 2025-09-11T14:05:55.000+02:00
Modifies the constructor of the Docker class to accept a reference to the Job object, enhancing performance by avoiding unnecessary copies.

This change supports better memory management and prepares the class for potential future modifications that may require direct manipulation of the job instance.
diff --git a/src/MultiFlexi/Executor/Docker.php b/src/MultiFlexi/Executor/Docker.php
@@ -27,9 +27,9 @@ class Docker extends Native implements \MultiFlexi\executor
     use \Ease\Logger\Logging;
     public const PULLCOMMAND = 'docker pull docker.io/vitexsoftware/debian:bookworm';
 
-    public function __construct($job)
+   public function __construct(\MultiFlexi\Job &$job)
     {
-        $this->job = $job;
+        parent::__construct($job);
     }
 
     public static function name(): string

Original file line number	Diff line number	Diff line change
`@@ -27,9 +27,9 @@ class Docker extends Native implements \MultiFlexi\executor`
`27`	`27`	`use \Ease\Logger\Logging;`
`28`	`28`	`public const PULLCOMMAND = 'docker pull docker.io/vitexsoftware/debian:bookworm';`
`29`	`29`
`30`		`- public function __construct($job)`
	`30`	`+ public function __construct(\MultiFlexi\Job &$job)`
`31`	`31`	`{`
`32`		`- $this->job = $job;`
	`32`	`+ parent::__construct($job);`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`public static function name(): string`