split the webservices/functions.php into user / group / institution functions for better code separation

[Robert Lyon]

Currently the classes mahara_user_external, mahara_group_external, and mahara_institution_external are all within the webservices/functions.php file.

I think as we add more functions to that file it will get quite large. So for easier editing will be worth breaking that up into a structure like:

webservices/functions/user.php - for user functions
webservices/functions/group.php - for group functions
webservices/functions/institution.php - for institution functions

this way if we want to add another set of functions say 'collection' it would be a matter of adding in
webservices/functions/collection.php

[Aaron Wells (u-aaronw)]

Good idea. We have too many giant files in Mahara already. :)

[Mahara Bot (dev-mahara)]

Patch for "master" branch: https://reviews.mahara.org/4110

[Robert Lyon (robertl-9)]

Ok, have gone almost with what I mentioned in original report except that the name of the file in the functions directory is based on the classname field in the external_functions table and therefore the class the file holds.

So we have:
webservices/functions/mahara_user_external.php - for user functions
webservices/functions/mahara_group_external.php - for group functions
webservices/functions/mahara_institution_exteranl.php - for institution functions

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/4110
Committed: http://gitorious.org/mahara/mahara/commit/dbe457bae7b45f921d5ae0058e6abb71fee2eda3
Submitter: Aaron Wells (<email address hidden>)
Branch: master

commit dbe457bae7b45f921d5ae0058e6abb71fee2eda3
Author: Robert Lyon <email address hidden>
Date: Mon Dec 22 16:33:55 2014 +1300

Separation of the functions (Bug #1393621)

Allowing each of the 'classname' classes listed in external_functions
to be it's own function file to make it easier for developers.

Change-Id: I9c4e3dfc198abdf2c274c8a7d569fce5bf431471
Signed-off-by: Robert Lyon <email address hidden>