Commit 93df9c4c by Chris Dodge

resolve merge conflicts

parent 042b7d5f
...@@ -13,7 +13,7 @@ ...@@ -13,7 +13,7 @@
<h2 class="info-course"> <h2 class="info-course">
<span class="sr">${_("Current Course:")}</span> <span class="sr">${_("Current Course:")}</span>
<a class="course-link" href="${reverse('course_index', kwargs=dict(org=ctx_loc.org, course=ctx_loc.course, name=ctx_loc.name))}"> <a class="course-link" href="${reverse('course_index', kwargs=dict(org=ctx_loc.org, course=ctx_loc.course, name=ctx_loc.name))}">
<span class="course-org">${ctx_loc.org}</span><span class="course-number">${ctx_loc.course}</span> <span class="course-org">${context_course.display_org_with_default}</span><span class="course-number">${context_course.display_number_with_default}</span>
<span class="course-title" title="${context_course.display_name_with_default}">${context_course.display_name_with_default}</span> <span class="course-title" title="${context_course.display_name_with_default}">${context_course.display_name_with_default}</span>
</a> </a>
</h2> </h2>
......
...@@ -362,6 +362,11 @@ class CourseFields(object): ...@@ -362,6 +362,11 @@ class CourseFields(object):
# Explicit comparison to True because we always want to return a bool. # Explicit comparison to True because we always want to return a bool.
hide_progress_tab = Boolean(help="DO NOT USE THIS", scope=Scope.settings) hide_progress_tab = Boolean(help="DO NOT USE THIS", scope=Scope.settings)
display_organization = String(help="An optional display string for the course organization that will get rendered in the LMS",
scope=Scope.settings)
display_coursenumber = String(help="An optional display string for the course number that will get rendered in the LMS",
scope=Scope.settings)
class CourseDescriptor(CourseFields, SequenceDescriptor): class CourseDescriptor(CourseFields, SequenceDescriptor):
module_class = SequenceModule module_class = SequenceModule
...@@ -934,5 +939,25 @@ class CourseDescriptor(CourseFields, SequenceDescriptor): ...@@ -934,5 +939,25 @@ class CourseDescriptor(CourseFields, SequenceDescriptor):
return self.location.course return self.location.course
@property @property
def display_number_with_default(self):
"""
Return a display course number if it has been specified, otherwise return the 'course' that is in the location
"""
if self.display_coursenumber:
return self.display_coursenumber
return self.location.course
@property
def org(self): def org(self):
return self.location.org return self.location.org
@property
def display_org_with_default(self):
"""
Return a display organization if it has been specified, otherwise return the 'org' that is in the location
"""
if self.display_organization:
return self.display_organization
return self.location.org
...@@ -95,7 +95,7 @@ def course_wiki_redirect(request, course_id): ...@@ -95,7 +95,7 @@ def course_wiki_redirect(request, course_id):
root, root,
course_slug, course_slug,
title=course_slug, title=course_slug,
content="This is the wiki for **{0}**'s _{1}_.".format(course.org, course.display_name_with_default), content="This is the wiki for **{0}**'s _{1}_.".format(course.display_org_with_default, course.display_name_with_default),
user_message="Course page automatically created.", user_message="Course page automatically created.",
user=None, user=None,
ip_address=None, ip_address=None,
......
...@@ -174,9 +174,9 @@ def get_course_about_section(course, section_key): ...@@ -174,9 +174,9 @@ def get_course_about_section(course, section_key):
elif section_key == "title": elif section_key == "title":
return course.display_name_with_default return course.display_name_with_default
elif section_key == "university": elif section_key == "university":
return course.location.org return course.display_org_with_default
elif section_key == "number": elif section_key == "number":
return course.number return course.display_number_with_default
raise KeyError("Invalid about key " + str(section_key)) raise KeyError("Invalid about key " + str(section_key))
......
...@@ -14,13 +14,13 @@ from courseware.courses import course_image_url, get_course_about_section ...@@ -14,13 +14,13 @@ from courseware.courses import course_image_url, get_course_about_section
<div class="inner-wrapper"> <div class="inner-wrapper">
<header class="course-preview"> <header class="course-preview">
<hgroup> <hgroup>
<h2><span class="course-number">${course.number}</span> ${get_course_about_section(course, 'title')}</h2> <h2><span class="course-number">${course.display_number_with_default}</span> ${get_course_about_section(course, 'title')}</h2>
</hgroup> </hgroup>
<div class="info-link">&#x2794;</div> <div class="info-link">&#x2794;</div>
</header> </header>
<section class="info"> <section class="info">
<div class="cover-image"> <div class="cover-image">
<img src="${course_image_url(course)}" alt="${course.number} ${get_course_about_section(course, 'title')} Cover Image" /> <img src="${course_image_url(course)}" alt="${course.display_number_with_default} ${get_course_about_section(course, 'title')} Cover Image" />
</div> </div>
<div class="desc"> <div class="desc">
<p>${get_course_about_section(course, 'short_description')}</p> <p>${get_course_about_section(course, 'short_description')}</p>
......
...@@ -66,7 +66,11 @@ ...@@ -66,7 +66,11 @@
<script src="${static.url('js/course_info.js')}"></script> <script src="${static.url('js/course_info.js')}"></script>
</%block> </%block>
<<<<<<< HEAD
<%block name="title"><title>${_("About {course.number}").format(course=course)}</title></%block> <%block name="title"><title>${_("About {course.number}").format(course=course)}</title></%block>
=======
<%block name="title"><title>About ${course.display_number_with_default}</title></%block>
>>>>>>> add display_coursenumber and display_organization fields on the CourseModule, with some property accessors. Update LMS/CMS pages to use those display strings as appropraite.
<section class="course-info"> <section class="course-info">
<header class="course-profile"> <header class="course-profile">
...@@ -75,7 +79,7 @@ ...@@ -75,7 +79,7 @@
<section class="intro"> <section class="intro">
<hgroup> <hgroup>
<h1> <h1>
${course.number}: ${get_course_about_section(course, "title")} ${course.display_number_with_default}: ${get_course_about_section(course, "title")}
% if not self.theme_enabled(): % if not self.theme_enabled():
<a href="#">${get_course_about_section(course, "university")}</a> <a href="#">${get_course_about_section(course, "university")}</a>
% endif % endif
...@@ -87,13 +91,21 @@ ...@@ -87,13 +91,21 @@
%if show_courseware_link: %if show_courseware_link:
<a href="${course_target}"> <a href="${course_target}">
%endif %endif
<<<<<<< HEAD
<span class="register disabled">${_("You are registered for this course {course.number}").format(course=course)}</span> <span class="register disabled">${_("You are registered for this course {course.number}").format(course=course)}</span>
=======
<span class="register disabled">You are registered for this course (${course.display_number_with_default})</span>
>>>>>>> add display_coursenumber and display_organization fields on the CourseModule, with some property accessors. Update LMS/CMS pages to use those display strings as appropraite.
%if show_courseware_link: %if show_courseware_link:
<strong>${_("View Courseware")}</strong> <strong>${_("View Courseware")}</strong>
</a> </a>
%endif %endif
%else: %else:
<<<<<<< HEAD
<a href="#" class="register">${_("Register for {course.number}").format(course=course)}</a> <a href="#" class="register">${_("Register for {course.number}").format(course=course)}</a>
=======
<a href="#" class="register">Register for ${course.display_number_with_default}</a>
>>>>>>> add display_coursenumber and display_organization fields on the CourseModule, with some property accessors. Update LMS/CMS pages to use those display strings as appropraite.
<div id="register_error"></div> <div id="register_error"></div>
%endif %endif
</div> </div>
...@@ -162,8 +174,13 @@ ...@@ -162,8 +174,13 @@
</header> </header>
<ol class="important-dates"> <ol class="important-dates">
<<<<<<< HEAD
<li><div class="icon course-number"></div><p>${_("Course Number")}</p><span class="course-number">${course.number}</span></li> <li><div class="icon course-number"></div><p>${_("Course Number")}</p><span class="course-number">${course.number}</span></li>
<li><div class="icon start"></div><p>${_("Classes Start")}</p><span class="start-date">${course.start_date_text}</span></li> <li><div class="icon start"></div><p>${_("Classes Start")}</p><span class="start-date">${course.start_date_text}</span></li>
=======
<li><div class="icon course-number"></div><p>Course Number</p><span class="course-number">${course.display_number_with_default}</span></li>
<li><div class="icon start"></div><p>Classes Start</p><span class="start-date">${course.start_date_text}</span></li>
>>>>>>> add display_coursenumber and display_organization fields on the CourseModule, with some property accessors. Update LMS/CMS pages to use those display strings as appropraite.
## We plan to ditch end_date (which is not stored in course metadata), ## We plan to ditch end_date (which is not stored in course metadata),
## but for backwards compatibility, show about/end_date blob if it exists. ## but for backwards compatibility, show about/end_date blob if it exists.
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
<%inherit file="/main.html" /> <%inherit file="/main.html" />
<%namespace name='static' file='/static_content.html'/> <%namespace name='static' file='/static_content.html'/>
<%block name="bodyclass">courseware ${course.css_class}</%block> <%block name="bodyclass">courseware ${course.css_class}</%block>
<%block name="title"><title>${_("{course_number} Courseware").format(course_number=course.number)}</title></%block> <%block name="title"><title>${_("{course_number} Courseware").format(course_number=course.display_number_with_default)}</title></%block>
<%block name="headextra"> <%block name="headextra">
<%static:css group='course'/> <%static:css group='course'/>
......
...@@ -7,7 +7,7 @@ ...@@ -7,7 +7,7 @@
<%static:css group='course'/> <%static:css group='course'/>
</%block> </%block>
<%block name="title"><title>${_("{course.number} Course Info").format(course=course)}</title></%block> <%block name="title"><title>${_("{course.number} Course Info").format(course=course.display_number_with_default)}</title></%block>
<%include file="/courseware/course_navigation.html" args="active_page='info'" /> <%include file="/courseware/course_navigation.html" args="active_page='info'" />
<%! <%!
......
...@@ -8,7 +8,7 @@ ...@@ -8,7 +8,7 @@
<%inherit file="../mktg_iframe.html" /> <%inherit file="../mktg_iframe.html" />
<%block name="title"><title>${_("About {course_number}").format(course_number=course.number)}</title></%block> <%block name="title"><title>${_("About {course_number}").format(course_number=course.display_number_with_default)}</title></%block>
<%block name="bodyclass">view-partial-mktgregister</%block> <%block name="bodyclass">view-partial-mktgregister</%block>
...@@ -52,7 +52,7 @@ ...@@ -52,7 +52,7 @@
<div class="action is-registered">${_("You Are Registered")}</div> <div class="action is-registered">${_("You Are Registered")}</div>
%endif %endif
%elif allow_registration: %elif allow_registration:
<a class="action action-register register" href="#">${_("Register for")} <strong>${course.number}</strong></a> <a class="action action-register register" href="#">${_("Register for")} <strong>${course.display_number_with_default}</strong></a>
%else: %else:
<div class="action registration-closed is-disabled">${_("Registration Is Closed")}</div> <div class="action registration-closed is-disabled">${_("Registration Is Closed")}</div>
%endif %endif
......
...@@ -8,7 +8,7 @@ ...@@ -8,7 +8,7 @@
<%namespace name="progress_graph" file="/courseware/progress_graph.js"/> <%namespace name="progress_graph" file="/courseware/progress_graph.js"/>
<%block name="title"><title>${_("{course_number} Progress").format(course_number=course.number)}</title></%block> <%block name="title"><title>${_("{course_number} Progress").format(course_number=course.display_number_with_default)}</title></%block>
<%! <%!
from django.core.urlresolvers import reverse from django.core.urlresolvers import reverse
......
...@@ -6,7 +6,7 @@ ...@@ -6,7 +6,7 @@
<%static:css group='course'/> <%static:css group='course'/>
</%block> </%block>
<%block name="title"><title>${course.number} ${tab['name']}</title></%block> <%block name="title"><title>${course.display_number_with_default} ${tab['name']}</title></%block>
<%include file="/courseware/course_navigation.html" args="active_page='static_tab_{0}'.format(tab['url_slug'])" /> <%include file="/courseware/course_navigation.html" args="active_page='static_tab_{0}'.format(tab['url_slug'])" />
......
...@@ -6,7 +6,7 @@ ...@@ -6,7 +6,7 @@
<%static:css group='course'/> <%static:css group='course'/>
</%block> </%block>
<%block name="title"><title>${_("{course.number} Course Info").format(course=course)}</title></%block> <%block name="title"><title>${_("{course.display_number_with_default} Course Info").format(course=course)}</title></%block>
<%include file="/courseware/course_navigation.html" args="active_page='syllabus'" /> <%include file="/courseware/course_navigation.html" args="active_page='syllabus'" />
<%! <%!
......
...@@ -140,11 +140,19 @@ ...@@ -140,11 +140,19 @@
% if course.id in show_courseware_links_for: % if course.id in show_courseware_links_for:
<a href="${course_target}" class="cover"> <a href="${course_target}" class="cover">
<<<<<<< HEAD
<img src="${course_image_url(course)}" alt="${_('{course_number} {course_name} Cover Image').format(course_number='${course.number}', course_name='${course.display_name_with_default}')}" /> <img src="${course_image_url(course)}" alt="${_('{course_number} {course_name} Cover Image').format(course_number='${course.number}', course_name='${course.display_name_with_default}')}" />
</a> </a>
% else: % else:
<div class="cover"> <div class="cover">
<img src="${course_image_url(course)}" alt="${_('{course_number} {course_name} Cover Image').format(course_number='${course.number}', course_name='${course.display_name_with_default}')}" /> <img src="${course_image_url(course)}" alt="${_('{course_number} {course_name} Cover Image').format(course_number='${course.number}', course_name='${course.display_name_with_default}')}" />
=======
<img src="${course_image_url(course)}" alt="${course.display_number_with_default} ${course.display_name_with_default} Cover Image" />
</a>
% else:
<div class="cover">
<img src="${course_image_url(course)}" alt="${course.display_number_with_default} ${course.display_name_with_default} Cover Image" />
>>>>>>> add display_coursenumber and display_organization fields on the CourseModule, with some property accessors. Update LMS/CMS pages to use those display strings as appropraite.
</div> </div>
% endif % endif
...@@ -162,9 +170,9 @@ ...@@ -162,9 +170,9 @@
<h2 class="university">${get_course_about_section(course, 'university')}</h2> <h2 class="university">${get_course_about_section(course, 'university')}</h2>
<h3> <h3>
% if course.id in show_courseware_links_for: % if course.id in show_courseware_links_for:
<a href="${course_target}">${course.number} ${course.display_name_with_default}</a> <a href="${course_target}">${course.display_number_with_default} ${course.display_name_with_default}</a>
% else: % else:
<span>${course.number} ${course.display_name_with_default}</span> <span>${course.display_number_with_default} ${course.display_name_with_default}</span>
% endif % endif
</h3> </h3>
</hgroup> </hgroup>
...@@ -197,6 +205,7 @@ ...@@ -197,6 +205,7 @@
% endif % endif
% if registration.is_rejected: % if registration.is_rejected:
<div class="message message-status is-shown exam-schedule"> <div class="message message-status is-shown exam-schedule">
<<<<<<< HEAD
<p class="message-copy"> <p class="message-copy">
<strong>${_("Your registration for the Pearson exam has been rejected. Please {link_start}see your registration status details{link_end}.").format( <strong>${_("Your registration for the Pearson exam has been rejected. Please {link_start}see your registration status details{link_end}.").format(
link_start='<a href="{url}" id="exam_register_link">'.format(url=testcenter_register_target), link_start='<a href="{url}" id="exam_register_link">'.format(url=testcenter_register_target),
...@@ -206,6 +215,9 @@ ...@@ -206,6 +215,9 @@
link_end='</a>', link_end='</a>',
email="exam-help@edx.org", email="exam-help@edx.org",
)} )}
=======
<p class="message-copy"><strong>Your registration for the Pearson exam has been rejected. Please <a href="${testcenter_register_target}" id="exam_register_link">see your registration status details</a></strong>. Otherwise <a class="contact-link" href="mailto:exam-help@edx.org?subject=Pearson VUE Exam - ${get_course_about_section(course, 'university')} ${course.display_number_with_default}">contact edX at exam-help@edx.org</a> for further help.</p>
>>>>>>> add display_coursenumber and display_organization fields on the CourseModule, with some property accessors. Update LMS/CMS pages to use those display strings as appropraite.
</div> </div>
% endif % endif
% if not registration.is_accepted and not registration.is_rejected: % if not registration.is_accepted and not registration.is_rejected:
......
...@@ -6,7 +6,7 @@ ...@@ -6,7 +6,7 @@
<%inherit file="../main.html" /> <%inherit file="../main.html" />
<%namespace name='static' file='../static_content.html'/> <%namespace name='static' file='../static_content.html'/>
<%block name="bodyclass">discussion</%block> <%block name="bodyclass">discussion</%block>
<%block name="title"><title>${_("Discussion - {course_number}").format(course_number=course.number) | h}</title></%block> <%block name="title"><title>${_("Discussion - {course_number}").format(course_number=course.display_number_with_default) | h}</title></%block>
<%block name="headextra"> <%block name="headextra">
<%static:css group='course'/> <%static:css group='course'/>
......
...@@ -7,7 +7,7 @@ ...@@ -7,7 +7,7 @@
<%inherit file="../main.html" /> <%inherit file="../main.html" />
<%namespace name='static' file='../static_content.html'/> <%namespace name='static' file='../static_content.html'/>
<%block name="bodyclass">discussion</%block> <%block name="bodyclass">discussion</%block>
<%block name="title"><title>${_("Discussion - {course_number}").format(course_number=course.number) | h}</title></%block> <%block name="title"><title>${_("Discussion - {course_number}").format(course_number=course.display_number_with_default) | h}</title></%block>
<%block name="headextra"> <%block name="headextra">
<%static:css group='course'/> <%static:css group='course'/>
......
...@@ -4,7 +4,7 @@ ...@@ -4,7 +4,7 @@
<%inherit file="../main.html" /> <%inherit file="../main.html" />
<%namespace name='static' file='../static_content.html'/> <%namespace name='static' file='../static_content.html'/>
<%block name="bodyclass">discussion</%block> <%block name="bodyclass">discussion</%block>
<%block name="title"><title>${_("Discussion - {course_number}").format(course_number=course.number) | h}</title></%block> <%block name="title"><title>${_("Discussion - {course_number}").format(course_number=course.display_number_with_default) | h}</title></%block>
<%block name="headextra"> <%block name="headextra">
<%static:css group='course'/> <%static:css group='course'/>
......
...@@ -7,7 +7,7 @@ ...@@ -7,7 +7,7 @@
<%static:css group='course'/> <%static:css group='course'/>
</%block> </%block>
<%block name="title"><title>${_("{course_number} Staff Grading").format(course_number=course.number)}</title></%block> <%block name="title"><title>${_("{course_number} Staff Grading").format(course_number=course.display_number_with_default)}</title></%block>
<%include file="/courseware/course_navigation.html" args="active_page='staff_grading'" /> <%include file="/courseware/course_navigation.html" args="active_page='staff_grading'" />
......
...@@ -50,7 +50,7 @@ site_status_msg = get_site_status_msg(course_id) ...@@ -50,7 +50,7 @@ site_status_msg = get_site_status_msg(course_id)
</h1> </h1>
% if course: % if course:
<h2><span class="provider">${course.org}:</span> ${course.number} ${course.display_name_with_default}</h2> <h2><span class="provider">${course.display_org_with_default}:</span> ${course.display_number_with_default} ${course.display_name_with_default}</h2>
% endif % endif
% if user.is_authenticated(): % if user.is_authenticated():
......
...@@ -7,7 +7,7 @@ ...@@ -7,7 +7,7 @@
<%static:css group='course'/> <%static:css group='course'/>
</%block> </%block>
<%block name="title"><title>${_("{course_number} Combined Notifications").format(course_number=course.number)}</title></%block> <%block name="title"><title>${_("{course_number} Combined Notifications").format(course_number=course.display_number_with_default)}</title></%block>
<%include file="/courseware/course_navigation.html" args="active_page='open_ended'" /> <%include file="/courseware/course_navigation.html" args="active_page='open_ended'" />
......
...@@ -7,7 +7,7 @@ ...@@ -7,7 +7,7 @@
<%static:css group='course'/> <%static:css group='course'/>
</%block> </%block>
<%block name="title"><title>${_("{course_number} Flagged Open Ended Problems").format(course_number=course.number)}</title></%block> <%block name="title"><title>${_("{course_number} Flagged Open Ended Problems").format(course_number=course.display_number_with_default)}</title></%block>
<%include file="/courseware/course_navigation.html" args="active_page='open_ended_flagged_problems'" /> <%include file="/courseware/course_navigation.html" args="active_page='open_ended_flagged_problems'" />
......
...@@ -7,7 +7,7 @@ ...@@ -7,7 +7,7 @@
<%static:css group='course'/> <%static:css group='course'/>
</%block> </%block>
<%block name="title"><title>${_("{course_number} Open Ended Problems").format(course_number=course.number)}</title></%block> <%block name="title"><title>${_("{course_number} Open Ended Problems").format(course_number=course.display_number_with_default)}</title></%block>
<%include file="/courseware/course_navigation.html" args="active_page='open_ended_problems'" /> <%include file="/courseware/course_navigation.html" args="active_page='open_ended_problems'" />
......
...@@ -2,7 +2,8 @@ ...@@ -2,7 +2,8 @@
<%inherit file="main.html" /> <%inherit file="main.html" />
<%namespace name='static' file='static_content.html'/> <%namespace name='static' file='static_content.html'/>
<%block name="title"><title>${_('{course_number} Textbook').format(course_number=course.number)}</title> <%block name="title"><title>${_('{course_number} Textbook').format(course_number=course.display_number_with_default)}</title>
</%block> </%block>
<%block name="headextra"> <%block name="headextra">
......
...@@ -5,7 +5,8 @@ ...@@ -5,7 +5,8 @@
<%block name="title"> <%block name="title">
<meta charset="utf-8"> <meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1"> <meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1">
<title>${_('{course_number} Textbook').format(course_number=course.number)}</title> <title>${_('{course_number} Textbook').format(course_number=course.display_number_with_default)}</title>
</%block> </%block>
<%block name="headextra"> <%block name="headextra">
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
<%inherit file="main.html" /> <%inherit file="main.html" />
<%namespace name='static' file='static_content.html'/> <%namespace name='static' file='static_content.html'/>
<%block name="title"><title>${_("{course_number} Textbook").format(course_number=course.number)}</title></%block> <%block name="title"><title>${_("{course_number} Textbook").format(course_number=course.display_number_with_default)}</title></%block>
<%block name="headextra"> <%block name="headextra">
<%static:css group='course'/> <%static:css group='course'/>
......
...@@ -95,7 +95,7 @@ ...@@ -95,7 +95,7 @@
<section class="introduction"> <section class="introduction">
<header> <header>
<hgroup> <hgroup>
<h2><a href="${reverse('dashboard')}">${get_course_about_section(course, 'university')} ${course.number} ${course.display_name_with_default}</a></h2> <h2><a href="${reverse('dashboard')}">${get_course_about_section(course, 'university')} ${course.display_number_with_default} ${course.display_name_with_default}</a></h2>
% if registration: % if registration:
<h1>${_('Your Pearson VUE Proctored Exam Registration')}</h1> <h1>${_('Your Pearson VUE Proctored Exam Registration')}</h1>
...@@ -442,7 +442,7 @@ ...@@ -442,7 +442,7 @@
% endif % endif
<div class="details details-course"> <div class="details details-course">
<h4>${_("About {university} {course_number}").format(university=get_course_about_section(course, 'university'), course_number=course.number)}</h4> <h4>${_("About {university} {course_number}").format(university=get_course_about_section(course, 'university'), course_number=course.display_number_with_default)}</h4>
<p> <p>
% if course.has_ended(): % if course.has_ended():
<span class="label">${_('Course Completed:')}</span> <span class="value">${course.end_date_text}</span> <span class="label">${_('Course Completed:')}</span> <span class="value">${course.end_date_text}</span>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment