Skip to content

Conversation

tusharnain
Copy link
Contributor

Overview

This PR improves the Helper::compileBlade method by replacing the use of eval() with the safer and Laravel-native Blade::render() method.

Changes

  • Removed the eval() approach to compile Blade strings.
  • Added Blade::render($str, $data) for string compilation.
  • Maintains existing functionality while improving security and maintainability.

Benefits

  • ✅ Safer: Eliminates potential security risks from eval().
  • ✅ Cleaner: Uses Laravel's built-in Blade rendering.
  • ✅ More maintainable and testable code.

Notes

  • No behavioral changes for existing functionality.
  • Fully backward compatible.

Copy link

sonarqubecloud bot commented Oct 7, 2025

@yajra yajra merged commit e2c8526 into yajra:master Oct 8, 2025
7 of 8 checks passed
@yajra
Copy link
Owner

yajra commented Oct 8, 2025

🎉 This PR is included in version 12.6.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@yajra yajra added the released label Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants