Skip to content

[6.x] Update logged_in to use the guard from the Statamic config files #11666

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.

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

Merged
merged 3 commits into from
Apr 24, 2025

Conversation

vmitchell85
Copy link

This PR updates the logged_in variable to use the correct guard from the users.php config file.

I didn't add any test as I couldn't determine a good way to do that (it's also a bit late so the brain might figure it out in the morning).

I don't expect this to break anything. I ran into this while using logged_in after following the guide regarding Using an Independent Authentication Guard.

@jasonvarga jasonvarga changed the title [5.x] Update logged_in to use the guard from the Statamic config files [6.x] Update logged_in to use the guard from the Statamic config files Apr 22, 2025
@jasonvarga jasonvarga changed the base branch from 5.x to master April 22, 2025 19:19
@jasonvarga
Copy link
Member

jasonvarga commented Apr 22, 2025

Since this does change some behavior potentially, I've changed this to target v6.

In the mean time, you can add a variable yourself. Throw this into your app service provider:

\Illuminate\Support\Facades\View::composer('*', function ($view) {
    $view->with(['logged_into_guard' => auth('yourguard')->check()]);
});

And use logged_into_guard instead of logged_in.

@jasonvarga jasonvarga merged commit 2225ed3 into statamic:master Apr 24, 2025
18 checks passed
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.

3 participants