Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

Wrong command order #28

Open
within25 opened this issue Jun 17, 2020 · 0 comments
Open

Wrong command order #28

within25 opened this issue Jun 17, 2020 · 0 comments

Comments

@within25
Copy link

    foreach ($this->chainedCommands as $chainedCommand) {
        $chainedCommand->determinePreOptimisedFileSize();
        $chainedCommand->setImagePath($this->imagePath);
        $chainedCommand->optimise();
        $chainedCommand->determinePostOptimisedFileSize();
    }

first must be setImagePath and only then determinePreOptimisedFileSize

we get an error
Unable to determine pre-optimized fileSize

because $ this-> imagePath is still empty

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant