Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Widget property current record #477

Open
baumannsven opened this issue Mar 19, 2019 · 1 comment
Open

Widget property current record #477

baumannsven opened this issue Mar 19, 2019 · 1 comment
Assignees
Milestone

Comments

@baumannsven
Copy link
Member

The widget property current record where use in the page tree. It is use if the picker builder not found the picker with the context page. There's no way that's gonna happen, I think.

So i think we can remove this.

@discordier What do you think?

See:

$widget->currentRecord = $model->getId();

$widget->currentRecord = $model->getId();

@baumannsven baumannsven added this to the Future milestone Mar 19, 2019
@baumannsven baumannsven self-assigned this Mar 19, 2019
@discordier
Copy link
Member

Nope, we have to keep it.
It is a Contao base property in widgets, see i.e. https://github.com/contao/core-bundle/search?q=currentRecord

Other widgets (i.e. https://github.com/terminal42/contao-dcawizard) also implement it.

@baumannsven baumannsven modified the milestones: Future, 2.2.0 Mar 21, 2020
@zonky2 zonky2 modified the milestones: 2.2.0, Future May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants