Skip to content

Conversation

arffsaad
Copy link

@arffsaad arffsaad commented Oct 8, 2025

Description of Problem

Using Leaf Core, noticed that I CANT use request()->all(). Turns out that method does not exist at all. Digging inside the Leaf\Http package in Request.php reveals that there are no such function called all(), but a function called body() that does what is needed.

Other than that, the release branch of Http has overlapping functions params() and getOrDefault() that do the exact same function. The document mentions the use of getOrDefault, but the main branch for Http module showed that getOrDefault() was dropped in favor of params()

Proposed Solution

To update the documentation to reflect the actual methods in Http Module.

Additional Information

None. Hope my PR has enough info and helps.

Copy link

netlify bot commented Oct 8, 2025

Deploy Preview for leaf3 ready!

Name Link
🔨 Latest commit 18fea6f
🔍 Latest deploy log https://app.netlify.com/projects/leaf3/deploys/68e636add9e59c0008f88d86
😎 Deploy Preview https://deploy-preview-128--leaf3.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 32 (🟢 up 1 from production)
Accessibility: 74 (no change from production)
Best Practices: 83 (no change from production)
SEO: 81 (no change from production)
PWA: 60 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

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