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

PHPDoc comment in MessageOriginChat mislead to wrong getter getChat(). #1472

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

r3bers
Copy link
Contributor

@r3bers r3bers commented Jun 25, 2024

Factory creates getter getSender_chat() because property name sender_chat in MessageOriginChat

? !
Type bug
BC Break yes / no
Fixed issues New.

Summary

Just simple quick fix because PHPDoc Comment was the same to MessageOriginChat and MessageOriginChannel but properties isn't

Factory creates getter getSender_chat() because property name sender_chat in MessageOriginChat
@TiiFuchs
Copy link
Member

It should be getSenderChat().
The snake case field names get converted to PascalCase.

Could you adjust it accordingly, please?

Factory creates getter getSenderChat() because property name sender_chat in MessageOriginChat
@r3bers
Copy link
Contributor Author

r3bers commented Jun 25, 2024

Oh Sorry, my bad. Too much python for today. Pushed getSenderChat()

Because bot has channel functionality and cleanup of telegram_updates table didn't clean lines with links to tables messages and edited_messages by post columns.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants