Skip to content

Conversation

@xificurk
Copy link
Contributor

@xificurk xificurk commented May 5, 2016

@xificurk
Copy link
Contributor Author

xificurk commented May 5, 2016

Hm, travis failed, but as far as I can tell, it's unrelated with this change.

@dg
Copy link
Member

dg commented May 5, 2016

Hmmm, there is not getParameters()? It extends Latte\Template.

@xificurk
Copy link
Contributor Author

xificurk commented May 5, 2016

There is, but it's returning parameters of VariableTemplate (which is empty array). You need to return parameters of the underlying latte template object.

@dg
Copy link
Member

dg commented May 5, 2016

Oh! Please can you fix typo in commit message: VariableTemplate

@xificurk xificurk force-pushed the variable-template-bc branch from 3c2fe0e to ff323a9 Compare May 5, 2016 08:39
@xificurk
Copy link
Contributor Author

xificurk commented May 5, 2016

Good catch! Fixed.

@dg
Copy link
Member

dg commented May 5, 2016

Thanks

@dg dg merged commit e9cc574 into nette:master May 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants