- ๐บ๐ธUnited States danflanagan8 St. Louis, US
The patch in #11 is not sufficient. Here's something else that needs to be addressed: LanguageSwitchingTest uses the stable theme, but the stable theme is deprecated.
https://git.drupalcode.org/project/lang_dropdown/-/blob/8.x-2.x/tests/sr...
Ideally we change that to
stark
. There's also the option of pulling in the contrib stable through composer, but using stark would be better. - ๐ฎ๐ณIndia prashant.c Dharamshala
Following errors/warnings also needs to be fixed:
------ ----------------------------------------------------------------------- Line src/Form/LanguageDropdownForm.php ------ ----------------------------------------------------------------------- 70 Property Drupal\lang_dropdown\Form\LanguageDropdownForm::$renderer has unknown class Drupal\lang_dropdown\Form\Drupal\Core\Render\RendererInterface as its type. ๐ก Learn more at https://phpstan.org/user-guide/discovering-symbols 188 Variable $language in empty() always exists and is not falsy. 203 Access to an undefined property Symfony\Component\HttpFoundation\RequestStack::$query. 231 Call to method renderPlain() on an unknown class Drupal\lang_dropdown\Form\Drupal\Core\Render\RendererInterface. ๐ก Learn more at https://phpstan.org/user-guide/discovering-symbols 365 Access to an undefined property Symfony\Component\HttpFoundation\RequestStack::$query. ------ ----------------------------------------------------------------------- ------ ------------------------------------------------------------------------------ Line src/Plugin/Block/LanguageDropdownBlock.php ------ ------------------------------------------------------------------------------ 682 Variable $code in PHPDoc tag @var does not exist. 682 Variable $values in PHPDoc tag @var does not exist. 763 Method Drupal\Core\Form\FormBuilderInterface::getForm() invoked with 4 parameters, 1 required. 812 Call to an undefined method Drupal\Core\TypedData\TranslationStatusInterface::getTranslationLanguages(). ------ ------------------------------------------------------------------------------ [ERROR] Found 9 errors
- ๐ช๐ธSpain Carlos Romero
Carlos Romero โ made their first commit to this issueโs fork.
- ๐ต๐ฑPoland ad0z
I am attaching patch with compatibility fixes related with #12 and #13.
For #12 as recommended I changed theme to "stark" and tests are passing on my local environment.
For #13 everything should be fixed exceptMethod Drupal\Core\Form\FormBuilderInterface::getForm() invoked with
4 parameters, 1 required.I don't know if we can do anything else than
// @phpstan-ignore-next-line
- ๐ต๐ฑPoland ad0z
I am attaching patch with compatibility fixes related with #12 and #13.
For #12 as recommended I changed theme to "stark" and tests are passing on my local environment.
For #13 everything should be fixed exceptMethod Drupal\Core\Form\FormBuilderInterface::getForm() invoked with
4 parameters, 1 required.I don't know if we can do anything else than
// @phpstan-ignore-next-line
- Status changed to Needs review
over 1 year ago 8:37am 17 May 2023 - last update
over 1 year ago 1 pass - ๐บ๐ธUnited States msypes
Note to others out there in Drupal-land:
The patch in #17 is meant for the 2.x-dev@dev branch. It cannot be applied to the 8.x-2.0 release. It does, however, apply cleanly to the dev branch. - ๐ฏ๐ดJordan Rajab Natshah Jordan
โ Released lang_dropdown-8.x-2.1 โ
- Status changed to Fixed
over 1 year ago 8:36pm 29 June 2023 - ๐ง๐ชBelgium bramvandenbulcke
This issue was preventing me from updating certain sites to Drupal 10. Thanks, Rajab!
Automatically closed - issue fixed for 2 weeks with no activity.