Moodle
  1. Moodle
  2. MDL-40353

Wrong redirection from course groups page

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.7, 2.4.4, 2.5
    • Fix Version/s: 2.3.8, 2.4.5, 2.5.1
    • Component/s: Groups
    • Labels:
    • Rank:
      51133

      Description

      Not a major (hard to reproduce) but still

      Inside the course I selected:
      Course Administration -> Users -> Groups
      then in another window I logged in as student and forgot about it.
      I returned to the first window and tried to click 'Add group' (or any button, or just refresh page), received an error with text "Array" and redirection to http://localhost/course/view.php, which does not exist because my site is installed in a subfolder.

        Activity

        Hide
        Marina Glancy added a comment -

        I did not correct the redirection url, I've just changed the capability check to the standard call to require_capability()

        Show
        Marina Glancy added a comment - I did not correct the redirection url, I've just changed the capability check to the standard call to require_capability()
        Hide
        Eloy Lafuente (stronk7) 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
        Eloy Lafuente (stronk7) 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
        Damyon Wiese added a comment -

        Thanks Marina, integrated to 23, 24, 25 and master.

        Show
        Damyon Wiese added a comment - Thanks Marina, integrated to 23, 24, 25 and master.
        Hide
        Andrew Davis added a comment -

        Works as described. Passing.

        Show
        Andrew Davis added a comment - Works as described. Passing.
        Hide
        Damyon Wiese added a comment -

        This issue is fixed! Hurray! Hurray!
        Your issue is fixed, what a wonderful day!

        Cheers, Damyon

        Show
        Damyon Wiese added a comment - This issue is fixed! Hurray! Hurray! Your issue is fixed, what a wonderful day! Cheers, Damyon

          People

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

            Dates

            • Created:
              Updated:
              Resolved: