-
-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Link caching #15
Link caching #15
Conversation
This reverts commit d58cd0d.
# Conflicts: # src/Api/Controllers/ScrapperController.php
$this->web = $web; | ||
$this->translator = $translator; | ||
$this->cache = $cache; | ||
$cacheTime = $settings->get('datlechin-link-preview.cache_time'); | ||
if (!is_numeric($cacheTime)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PSR-12 here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please elaborate?
Checked the code against PSR-12 and found no issue in the marked segment. Are you referring to an empty line before the if-statement?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a space after !
by the way, I'll update it later
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Strange I set up my IDE to conform to PSR-12 an even ran PHP_CodeSniffer with the PSR12 preset and the space after the negation operator is removed every time I autoformat using the tool.
Caches successfully visited links using the default cache provider according to the set cache time in the backend.
serializeToForm