Skip to content

Rename Form::setModel() method to Form::setEntity()#2239

Merged
mvorisek merged 28 commits intodevelopfrom
form_setEntity
Jan 6, 2025
Merged

Rename Form::setModel() method to Form::setEntity()#2239
mvorisek merged 28 commits intodevelopfrom
form_setEntity

Conversation

@mvorisek
Copy link
Copy Markdown
Member

@mvorisek mvorisek commented Jan 3, 2025

continue #2192

mainly for Form class, but also for Card, CardSection, CardTable, Form\AbstractLayout and InlineEdit classes

and remove $model/$entity properties from View

@mvorisek mvorisek added the MAJOR label Jan 3, 2025
@mvorisek mvorisek force-pushed the form_setEntity branch 6 times, most recently from 54f9db6 to 0511b82 Compare January 5, 2025 17:09
@mvorisek mvorisek marked this pull request as ready for review January 5, 2025 19:12
@mvorisek mvorisek force-pushed the form_setEntity branch 4 times, most recently from c6921b6 to 2c0fab0 Compare January 5, 2025 20:24
Copy link
Copy Markdown
Member

@DarkSide666 DarkSide666 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mvorisek mvorisek merged commit a8e4d3a into develop Jan 6, 2025
@mvorisek mvorisek deleted the form_setEntity branch January 6, 2025 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants