Skip to content
This repository has been archived by the owner on Jun 4, 2024. It is now read-only.

Properties that are marked as readOnly are not read only #4

Open
ItsReddi opened this issue Oct 23, 2019 · 1 comment
Open

Properties that are marked as readOnly are not read only #4

ItsReddi opened this issue Oct 23, 2019 · 1 comment
Assignees
@cebe
Labels
generator-feature Functionality missing from the generator
Milestone
2.0

Comments

@ItsReddi
Copy link

The generated code does not respect the "readOnly" property.
This is because of the generated model rules. and the requestbody is loaded against the DatabaseModel.

It could be a good idea to create a validator for each endpoint to be able to mix the same property on different endpoints.
@cebe
Copy link
Owner

cebe commented Oct 23, 2019

yeah, validation should use the schema definition instead of yii model rules, this is currently not implemented.

@cebe cebe added the generator-feature Functionality missing from the generator label Oct 23, 2019
@cebe cebe added this to the 2.0 milestone Aug 3, 2020
cebe pushed a commit that referenced this issue Jan 5, 2023
@SOHELAHMED7
Merge pull request #4 from SOHELAHMED7/104-remove-id-from-rules-metho…
86ae33d 
…d-from-base-model-during-code-generation

Fix 104- Remove id from rules() method from base model during code generation
@cebe cebe moved this to 🆕 New in yii2-openapi Project Jan 27, 2023
@cebe cebe moved this from 🆕 New to 📋 Backlog in yii2-openapi Project Jan 27, 2023
@cebe cebe mentioned this issue Jan 27, 2023
Open
@cebe cebe moved this from 📋 Backlog to 🔢 Concept Needed in yii2-openapi Project Jan 27, 2023
@cebe cebe self-assigned this Jan 27, 2023
@siggi-k siggi-k mentioned this issue May 27, 2024
Closed
@SOHELAHMED7 SOHELAHMED7 mentioned this issue Jul 4, 2024
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@cebe cebe

Labels
generator-feature Functionality missing from the generator
Projects
yii2-openapi Project
Status: 🔢 Concept Needed
Milestone
2.0
Development

No branches or pull requests
2 participants
@cebe @ItsReddi