fix: prevent infinite loop in word_wrap() with a character limit below 2

[mdalikadar]

Description

word_wrap() never returns when $charlim is smaller than 2.

The loop that splits over-length words takes $charlim - 1 characters per
iteration:

• $charlim = 1: the chunk length is 0, so $line never shrinks and the
  while (mb_strlen($line) > $charlim) loop runs forever.
• $charlim <= 0: the length check is always true, even for an empty line,
  so the loop never ends.

Reproduce:

helper('text');
word_wrap('aaaa bbbb', 1); // hangs until max_execution_time / forever on CLI

Changes:

• Clamp $charlim to at least 1 at the start of the function.
• Always consume at least one character per iteration of the splitting loop.

Behavior for $charlim >= 2 is unchanged. For $charlim <= 1, which previously
hung, word_wrap('aaaa bbbb', 1) now returns "aaa\na\nbbb\nb", the same
chunking the function applies for larger limits.

Added regression tests for limits of 1, 0 and a negative value. The new test
hangs without the fix and passes with it.

This is a bug fix with no signature or documented-behavior change, so it targets
develop.

Checklist:

• Securely signed commits
• Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
• Unit testing, with >80% coverage
• User guide updated (not needed: no documented behavior changed)
• Conforms to style guide

[Copilot]

Copilot review overview

🟡 Changes recommended

The new limit-1 behavior terminates but does not wrap at the documented character count.

Review effort: Balanced
Findings: 1

Open (1)

• Ensure word_wrap enforces one-character line limits · New

What changed in this PR

Prevents word_wrap() from hanging when the character limit is below 2.

Changes:

• Clamps limits to at least 1.
• Ensures each loop iteration consumes a character.
• Adds regression coverage for limits 1, 0, and -5.

The fix terminates, but limit 1 still produces lines longer than one character. Focused PHP verification confirmed a maximum line length of 3; PHPUnit and full validation remain for CI.

File	Description
system/​Helpers/​text_helper.php	Adds limit normalization and guaranteed loop progress.
tests/​system/​Helpers/​TextHelperTest.php	Tests small and negative limits.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.