Skip to content

Add recomendation for memory limit#588

Open
rdeutz wants to merge 1 commit intojoomla:mainfrom
rdeutz:memory-limit-recomendation
Open

Add recomendation for memory limit#588
rdeutz wants to merge 1 commit intojoomla:mainfrom
rdeutz:memory-limit-recomendation

Conversation

@rdeutz
Copy link
Contributor

@rdeutz rdeutz commented Feb 23, 2026

Added because of: joomla/joomla-cms#46921

@qodo-code-review
Copy link
Contributor

Review Summary by Qodo

Add recommended PHP memory limit to documentation

📝 Documentation

Grey Divider

Walkthroughs

Description
• Add recommended PHP memory limit documentation
• Specify 256KB as recommended memory limit value
• Update technical requirements in two sections
Diagram
flowchart 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"]
Loading

Grey Divider

File Changes

1. docs/get-started/technical-requirements.md 📝 Documentation +2/-0

Add PHP memory limit recommendations

• Added "Recommended PHP Memory Limit: 256KB" to getting started section
• Added "Recommended PHP Memory Limit: 256KB" to requirements table section
• Addresses GitHub issue #46921 regarding memory limit recommendations

docs/get-started/technical-requirements.md


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Contributor

qodo-code-review bot commented Feb 23, 2026

Code Review by Qodo

🐞 Bugs (1) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider


Action required

1. Memory limit unit typo 🐞 Bug ✓ Correctness
Description
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.
Code

docs/get-started/technical-requirements.md[28]

+**Recommended PHP Memory Limit:** 256KB  
Evidence
The technical requirements documentation now states a recommended PHP memory limit of 256KB in both
Joomla 6.x and 5.x sections; this is the published guidance users will follow.

docs/get-started/technical-requirements.md[26-30]
docs/get-started/technical-requirements.md[46-49]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### 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


Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo


**Required PHP Modules:** json, simplexml, dom, zlib, gd, mysqlnd or pdo_mysql or pdo_pgsql
**Recommended PHP Modules:** mbstring
**Recommended PHP Memory Limit:** 256KB
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Action required

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

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't it be MB and not KB?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants