Conversation
Review Summary by QodoAdd recommended PHP memory limit to documentation
WalkthroughsDescription• Add recommended PHP memory limit documentation • Specify 256KB as recommended memory limit value • Update technical requirements in two sections Diagramflowchart LR
A["Technical Requirements Doc"] -- "Add memory limit recommendation" --> B["256KB Memory Limit Guidance"]
B -- "Applied to" --> C["Getting Started Section"]
B -- "Applied to" --> D["Requirements Table Section"]
File Changes1. docs/get-started/technical-requirements.md
|
Code Review by Qodo
1. Memory limit unit typo
|
|
|
||
| **Required PHP Modules:** json, simplexml, dom, zlib, gd, mysqlnd or pdo_mysql or pdo_pgsql | ||
| **Recommended PHP Modules:** mbstring | ||
| **Recommended PHP Memory Limit:** 256KB |
There was a problem hiding this comment.
1. Memory limit unit typo 🐞 Bug ✓ Correctness
The docs recommend a PHP memory limit of 256KB, which is effectively unusable for Joomla and will mislead users into configuring a far-too-low value. This can cause immediate out-of-memory failures during installation or runtime and create avoidable support load.
Agent Prompt
### Issue description
The docs currently recommend `**Recommended PHP Memory Limit:** 256KB`, which is almost certainly a unit typo and will lead users to configure an unusably low memory limit.
### Issue Context
This line appears in both the Joomla 6.x and Joomla 5.x requirement sections.
### Fix Focus Areas
- docs/get-started/technical-requirements.md[26-30]
- docs/get-started/technical-requirements.md[46-49]
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
There was a problem hiding this comment.
Shouldn't it be MB and not KB?
There was a problem hiding this comment.
Ask❓
Shouldn't it be MB and not KB?
Answer:
Yes—this should be MB, not KB.
256KB is far too low for Joomla and is almost certainly a unit typo. A recommendation like 256MB (or whatever value the linked issue/official guidance intends) is consistent with real-world PHP memory needs and avoids users misconfiguring PHP and hitting immediate out-of-memory errors.
Suggested change in both places:
**Recommended PHP Memory Limit:** 256MB
Also consider whether the project wants to recommend a higher value (e.g., 512MB) depending on typical extensions/workloads, but at minimum the unit should be MB.
Added because of: joomla/joomla-cms#46921