Remove unused property, fixes #79 #80

Merged
merged 1 commit into from Apr 4, 2013

Conversation

Projects
None yet
2 participants
Contributor

Seldaek commented Apr 4, 2013

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 added a commit that referenced this pull request Apr 4, 2013

Merge pull request #80 from Seldaek/patch-1
Remove unused property, fixes #79

@pablodip pablodip merged commit 4fded9a into whiteoctober:master Apr 4, 2013

1 check was pending

default The Travis build is in progress
Details
Contributor

pablodip commented Apr 4, 2013

Thanks! ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment