Skip to content
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

Statistics - selected time period throws exeption #41

Closed
DTM-TYPO3 opened this issue Jun 22, 2023 · 4 comments
Closed

Statistics - selected time period throws exeption #41

DTM-TYPO3 opened this issue Jun 22, 2023 · 4 comments

Comments

@DTM-TYPO3
Copy link

Hello,

I`m using the statistics module of mindshape_cookie_consent. When I change the time period in die select-field above, an exception is thrown saying:

Call to undefined method TYPO3\CMS\Backend\Template\ModuleTemplate::assign()

in mindshape_cookie_consent/Classes/Controller/Backend/StatisticController.php

Also the pagination doesn´t work. When clicking forward through the pages, it jumpes back to the initial selection.

Tested on Typo3 11.5.28 with PHP 8.1 and mindshape_cookie_consent 3.0.1

Thanks in advance.
Stefanie

@DTM-TYPO3 DTM-TYPO3 changed the title Statistic - selected time period throws exeption Statistics - selected time period throws exeption Jun 22, 2023
@featdd
Copy link
Member

featdd commented Jun 22, 2023

Hi @DTM-TYPO3,

thankts for the hint, I could fix the exception here: 33317f0

But I struggle reproduce the issue with the pagination, have you overriden the backend templates?
The extension uses "georgringer/numbered-pagination" for the cropped pagination.

Greetings
Daniel

@DTM-TYPO3
Copy link
Author

Hi @featdd,
thanks for the quick fix. Changing the time periods is working now.

As for pagination, I´ve not changed any templates and the EXT numbered-pagination is installed in version 1.0.5 (latest, also tested with TER V1.0.2). Nevertheless, it jumpes back to the initial selection of the module.

Btw. there is no hint in dependencies in mindshape_cookie_consent that the EXT is required.

Regards
Stefanie

@featdd
Copy link
Member

featdd commented Oct 29, 2023

Hi @DTM-TYPO3,

sorry for the late response, do you use a non composer instance for your TYPO3?
That might be the issue, I declared the missing dependency in the ext_emconf here: c66cd74

Greetings
Daniel

@featdd featdd closed this as completed Oct 29, 2023
@featdd
Copy link
Member

featdd commented Nov 10, 2023

Hi @DTM-TYPO3,

an update with this fix is now released in TER.

Greetings
Daniel

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

No branches or pull requests

2 participants