Skip to content

[DomCrawler] fixes typo Form::getFields() -> Form::getFiles() #20077

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

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

mvhirsch
Copy link
Contributor

Looks like this method does not exist in Form: https://github.com/symfony/symfony/blob/5.4/src/Symfony/Component/DomCrawler/Form.php

I think it never did. Looking at v2.8 it does not exist, same's true for 3.4, 4.4, 5.4, 6.4, 7.0 ...

@carsonbot carsonbot added this to the 5.4 milestone Jul 29, 2024
@carsonbot carsonbot changed the title Bugfix: removes unknown method Form::getFields() [DomCrawler] Bugfix: removes unknown method Form::getFields() Jul 29, 2024
@OskarStark OskarStark added the bug label Jul 29, 2024
@OskarStark OskarStark changed the title [DomCrawler] Bugfix: removes unknown method Form::getFields() [DomCrawler] remove unknown method Form::getFields() Jul 29, 2024
@smnandre
Copy link
Member

smnandre commented Aug 2, 2024

Wonder if this was not supposed to be getFiles

@mvhirsch
Copy link
Contributor Author

mvhirsch commented Aug 5, 2024

Wonder if this was not supposed to be getFiles

Maybe yes. Should I just rename instead of removing it @smnandre ?

@xabbuh
Copy link
Member

xabbuh commented Aug 5, 2024

renaming getFields() to getFiles() looks good to me

@mvhirsch mvhirsch changed the title [DomCrawler] remove unknown method Form::getFields() [DomCrawler] fixes list of helpful methods (Form::getFields() -> Form::getFiles()) Aug 6, 2024
@mvhirsch mvhirsch changed the title [DomCrawler] fixes list of helpful methods (Form::getFields() -> Form::getFiles()) [DomCrawler] fixes typo Form::getFields() -> Form::getFiles() Aug 6, 2024
@xabbuh xabbuh force-pushed the bugfix/removes-mention branch from fb3498d to 8cfb8e0 Compare August 6, 2024 08:15
@xabbuh
Copy link
Member

xabbuh commented Aug 6, 2024

Good catch, thanks @mvhirsch.

@xabbuh xabbuh merged commit c321bf2 into symfony:5.4 Aug 6, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants