Skip to content

Many Bugs fixed and new methods#422

Merged
papac merged 1 commit into
5.xfrom
refactor/code-base
Jun 22, 2026
Merged

Many Bugs fixed and new methods#422
papac merged 1 commit into
5.xfrom
refactor/code-base

Conversation

@papac

@papac papac commented Jun 22, 2026

Copy link
Copy Markdown
Member
  • file() — the broken !is_uploaded_file(...) === UPLOAD_ERR_OK check (always false due to precedence) now correctly rejects non-uploaded files while preserving the array/multi-file path.
  • locale() — the delimiter-less regex ('^([a-z]+)[-]?/i') that warned and never matched is now '/([a-z]+)[-]?/i', with null guards for a missing header.
  • query() / post() — return type was array but returned scalars (guaranteed TypeError); now mixed with a $default argument matching get()'s contract.
  • time() — read the non-existent $_SESSION['REQUEST_TIME']; now returns an int from $_SERVER['REQUEST_TIME'].
  • old() — $fallback had no default, fataling on old('x'); now defaults to null (also fixed the fullback typo).
  • hostname() / path() — guarded against undefined $_SERVER indices (CLI/tests) and fixed strpos truthiness with !== false.

Methods added:

isMethod(string) — match the request method.
input(key, default) — alias of get().
boolean(key, default) — coerce an input to bool (1/true/on/yes). bearerToken() — extract the token from an Authorization: Bearer … header.

- file() — the broken !is_uploaded_file(...) === UPLOAD_ERR_OK check (always false due to precedence) now correctly rejects non-uploaded files while preserving the array/multi-file path.
- locale() — the delimiter-less regex ('^([a-z]+)[-_]?/i') that warned and never matched is now '/([a-z]+)[-_]?/i', with null guards for a missing header.
- query() / post() — return type was array but returned scalars (guaranteed TypeError); now mixed with a $default argument matching get()'s contract.
- time() — read the non-existent $_SESSION['REQUEST_TIME']; now returns an int from $_SERVER['REQUEST_TIME'].
- old() — $fallback had no default, fataling on old('x'); now defaults to null (also fixed the fullback typo).
- hostname() / path() — guarded against undefined $_SERVER indices (CLI/tests) and fixed strpos truthiness with !== false.

Methods added:

isMethod(string) — match the request method.
input(key, default) — alias of get().
boolean(key, default) — coerce an input to bool (1/true/on/yes).
bearerToken() — extract the token from an Authorization: Bearer … header.
@papac papac merged commit bcb44d9 into 5.x Jun 22, 2026
9 checks passed
@papac papac deleted the refactor/code-base branch June 22, 2026 01:49
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.

1 participant