From 3be5a98a8dfce123889f34d80fb16d161db13d97 Mon Sep 17 00:00:00 2001
From: sam marshall <s.marshall@open.ac.uk>
Date: Mon, 7 Aug 2023 12:19:44 +0100
Subject: [PATCH] MDL-78927 core_user: Profile page missing breadcrumbs if no
 id=

---
 user/profile.php | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/user/profile.php b/user/profile.php
index be5ce4851ae..2de644ed536 100644
--- a/user/profile.php
+++ b/user/profile.php
@@ -42,8 +42,6 @@ $userid         = optional_param('id', 0, PARAM_INT);
 $edit           = optional_param('edit', null, PARAM_BOOL);    // Turn editing on and off.
 $reset          = optional_param('reset', null, PARAM_BOOL);
 
-$PAGE->set_url('/user/profile.php', array('id' => $userid));
-
 if (!empty($CFG->forceloginforprofiles)) {
     require_login();
     if (isguestuser()) {
@@ -60,6 +58,11 @@ if (!empty($CFG->forceloginforprofiles)) {
 }
 
 $userid = $userid ? $userid : $USER->id;       // Owner of the page.
+
+// Even if the user didn't supply a userid, we treat page URL as if they did; this is needed
+// so navigation works correctly.
+$PAGE->set_url('/user/profile.php', ['id' => $userid]);
+
 if ((!$user = $DB->get_record('user', array('id' => $userid))) || ($user->deleted)) {
     $PAGE->set_context(context_system::instance());
     echo $OUTPUT->header();