Skip to content
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

Error with Content Culture Picker and TheTheme #15673

Closed
agriffard opened this issue Apr 6, 2024 · 4 comments · Fixed by #15681
Closed

Error with Content Culture Picker and TheTheme #15673

agriffard opened this issue Apr 6, 2024 · 4 comments · Fixed by #15681
Labels
Milestone

Comments

@agriffard
Copy link
Member

When I enable the Content Culture Picker and use the Default theme, there is an error on the home page (Saas recipe):

RuntimeBinderException: 'OrchardCore.DisplayManagement.Views.ShapeViewModel<OrchardCore.Admin.Models.Navbar>' does not contain a definition for 'CurrentCulture'

AspNetCoreGeneratedDocument.Views_ContentCulturePicker.ExecuteAsync() in ContentCulturePicker.cshtml
line 23            if (culture.Name != Model.CurrentCulture.Name)

image

@hishamco
Copy link
Member

hishamco commented Apr 7, 2024

Is the navbar shape added?

@agriffard
Copy link
Member Author

Is the navbar shape added?

Yes, it is using the TheTheme:

@agriffard agriffard changed the title Error with Content Culture Picker and Default theme Error with Content Culture Picker and TheTheme Apr 7, 2024
@agriffard
Copy link
Member Author

agriffard commented Apr 7, 2024

I tested on another project with a theme copy/pasted from TheTheme and I have the same error using the preview packages version 1.9.0-preview-18167

but this time, the exception is earlier, on line 14:

AspNetCoreGeneratedDocument.Views_Layout.ExecuteAsync() in Layout.cshtml
14 var navbar = await DisplayAsync(await DisplayManager.BuildDisplayAsync<Navbar>(UpdateModelAccessor.ModelUpdater));

@hishamco
Copy link
Member

hishamco commented Apr 7, 2024

/cc @MikeAlhayek

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 a pull request may close this issue.

3 participants