Jeroen De Dauw has submitted this change and it was merged.
Change subject: Show links to ambassador profiles based on right instead of
enrollment
......................................................................
Show links to ambassador profiles based on right instead of enrollment
Bug: 43929
Change-Id: Ie34bbe86918dcee85cd8be2df1700aeb97bad7f5
---
M includes/Menu.php
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Jeroen De Dauw: Verified; Looks good to me, approved
diff --git a/includes/Menu.php b/includes/Menu.php
index 8a6f6a3..8393abb 100644
--- a/includes/Menu.php
+++ b/includes/Menu.php
@@ -110,11 +110,11 @@
$items['ep-nav-mycourses'] = SpecialPage::getTitleFor(
'MyCourses' );
}
- if ( OA::newFromUser( $user )->hasCourse() ) {
+ if ( $user->isAllowed( 'eponline' ) ) {
$items['ep-nav-oaprofile'] = SpecialPage::getTitleFor(
'OnlineAmbassadorProfile' );
}
- if ( CA::newFromUser( $user )->hasCourse() ) {
+ if ( $user->isAllowed( 'epcampus' ) ) {
$items['ep-nav-caprofile'] = SpecialPage::getTitleFor(
'CampusAmbassadorProfile' );
}
--
To view, visit https://gerrit.wikimedia.org/r/51793
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie34bbe86918dcee85cd8be2df1700aeb97bad7f5
Gerrit-PatchSet: 7
Gerrit-Project: mediawiki/extensions/EducationProgram
Gerrit-Branch: master
Gerrit-Owner: Jeroen De Dauw <[email protected]>
Gerrit-Reviewer: CSteipp <[email protected]>
Gerrit-Reviewer: Jeroen De Dauw <[email protected]>
Gerrit-Reviewer: Ragesoss <[email protected]>
Gerrit-Reviewer: Reedy <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
