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

Fix Buggy thumbnail url generation #1928 #1938

Merged
merged 4 commits into from
Oct 4, 2020
Merged

Fix Buggy thumbnail url generation #1928 #1938

merged 4 commits into from
Oct 4, 2020

Conversation

luistar15
Copy link
Contributor

  • Do not split parameters using "x", could cause an other bug when the location or fit contains an "x".
  • Do not use "parameter=value" syntax anymore.
  • Assumes always sending both "WIDTH×HEIGHT" parameters.

@luistar15
Copy link
Contributor Author

luistar15 commented Oct 3, 2020

How do I proceed?
Do I need to send another pull request fixing the Coding Standard issues? or can it be fixed automatically?
Edit: I think I figured it out.

@bobdenotter
Copy link
Member

Do not split parameters using "x", could cause an other bug when the location or fit contains an "x".

Right, the current str_replace was too lazy, so it needs to be removed. From experience i know that a lot of people have no clue how to input an ×, so I think we should breing that back in a separate PR. A simple regex [0-9]x[0-9] should already do the trick.

I've nudged Travis. If it passes, i'll merge this in! 👍

@bobdenotter bobdenotter merged commit 15cccc9 into bolt:master Oct 4, 2020
@bobdenotter
Copy link
Member

Thanks again, @luistar15

@luistar15 luistar15 deleted the bugfix-thumbnail-url branch October 4, 2020 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants