New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: Ability to pass static data to a blade render #3067

Merged

yajra merged 1 commit into yajra:master from OzanKurt:patch-8

Sep 29, 2023

Contributor

OzanKurt commented Sep 18, 2023

Usage:

$builder->editColumn('permissions_display', [
    'admin.roles.data-table.permissions-display', ['groupedPermissions' => $groupedPermissions],
]);

I couldn't decide if I should make it an associative array or not, but this seems simple enough. If the $content is an array get the split it and add the second value to its view data.

Extremely straightforward code for mini extra function.

if (is_array($content)) {
    [$view, $viewData] = $content;

    return static::compileBlade($view, static::getMixedValue($data, $param)  + $viewData);
}


          Update Helper.php

dac4426

sonarcloud bot commented Sep 18, 2023

SonarCloud Quality Gate failed.

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
13.7% Duplication

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

Contributor Author

OzanKurt commented Sep 29, 2023

@yajra Any luck about this? 😄

yajra approved these changes

View reviewed changes

Owner

yajra left a comment

Looks good and does not break any tests. It would be nice if we could also document this in https://github.com/yajra/laravel-datatables-docs/blob/10.0/edit-column.md. Thanks!

yajra changed the title ~~Ability to pass static data to a blade render~~ feat: Ability to pass static data to a blade render

yajra merged commit 39923f8 into yajra:master

4 of 5 checks passed

Owner

yajra commented Sep 29, 2023

Released on v10.9.0 🚀 Thanks!

OzanKurt mentioned this pull request

Updated Docs yajra/laravel-datatables-docs#96

Merged

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

Labels

None yet