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

Group together subgroups of courses #65

Open
marcoscale98 opened this issue Nov 30, 2020 · 5 comments
Open

Group together subgroups of courses #65

marcoscale98 opened this issue Nov 30, 2020 · 5 comments
Labels
Elaborate enhancement New feature or request (But it is costly, PR is seen with pleasure.)

Comments

@marcoscale98
Copy link

My moodle courses are diveded in "year of course/Graduation Year" (it is difficult to put into words, but it is something logical). Could you do the same grouping in folders in automatic? At the moment my courses are all together in the main folder.

@marcoscale98 marcoscale98 added the enhancement New feature or request label Nov 30, 2020
@C0D3D3V
Copy link
Owner

C0D3D3V commented Nov 30, 2020

Can you send a screenshot of the grouping? (Also you can add me on Telegram (Username: c0d3d3v) if you don't want to post them here)

@marcoscale98
Copy link
Author

Division into academic years
image
Division into graduation years
image
Division into paths
image
Courses
image

@marcoscale98 marcoscale98 mentioned this issue Jan 14, 2021
1 task
@C0D3D3V
Copy link
Owner

C0D3D3V commented Jan 18, 2021

This is kind of related to #6 because we have to follow the course tree.

To find out the category of a course there are basically only two functions: core_course_get_courses_by_field and core_course_get_categories. But a call to core_course_get_categories can take several seconds to minutes in the worst case in a large Moodle.

What we would do is adjust the filter to get the categories of courses you are enrolled in, that would simplify the call.

But I am not sure if this function is useful. Because you can just rename a course as you want in the configuration.

@marcoscale98
Copy link
Author

To find out the category of a course there are basically only two functions: core_course_get_courses_by_field and core_course_get_categories. But a call to core_course_get_categories can take several seconds to minutes in the worst case in a large Moodle.

I think you can allow this with a parameter in initial configuration. My moodle is always quick to load

@C0D3D3V C0D3D3V added Elaborate enhancement New feature or request (But it is costly, PR is seen with pleasure.) and removed enhancement New feature or request labels Oct 6, 2021
@C0D3D3V
Copy link
Owner

C0D3D3V commented Mar 18, 2023

If the API call takes to long, we could call the API once to get the sturcture and save it in the config for every configuered course. So you do not need to make the call every time you run moodle-dl. Since the structure will only very rarly change, this should be fine...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Elaborate enhancement New feature or request (But it is costly, PR is seen with pleasure.)
Projects
None yet
Development

No branches or pull requests

2 participants