Skip to content
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

Exclude path images (duplicate images) #13

Closed
mrownicki opened this issue Dec 17, 2018 · 8 comments
Closed

Exclude path images (duplicate images) #13

mrownicki opened this issue Dec 17, 2018 · 8 comments

Comments

@mrownicki
Copy link

mrownicki commented Dec 17, 2018

Hi.
This is not Issues maybe a little.
If any gallery create optimize image in src in own breakpoints.
Duplicating images. Even if they are not webp This profit will be minimal at the expense of the space occupied.

Maybe gzip be smart and discover image src set or option for exclude patch cache images for gallery support src set.

DJ mediatools, econa, droppic, JUltraNewsPRO using src set.

@tbela99
Copy link
Owner

tbela99 commented Dec 17, 2018

Do you mean srcet is overriden?

@mrownicki
Copy link
Author

No, I did not overtake my thought before I tested it.
But, now I test in mediatools and get error now.
Warning: file_get_contents(/home/xvqmcyxf/XXXX/cache/z/app/XXXX/worker_version): failed to open stream: No such file or directory in /home/xvqmcyxf/XXXX/plugins/system/gzip/gzip.php on line 239
Fatal error: Paletter image not supported by webp in /home/xvqmcyxf/XXXX/plugins/system/gzip/helper.php on line 2521

Article with gallery dj mediatools

@tbela99
Copy link
Owner

tbela99 commented Dec 17, 2018

I suggest you clear your cache from the joomla admin backend

@mrownicki
Copy link
Author

Cleaning cache. Not working.
If I report error switch to default i have error 500 on article.
I try add gallery to menu this same error.

Ps. Joomla 3.9.1 PHP 7.2.13
Dj media tools 2.11.0
Framework T3 + Template t3_bs3_blank free
GZIP conf.
security
criticas-css
webpush
swc
swn
app
image
css
js
cdn
http

@tbela99
Copy link
Owner

tbela99 commented Apr 29, 2019

Now I understand your issue :D
you want some kind of file path exclusion. Am I right?

@mrownicki
Copy link
Author

yes :)

@tbela99
Copy link
Owner

tbela99 commented Apr 29, 2019

Haha OK :)

I will take a look at that anytime soon

tbela99 added a commit that referenced this issue May 20, 2019
tbela99 added a commit that referenced this issue May 20, 2019
#13 Ignore images that match a pattern
@tbela99
Copy link
Owner

tbela99 commented May 20, 2019

The issue has been fixed

@tbela99 tbela99 closed this as completed May 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants