Skip to content

Commit 00d2414

Browse files
committed
Return errors when an operation fails
1 parent e9e3110 commit 00d2414

1 file changed

Lines changed: 24 additions & 10 deletions

File tree

src/WoolTrait.php

Lines changed: 24 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,15 @@ public function mkdir(array $payload)
1717
if (
1818
@mkdir(
1919
$payload['path'],
20-
(new PermissionFlagResolver())->resolve($payload['mode'])
20+
octdec($payload['mode'])
2121
)
2222
) {
2323
return \React\Promise\resolve([]);
2424
}
2525

26-
return \React\Promise\reject([]);
26+
return \React\Promise\reject([
27+
'error' => error_get_last(),
28+
]);
2729
}
2830

2931
/**
@@ -32,11 +34,13 @@ public function mkdir(array $payload)
3234
*/
3335
public function rmdir(array $payload)
3436
{
35-
if (rmdir($payload['path'])) {
37+
if (@rmdir($payload['path'])) {
3638
return \React\Promise\resolve([]);
3739
}
3840

39-
return \React\Promise\reject([]);
41+
return \React\Promise\reject([
42+
'error' => error_get_last(),
43+
]);
4044
}
4145

4246
/**
@@ -49,7 +53,9 @@ public function unlink(array $payload)
4953
return \React\Promise\resolve([]);
5054
}
5155

52-
return \React\Promise\reject([]);
56+
return \React\Promise\reject([
57+
'error' => error_get_last(),
58+
]);
5359
}
5460

5561
/**
@@ -62,7 +68,9 @@ public function chmod(array $payload)
6268
return \React\Promise\resolve([]);
6369
}
6470

65-
return \React\Promise\reject([]);
71+
return \React\Promise\reject([
72+
'error' => error_get_last(),
73+
]);
6674
}
6775

6876
/**
@@ -77,7 +85,9 @@ public function chown(array $payload)
7785
return \React\Promise\resolve([]);
7886
}
7987

80-
return \React\Promise\reject([]);
88+
return \React\Promise\reject([
89+
'error' => error_get_last(),
90+
]);
8191
}
8292

8393
/**
@@ -87,7 +97,9 @@ public function chown(array $payload)
8797
public function stat(array $payload)
8898
{
8999
if (!file_exists($payload['path'])) {
90-
return \React\Promise\reject([]);
100+
return \React\Promise\reject([
101+
'error' => 'Path doesn\'t exist',
102+
]);
91103
}
92104

93105
$stat = lstat($payload['path']);
@@ -148,7 +160,7 @@ public function open(array $payload)
148160
public function touch(array $payload)
149161
{
150162
return \React\Promise\resolve([
151-
touch($payload['path']),
163+
touch($payload['path']) && chmod($payload['path'], octdec($payload['mode'])),
152164
]);
153165
}
154166

@@ -199,7 +211,9 @@ public function rename(array $payload)
199211
return \React\Promise\resolve([]);
200212
}
201213

202-
return \React\Promise\reject([]);
214+
return \React\Promise\reject([
215+
'error' => error_get_last(),
216+
]);
203217
}
204218

205219
/**

0 commit comments

Comments
 (0)