- Issue created by @f.mazeikis
- ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
This isn't quite what I meant. I meant something much simpler!
Starting to use validation constraints for SDC
*.component.yml
files would be going too far โ that's what the SDC JSON schema is already doing, these are simply the additional requirements that XB has.Maybe we could convert that all to validation constraints, but that seems overkill ๐
- ๐ฎ๐ณIndia Akhil Babu Chengannur
akhil babu โ made their first commit to this issueโs fork.
- Merge request !296#3473289: XB Component config entity's `status` may only be `true` if it meets all of XB's requirements โ (Open) created by Akhil Babu
- ๐ฎ๐ณIndia Akhil Babu Chengannur
Added a validation constarint to check if 'status' is TRUE.
Now updating the tests - Status changed to Needs work
2 months ago 11:23am 12 September 2024 - ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
@kristen pol in #4: that
iff
wasn't a typo: https://en.wikipedia.org/wiki/If_and_only_if@akhil babu Thanks for getting started on this! But you only added a constraint that simply refuses to accept
false
as a value. That's not what's described in the issue summary:# @todo Write validation constraint that verifies that IF this value is `true`, that \Drupal\experience_builder\Plugin\ComponentPluginManager::componentMeetsRequirements() also is met.
You're well on your way to that though โ you need to update the validator you wrote to call that method ๐
- Assigned to Akhil Babu
- Issue was unassigned.
- Status changed to Needs review
about 2 months ago 9:51am 17 September 2024 - ๐ฎ๐ณIndia Akhil Babu Chengannur
Thanks for the guidance @wim leers. I have updated the validator code.
Drupal\Tests\experience_builder\Kernel\ComponentTest::testObsoleteStatusHandling
still fails. I am not sure if this test should be removed or updated. Moving to needs review for feedback. - ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
๐ Prepare for multiple component types: ComponentTreeStructure should contain Component config entity IDs, not SDC IDs Fixed landed, which requires a small change here.
- ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
I'll first tackle the tricky config validation bits.
- ๐ง๐ชBelgium wim leers Ghent ๐ง๐ช๐ช๐บ
Didn't get to it a week ago; won't have time for it in the coming days; higher priorities to deal with.