-
Notifications
You must be signed in to change notification settings - Fork 228
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
BUG: Exceptions when using UncacheViewHelper on TYPO3 12 #1894
Comments
I need a bit more context. As mentioned in #1872, the ViewHelper has been tested on v12 and is working. There must be something specific about your use case that causes the Request held by RenderingContext to be an unexpected class, so please describe a bit about what you're actually doing. |
No official release nor the current development branch includes the pull request's changes that fixes the viewhelper in typo3 v12. Since there was no coresponding issue to track this bug, I created this one here, in hopes for the pull request to be merged. So once the mentioned pull request gets merged, this issue would be solved. |
I have to first understand the cause for the issue - like I said, this doesn't happen in the test cases that I have. The pull request as-is cannot be merged since it breaks backwards compatibility but I can't know what the right way to solve that problem is, without first knowing why this appears to be a problem for you and not for me. And since you posted a screenshot of the error I assume that you've got some use case set up that would trigger the issue. Which is why I asked you (like I also asked @sk-foresite) to provide more information about the context. Otherwise I cannot solve the problem for you! |
The context is in the frontend and the viewhelper is used in a partial template of the websites html templates. Thus it is not used by a content element plugin. Maybe that is what is different? P.S. I am using the uncache viewhelper to display the name of the logged in user in the header. |
So, sounds like |
Correct, |
Thanks - I'll see if I can reproduce it and take it from there! |
Not sure if that is related in some way: Can someone check if there is in error when using the admin panel? |
Were you able to replicate the issue? |
I have prepared a new pull request #1898. It fixes this issue and another regression that came with the v7 update, tho it only affected Typo3 v10 It should not break backward compatiblity with both v10 and v11 in either context (with or without extbase) |
@NamelessCoder I saw the fix has been done in the development branch. When will you release a version with the fix? |
@lexwebmaker That depends, I'd like to include a few more fixes before that happens, but probably some time this weekend or during next week! |
I have checked that the bug exists in the
dev-development
branchYes
I have checked that there are no already open issues or recently closed issues about this bug
Yes
Describe the bug
On Typo3, the uncache viewhelper (v:render.uncache) is not working, throwing the following error:
FluidTYPO3\Vhs\Utility\RequestResolver::resolveRequestFromRenderingContext(): Return value must be of type TYPO3\CMS\Extbase\Mvc\RequestInterface, TYPO3\CMS\Core\Http\ServerRequest returned
To Reproduce
Try rendering a partial template using the v:render.uncache viewhelper in a template on a Typo3 v12 setup.
Expected behavior
Show a fresh render of the partial view
Screenshots
Additional context
A patch fixing both the mentioned issue and the subsequent error
Given request must have an attribute "extbase" of type ExtbaseAttribute
already exists (see #1872) and I can confirm that it fixes the viewhelper on Typo3 v12.The text was updated successfully, but these errors were encountered: