Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-778 META: Migrate to Ionic
  3. MOBILE-1049

Call to core_user_get_users_by_field has wrong parameters

    XMLWordPrintable

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Users
    • Labels:
    • Testing Instructions:
      Hide

      The problem for testing this is that core_user_get_users_by_field is not on Mobile services, so it has to be added. To do so:

      1. Add that function to lib/db/services.php on your Moodle.
      2. Increase version.php and update Moodle.

      The links to complete profile are only in Messages addon, so you'd have to manually add that addon or wait for it to be integrated.

      Once all that is done, you can test it:

      1. Go to Messages -> Contacts.
      2. Click on a contact and check you can see the complete profile. You can add console.log to make sure the right WS is used.
      Show
      The problem for testing this is that core_user_get_users_by_field is not on Mobile services, so it has to be added. To do so: Add that function to lib/db/services.php on your Moodle. Increase version.php and update Moodle. The links to complete profile are only in Messages addon, so you'd have to manually add that addon or wait for it to be integrated. Once all that is done, you can test it: Go to Messages -> Contacts. Click on a contact and check you can see the complete profile. You can add console.log to make sure the right WS is used.
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Pull Master Branch:

      Description

      On user profile, the call to core_user_get_users_by_field is using wrong parameters.

      We're not using this function because it's not on Mobile services, but we should since core_user_get_users_by_id is deprecated. Also, by_id said a teacher couldn't see a student profile, but using by_field I could see it.

        Attachments

          Activity

            People

            Assignee:
            dpalou Dani Palou
            Reporter:
            dpalou Dani Palou
            Peer reviewer:
            Frédéric Massart Frédéric Massart
            Integrator:
            Juan Leyva Juan Leyva
            Participants:
            Component watchers:
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              31/Jul/15