Status report wrongly warns of APCu memory limit when admin language is not English

This will need tests to show the test.

Also was previously tagged for IS update.

Did not test the issue.

This will need tests to show the test.

See note #5.

Missed that.

Drupal 9.5.10

Drupal 10.2.0

To see this issue on a multilingual site, the string @size MB must be localized.

My solution fixes the issue while respecting the user language, however, I think the best solution is to make the byte comparison work correctly when comparing two byte representations in two different languages, or to just compare the byte size as numbers, not as strings.

@alayham Yeah, the suggestion all along has been to avoid any localization and compare number/byte sizes directly.

https://git.drupalcode.org/project/drupal/-/merge_requests/670/diffs#not...

This was only in needs work because it doesn't have tests which is still true.

Taran2L → changed the visibility of the branch 3142928-status-report-wrongly-d10.2 to hidden.

Taran2L → changed the visibility of the branch 3142928-status-report-wrongly-d10 to hidden.

Taran2L → changed the visibility of the branch 11.x to hidden.

I have same problem as #7

$apcu_actual_size = ByteSizeMarkup::create($memory_info['seg_size'], 'en');
doesn't help

if ($memory_info['seg_size'] < Bytes::toNumber($apcu_recommended_size))
doesn't help too

hi.
this patch work for me in drupal 10.2.7:
status-report-apcu-3142928.patch

go to /core/modules/system/
then edit system.install
in line 427 remove
$apcu_actual_size = ByteSizeMarkup::create($memory_info['seg_size']);
replace with
$apcu_actual_size = ByteSizeMarkup::create($memory_info['seg_size'], 'en');

Per #5 this isn't going to be testable in a useful way - we can't fake the apcu memory limit for system_requirements(). Also think this is a good example where the new policy on https://www.drupal.org/about/core/policies/core-change-policies/core-gat... → should apply.

Can the issue summary be updated with least proposed solution and before/after screenshots.

Also the 1 MR still visible is for 9.1.x and still a number of patches so not sure which is to be reviewed.