Romern / syncMyMoodle

Synchronization client for RWTH Moodle
GNU General Public License v3.0
72 stars 18 forks source link

Add option to remove prefix "(VO)" or "(VU)" #61

Open autoantwort opened 3 years ago

autoantwort commented 3 years ago

Or make this the default for everybody.
Reason:
When I am in the file browser I want to type the name of the course to select the corresponding folder, but this is currently not possible.

HarHarLinks commented 3 years ago

while at it, the semester prefix 21ss/ etc could also be optional

septatrix commented 3 years ago

Hm Moodle (or rather the way the ITC names moodle courses) does not easily provide that information but I think a regex with a named capture group could solve this pretty well. The default might be (?P<course_name>.*) but one could overwrite that in the config

n0toose commented 2 years ago

Oh, I did something like that just yesterday. This should be fixable somewhere around here, I am just not sure about the full set of possible prefixes. (e.g. "SK" and "AG" and "RE", for "Sprachkurs", "Arbeitsgruppe" and "Rechenübung" respectively) also exist.

https://github.com/Romern/syncMyMoodle/blob/f8f10da7cf6f79cf25711dd690e756938c3ebf77/syncmymoodle/__main__.py#L760

n0toose commented 2 years ago

Feedback: Mind the conflicts. (e.g. Downloading all of the material from the Informatik/Computer Science B.Sc. course "Software Quality Assurance" that way would mean that both folders would have the same name without the prefix.)

n0toose commented 2 years ago

Idea: What if we were to put the suffixes in the end of the directory name?