? ? Pending

User tests: Successful: Unsuccessful:

avatar HLeithner
HLeithner
14 Nov 2020

Cast memory from string to float.

Using the debug plugin on php 8 raises an error.

avatar HLeithner HLeithner - open - 14 Nov 2020
avatar HLeithner HLeithner - change - 14 Nov 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Nov 2020
Category Front End Plugins
avatar SharkyKZ
SharkyKZ - comment - 14 Nov 2020

Didn't #30582 fix this?

avatar HLeithner
HLeithner - comment - 14 Nov 2020

how should it fix it ($currentMem > $this->previousMem ? '+' : '') . ($currentMem - $this->previousMem) ends in a string.

avatar SharkyKZ
SharkyKZ - comment - 14 Nov 2020

That shouldn't be an issue, as long as the string is formatted as a number. PHP will cast it to int or float in such case.

avatar HLeithner
HLeithner - comment - 14 Nov 2020

Please try it your self I got a type error.

avatar alikon
alikon - comment - 16 Nov 2020

before apply pr with php 8.0 rc3
image

after pr
no more error

avatar alikon alikon - test_item - 16 Nov 2020 - Tested successfully
avatar alikon
alikon - comment - 16 Nov 2020

I have tested this item successfully on 29ee6e6


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31404.

avatar SharkyKZ
SharkyKZ - comment - 17 Nov 2020

Did you test on latest staging with #30582 already included?

avatar alikon
alikon - comment - 18 Nov 2020

re-tested with the current staging and no error
sorry my previous test was made on an old staging ie without #30582

avatar alikon alikon - test_item - 18 Nov 2020 - Not tested
avatar alikon
alikon - comment - 18 Nov 2020

I have not tested this item.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31404.

avatar HLeithner
HLeithner - comment - 18 Nov 2020

Ok seems @SharkyKZ is right this pr is not needed with the mentioned pr. Closing it...

avatar HLeithner HLeithner - close - 18 Nov 2020
avatar HLeithner HLeithner - change - 18 Nov 2020
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2020-11-18 21:13:27
Closed_By HLeithner
Labels Added: ? ?

Add a Comment

Login with GitHub to post a comment