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:

          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".

            Gliffy Diagrams

              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: