Get text from docx files #81
Conversation
|
👋 @bbrands02
|
|
🔍 Vulnerabilities of
|
| digest | sha256:3c9dea73b2edfd35952fd664a69fd5aa70e72e2464f34b6ecb641c42e192713d |
| vulnerabilities |      |
| size | 247 MB |
| packages | 312 |
📦 Base Image php:8.2-fpm-alpine
| also known as |
|
| digest | sha256:c5d9ca92dd6998511c65f223359eec2b0eb1cef311bed2aa2c3778657de8ab28 |
| vulnerabilities |  |
Description | ||||||||
Description
| ||||||||
Description |
src/Service/FileService.php
Outdated
| $tempFilePath = tempnam(sys_get_temp_dir(), 'docx'); | ||
| if ($tempFilePath === false) { | ||
| $this->logger->error('Failed to create a temporary file '.$file->getName()); | ||
| $this->style && $this->style->error('Failed to create a temporary file '.$file->getName()); | ||
| } | ||
|
|
||
| file_put_contents($tempFilePath, $base64Decoded); |
There was a problem hiding this comment.
Maybe you could consider moving this to a separate function, seems to me that this will happen more often.
src/Service/FileService.php
Outdated
| $text = null; | ||
| } | ||
| break; | ||
| case 'application/vnd.openxmlformats-officedocument.wordprocessingml.document': |
There was a problem hiding this comment.
I'd be in favor of putting everything in a case in a separate function, although I could imagine only putting the contents of try in a separate function.
bbrands02
force-pushed
the
feature/WOO-49/extract-text-from-docx
branch
from
8514fdb to
b683fd7
Compare
No description provided.