bosss Posted September 15, 2022 Posted September 15, 2022 Movies (app) updated 2 days ago and works ok with 4.7.2 (was locked - efter instal IPS 4.7.1) Now update to 4.7.2.1 and Movies was disable again. Some tips or maybe @Adriano Faria
Adriano Faria Posted September 15, 2022 Posted September 15, 2022 New version was released yesterday to make it compatible with 4.7.2. Update again. Ooops, 4.2.7.1? Oh God, here we go again! Let me test it. Ioannis D and SeNioR- 2
Adriano Faria Posted September 15, 2022 Posted September 15, 2022 (edited) My bad... the block changed its position: @Matt, it was fine a couple of days ago. Please don't say we'll have to go through every app again! 😞 Edited September 15, 2022 by Adriano Faria SeNioR-, AlexJ, Ibai and 2 others 5
Davyc Posted September 15, 2022 Posted September 15, 2022 I'm pleased I didn't update again - there's far too many people posting issues for this to be updated until everything is sorted. My sympathies go out to everyone affected, clients and devs. Duken, My Sharona, Ibai and 1 other 4
Ioannis D Posted September 15, 2022 Posted September 15, 2022 I'm in the same position with 2 apps (Movies & VideoBox) but this time they haven't locked, only the warning at PHP8 Compatibility Scanner box exists.
Jim M Posted September 15, 2022 Posted September 15, 2022 4.2.7.1 will allow you to enable add-ons still. They will no longer be locked. Ioannis D 1
Adriano Faria Posted September 15, 2022 Posted September 15, 2022 Just now, Jim M said: 4.2.7.1 will allow you to enable add-ons still. They will no longer be locked. But will end up on an error. That's not the problem here. Scanner is flagging same issues and a new one! Ibai and DamonT 2
Davyc Posted September 15, 2022 Posted September 15, 2022 1 hour ago, bosss said: Now update to 4.7.2.1 and Movies was disable again. Looks as if you've also lost a lot of your font awesome icons @bosss Also missing in other areas too. Not sure if this is an addon that's been disabled or anything to do with the update but I would check this out.
bosss Posted September 15, 2022 Author Posted September 15, 2022 9 minutes ago, Davyc said: Looks as if you've also lost a lot of your font awesome icons @bosss Also missing in other areas too. Not sure if this is an addon that's been disabled or anything to do with the update but I would check this out. I see them ..... like guest and admin
Management Matt Posted September 15, 2022 Management Posted September 15, 2022 Show me your code and the base code. I did see we used \IPS\node\Controller instead of an uppercase “Node”. Is that what you’re seeing?
Adriano Faria Posted September 15, 2022 Posted September 15, 2022 5 minutes ago, Matt said: Show me your code and the base code. I did see we used \IPS\node\Controller instead of an uppercase “Node”. Is that what you’re seeing? Mine: protected function _createOrUpdate( \IPS\node\Model $node ) \IPS\Node\Api\NodeController: protected function _createOrUpdate( \IPS\Node\Model $node ) Yes. So now will have to update everything again? 4.7.2 is protected function _createOrUpdate( \IPS\node\Model $node ) Duken, SeNioR- and AlexJ 2 1
Stuart Silvester Posted September 15, 2022 Posted September 15, 2022 23 minutes ago, Adriano Faria said: Mine: protected function _createOrUpdate( \IPS\node\Model $node ) \IPS\Node\Api\NodeController: protected function _createOrUpdate( \IPS\Node\Model $node ) Yes. So now will have to update everything again? 4.7.2 is protected function _createOrUpdate( \IPS\node\Model $node ) Unfortunately that's the nature of these types of things now causing fatal errors. Matt did a pass of some stuff that wasn't consistent in our own code but in future non-major releases we're going to be mindful to avoid changing method signatures unless we really need to - sometimes it's just not possible to avoid.
Adriano Faria Posted September 15, 2022 Posted September 15, 2022 For the sake of my head/health, I need to take a break. Hisashi, Davyc, media and 6 others 6 3
Michael.J Posted September 15, 2022 Posted September 15, 2022 (edited) 1 hour ago, Adriano Faria said: For the sake of my head/health, I need to take a break. Yep 😞 I checked and updated nearly 100 apps and plugins already. Edited September 15, 2022 by Michael.J Davyc, DamonT, SeNioR- and 1 other 3 1
Duken Posted September 16, 2022 Posted September 16, 2022 Again..... my upgrade stays in de que. Hope the developers can have a good night of sleep. 😶 SeNioR- 1
Adriano Faria Posted September 19, 2022 Posted September 19, 2022 @Matt, can't this be avoided somehow? I'm using a resource that has a hook to another that isn't installed. 3 pages of log entries about this in 10 minutes I'm working on it. Users will freak out with so many log entries; support will say it's a 3rd-party issue; we will say it's a scanner overkill, bla bla bla... media and SeNioR- 2
Stuart Silvester Posted September 19, 2022 Posted September 19, 2022 5 minutes ago, Adriano Faria said: @Matt, can't this be avoided somehow? I'm using a resource that has a hook to another that isn't installed. 3 pages of log entries about this in 10 minutes I'm working on it. Users will freak out with so many log entries; support will say it's a 3rd-party issue; we will say it's a scanner overkill, bla bla bla... We'll be decreasing the verbosity of some of the logging in future releases Adriano Faria 1
Recommended Posts