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

get active (visible) courses only #787

Open
wants to merge 1 commit into
base: MOODLE_404_STABLE
Choose a base branch
from

Conversation

Ferenc-Frank-Lengyel
Copy link
Contributor

this is a possible solution to resolve issue #772

@danmarsden
Copy link
Owner

That looks like it may hide the tab for all users - and not just the student themselves?

I think there is a case where the teacher should still be able to see the report for courses that are hidden so we might still need to retain that access somehow.

I think we should check inside attendance_get_user_courses_attendances to see if the currently logged in user has the viewhiddencouress capabiliity and if so, get all users courses, and if not - get only the "active" courses.

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