Moodle
  1. Moodle
  2. MDL-31753

Community finder uses wrong courseid when in "My Moodle" context

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 2.2.1, 2.3.1, 2.4
        • Fix Version/s: 2.2.5, 2.3.2
        • Component/s: Blocks
        • Labels:
        • Testing Instructions:
          Hide

          Prerequesite:

          • You need a user who has an ID which doens't have a matching coursed to demonstrate the problem. (you might need to delete a few courses, or create a few users to this)
          • Its easiest to assign this user as an admin for testing
          1. Login as aforementioned user
          2. Visit yourmoodle/my/
          3. Add the community finder block to mymoodle page
          4. Click on the search button in the block
          5. VERIFY: that a search form comes up without errors
          6. Add the community finder block to the front page
          7. Click on the search button in the block
          8. VERIFY: that a search form comes up without errors
          Show
          Prerequesite: You need a user who has an ID which doens't have a matching coursed to demonstrate the problem. (you might need to delete a few courses, or create a few users to this) Its easiest to assign this user as an admin for testing Login as aforementioned user Visit yourmoodle/my/ Add the community finder block to mymoodle page Click on the search button in the block VERIFY: that a search form comes up without errors Add the community finder block to the front page Click on the search button in the block VERIFY: that a search form comes up without errors
        • Difficulty:
          Easy
        • Affected Branches:
          MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
        • Fixed Branches:
          MOODLE_22_STABLE, MOODLE_23_STABLE
        • Pull from Repository:
        • Rank:
          38352

          Description

          Community finder block creates a link to search interface that includes courseid argument. The value for courseid is wrong when block is added to "My Moodle" page - it equals to userid instead of "1".

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Dan Poltawski added a comment -

            Added testing instructions and submitting for integration.

            Tomasz, if you are able to rebase this on the latest 22_STABLE it'd be helpful.

            INTEGRATOR: Can be cherry-picked to 23 and master.

            Show
            Dan Poltawski added a comment - Added testing instructions and submitting for integration. Tomasz, if you are able to rebase this on the latest 22_STABLE it'd be helpful. INTEGRATOR: Can be cherry-picked to 23 and master.
            Hide
            Permalink
            Dan Poltawski added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            TIA and ciao

            Show
            Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
            Hide
            Permalink
            Eloy Lafuente (stronk7) added a comment -

            Integrated (22, 23 & master), thanks!

            Show
            Eloy Lafuente (stronk7) added a comment - Integrated (22, 23 & master), thanks!
            Hide
            Permalink
            Tomasz Muras added a comment - - edited

            Hi, I just came back - I see that you don't need re-basing for 22_STABLE anymore?

            Show
            Tomasz Muras added a comment - - edited Hi, I just came back - I see that you don't need re-basing for 22_STABLE anymore?
            Hide
            Permalink
            Dan Poltawski added a comment -

            No, thanks Tomasz - it cherry-picked cleanly (I imagine).

            Show
            Dan Poltawski added a comment - No, thanks Tomasz - it cherry-picked cleanly (I imagine).
            Hide
            Permalink
            Jason Fowler added a comment -

            All good Dan

            Show
            Jason Fowler added a comment - All good Dan
            Hide
            Permalink
            Eloy Lafuente (stronk7) added a comment -

            Fixed STOP Closed STOP Thanks STOP

            Yay, imagination! Ciao

            Show
            Eloy Lafuente (stronk7) added a comment - Fixed STOP Closed STOP Thanks STOP Yay, imagination! Ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: