Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Remove unused property, fixes #79 #80

Merged
merged 1 commit into from

2 participants

@Seldaek

So what happened is that you removed the private $nbPages;, which means the property wasn't declared anymore. Setting it to null declares a new public property (yay dynamic languages:p), and then twig sees that and reads it out instead of looking for a getter as it did before when it was private.

@pablodip pablodip merged commit 4fded9a into from
@pablodip
Owner

Thanks! ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Apr 4, 2013
  1. @Seldaek
This page is out of date. Refresh to see the latest.
Showing with 0 additions and 1 deletion.
  1. +0 −1  src/Pagerfanta/Pagerfanta.php
View
1  src/Pagerfanta/Pagerfanta.php
@@ -151,7 +151,6 @@ private function resetForMaxPerPageChange()
{
$this->currentPageResults = null;
$this->nbResults = null;
- $this->nbPages = null;
}
/**
Something went wrong with that request. Please try again.