- Issue created by @penyaskito
- ๐ฎ๐ณIndia jeeva r
jeeva r โ made their first commit to this issueโs fork.
- Merge request !12277Add constraint for Current Theme Condition theme so it can be FullyValidatable โ (Open) created by Unnamed author
- ๐ฎ๐ณIndia jeeva r
please review this https://git.drupalcode.org/project/drupal/-/merge_requests/12277
- ๐ช๐ธSpain penyaskito Seville ๐, Spain ๐ช๐ธ, UTC+2 ๐ช๐บ
Fix looks good, so good that it broke the tests that assumed a theme that it's not installed. We need to fix those tests.
- ๐ช๐ธSpain penyaskito Seville ๐, Spain ๐ช๐ธ, UTC+2 ๐ช๐บ
- ๐บ๐ธUnited States smustgrave
Seems like valid conditions.
Installed the condition_test module and used configuration_inspector
- First commit to issue fork.
- ๐ฎ๐ณIndia jeeva r
@smustgrave, if the testing was completed and reviewed, please update this issue status.. thank you.
- ๐ฌ๐งUnited Kingdom alexpott ๐ช๐บ๐
If we add this validation then we also need to add the dependency :)
We need to add something like:
/** * {@inheritdoc} */ public function calculateDependencies() { if (!$this->configuration['theme']) { return []; } return ['theme' => [$this->configuration['theme']]]; }
to \Drupal\system\Plugin\Condition\CurrentThemeCondition
- ๐ฌ๐งUnited Kingdom alexpott ๐ช๐บ๐
And we need an update function to at least save all blocks where the condition plugin is used...
- ๐บ๐ธUnited States smustgrave
And we need an update function to at least save all blocks where the condition plugin is used...
Still appears needed.