Make all entities identifiable #59
No reviewers
Labels
No labels
Compat/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Research
Kind/Security
Kind/Testing
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Blocked
Status
Need More Info
No project
No assignees
1 participant
Notifications
Total time spent: 9 minutes 51 seconds
Due date
c.fahner
9 minutes 51 seconds
No due date set.
Dependencies
No dependencies set
Reference
slendium/ocd!59
Loading…
Reference in a new issue
No description provided.
Delete branch "issues/46"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Resolves #46
Update Entity to add $id property
Remove Entity\Identifiable interface
@ -11,0 +8,4 @@* Objects are not required to implement this interface to generate a {@see Schema}.* In this case the schema won't contain an automatically generated primary key.** Further configuration of the ID field requires adding the {@see Schema\IdOptions} attribute.Explain where to add this attribute
@ -31,3 +21,1 @@public function test_fromConstructorParameters_shouldSetDefaultIdGeneratorWithoutIdField_whenEntityIsIdentifiableAndDeclaresIdField(): void {$sut = Schema::fromConstructorParameters(SchemaTest\EmptyIdentifiableEntity::class);public function test_fromConstructorParameters_shouldSetDefaultIdGeneratorWithoutIdField(): void {Rename "shouldSetDefaultIdGenerator
WithoutIdField"@ -11,4 +0,0 @@* @author C. Fahner* @copyright Slendium 2026*/final class NonEntity implements Entity\Identifiable {Check if the non-entity case (with fields) is still tested. From what I can tell only the EmptyObject case is tested.