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

Add more information to MissingMandatoryField exception #74

Closed
BlobbyBob opened this issue Dec 22, 2017 · 1 comment
Closed

Add more information to MissingMandatoryField exception #74

BlobbyBob opened this issue Dec 22, 2017 · 1 comment
Assignees
Labels

Comments

@BlobbyBob
Copy link

Issue

When getting a MissingMandatoryField exception it can be hard to identify from which Telegram Method it came, since the stack trace only shows the TelegramMethods class as source.

Suggestion:

Change the error message format to:
'The field "%s" in ' . get_class($this) . ' is mandatory and empty, please correct' (Abstracts/TelegramMethods.php, l. 78)

@unreal4u unreal4u self-assigned this Dec 22, 2017
@unreal4u
Copy link
Owner

Good suggestion!

Will include the change, however I will be kind of busy this weekend, so I think I will pick this up on Monday 25th.

Greetings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants