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

Undefined index: page #369

Closed
Mte90 opened this issue Nov 2, 2020 · 7 comments · Fixed by #363
Closed

Undefined index: page #369

Mte90 opened this issue Nov 2, 2020 · 7 comments · Fixed by #363

Comments

@Mte90
Copy link
Contributor

Mte90 commented Nov 2, 2020

An user reported a bug with xhgui,

$page = (int)$options['page'];

From here https://github.com/perftools/xhgui/blob/master/src/Xhgui/Searcher/PdoSearcher.php#L101 it is not clear if that key exists.

@glensc
Copy link
Contributor

glensc commented Nov 4, 2020

🚧 Never link to branches, share a permalink 🚧

NOTE: edit your post and fix it with a permalink!

@Mte90
Copy link
Contributor Author

Mte90 commented Nov 4, 2020

Sorry I forgot it again

@glensc
Copy link
Contributor

glensc commented Nov 4, 2020

This PR covers this topic: #363

but I have not able to test the branch other than making the CI pass.

@Mte90
Copy link
Contributor Author

Mte90 commented Nov 4, 2020

I don't see any error.

@glensc
Copy link
Contributor

glensc commented Nov 4, 2020

I don't see any error.

???

@Mte90
Copy link
Contributor Author

Mte90 commented Nov 4, 2020

Sorry I meant that I tested with that branch and I don't see any error.

@glensc
Copy link
Contributor

glensc commented Nov 4, 2020

You could have said that in that PR then, would be made more sense.

merging it then.

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 a pull request may close this issue.

2 participants