Problem/Motivation

Call to undefined method Drupal\Core\Language\LanguageManager::getLanguageConfigOverride() in Drupal\simple_favs\Form\SimpleFavsSettingsForm->buildForm() (line 58 of .... /modules/contrib/simple_favs/src/Form/SimpleFavsSettingsForm.php).

Steps to reproduce

install on 11.4.4

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments

lblabr created an issue. See original summary.

lblabr’s picture

lblabr commented

i didn't hat the language-module installed, so the configuration failed

joseph.olstad’s picture

joseph.olstad
Primary language French
commented
Status: Active » Closed (works as designed)

ok sounds like the issue is fixed?

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

lblabr’s picture

lblabr commented

not really, configuration does work, but fav's links are wrong (incl. language in url)
but in the system the nodes are adressed without language
so it seems that language is added somewhere

lblabr’s picture

lblabr commented
Status: Closed (works as designed) » Needs review
joseph.olstad’s picture

joseph.olstad
Primary language French
commented

ok ya, simple_favs will work if you add a language configuration to your site for example /en/node/1
the /en/ is the language code for english
whatever your language is you can add that , even if it's just one language it works, simple_favs however is designed to support numerous languages however will support just one if you do this.

Otherwise work could be done to support without language prefix but it would have to preserve the support of many or one with a prefix.

lblabr’s picture

lblabr commented

and the module needs to have the language module installed?

=> changig that would be a feature? right?

joseph.olstad’s picture

joseph.olstad
Primary language French
commented
Status: Needs review » Closed (duplicate)
Parent issue: » #3561865: Support multilingual sites with language undefined !?

There is a patch fix for the issue you're reporting. I'm not merging it because it breaks multilingual support. However the patch works with monolingual sans language prefix

#3561865: Support multilingual sites with language undefined !?

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

joseph.olstad’s picture

joseph.olstad
Primary language French
commented

I recommend that until we find a better solution to "use the patch"