- Issue created by @wim leers
- Merge request !481issue #3494363: Move config entity classes from src/Entity to src/Config/Entity. → (Closed) created by Unnamed author
Move the classes from \Drupal\experience_builder\Entity` to `\Drupal\experience_builder\Config\Entity`. After doing this and changes the all classes name the error is coming.
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Ugh … I was wrong 🙈😬
Due to
public function __construct(…) { parent::__construct('Entity', $namespaces, $module_handler, 'Drupal\Core\Entity\EntityInterface'); … $this->discovery = new AnnotatedClassDiscovery('Entity', $namespaces, 'Drupal\Core\Entity\Annotation\EntityType'); … }
in
\Drupal\Core\Entity\EntityTypeManager::__construct()
, it is impossible to put (content or config or otherwise) entity definitions in any other place thansrc/Entity
! 🤯This is highly misleading in Drupal core, because, these critical files do live in a
…/Config/Entity/…
subdirectory:Drupal\Core\Config\Entity\ConfigEntityInterface
\Drupal\Core\Config\Entity\ConfigEntityBase
- …
😬
- Merge request !482Resolve #3494363 "Content vs config entities owners" → (Merged) created by wim leers
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Sorry to have wasted some of your time, @utkarsh_kumar_singh 🙈
This was a lesson for me as well, I learned something new about Drupal core here! 😅
Your effort does not go unnoticed nor unappreciated though: you will be credited for this issue! 😊
- First commit to issue fork.
-
wim leers →
committed 73248450 on 0.x
Issue #3494363 by wim leers, utkarsh_kumar_singh, longwave: CODEOWNERS:...
-
wim leers →
committed 73248450 on 0.x