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.

Sign up for GitHub

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

Jump to bottom

Illegal offset type exception should mention the type #10432

Closed
mvorisek opened this issue Jan 24, 2023 · 2 comments · Fixed by #10504
Closed

Illegal offset type exception should mention the type #10432

mvorisek opened this issue Jan 24, 2023 · 2 comments · Fixed by #10504
Labels
Category: Engine Feature

Comments

@mvorisek
Copy link
Contributor

Description

https://3v4l.org/m069E
@marcosmarcolin
Copy link
Contributor

marcosmarcolin commented Jan 24, 2023 •
edited
Loading

I agree with you! @cmb69 Do you think this is viable?

If yes, I would like to try to apply this change.

@cmb69
Copy link
Member

cmb69 commented Jan 24, 2023

If yes, I would like to try to apply this change.

Yeah, sure, give it a try. It is not my decision, though, whether a respective PR will be accepted (albeit it is not unlikely; looks like a useful feature).

@marcosmarcolin marcosmarcolin mentioned this issue Jan 25, 2023
Closed
@marcosmarcolin marcosmarcolin mentioned this issue Feb 4, 2023
Merged
@Girgias Girgias linked a pull request Feb 5, 2023 that will close this issue
Inform type in messsage: 'Illegal offset type' #10504
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Category: Engine Feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Inform type in messsage: 'Illegal offset type' marcosmarcolin/php-src
4 participants
@mvorisek @cmb69 @Girgias @marcosmarcolin