Skip to content
Snippets Groups Projects
Commit 6416bb53 authored by Bastian Waidelich's avatar Bastian Waidelich
Browse files

[+FEATURE] Fluid (ViewHelpers/Widget): Slightly improved paginate widget: The...

[+FEATURE] Fluid (ViewHelpers/Widget): Slightly improved paginate widget: The link to the first page now skips the currentPage argument resulting in nicer URLs and less cHashes
[+FEATURE] Fluid (ViewHelpers): excluding obsolete widget arguments & cHash from widget links if they're not explicitly set
parent f25e0544
No related merge requests found
......@@ -117,6 +117,7 @@ class Tx_Fluid_ViewHelpers_Link_WidgetViewHelper extends Tx_Fluid_Core_ViewHelpe
->setSection($this->arguments['section'])
->setCreateAbsoluteUri(TRUE)
->setAddQueryString(TRUE)
->setArgumentsToBeExcludedFromQueryString(array($argumentPrefix, 'cHash'))
->setFormat($this->arguments['format'])
->build();
}
......
......@@ -95,6 +95,7 @@ class Tx_Fluid_ViewHelpers_Uri_WidgetViewHelper extends Tx_Fluid_Core_ViewHelper
->setSection($this->arguments['section'])
->setCreateAbsoluteUri(TRUE)
->setAddQueryString(TRUE)
->setArgumentsToBeExcludedFromQueryString(array($argumentPrefix, 'cHash'))
->setFormat($this->arguments['format'])
->build();
}
......
......@@ -12,7 +12,14 @@
<ul class="f3-widget-paginator">
<f:if condition="{pagination.previousPage}">
<li class="previous">
<f:link.widget arguments="{currentPage: pagination.previousPage}">previous</f:link.widget>
<f:if condition="{pagination.previousPage} > 1">
<f:then>
<f:link.widget arguments="{currentPage: pagination.previousPage}">previous</f:link.widget>
</f:then>
<f:else>
<f:link.widget>previous</f:link.widget>
</f:else>
</f:if>
</li>
</f:if>
<f:for each="{pagination.pages}" as="page">
......@@ -24,7 +31,14 @@
</f:then>
<f:else>
<li>
<f:link.widget arguments="{currentPage: page.number}">{page.number}</f:link.widget>
<f:if condition="{page.number} > 1">
<f:then>
<f:link.widget arguments="{currentPage: page.number}">{page.number}</f:link.widget>
</f:then>
<f:else>
<f:link.widget>{page.number}</f:link.widget>
</f:else>
</f:if>
</li>
</f:else>
</f:if>
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment