Does this work for Concrete 5.8?

Permalink Browser Info Environment
Hi, will this add-on work on Concrete 5.8?

Cheers!

Type: Pre-Sale
Status: In Progress
ZillionProductions
View Replies:
ZillionProductions replied on at Permalink Reply
ZillionProductions
Hi, could you please let me know if this works for C5.8 as well? Thanks.
crivers replied on at Permalink Reply
crivers
Hi,

Yes, this has been tested and proven to work with the latest version of Concrete5.8. Let me know if you need any assistance getting started.

Cheers!
xxcriversxx
helenmarie replied on at Permalink Reply
helenmarie
Hi, I have just purchased and tried to install this into version 8.1 and it throws this error?

Declaration of Concrete\Package\Instafolio\Block\Instafolio\Controller::registerViewAssets() should be compatible with Concrete\Core\Block\BlockController::registerViewAssets($outputContent = '')

There is also an error when trying to uninstall?
helenmarie replied on at Permalink Reply
helenmarie
After a search I found a thread that said in php7 you may have to change

public function registerViewAssets()


to

public function registerViewAssets($outputContent = '')


For anyone else with this issue:

1. I removed the package manually
2. Made the change in the blocks/instafolio/controller.php file
3. Manually re-uploaded the package
4. Installed it via the dashboard
5. It worked!
crivers replied on at Permalink Reply
crivers
Hi, thanks for reaching out and thanks for looking into this yourself. This is definitely directly related to that version of php. Not many have yet adopted php version 7. If you don't mind me asking, who are you using for hosting?
bhans replied on at Permalink Reply
bhans
So, is this working in Concrete5 8.5.1 and PHP 7.2.20?
baysmedia replied on at Permalink Reply
baysmedia
Yes to both - I'm running in production on C5 8.5.2 and PHP 7.2

concrete5 Environment Information

irrelevant

Browser User-Agent String

Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_3) AppleWebKit/602.4.8 (KHTML, like Gecko) Version/10.0.3 Safari/602.4.8

Hide Post Content

This will replace the post content with the message: "Content has been removed by an Administrator"

Hide Content

Request Refund

You have not specified a license for this support ticket. You must have a valid license assigned to a support ticket to request a refund.