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

Learner: Single course ID returns string instead of array of string, causes WP_Query error #6422

Open
jmalko opened this issue Jan 24, 2023 · 2 comments
Labels
[Status] Needs Author Reply Requires response from the author [Type] Bug

Comments

@jmalko
Copy link
Contributor

jmalko commented Jan 24, 2023

Am submitting a PR, here's the issue:

Steps to Reproduce

  1. Attempt to view a certificate
  2. See type error within WP_Query

What I Expected

  1. Wordpress expects the ['post__in'] arg to be an array

What Happened Instead

  1. Sensei_Learner methods return a string in the case of looking at a single course

PHP / WordPress / Sensei LMS version

Wordpress 6.1
Sensei 4.10

Browser / OS version

Any

Screenshot / Video

image

@jmalko jmalko added [Status] Needs Triage Issue needs to be triaged [Type] Bug labels Jan 24, 2023
@jmalko
Copy link
Contributor Author

jmalko commented Jan 24, 2023

#6423

@donnapep
Copy link
Collaborator

@jmalko Thanks for the report! Unfortunately, I haven't been able to reproduce this error by viewing a certificate from either WP admin or on the frontend. Are there any additional details you could provide to enable us to reproduce it? Thx!

@donnapep donnapep added [Status] Needs Author Reply Requires response from the author and removed [Status] Needs Triage Issue needs to be triaged labels May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Status] Needs Author Reply Requires response from the author [Type] Bug
Projects
None yet
Development

No branches or pull requests

2 participants