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 transliteration. #253
Fix transliteration. #253
Conversation
Add `handlesUrlization` to handler interface for handlers that handle Urlization so that we don't re-urlize and remove special characters needed by that specific handler.
/** | ||
* {@inheritDoc} | ||
*/ | ||
public function handlesUrlization(){ |
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.
the curly brace should be on its own line
@@ -387,4 +388,5 @@ private function filterSimilarSlugs(array &$slugs, array &$config, $prefered) |
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.
am I seeing tabs? etc \t
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.
No I don't think those are tabs, but didn't realize my IDE messed up the code this much and changed all the indentations. Will correct these. Sorry about that.
@@ -21,8 +21,9 @@ | |||
* @link http://www.gediminasm.org | |||
* @license MIT License (http://www.opensource.org/licenses/mit-license.php) | |||
*/ | |||
class SluggableListener extends MappedEventSubscriber | |||
class SluggableListener extends MappedEventSubscriber |
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.
you should also remove the trailing whitespaces added here
thanks! |
Add
handlesUrlization
to handler interface for handlers that handle Urlization so that we don't re-urlize and remove special characters needed by that specific handler.Tests are now all passing (except one that was previously getting an error already) with transliteration separated from urlization.