search

marxjohnson / moodle-block_quickcourselist

Moodle block providing a quick method of searching course pages
http://moodle.org/mod/data/view.php?d=13&rid=2334
6 stars 7 forks source link

Up #5

Closed alekiy closed 10 years ago

alekiy commented 10 years ago

Hello Marx,

here is an Pull&Merge Request for you. I added the possibility to set whether to use the shortname, fullname or shortname: fullname when displaying the search results (see #2) Additional I added the wished applicable_formats function (see #3).

Best wishes Alex

marxjohnson commented 10 years ago

Hi @alekiy, Sorry for not getting back to you about this sooner. I'll try and get this tested and merged this week.
I'm just a bit confused about commit e10eae8. can you explain what it does?

alekiy commented 10 years ago

Hey Marx,

you can ignore the first commit. It was somehow pushed :(

marxjohnson commented 10 years ago

I've now merged the changes to enable the configuration thank you for the contribution. The applicable formats function was in the wrong place and I don't think it's the best solution to the problem but I'll add my thoughts to #3.