#6793: Adding a content-independent culture selector shape for the front-end #8784
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# Conflicts: # src/Orchard.Web/Modules/Orchard.Localization/Orchard.Localization.csproj # src/Orchard.Web/Modules/Orchard.Localization/Services/LocalizationService.cs
…chard.Localization to Orchard.Framework
…f the culture to the query string
BenedekFarkas
commented
Apr 15, 2024
src/Orchard/Utility/Extensions/NameValueCollectionExtensions.cs
Outdated
Show resolved
Hide resolved
|
|
||
| ILocalizableAspect localizationPart = null, siteCultureLocalizationPart = null; | ||
| foreach (LocalizationPart l in localizations) { | ||
| if (l.Culture.Culture == cultureName) { |
There was a problem hiding this comment.
We are currently normalizing these in OC, should it compare with ignore case?
There was a problem hiding this comment.
Good idea! Updated in a5e4f1c.
| } | ||
|
|
||
| ContentItem localizedContentItem = localizationPart.ContentItem; | ||
| localizedRoute = localizedContentItem.Parts.Single(p => p is AutoroutePart).As<AutoroutePart>(); |
There was a problem hiding this comment.
Single is evil, it will throw if it's not exactly one element. But you might also want that.
There was a problem hiding this comment.
Thanks, I didn't notice it. As far as I can tell, it's not necessary to use it here. .Single is used in some other parts in Orchard (mostly in tests, but not only) to find a part by its name, but when the type is known, .As<> should be fine, since you can't have more than one of a (statically typed) part. Fixed in bf9b9b0.
sebastienros
approved these changes
Apr 18, 2024
BenedekFarkas
changed the title
BenedekFarkas
added a commit
that referenced
this pull request
Apr 19, 2024
* 6748: Stricter file and folder name validation (#6792) * Media Library: More strict file and folder name validation, fixes #6748 * Resetting MediaLibraryService changes to 1.10.x * Code styling in FileSystemStorageProvider * Adding string file and folder name validation to FileSystemStorageProvider, so that MediaLibrary components don't need to do it separately * Applying the same file and folder name validation to AzureFileSystem too * Code styling and fixes in AzureFileSystem, MediaLibrary and IStorageProvider * Simplifying invalid character detection * Code styling * Adding InvalidNameCharacterException to be able to handle invalid characters precisely at various user-facing components * Updating MediaLibrary not to log an error when a file can't be uploaded due to invalid characters --------- Co-authored-by: Lombiq <github@lombiq.com> * #6793: Adding a content-independent culture selector shape for the front-end (#8784) * Adds a new CultureSelector shape for front-end * fixed query string culture change * Moving NameValueCollectionExtensions from Orchard.DynamicForms and Orchard.Localization to Orchard.Framework * Code styling * Simplifying UserCultureSelectorController and removing the addition of the culture to the query string * EOF empty lines and code styling * Fixing that the main Orchard.Localization should depend on Orchard.Autoroute * Code styling in LocalizationService * Updating LocalizationService to not have to use IEnumerable.Single * Matching culture name matching in LocalizationService culture- and casing-invariant --------- Co-authored-by: Sergio Navarro <jersio@hotmail.com> Co-authored-by: psp589 <pablosanchez589@gmail.com> * #8640: Fixing consistency between different Enumeration Field flavors' data storage (#8789) * Reworking EnumerationField's logic to store/retrieve its (selected) values * Fixing exception when creating new item with CheckboxList flavor, adding more nullchecks and compactness * Code styling in EnumerationFieldDriver * Code styling in EnumerationField editor template * Fixing that EnumerationFieldDriver and the EnumerationField editor template should read SelectedValues instead of Values directly --------- Co-authored-by: Matteo Piovanelli <MatteoPiovanelli-Laser@users.noreply.github.com> * Fixing merge --------- Co-authored-by: Lombiq <github@lombiq.com> Co-authored-by: Sergio Navarro <jersio@hotmail.com> Co-authored-by: psp589 <pablosanchez589@gmail.com> Co-authored-by: Matteo Piovanelli <MatteoPiovanelli-Laser@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #6793
Originally developed by Sergio from XK Project.