These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
1 | <?php |
||
2 | namespace Robo\Task\File; |
||
3 | |||
4 | use Iterator; |
||
5 | use Robo\Common\ResourceExistenceChecker; |
||
6 | use Robo\Result; |
||
7 | use Robo\Task\BaseTask; |
||
8 | |||
9 | /** |
||
10 | * Merges files into one. Used for preparing assets. |
||
11 | * |
||
12 | * ``` php |
||
13 | * <?php |
||
14 | * $this->taskConcat([ |
||
15 | * 'web/assets/screen.css', |
||
16 | * 'web/assets/print.css', |
||
17 | * 'web/assets/theme.css' |
||
18 | * ]) |
||
19 | * ->to('web/assets/style.css') |
||
20 | * ->run() |
||
21 | * ?> |
||
22 | * ``` |
||
23 | */ |
||
24 | class Concat extends BaseTask |
||
25 | { |
||
26 | use ResourceExistenceChecker; |
||
27 | |||
28 | /** |
||
29 | * @var array|Iterator |
||
30 | */ |
||
31 | protected $files; |
||
32 | |||
33 | /** |
||
34 | * @var string |
||
35 | */ |
||
36 | protected $dst; |
||
37 | |||
38 | /** |
||
39 | * Constructor. |
||
40 | * |
||
41 | * @param array|Iterator $files |
||
42 | */ |
||
43 | public function __construct($files) |
||
44 | { |
||
45 | $this->files = $files; |
||
46 | } |
||
47 | |||
48 | /** |
||
49 | * set the destination file |
||
50 | * |
||
51 | * @param string $dst |
||
52 | * |
||
53 | * @return $this |
||
54 | */ |
||
55 | public function to($dst) |
||
56 | { |
||
57 | $this->dst = $dst; |
||
58 | |||
59 | return $this; |
||
60 | } |
||
61 | |||
62 | /** |
||
63 | * {@inheritdoc} |
||
64 | */ |
||
65 | public function run() |
||
66 | { |
||
67 | if (is_null($this->dst) || "" === $this->dst) { |
||
68 | return Result::error($this, 'You must specify a destination file with to() method.'); |
||
69 | } |
||
70 | |||
71 | if (!$this->checkResources($this->files, 'file')) { |
||
0 ignored issues
–
show
|
|||
72 | return Result::error($this, 'Source files are missing!'); |
||
73 | } |
||
74 | |||
75 | View Code Duplication | if (file_exists($this->dst) && !is_writable($this->dst)) { |
|
76 | return Result::error($this, 'Destination already exists and cannot be overwritten.'); |
||
77 | } |
||
78 | |||
79 | $dump = ''; |
||
80 | |||
81 | foreach ($this->files as $path) { |
||
82 | foreach (glob($path) as $file) { |
||
83 | $dump .= file_get_contents($file) . "\n"; |
||
84 | } |
||
85 | } |
||
86 | |||
87 | $this->printTaskInfo('Writing {destination}', ['destination' => $this->dst]); |
||
88 | |||
89 | $dst = $this->dst . '.part'; |
||
90 | $write_result = file_put_contents($dst, $dump); |
||
91 | |||
92 | if (false === $write_result) { |
||
93 | @unlink($dst); |
||
94 | return Result::error($this, 'File write failed.'); |
||
95 | } |
||
96 | // Cannot be cross-volume; should always succeed. |
||
97 | @rename($dst, $this->dst); |
||
98 | |||
99 | return Result::success($this); |
||
100 | } |
||
101 | } |
||
102 |
It seems like the type of the argument is not accepted by the function/method which you are calling.
In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.
We suggest to add an explicit type cast like in the following example: