Since most implementations of Drupal for elearning uses Organic Groups, Certify needs to be aware of this.
I am not sure how to do this. As it is now, the code uses module_exists('og') for a few conditionals (namely the users on the instructors overview page), but others (book and quiz listing) does not. This might be a "good enough" technique for the 1.0 release (the book and quiz listing has to be added ofcourse), but does anyone have a better idea on how to accomplish this? Exposing hooks for this might need a LOT of hooks for all conditions types.
The current implementation also enforces the OG filtering, as long as OG is enabled. I guess there should at the very least be a switch for it on the settings page for the module.
Any suggestions are welcome!
Comments
Comment #1
fuzzy76 commentedCommited code for a settings switch, and added some !TODO markers to the places this needs to be fixed.
Comment #2
fuzzy76 commentedIssues that are parts of this are #942408: Available quizzes needs to be filtered by OG and #942406: Available books needs to filter when in OG
Comment #3
fuzzy76 commentedClosing this, as the two issues mentioned above are the only pieces of the puzzle left as far as I can tell. Filtering per OG in certify blocks was committed to 2.0beta3.