php7 compatibility
Permalink Browser Info Environment
Hi Sebastien,
When running php7 i'm getting this error:
I'm not using the piechart block, but it's in an old page version. When trying to delete that version, i'm getting the same error.
I've seen similar errors in the forum, so i guess it's a universal error for register functions.
Could you look into this for making the theme compatible with php7?
Also i get this error when trying to get to the theme dashboard page:
I would like to run php7 for better speed performance.
When running php7 i'm getting this error:
Declaration of Concrete\Package\ThemeAnitya\Block\PieChart\Controller::registerViewAssets() should be compatible with Concrete\Core\Block\BlockController::registerViewAssets($outputContent = '')
I'm not using the piechart block, but it's in an old page version. When trying to delete that version, i'm getting the same error.
I've seen similar errors in the forum, so i guess it's a universal error for register functions.
Could you look into this for making the theme compatible with php7?
Also i get this error when trying to get to the theme dashboard page:
Fatal error: Function name must be a string
I would like to run php7 for better speed performance.
Type: | Ticket |
---|---|
Status: | Archived |
To get rid of this error you can change a line in this file:
\packages\theme_super_mint\blocks\pie_chart\controller.php
Find this line:
And change it to this:
If i remember right, there was the same error for another block. You can get rid of it changing the same code in the other block controller.
\packages\theme_super_mint\blocks\pie_chart\controller.php
Find this line:
public function registerViewAssets() {
And change it to this:
public function registerViewAssets($outputContent = '') {
If i remember right, there was the same error for another block. You can get rid of it changing the same code in the other block controller.
Attention: Since there has been no activity on this issue for two weeks, this issue has been automatically archived.
To re-open this issue, reply to this message.
To re-open this issue, reply to this message.
I posted a description (and the solution) of this problem here:https://www.concrete5.org/community/forums/block_requests/block-zoom...
Attention: Since there has been no activity on this issue for two weeks, this issue has been automatically archived.
To re-open this issue, reply to this message.
To re-open this issue, reply to this message.
Declaration of Concrete\Package\ThemeSuperMint\Block\PieChart\Controller::registerViewAssets() should be compatible with Concrete\Core\Block\BlockController::registerViewAssets($outputContent = '')