-
Notifications
You must be signed in to change notification settings - Fork 60
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
IBX-1999: Added pagination to URL Wildcard tab in URL Management #2030
Conversation
src/bundle/Resources/views/themes/admin/url_wildcard/list.html.twig
Outdated
Show resolved
Hide resolved
@@ -73,8 +73,8 @@ | |||
</tr> | |||
</thead> | |||
<tbody> | |||
{% if url_wildcards|length %} | |||
{% for url_wildcard in url_wildcards %} | |||
{% if pager.currentPageResults|length %} |
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.
{% if pager.currentPageResults|length %} | |
{% if pager|length %} |
Pagerfanta objects are countable. Also you use getNbResults method.
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.
changed to count is same as(0)
like in another tabs in admin-ui
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.
Tests should be updated, but otherwise than that 👍
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Checklist:
$ composer fix-cs
)