This is a snapshot of Indico's old Trac site. Any information contained herein is most probably outdated. Access our new GitHub site here.

#1535 closed defect (fixed)

Obsolete time zone info in conference Overview page

Reported by: s.hesselbach Owned by:
Priority: low Milestone: v1.2
Component: General Version: 1.1
Keywords: Cc:

Description

Because nobody replied to my email on the list I open this ticket to make sure it won't be forgotten. To correct the problem only one line of code in .../MaKaC/webinterface/pages/conferences.py has to be changed.

In conferences which start in one month and in another one, e.g.

http://indico.cern.ch/event/286275/

the time zone info appears also the <div class="date"> in addition to the <div class="timezone">:
...
<div class="confSubTitle" >

<div class="datePlace">

<div class="date">from 6 January 2014 to 13 March 2014 (Europe/Zurich?)</div>
<div class="place">European Scientific Institute - Archamps</div>
<div class="timezone">Europe/Zurich? timezone</div>

...

I think the time zone info in the first line

<div class="date">from 6 January 2014 to 13 March 2014 (Europe/Zurich?)</div>

should go away as is the case in conferences which start and end in the same month: http://indico.cern.ch/event/306859/

I have found the place where to correct it:

MaKaC/webinterface/pages/conferences.py

in line 468 for the current master branch in the GIT.
(class WConfDisplayFrame; in function getVars), see attached patch.

This patch is applied to our installation, see e.g.

http://indico.gsi.de/event/stori14

I have checked that the timezone info still appears also for meetings and lectures.

Attachments (1)

conferences.patch (762 bytes) - added by s.hesselbach 17 months ago.
Patch for MaKaC/webinterface/pages/conferences.py

Download all attachments as: .zip

Change History (5)

Changed 17 months ago by s.hesselbach

Patch for MaKaC/webinterface/pages/conferences.py

comment:1 Changed 17 months ago by pferreir

It's merged, thanks!

https://github.com/indico/indico/commit/d3d0414874c680cc2df2456a4090cb02a1aa1a2c

I should have said "Patch contributed", though :/

comment:2 Changed 17 months ago by pferreir

  • Status changed from new to awaiting_merge

comment:3 Changed 17 months ago by pferreir

  • Status changed from awaiting_merge to merging

comment:4 Changed 17 months ago by pferreir

  • Resolution set to fixed
  • Status changed from merging to closed
Note: See TracTickets for help on using tickets.