Uploaded image for project: 'OpenOLAT'
  1. OpenOLAT
  2. OO-1447

Better display of lifecycle in My courses and detail page

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 10.2
    • Fix Version/s: 10.2.1
    • Component/s: None
    • Labels:
      None

      Description

      One thing to discuss:
      For our cases it's better to display the semester instead of start and end date both in My courses and in detail (info) page of a course. If semester is set as lifecycle by an author and the semester is configured with a start and end date, everything is displayed in My courses list. I think, this is confusing. Every student knows when semester starts and ends, so the information about start and end date is not important.

      That's my patch which also corrects a velocity syntax error (details.html: $!v -> !$v):

      diff -r 460fad2a6bec src/main/java/org/olat/repository/ui/list/_content/details.html
      --- a/src/main/java/org/olat/repository/ui/list/_content/details.html	Fri Feb 20 14:49:10 2015 +0100
      +++ b/src/main/java/org/olat/repository/ui/list/_content/details.html	Fri Feb 20 15:41:11 2015 +0100
      @@ -41,9 +41,11 @@
       					#if($v.lifecycle || $v.lifecycle.softKey || $row.lifecycle.start || $row.lifecycle.end)
       					<div title="$r.translateInAttribute("cif.dates")">
       						<i class="o_icon o_icon_lifecycle"></i> 
      -						#if($!v.lifecycle.private && $v.lifecycle.softKey) $v.lifecycle.softKey #end
      -						#if($v.lifecycle.validFrom) $r.formatDate($v.lifecycle.validFrom) #end
      -						#if($v.lifecycle.validTo) - $r.formatDate($v.lifecycle.validTo) #end
      +						#if(!$v.lifecycle.privateCycle) $v.lifecycle.softKey
      +						#else
      +							#if($v.lifecycle.validFrom) $r.formatDate($v.lifecycle.validFrom) #end
      +							#if($v.lifecycle.validTo) - $r.formatDate($v.lifecycle.validTo) #end
      +						#end
       					</div>
       					#end
       					#if($v.expenditureOfWork && !$v.expenditureOfWork.empty)
      diff -r 460fad2a6bec src/main/java/org/olat/repository/ui/list/_content/row_1.html
      --- a/src/main/java/org/olat/repository/ui/list/_content/row_1.html	Fri Feb 20 14:49:10 2015 +0100
      +++ b/src/main/java/org/olat/repository/ui/list/_content/row_1.html	Fri Feb 20 15:41:11 2015 +0100
      @@ -23,10 +23,11 @@
       		#if($row.lifecycle || $row.lifecycleSoftKey || $row.lifecycleStart || $row.lifecycleEnd)
       			<div class="o_lifecycle #if ($row.isActive()) o_active #end">
       				<i class="o_icon o_icon_lifecycle"></i>
      -				#if($row.lifecycle) $row.lifecycle #end
      -				#if($row.lifecycleSoftKey) $row.lifecycleSoftKey #end
      -				#if($row.lifecycleStart) $r.formatDate($row.lifecycleStart) #end
      -				#if($row.lifecycleEnd) - $r.formatDate($row.lifecycleEnd) #end
      +				#if($row.lifecycleSoftKey) $row.lifecycleSoftKey
      +				#else
      +					#if($row.lifecycleStart) $r.formatDate($row.lifecycleStart) #end
      +					#if($row.lifecycleEnd) - $r.formatDate($row.lifecycleEnd) #end
      +				#end
       			</div>
       		#end
       		<div class="o_desc">
      

        Attachments

          Activity

            People

            Assignee:
            srosse Stéphane Rossé
            Reporter:
            stephan Stephan Clemenz
            Tester:
            Florian Gnägi
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 1 hour, 10 minutes
                1h 10m