Details

    • Type: Task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Web Services
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      add new groups

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              tusefomal Tusefomal added a comment -

              The current implementation of this function is here:

              http://cvs.moodle.org/contrib/patches/dfws/webservice/apis/group.lib.php?revision=1.1&view=markup

              But for the API is necessary a massive function, I'm working on the new one but I have to know the params. By now, function gets courseid, groupname and groupdescription, but this params force to use an internal id course identifier. In other functions like get_users (http://tracker.moodle.org/browse/MDL-12887) and get_courses (http://tracker.moodle.org/browse/MDL-13128) the philosophy is to use a criteria param which determine the kind of identifier (id, idnumber...). In the other hand, I think that description is not a necessary param and can be deleted or threated in a group modifier function.

              My proposal is:

              /**

              • add a group to specified course
              • @param string criteria {idnumber, id, shortname}
              • @param string course: course key
              • @param string groupname
              • @return associative array groupname => integer (identifier if ok or -1 if error)
                */
                array add_groups (criteria,course,groupnames)

              Group name in a course are unique and it's possible to use group name and course to identify it.

              Show
              tusefomal Tusefomal added a comment - The current implementation of this function is here: http://cvs.moodle.org/contrib/patches/dfws/webservice/apis/group.lib.php?revision=1.1&view=markup But for the API is necessary a massive function, I'm working on the new one but I have to know the params. By now, function gets courseid, groupname and groupdescription, but this params force to use an internal id course identifier. In other functions like get_users ( http://tracker.moodle.org/browse/MDL-12887 ) and get_courses ( http://tracker.moodle.org/browse/MDL-13128 ) the philosophy is to use a criteria param which determine the kind of identifier (id, idnumber...). In the other hand, I think that description is not a necessary param and can be deleted or threated in a group modifier function. My proposal is: /** add a group to specified course @param string criteria {idnumber, id, shortname} @param string course: course key @param string groupname @return associative array groupname => integer (identifier if ok or -1 if error) */ array add_groups (criteria,course,groupnames) Group name in a course are unique and it's possible to use group name and course to identify it.
              Hide
              dcastro David Castro added a comment -

              Implemented and saved on CVS.

              Show
              dcastro David Castro added a comment - Implemented and saved on CVS.
              Hide
              pigui Jordi Piguillem Poch added a comment -

              I'm using groups_create_group() function to implement this WS.

              I was thinking about checking that the courseid really exist before calling it, but it would be better to check it inside groups_create_group()....

              Any clue or comment?

              Show
              pigui Jordi Piguillem Poch added a comment - I'm using groups_create_group() function to implement this WS. I was thinking about checking that the courseid really exist before calling it, but it would be better to check it inside groups_create_group().... Any clue or comment?

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    24/Nov/10