Skip to content

Commit e82bd16

Browse files
committed
Remove unneeded Windows logic
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
1 parent 23a5d6c commit e82bd16

2 files changed

Lines changed: 0 additions & 32 deletions

File tree

lib/private/LargeFileHelper.php

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -137,23 +137,6 @@ public function getFileSizeViaCurl($fileName) {
137137
return null;
138138
}
139139

140-
/**
141-
* @brief Tries to get the size of a file via the Windows DOM extension.
142-
*
143-
* @param string $filename Path to the file.
144-
*
145-
* @return null|int|float Number of bytes as number (float or int) or
146-
* null on failure.
147-
*/
148-
public function getFileSizeViaCOM($filename) {
149-
if (class_exists('COM')) {
150-
$fsObj = new \COM("Scripting.FileSystemObject");
151-
$file = $fsObj->GetFile($filename);
152-
return 0 + $file->Size;
153-
}
154-
return null;
155-
}
156-
157140
/**
158141
* @brief Tries to get the size of a file via an exec() call.
159142
*

tests/lib/LargeFileHelperGetFileSizeTest.php

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -54,21 +54,6 @@ public function testGetFileSizeViaCurl($filename, $fileSize) {
5454
);
5555
}
5656

57-
/**
58-
* @dataProvider dataFileNameProvider
59-
*/
60-
public function testGetFileSizeViaCOM($filename, $fileSize) {
61-
if (!extension_loaded('COM')) {
62-
$this->markTestSkipped(
63-
'The PHP Windows COM extension is required for this test.'
64-
);
65-
}
66-
$this->assertSame(
67-
$fileSize,
68-
$this->helper->getFileSizeViaCOM($filename)
69-
);
70-
}
71-
7257
/**
7358
* @dataProvider dataFileNameProvider
7459
*/

0 commit comments

Comments
 (0)